[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 07/18] xen: add event channel interface for XenD

From: Anthony PERARD
Subject: Re: [Qemu-devel] [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

reply via email to

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