Messages in this thread | | | From | Justin He <> | Subject | RE: [PATCH 02/14] d_path: saner calling conventions for __dentry_path() | Date | Wed, 7 Jul 2021 04:52:41 +0000 |
| |
> -----Original Message----- > From: Justin He > Sent: Friday, June 25, 2021 5:33 PM > To: Al Viro <viro@zeniv.linux.org.uk>; Linus Torvalds <torvalds@linux- > foundation.org> > Cc: Petr Mladek <pmladek@suse.com>; Steven Rostedt <rostedt@goodmis.org>; > Sergey Senozhatsky <senozhatsky@chromium.org>; Andy Shevchenko > <andriy.shevchenko@linux.intel.com>; Rasmus Villemoes > <linux@rasmusvillemoes.dk>; Jonathan Corbet <corbet@lwn.net>; Heiko > Carstens <hca@linux.ibm.com>; Vasily Gorbik <gor@linux.ibm.com>; Christian > Borntraeger <borntraeger@de.ibm.com>; Eric W . Biederman > <ebiederm@xmission.com>; Darrick J. Wong <darrick.wong@oracle.com>; Peter > Zijlstra (Intel) <peterz@infradead.org>; Ira Weiny <ira.weiny@intel.com>; > Eric Biggers <ebiggers@google.com>; Ahmed S. Darwish > <a.darwish@linutronix.de>; open list:DOCUMENTATION <linux- > doc@vger.kernel.org>; Linux Kernel Mailing List <linux- > kernel@vger.kernel.org>; linux-s390 <linux-s390@vger.kernel.org>; linux- > fsdevel <linux-fsdevel@vger.kernel.org> > Subject: RE: [PATCH 02/14] d_path: saner calling conventions for > __dentry_path() > > Hi Al > > > -----Original Message----- > > From: Al Viro <viro@ftp.linux.org.uk> On Behalf Of Al Viro > > Sent: Wednesday, May 19, 2021 8:49 AM > > To: Linus Torvalds <torvalds@linux-foundation.org> > > Cc: Justin He <Justin.He@arm.com>; Petr Mladek <pmladek@suse.com>; Steven > > Rostedt <rostedt@goodmis.org>; Sergey Senozhatsky > > <senozhatsky@chromium.org>; Andy Shevchenko > > <andriy.shevchenko@linux.intel.com>; Rasmus Villemoes > > <linux@rasmusvillemoes.dk>; Jonathan Corbet <corbet@lwn.net>; Heiko > > Carstens <hca@linux.ibm.com>; Vasily Gorbik <gor@linux.ibm.com>; > Christian > > Borntraeger <borntraeger@de.ibm.com>; Eric W . Biederman > > <ebiederm@xmission.com>; Darrick J. Wong <darrick.wong@oracle.com>; Peter > > Zijlstra (Intel) <peterz@infradead.org>; Ira Weiny <ira.weiny@intel.com>; > > Eric Biggers <ebiggers@google.com>; Ahmed S. Darwish > > <a.darwish@linutronix.de>; open list:DOCUMENTATION <linux- > > doc@vger.kernel.org>; Linux Kernel Mailing List <linux- > > kernel@vger.kernel.org>; linux-s390 <linux-s390@vger.kernel.org>; linux- > > fsdevel <linux-fsdevel@vger.kernel.org> > > Subject: [PATCH 02/14] d_path: saner calling conventions for > __dentry_path() > > > > 1) lift NUL-termination into the callers > > 2) pass pointer to the end of buffer instead of that to beginning. > > > > (1) allows to simplify dentry_path() - we don't need to play silly > > games with restoring the leading / of "//deleted" after __dentry_path() > > would've overwritten it with NUL. > > > > We also do not need to check if (either) prepend() in there fails - > > if the buffer is not large enough, we'll end with negative buflen > > after prepend() and __dentry_path() will return the right value > > (ERR_PTR(-ENAMETOOLONG)) just fine. > > > > Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> > > --- > > fs/d_path.c | 33 +++++++++++++-------------------- > > 1 file changed, 13 insertions(+), 20 deletions(-) > > > > diff --git a/fs/d_path.c b/fs/d_path.c > > index 01df5dfa1f88..1a1cf05e7780 100644 > > --- a/fs/d_path.c > > +++ b/fs/d_path.c > > @@ -326,22 +326,21 @@ char *simple_dname(struct dentry *dentry, char > > *buffer, int buflen) > > /* > > * Write full pathname from the root of the filesystem into the buffer. > > */ > I suggest adding the comments to remind NUL terminator should be prepended > before invoking __dentry_path() > Except for my suggestion about adding comments
Reviewed-by: Jia He <justin.he@arm.com>
| |