From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesper Nilsson Subject: Re: [PATCH] VFS: Convert file->f_dentry->d_inode to file_inode() [ver #2] Date: Fri, 16 Jan 2015 09:34:33 +0100 Message-ID: <20150116083433.GZ26817@axis.com> References: <20150113175753.29073.20788.stgit@warthog.procyon.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jesper Nilsson , "viro@ZenIV.linux.org.uk" , "linux-fsdevel@vger.kernel.org" , "linux-kernel@vger.kernel.org" , linux-cris-kernel To: David Howells Return-path: Content-Disposition: inline In-Reply-To: <20150113175753.29073.20788.stgit@warthog.procyon.org.uk> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Tue, Jan 13, 2015 at 06:57:53PM +0100, David Howells wrote: > Convert file->f_dentry->d_inode to file_inode() so as to get layered > filesystems right. > > Found with: git grep '[.>]f_dentry' Yes, compile works with this patch. Do you want me to take it through the CRIS tree or do you have any other tree in mind? > Signed-off-by: David Howells In any case: Reviewed-by: Jesper Nilsson > arch/cris/arch-v32/drivers/sync_serial.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/cris/arch-v32/drivers/sync_serial.c b/arch/cris/arch-v32/drivers/sync_serial.c > index 08a313fc2241..f772068d9e79 100644 > --- a/arch/cris/arch-v32/drivers/sync_serial.c > +++ b/arch/cris/arch-v32/drivers/sync_serial.c > @@ -604,7 +604,7 @@ static ssize_t __sync_serial_read(struct file *file, > struct timespec *ts) > { > unsigned long flags; > - int dev = MINOR(file->f_dentry->d_inode->i_rdev); > + int dev = MINOR(file_inode(file)->i_rdev); > int avail; > struct sync_port *port; > unsigned char *start; > /^JN - Jesper Nilsson -- Jesper Nilsson -- jesper.nilsson@axis.com