Re: [PATCH 3/3] eventfd: add internal reference counting to fixnotifier race conditions

From: Michael S. Tsirkin
Date: Tue Jun 23 2009 - 11:05:57 EST


On Tue, Jun 23, 2009 at 07:35:00AM -0700, Davide Libenzi wrote:
> On Tue, 23 Jun 2009, Gregory Haskins wrote:
>
> > Davide Libenzi wrote:
> > > On Mon, 22 Jun 2009, Gregory Haskins wrote:
> > >
> > >
> > >> To be honest, I am not sure. I would guess its not a *huge* deal,
> > >> though it was obviously enough of a concern to at least discuss it. I
> > >> can definitely say that I think the other issues which are being fixed
> > >> are substantially more important.
> > >>
> > >
> > > Ok then, will repost the revised patch later today.
> > >
> >
> > Ok sounds good. I did have a chance to take a closer look at your
> > proposal for the key data, and I think it makes a lot of sense. Do you
> > see it as a problem if we defer adding that? Or should we try to add
> > that notion now?
>
> That would need to go eventually via mainline, after some discussion. But
> yes, I believe that using the "key" as simple bitmask is a little
> restrictive.
>
>
> - Davide

I agree.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/