Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Jason Baron
Hi Michael, On 03/14/2016 07:26 PM, Michael Kerrisk (man-pages) wrote: > Hi Jason, > > On 03/15/2016 11:35 AM, Jason Baron wrote: >> Hi Michael, >> >> On 03/14/2016 05:03 PM, Michael Kerrisk (man-pages) wrote: >>> Hi Jason, >>> >>> On 03/15/2016 09:01 AM, Michael Kerrisk (man-pages) wrote: H

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Michael Kerrisk (man-pages)
Hi Jason, On 03/15/2016 11:35 AM, Jason Baron wrote: > Hi Michael, > > On 03/14/2016 05:03 PM, Michael Kerrisk (man-pages) wrote: >> Hi Jason, >> >> On 03/15/2016 09:01 AM, Michael Kerrisk (man-pages) wrote: >>> Hi Jason, >>> >>> On 03/15/2016 08:32 AM, Jason Baron wrote: On 03/14/

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Madars Vitolins
Hi Jason and Michael, Hmm... I tried to play with those pipe samples bellow, but even with sleep I got that all process wakeups (maybe I miss something too), also tried with EPOLLIN. On same bases I created sample with Posix Queues with EPOLLIN | EPOLLEXCLUSIVE and the goods news are that it

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Jason Baron
Hi Michael, On 03/14/2016 05:03 PM, Michael Kerrisk (man-pages) wrote: > Hi Jason, > > On 03/15/2016 09:01 AM, Michael Kerrisk (man-pages) wrote: >> Hi Jason, >> >> On 03/15/2016 08:32 AM, Jason Baron wrote: >>> >>> >>> On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: [Restoring CC

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Michael Kerrisk (man-pages)
Hi Jason, On 03/15/2016 09:01 AM, Michael Kerrisk (man-pages) wrote: > Hi Jason, > > On 03/15/2016 08:32 AM, Jason Baron wrote: >> >> >> On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: >>> [Restoring CC, which I see I accidentally dropped, one iteration back.] [...] >>> Returning to

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Michael Kerrisk (man-pages)
Hi Jason, On 03/15/2016 08:32 AM, Jason Baron wrote: > > > On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: >> [Restoring CC, which I see I accidentally dropped, one iteration back.] [...] values in events yield an error. EPOLLEXCLUSIVE may be

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Jason Baron
On 03/14/2016 01:47 PM, Michael Kerrisk (man-pages) wrote: > [Restoring CC, which I see I accidentally dropped, one iteration back.] > > Hi Jason, > > Thanks for the review. I've tweaked one piece to respond to your > feedback. But I also have another new question below. > > On 03/15/2016 03:5

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-14 Thread Michael Kerrisk (man-pages)
[Restoring CC, which I see I accidentally dropped, one iteration back.] Hi Jason, Thanks for the review. I've tweaked one piece to respond to your feedback. But I also have another new question below. On 03/15/2016 03:55 AM, Jason Baron wrote: > On 03/11/2016 06:25 PM, Michael Kerrisk (man-pages

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-11 Thread Michael Kerrisk (man-pages)
>> By the way, in the code you have >> >> case EPOLL_CTL_MOD: >> if (epi) { >> if (!(epi->event.events & EPOLLEXCLUSIVE)) { >> epds.events |= POLLERR | POLLHUP; >> error = ep_modify(ep, e

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-10 Thread Jason Baron
On 03/10/2016 02:58 PM, Michael Kerrisk (man-pages) wrote: > On 03/10/2016 07:53 PM, Jason Baron wrote: >> Hi Michael, >> >> On 01/29/2016 03:14 AM, Michael Kerrisk (man-pages) wrote: >>> Hello Jason, >>> On 01/28/2016 06:57 PM, Jason Baron wrote: Hi, On 01/28/2016 02:16 AM, Michae

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-10 Thread Michael Kerrisk (man-pages)
On 03/10/2016 07:53 PM, Jason Baron wrote: > Hi Michael, > > On 01/29/2016 03:14 AM, Michael Kerrisk (man-pages) wrote: >> Hello Jason, >> On 01/28/2016 06:57 PM, Jason Baron wrote: >>> Hi, >>> >>> On 01/28/2016 02:16 AM, Michael Kerrisk (man-pages) wrote: Hi Jason, On 12/08/2015 04

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-10 Thread Michael Kerrisk (man-pages)
Hi Jason, > Ok, here's some updated text: > > EPOLLEXCLUSIVE > > Sets an exclusive wakeup mode for the epfd file descriptor that is being > attached to the target file descriptor, fd. When a wakeup event occurs > and multiple epfd file descriptors are attached to the same target file > using EPO

Re: [PATCH] epoll: add exclusive wakeups flag

2016-03-10 Thread Jason Baron
Hi Michael, On 01/29/2016 03:14 AM, Michael Kerrisk (man-pages) wrote: > Hello Jason, > On 01/28/2016 06:57 PM, Jason Baron wrote: >> Hi, >> >> On 01/28/2016 02:16 AM, Michael Kerrisk (man-pages) wrote: >>> Hi Jason, >>> >>> On 12/08/2015 04:23 AM, Jason Baron wrote: Hi, Re-post of

Re: [PATCH] epoll: add exclusive wakeups flag

2016-02-01 Thread Jason Baron
On 01/29/2016 03:14 AM, Michael Kerrisk (man-pages) wrote: > Hello Jason, > On 01/28/2016 06:57 PM, Jason Baron wrote: >> Hi, >> >> On 01/28/2016 02:16 AM, Michael Kerrisk (man-pages) wrote: >>> Hi Jason, >>> >>> On 12/08/2015 04:23 AM, Jason Baron wrote: Hi, Re-post of an old seri

Re: [PATCH] epoll: add exclusive wakeups flag

2016-01-29 Thread Michael Kerrisk (man-pages)
Hello Jason, On 01/28/2016 06:57 PM, Jason Baron wrote: > Hi, > > On 01/28/2016 02:16 AM, Michael Kerrisk (man-pages) wrote: >> Hi Jason, >> >> On 12/08/2015 04:23 AM, Jason Baron wrote: >>> Hi, >>> >>> Re-post of an old series addressing thundering herd issues when sharing >>> an event source fd

Re: [PATCH] epoll: add exclusive wakeups flag

2016-01-28 Thread Jason Baron
Hi, On 01/28/2016 02:16 AM, Michael Kerrisk (man-pages) wrote: > Hi Jason, > > On 12/08/2015 04:23 AM, Jason Baron wrote: >> Hi, >> >> Re-post of an old series addressing thundering herd issues when sharing >> an event source fd amongst multiple epoll fds. Last posting was here >> for reference:

Re: [PATCH] epoll: add exclusive wakeups flag

2016-01-27 Thread Michael Kerrisk (man-pages)
Hi Jason, On 12/08/2015 04:23 AM, Jason Baron wrote: > Hi, > > Re-post of an old series addressing thundering herd issues when sharing > an event source fd amongst multiple epoll fds. Last posting was here > for reference: https://lkml.org/lkml/2015/2/25/56 > > The patch herein drops the core s