[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 07/18] xen: add event channel interface for XenD
From: |
Anthony PERARD |
Subject: |
Re: [Qemu-block] [PATCH 07/18] xen: add event channel interface for XenDevice-s |
Date: |
Tue, 4 Dec 2018 14:24:59 +0000 |
User-agent: |
Mutt/1.11.0 (2018-11-25) |
On Mon, Dec 03, 2018 at 04:24:24PM +0000, Anthony PERARD wrote:
> On Wed, Nov 21, 2018 at 03:12:00PM +0000, Paul Durrant wrote:
> > +static void xen_device_event(void *opaque)
> > +{
> > + XenDevice *xendev = opaque;
> > + unsigned long port = xenevtchn_pending(xendev->xeh);
> > +
> > + notifier_list_notify(&xendev->event_notifiers, (void *)port);
>
> I wonder if a Notifier is a good fit for XenDevice, like here for the
> events or the xenstore watches in previous patches, as NotifierLists are
> normaly used when every Notifiers want to do something, but here there
> is only one that is going to do something. But I guess it might not be
> much better to write a loop in here rather than use the one in
> notifier_list_notify.
I've seen that you use GHashTable in a following patch, wouldn't that be
useful to use for xenstore watches and evtchn events as well?
--
Anthony PERARD