Re: RFC: poll change

Andrea Arcangeli (andrea@suse.de)
Wed, 15 Aug 2001 16:13:18 +0200


On Tue, Aug 14, 2001 at 05:16:09PM -0700, David S. Miller wrote:
> From: Andrea Arcangeli <andrea@suse.de>
> Date: Wed, 15 Aug 2001 02:11:10 +0200
>
> On Tue, Aug 14, 2001 at 04:53:43PM -0700, Tim Hockin wrote:
> > - if (nfds > NR_OPEN)
> > + if (nfds > current->rlim[RLIMIT_NOFILE].rlim_cur)
>
> Here SuS speaks about OPEN_MAX, not sure if OPEN_MAX corresponds to the
> current ulimit or to the absolute maximum (to me it sounds more like our
> NR_OPEN).
>
> Right, and our equivalent is "NR_OPEN".

I was backporting the new version to 2.2 and I noticed that by using
NR_OPEN an luser will actually be able to lock into RAM something of the
order of the dozen mbytes per process. So I'm wondering that it would be
saner to use the rlimit instead, after all I don't see much of a value
to use NR_OPEN instead of the rlimit (even if strictly speaking SuS asks
us to use NR_OPEN). Any weird program (if any) that would depend on
NR_OPEN instead of the rlimit can be easily fixed with a one liner at
most. So I guess I'd be more happy with the rlimit instead of NR_OPEN.
Comments?

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