Messages in this thread | | | From | "Bron Gondwana" <> | Subject | Re: [patch 016/104] epoll: introduce resource usage limits | Date | Wed, 28 Jan 2009 20:24:53 +1100 |
| |
On Tue, 27 Jan 2009 22:57 -0800, "Davide Libenzi" <davidel@xmailserver.org> wrote: > On Tue, 27 Jan 2009, Davide Libenzi wrote: > > > Or, like Willy said, make (for -stable) the default unlimited, and > > let sysadmins to put the bounds if they feel the DoS can apply to them. > > Whose patch follows ...
ACK.
Solves my problem and my "advocate of the poor suffering sysadmins who have to track down why their stuff suddenly broke with a stable update hat" problem as well.
One wondering...
> - if (unlikely(atomic_read(&user->epoll_devs) >= > - max_user_instances)) > + if (atomic_read(&user->epoll_devs) >= max_user_instances) > goto free_uid;
Any reason this has become _less_ unlikely()?
Thanks,
Bron.
-- Bron Gondwana brong@fastmail.fm
| |