Re: [patch] inotify.

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

 



On Fri, Jun 17, 2005 at 02:38:06PM -0400, Robert Love wrote:
> > You are using ioctl as an really bad syscall multiplexer.  You're
> > not using the file descriptor it's called on at all, so it does not qualify
> > as a valid ioctl() usage even under the most lax rules.
> 
> We provide two different ioctl commands, it is not a bad multiplexer.
> We have discussed this before.
> 
> We do use the fd.  It maps back to the inotify device.

inotify does indeed use file->private data to retrieve the inotify_dev
structure.  Of which by design exists a single instance only.  So yes,
you do not use the file descriptor at all.

This really looks like copied from the ioctl abuse 101 example code.
We are beating up driver writers for stupidities like that all the time,
and inotify is not going to get a special treatment just because it's
core code.

> > Also you claimed the resource shortage for the proposed architecture
> > with just a single syscall, aka one watch per fd without showing any
> > reasons why that would be true, in fact by any means there's no reason
> > to believe file descriptors are a rare ressource in a modern Linux system.
> 
> It is not implausible to believe that a system might have the default
> maximum for file descriptors (not very high) but allow a _much_ greater
> number of inotify watches (32k, say).

And a default limit matters exactly how?

> Everything to you is "really bad" and "totally unacceptable".  Chill
> out.  Stop ranting so much and enjoy life.

Thanks a lot, I'm enjoying life a lot when I don't happen to have to deal
with ignorant people :)

-
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