public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jason Baron <jbaron@akamai.com>
To: Eric Wong <normalperson@yhbt.net>, Madars Vitolins <m@silodev.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: epoll and multiple processes - eliminate unneeded process wake-ups
Date: Tue, 04 Aug 2015 11:02:24 -0400	[thread overview]
Message-ID: <55C0D400.3070904@akamai.com> (raw)
In-Reply-To: <20150803234842.GA21995@dcvr.yhbt.net>



On 08/03/2015 07:48 PM, Eric Wong wrote:
> Madars Vitolins <m@silodev.com> wrote:
>> Hi Folks,
>>
>> I am developing kind of open systems application, which uses
>> multiple processes/executables where each of them monitors some set
>> of resources (in this case POSIX Queues) via epoll interface. For
>> example when 10 processes on same queue are in state of epoll_wait()
>> and one message arrives, all 10 processes gets woken up and all of
>> them tries to read the message from Q. One succeeds, the others gets
>> EAGAIN error. The problem is with those others, which generates
>> extra context switches - useless CPU usage. With more processes
>> inefficiency gets higher.
>>
>> I tried to use EPOLLONESHOT, but no help. Seems this is suitable for
>> multi-threaded application and not for multi-process application.
> 
> Correct.  Most FDs are not shared across processes.
> 
>> Ideal mechanism for this would be:
>> 1. If multiple epoll sets in kernel matches same event and one or
>> more processes are in state of epoll_wait() - then send event only
>> to one waiter.
>> 2. If none of processes are in wait state, then send the event to
>> all epoll sets (as it is currently). Then the first free process
>> will grab the event.
> 
> Jason Baron was working on this (search LKML archives for
> EPOLLEXCLUSIVE, EPOLLROUNDROBIN, EPOLL_ROTATE)
> 
> However, I was unconvinced about modifying epoll.
> 
> Perhaps I may be more easily convinced about your mqueue case than his
> case for listen sockets, though[*]
> 

Yeah, so I implemented an 'EPOLL_ROTATE' mode, where you could have
multiple epoll fds (or epoll sets) attached to the same wakeup source,
and have the wakeups 'rotate' among the epoll sets. The wakeup
essentially walks the list of waiters, wakes up the first thread
that is actively in epoll_wait(), stops and moves the woken up
epoll set to the end of the list. So it attempts to balance
the wakeups among the epoll sets, I think in the way that you
were describing.

Here is the patchset:

https://lkml.org/lkml/2015/2/24/667

The test program shows how to use the API. Essentially, you
have to create a 'dummy' epoll fd with the 'EPOLL_ROTATE' flag,
which you then attach to you're shared wakeup source and
then to your epoll sets. Please let me know if its unclear.

Thanks,

-Jason

> Typical applications have few (probably only one) listen sockets or
> POSIX mqueues; so I would rather use dedicated threads to issue
> blocking syscalls (accept4 or mq_timedreceive).
> 
> Making blocking syscalls allows exclusive wakeups to avoid thundering
> herds.
> 
>> How do you think, would it be real to implement this? How about
>> concurrency?
>> Can you please give me some hints from which points in code to start
>> to implement these changes?
> 
> For now, I suggest dedicating a thread in each process to do
> mq_timedreceive/mq_receive, assuming you only have a small amount
> of queues in your system.
> 
> 
> [*] mq_timedreceive may copy a largish buffer which benefits from
>     staying on the same CPU as much as possible.
>     Contrary, accept4 only creates a client socket.  With a C10K+
>     socket server (e.g. http/memcached/DB), a typical new client
>     socket spends a fair amount of time idle.  Thus I don't believe
>     memory locality inside the kernel is much concern when there's
>     thousands of accepted client sockets.
> 

  reply	other threads:[~2015-08-04 15:03 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-13 12:34 epoll and multiple processes - eliminate unneeded process wake-ups Madars Vitolins
2015-07-15 13:07 ` Madars Vitolins
2015-08-03 23:48 ` Eric Wong
2015-08-04 15:02   ` Jason Baron [this message]
2015-08-05 11:06     ` Madars Vitolins
2015-08-05 13:32       ` Jason Baron
  -- strict thread matches above, loose matches on Subject: below --
2015-11-28 22:54 Madars Vitolins
2015-11-30 19:45 ` Jason Baron
2015-11-30 21:28   ` Madars Vitolins
2015-12-01 20:11     ` Jason Baron
2015-12-05 11:47       ` Madars Vitolins

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55C0D400.3070904@akamai.com \
    --to=jbaron@akamai.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m@silodev.com \
    --cc=normalperson@yhbt.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox