[RFC][PATCH] Inotify kernel API
serue at us.ibm.com
serue at us.ibm.com
Fri Aug 26 17:10:18 UTC 2005
Quoting Chris Wright (chrisw at osdl.org):
> * Robert Love (rml at novell.com) wrote:
> > On Fri, 2005-08-26 at 11:48 -0400, John McCutchan wrote:
> >
> > > There is nothing in inotify_watch that is useful to other parts of the
> > > kernel other than the inode, which will be passed to the callback.
> >
> > Also, I don't see anything wrong with not exposing data structures and
> > proving access only via abstracted interfaces.
>
> There's no compelling reason (that I see), to make it typeless. No need
> to expose irrelevant internals, but should be simple enough to create
> a data structure that's useable and/or extensible by the client.
I agree - so long as there is no need to make it typeless, exposing the
data structure:
1. makes it much easier for a new person to investigate and
learn the interface (and == easier code review)
2. iiuc, should make it easier to detect errors with sparse
or other code checkers.
Just my pov... I'm sure someone feels the opposite particularly about
my point (1) :)
-serge
More information about the Linux-audit
mailing list