On Sun, Aug 26, 2012 at 04:05:20PM +0100, Al Viro wrote:
>
> Applied, with a couple of changes:
> * there's no need for those games with ihold/iput - opened file pins
>its inode down just fine, TYVM.
Ah, yeah, thanks!
> * struct fd_info is pointless in that form - the last argument
>
On Sun, Aug 26, 2012 at 06:28:20PM +0400, Cyrill Gorcunov wrote:
> On Sun, Aug 26, 2012 at 03:46:53AM +0100, Al Viro wrote:
> >
> > IMO doing that at open() time is just a headache for no good reason -
> > resolving descriptor to struct file * at read() time as we do now
> > is much saner.
On Sun, Aug 26, 2012 at 03:46:53AM +0100, Al Viro wrote:
>
> IMO doing that at open() time is just a headache for no good reason -
> resolving descriptor to struct file * at read() time as we do now
> is much saner. Better do that in your ->show(), since you are using
> a single-shot iterator
On Sun, Aug 26, 2012 at 03:46:53AM +0100, Al Viro wrote:
> On Thu, Aug 23, 2012 at 02:43:25PM +0400, Cyrill Gorcunov wrote:
> > This patch converts /proc/pid/fdinfo/ handling routines to seq-file which
> > is needed to extend seq operations and plug in auxiliary fdinfo provides
> > from subsystems
On Sun, Aug 26, 2012 at 03:46:53AM +0100, Al Viro wrote:
On Thu, Aug 23, 2012 at 02:43:25PM +0400, Cyrill Gorcunov wrote:
This patch converts /proc/pid/fdinfo/ handling routines to seq-file which
is needed to extend seq operations and plug in auxiliary fdinfo provides
from subsystems like
On Sun, Aug 26, 2012 at 03:46:53AM +0100, Al Viro wrote:
IMO doing that at open() time is just a headache for no good reason -
resolving descriptor to struct file * at read() time as we do now
is much saner. Better do that in your -show(), since you are using
a single-shot iterator
On Sun, Aug 26, 2012 at 06:28:20PM +0400, Cyrill Gorcunov wrote:
On Sun, Aug 26, 2012 at 03:46:53AM +0100, Al Viro wrote:
IMO doing that at open() time is just a headache for no good reason -
resolving descriptor to struct file * at read() time as we do now
is much saner. Better do that
On Sun, Aug 26, 2012 at 04:05:20PM +0100, Al Viro wrote:
Applied, with a couple of changes:
* there's no need for those games with ihold/iput - opened file pins
its inode down just fine, TYVM.
Ah, yeah, thanks!
* struct fd_info is pointless in that form - the last argument
of
On Thu, Aug 23, 2012 at 02:43:25PM +0400, Cyrill Gorcunov wrote:
> This patch converts /proc/pid/fdinfo/ handling routines to seq-file which
> is needed to extend seq operations and plug in auxiliary fdinfo provides
> from subsystems like eventfd/eventpoll/fsnotify.
>
> Note the proc_fd_link no
On Thu, Aug 23, 2012 at 02:43:25PM +0400, Cyrill Gorcunov wrote:
This patch converts /proc/pid/fdinfo/ handling routines to seq-file which
is needed to extend seq operations and plug in auxiliary fdinfo provides
from subsystems like eventfd/eventpoll/fsnotify.
Note the proc_fd_link no longer
This patch converts /proc/pid/fdinfo/ handling routines to seq-file which
is needed to extend seq operations and plug in auxiliary fdinfo provides
from subsystems like eventfd/eventpoll/fsnotify.
Note the proc_fd_link no longer call for proc_fd_info, simply because
proc_fd_info is converted to
This patch converts /proc/pid/fdinfo/ handling routines to seq-file which
is needed to extend seq operations and plug in auxiliary fdinfo provides
from subsystems like eventfd/eventpoll/fsnotify.
Note the proc_fd_link no longer call for proc_fd_info, simply because
proc_fd_info is converted to
12 matches
Mail list logo
