Re: [patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-30 Thread Linus Torvalds
On Fri, 30 Mar 2007, Andrew Morton wrote: > > > > Ok, it was panincing, and someone made me change it. Would you please > > agree? > > The system can survive w/out, but it'll be a broken system WRT userspace. > > I'd say panic. There's no much point in limping along with an > incorrectly-work

Re: [patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-30 Thread Davide Libenzi
On Fri, 30 Mar 2007, Andrew Morton wrote: > I'd say panic. There's no much point in limping along with an > incorrectly-working kernel, only to have some small number of apps fail > mysteriously later on. Panic it is ... > > > Can we make this optional if CONFIG_EMBEDDED? You plan on converti

Re: [patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-30 Thread Andrew Morton
On Fri, 30 Mar 2007 15:44:15 -0700 (PDT) Davide Libenzi wrote: > On Fri, 30 Mar 2007, Andrew Morton wrote: > > > > +#include > > > + > > > + > > > + > > > > Too many blank lines > > It'd be interesting to know how much is enough. You use one, ppl says it > is too dense. You use more, ppl say

Re: [patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-30 Thread Randy Dunlap
On Fri, 30 Mar 2007 15:44:15 -0700 (PDT) Davide Libenzi wrote: > On Fri, 30 Mar 2007, Andrew Morton wrote: > > > > +#include > > > + > > > + > > > + > > > > Too many blank lines > > It'd be interesting to know how much is enough. You use one, ppl says it > is too dense. You use more, ppl says

Re: [patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-30 Thread Davide Libenzi
On Fri, 30 Mar 2007, Andrew Morton wrote: > > +#include > > + > > + > > + > > Too many blank lines It'd be interesting to know how much is enough. You use one, ppl says it is too dense. You use more, ppl says it's too much. There's the one-line rule for inter-function spacing, but what's the

Re: [patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-30 Thread Andrew Morton
On Tue, 20 Mar 2007 11:37:14 -0700 Davide Libenzi wrote: > This patch add an anonymous inode source, to be used for files that need > and inode only in order to create a file*. We do not care of having an > inode for each file, and we do not even care of having different names in > the associa

[patch 1/13] signal/timer/event fds v8 - anonymous inode source ...

2007-03-20 Thread Davide Libenzi
This patch add an anonymous inode source, to be used for files that need and inode only in order to create a file*. We do not care of having an inode for each file, and we do not even care of having different names in the associated dentries (dentry names will be same for classes of file*). This