From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46547) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XyP00-0000qk-62 for qemu-devel@nongnu.org; Tue, 09 Dec 2014 12:55:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XyOzv-0000CD-Fz for qemu-devel@nongnu.org; Tue, 09 Dec 2014 12:55:00 -0500 Received: from cantor2.suse.de ([195.135.220.15]:49147 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XyOzv-0000C7-5d for qemu-devel@nongnu.org; Tue, 09 Dec 2014 12:54:55 -0500 Message-ID: <5487376D.1010406@suse.de> Date: Tue, 09 Dec 2014 18:54:53 +0100 From: =?windows-1252?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1417776605-36309-1-git-send-email-paul.durrant@citrix.com> <1417776605-36309-2-git-send-email-paul.durrant@citrix.com> <54819ABB.4030600@redhat.com> <9AAE0902D5BC7E449B7C8E4E778ABCD025780787@AMSPEX01CL01.citrite.net> <54873414.8040401@redhat.com> In-Reply-To: <54873414.8040401@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v5 1/2] Add device listener interface List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paul Durrant , "qemu-devel@nongnu.org" , Stefano Stabellini Cc: Peter Crosthwaite , Thomas Huth , "Michael S. Tsirkin" , Markus Armbruster , Christian Borntraeger , Igor Mammedov , Paolo Bonzini Am 09.12.2014 um 18:40 schrieb Paolo Bonzini: > On 08/12/2014 12:12, Paul Durrant wrote: >>> -----Original Message----- >>> From: Paolo Bonzini [mailto:pbonzini@redhat.com] >>> Sent: 05 December 2014 11:45 >>> To: Paul Durrant; qemu-devel@nongnu.org >>> Cc: Michael S. Tsirkin; Andreas Faerber"; Peter Crosthwaite; Igor >>> Mammedov; Markus Armbruster; Thomas Huth; Christian Borntraeger >>> Subject: Re: [PATCH v5 1/2] Add device listener interface >>> >>> >>> >>> On 05/12/2014 11:50, Paul Durrant wrote: >>>> The Xen ioreq-server API, introduced in Xen 4.5, requires that PCI d= evice >>>> models explicitly register with Xen for config space accesses. This = patch >>>> adds a listener interface into qdev-core which can be used by the Xe= n >>>> interface code to monitor for arrival and departure of PCI devices. >>>> >>>> Signed-off-by: Paul Durrant >>>> Cc: Michael S. Tsirkin >>>> Cc: Andreas Faerber" >>>> Cc: Paolo Bonzini >>>> Cc: Peter Crosthwaite >>>> Cc: Igor Mammedov >>>> Cc: Markus Armbruster >>>> Cc: Thomas Huth >>>> Cc: Christian Borntraeger >>>> --- >>>> hw/core/qdev.c | 53 >>> +++++++++++++++++++++++++++++++++++++++++++++++ >>>> include/hw/qdev-core.h | 10 +++++++++ >>>> include/qemu/typedefs.h | 1 + >>>> 3 files changed, 64 insertions(+) >>>> >>>> diff --git a/hw/core/qdev.c b/hw/core/qdev.c >>>> index 35fd00d..76ff9ef 100644 >>>> --- a/hw/core/qdev.c >>>> +++ b/hw/core/qdev.c >>>> @@ -189,6 +189,56 @@ int qdev_init(DeviceState *dev) >>>> return 0; >>>> } >>>> >>>> +static QTAILQ_HEAD(device_listeners, DeviceListener) device_listene= rs >>>> + =3D QTAILQ_HEAD_INITIALIZER(device_listeners); >>>> + >>>> +enum ListenerDirection { Forward, Reverse }; >>>> + >>>> +#define DEVICE_LISTENER_CALL(_callback, _direction, _args...) \ >>>> + do { \ >>>> + DeviceListener *_listener; \ >>>> + \ >>>> + switch (_direction) { \ >>>> + case Forward: \ >>>> + QTAILQ_FOREACH(_listener, &device_listeners, link) { \ >>>> + if (_listener->_callback) { \ >>>> + _listener->_callback(_listener, ##_args); \ >>>> + } \ >>>> + } \ >>>> + break; \ >>>> + case Reverse: \ >>>> + QTAILQ_FOREACH_REVERSE(_listener, &device_listeners, \ >>>> + device_listeners, link) { \ >>>> + if (_listener->_callback) { \ >>>> + _listener->_callback(_listener, ##_args); \ >>>> + } \ >>>> + } \ >>>> + break; \ >>>> + default: \ >>>> + abort(); \ >>>> + } \ >>>> + } while (0) >>>> + >>>> +static int device_listener_add(DeviceState *dev, void *opaque) >>>> +{ >>>> + DEVICE_LISTENER_CALL(realize, Forward, dev); >>>> + >>>> + return 0; >>>> +} >>>> + >>>> +void device_listener_register(DeviceListener *listener) >>>> +{ >>>> + QTAILQ_INSERT_TAIL(&device_listeners, listener, link); >>>> + >>>> + qbus_walk_children(sysbus_get_default(), NULL, NULL, >>> device_listener_add, >>>> + NULL, NULL); >>>> +} >>>> + >>>> +void device_listener_unregister(DeviceListener *listener) >>>> +{ >>>> + QTAILQ_REMOVE(&device_listeners, listener, link); >>>> +} >>>> + >>>> static void device_realize(DeviceState *dev, Error **errp) >>>> { >>>> DeviceClass *dc =3D DEVICE_GET_CLASS(dev); >>>> @@ -994,6 +1044,8 @@ static void device_set_realized(Object *obj, bo= ol >>> value, Error **errp) >>>> goto fail; >>>> } >>>> >>>> + DEVICE_LISTENER_CALL(realize, Forward, dev); >>>> + >>>> hotplug_ctrl =3D qdev_get_hotplug_handler(dev); >>>> if (hotplug_ctrl) { >>>> hotplug_handler_plug(hotplug_ctrl, dev, &local_err); >>>> @@ -1035,6 +1087,7 @@ static void device_set_realized(Object *obj, b= ool >>> value, Error **errp) >>>> dc->unrealize(dev, local_errp); >>>> } >>>> dev->pending_deleted_event =3D true; >>>> + DEVICE_LISTENER_CALL(unrealize, Reverse, dev); >>>> } >>>> >>>> if (local_err !=3D NULL) { >>>> diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h >>>> index 589bbe7..15a226f 100644 >>>> --- a/include/hw/qdev-core.h >>>> +++ b/include/hw/qdev-core.h >>>> @@ -165,6 +165,12 @@ struct DeviceState { >>>> int alias_required_for_version; >>>> }; >>>> >>>> +struct DeviceListener { >>>> + void (*realize)(DeviceListener *listener, DeviceState *dev); >>>> + void (*unrealize)(DeviceListener *listener, DeviceState *dev); >>>> + QTAILQ_ENTRY(DeviceListener) link; >>>> +}; >>>> + >>>> #define TYPE_BUS "bus" >>>> #define BUS(obj) OBJECT_CHECK(BusState, (obj), TYPE_BUS) >>>> #define BUS_CLASS(klass) OBJECT_CLASS_CHECK(BusClass, (klass), >>> TYPE_BUS) >>>> @@ -376,4 +382,8 @@ static inline bool qbus_is_hotpluggable(BusState >>> *bus) >>>> { >>>> return bus->hotplug_handler; >>>> } >>>> + >>>> +void device_listener_register(DeviceListener *listener); >>>> +void device_listener_unregister(DeviceListener *listener); >>>> + >>>> #endif >>>> diff --git a/include/qemu/typedefs.h b/include/qemu/typedefs.h >>>> index 3475177..4bb4938 100644 >>>> --- a/include/qemu/typedefs.h >>>> +++ b/include/qemu/typedefs.h >>>> @@ -20,6 +20,7 @@ typedef struct Property Property; >>>> typedef struct PropertyInfo PropertyInfo; >>>> typedef struct CompatProperty CompatProperty; >>>> typedef struct DeviceState DeviceState; >>>> +typedef struct DeviceListener DeviceListener; >>>> typedef struct BusState BusState; >>>> typedef struct BusClass BusClass; >>>> >>>> >>> >>> Reviewed-by: Paolo Bonzini >>> >> >> Great. Thanks for that. All I need now is an ack on this from a mainta= iner. >=20 > I think my reviewed-by is pretty close to an Ack. >=20 > Stefano, feel free to pick this patch up together with 2/2. In that case, Reviewed-by: Andreas F=E4rber If I should still pick it up, just let me know. Regards, Andreas --=20 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 21284 AG N=FCrnbe= rg