Re: [patch] inotify.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Freedag 17 Juni 2005 23:40, Robert Love wrote:
> On Fri, 2005-06-17 at 14:33 -0700, Andrew Morton wrote:
> 
> > So.  It's not too late.  Please spend an hour and write up the Inofity
> > Implementation FAQ?  You probably remember and fully understand what all of
> > our objections are and I know that you have explanations and rebuttals at
> > hand.
> 
> I wrote this up the first time you asked:
> 
>         Documentation/filesystems/inotify.txt
> 
> It sounds like its never been addressed because some people don't take
> different views as an acceptable solution.

Yes, this is the file that I quoted in my mail. However, "because adding
three or four new system calls that mirrored open, close, and ioctl seemed
silly" doesn't really explain the choice.
An explanation along the lines of "neither ioctl on cdev nor a syscall
based approach is made everyone happy, so we decided to stick with the
one that is already used" might give a little more insight.

However, this still does not explain why the choice here is different
from epoll, as I clearly remember the decision to move from /dev/epoll
to the final sys_epoll* interface [1]. If it's just a matter of different
views like "ioctl is ugly" vs "syscall is silly", I would have gone for a
sys_inotify_create()/sys_inotify_ctl() based interface simply for the
reason of consistency with existing interfaces like epoll or mq_open().

Of course, at this point, compatibility with the existing user base
might already be a much stronger argument.

	Arnd <><

[1] http://lwn.net/Articles/13264/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux