[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <44EBB6FD.3080005@symas.com>
Date: Tue, 22 Aug 2006 19:01:33 -0700
From: Howard Chu <hyc@...as.com>
To: Nicholas Miell <nmiell@...cast.net>
CC: David Miller <davem@...emloft.net>, rdunlap@...otime.net,
johnpol@....mipt.ru, linux-kernel@...r.kernel.org,
drepper@...hat.com, akpm@...l.org, netdev@...r.kernel.org,
zach.brown@...cle.com, hch@...radead.org
Subject: Re: The Proposed Linux kevent API
Nicholas Miell wrote:
> Having looked all this over to figure out what it actually does, I can
> make the following comments:
>
> - there's a distinct lack of any sort of commenting beyond brief
> descriptions of what the occasional function is supposed to do
>
> - the kevent interface is all the horror of the BSD kqueue interface,
> but with no compatibility with the BSD kqueue interface.
>
> - lots of parameters from userspace go unsanitized, although I'm not
> sure if this will actually cause problems. At the very least, there
> should be checks for unknown flags and use of reserved fields, lest
> somebody start using them for their own purposes and then their app
> breaks when a newer version of the kernel starts using them itself.
>
Which reminds me, why go through the trouble of copying the structs back
and forth between userspace and kernel space? Why not map the struct
array and leave it in place, as I proposed back here?
http://groups.google.com/group/linux.kernel/browse_frm/
thread/57847cfedb61bdd5/8d02afa60a8f83af?lnk=gst&q=equeue&rnum=
1#8d02afa60a8f83af
--
-- Howard Chu
Chief Architect, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc
OpenLDAP Core Team http://www.openldap.org/project/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists