Re: [PATCH uq/master 2/9] event_notifier: remove event_notifier_test

2012-07-12 Thread Paolo Bonzini
Il 12/07/2012 13:04, Avi Kivity ha scritto: > Right, it's useless. I'll adjust the comment (and the whitespace fix) > and apply. Ok, thanks very much! Paolo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo inf

Re: [PATCH uq/master 2/9] event_notifier: remove event_notifier_test

2012-07-12 Thread Avi Kivity
On 07/12/2012 01:30 PM, Paolo Bonzini wrote: > Il 12/07/2012 11:10, Avi Kivity ha scritto: >> On 07/05/2012 06:16 PM, Paolo Bonzini wrote: >>> This is broken; since the eventfd is used in nonblocking mode there >>> is a race between reading and writing. >>> >> >>> diff --git a/event_notifier.c b/e

Re: [PATCH uq/master 2/9] event_notifier: remove event_notifier_test

2012-07-12 Thread Paolo Bonzini
Il 12/07/2012 11:10, Avi Kivity ha scritto: > On 07/05/2012 06:16 PM, Paolo Bonzini wrote: >> This is broken; since the eventfd is used in nonblocking mode there >> is a race between reading and writing. >> > >> diff --git a/event_notifier.c b/event_notifier.c >> index 2b210f4..c339bfe 100644 >> -

Re: [PATCH uq/master 2/9] event_notifier: remove event_notifier_test

2012-07-12 Thread Avi Kivity
On 07/05/2012 06:16 PM, Paolo Bonzini wrote: > This is broken; since the eventfd is used in nonblocking mode there > is a race between reading and writing. > > diff --git a/event_notifier.c b/event_notifier.c > index 2b210f4..c339bfe 100644 > --- a/event_notifier.c > +++ b/event_notifier.c > @@ -

[PATCH uq/master 2/9] event_notifier: remove event_notifier_test

2012-07-05 Thread Paolo Bonzini
This is broken; since the eventfd is used in nonblocking mode there is a race between reading and writing. Signed-off-by: Paolo Bonzini --- event_notifier.c | 15 --- event_notifier.h |1 - 2 files changed, 16 deletions(-) diff --git a/event_notifier.c b/event_notifier.c index