qemu-devel
[Top][All Lists]
Advanced

[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>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]