[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/10] qdev: rename qdev_unplug to qdev_unplug_r
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 08/10] qdev: rename qdev_unplug to qdev_unplug_req |
Date: |
Fri, 24 Aug 2012 16:48:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120717 Thunderbird/14.0 |
Il 24/08/2012 11:49, Liu Ping Fan ha scritto:
> From: Liu Ping Fan <address@hidden>
>
> Unplug divides into two steps: request and complete
> The name "req" show more clearly about its meaning
But not enough. Let's make it "qdev_request_unplug", and change the
method name in the DeviceClass too.
> and as to the complete, it is qdev_delete_subtree()
Completing unplug is just qdev_free after commit da5a44e (qom:
object_delete should unparent the object first, 2012-08-22).
> Also adding ret to indicate the request can be eject or not
This is also given by the Error object. Returning the same thing twice
is guaranteed to give bugs because the two get out of sync.
Paolo
> Signed-off-by: Liu Ping Fan <address@hidden>
- [Qemu-devel] [PATCH 05/10] qdev: finalize of qbus, qdev will not the right place to free children, (continued)
- [Qemu-devel] [PATCH 05/10] qdev: finalize of qbus, qdev will not the right place to free children, Liu Ping Fan, 2012/08/24
- [Qemu-devel] [PATCH 06/10] qom: expose object_property_del_child, Liu Ping Fan, 2012/08/24
- [Qemu-devel] [PATCH 07/10] unplug: using new intf qdev_delete_subtree in acpi_piix_eject_slot, Liu Ping Fan, 2012/08/24
- [Qemu-devel] [PATCH 09/10] mon: release dev's ref hold by qdev_get_peripheral, Liu Ping Fan, 2012/08/24
- [Qemu-devel] [PATCH 10/10] qdev: fix create in place obj's life cycle problem, Liu Ping Fan, 2012/08/24
- [Qemu-devel] [PATCH 08/10] qdev: rename qdev_unplug to qdev_unplug_req, Liu Ping Fan, 2012/08/24
- Re: [Qemu-devel] [PATCH 08/10] qdev: rename qdev_unplug to qdev_unplug_req,
Paolo Bonzini <=