Re: [GIT PULL] readlink cleanup

2016-12-11 Thread Miklos Szeredi
On Sun, Dec 11, 2016 at 3:39 AM, Al Viro wrote: > On Sun, Dec 11, 2016 at 02:36:20AM +, Al Viro wrote: > >> I'm still not sure what does "vfs: convert ->readlink to same signature as >> ->get_link" buy us. If anything, the result appears to be more complex - >> you make freeing that buffer de

Re: [GIT PULL] readlink cleanup

2016-12-10 Thread Al Viro
On Sun, Dec 11, 2016 at 02:36:20AM +, Al Viro wrote: > I'm still not sure what does "vfs: convert ->readlink to same signature as > ->get_link" buy us. If anything, the result appears to be more complex - > you make freeing that buffer delayed (and introduce a dynamically allocated > buffer i

Re: [GIT PULL] readlink cleanup

2016-12-10 Thread Al Viro
On Sat, Dec 10, 2016 at 09:41:45PM +0100, Miklos Szeredi wrote: > Hi Al, > > Please pull from: > > git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git readlink > > This is a rework of the readlink cleanup patchset. Now readlink(2) does the > following: > > - if i_op->readlink() i

[GIT PULL] readlink cleanup

2016-12-10 Thread Miklos Szeredi
Hi Al, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git readlink This is a rework of the readlink cleanup patchset. Now readlink(2) does the following: - if i_op->readlink() is non-NULL (only proc and afs mountpoints for now) then it calls that; - otherwi