[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release ob
From: |
Avi Kivity |
Subject: |
Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async |
Date: |
Tue, 11 Sep 2012 12:37:48 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120828 Thunderbird/15.0 |
On 09/11/2012 12:32 PM, liu ping fan wrote:
> On Tue, Sep 11, 2012 at 4:32 PM, Avi Kivity <address@hidden> wrote:
>> On 09/11/2012 10:51 AM, Liu Ping Fan wrote:
>>> From: Liu Ping Fan <address@hidden>
>>>
>>> DeviceState will be protected by refcnt from disappearing during
>>> dispatching. But when refcnt comes down to zero, DeviceState may
>>> be still in use by iohandler, timer etc in main loop, we just delay
>>> its free untill no reader.
>>>
>>
>> How can this be? We elevate the refcount while dispatching I/O. If we
>> have similar problems with the timer, we need to employ a similar solution.
>>
> Yes, at the next step, plan to covert iohandler, timer etc to use
> refcount as memory. Here just a temp solution.
I prefer not to ever introduce it.
What we can do is introduce a sub-region for e1000's mmio that will take
only the device lock, and let original region use the old dispatch path
(and also take the device lock). As we thread the various subsystems
e1000 uses, we can expand the sub-region until it covers all of e1000's
functions, then fold it back into the main region.
To start with the sub-region can only include registers that call no
qemu infrastructure code: simple read/writes.
--
error compiling committee.c: too many arguments to function
[Qemu-devel] [PATCH V3 11/11] vcpu: push mmio dispatcher out of big lock, Liu Ping Fan, 2012/09/11
[Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, Liu Ping Fan, 2012/09/11
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, Avi Kivity, 2012/09/11
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, liu ping fan, 2012/09/11
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async,
Avi Kivity <=
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, liu ping fan, 2012/09/13
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, Avi Kivity, 2012/09/13
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, liu ping fan, 2012/09/13
- Re: [Qemu-devel] [PATCH V3 08/11] qom: introduce reclaimer to release obj in async, Avi Kivity, 2012/09/13
[Qemu-devel] [PATCH V3 09/11] vcpu: make QemuThread as tls to store thread-self info, Liu Ping Fan, 2012/09/11
[Qemu-devel] [PATCH V3 06/11] memory: make mmio dispatch able to be out of biglock, Liu Ping Fan, 2012/09/11