Re: [PATCH] xfs_fsr: Get the last mount on a specific mount point

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

On Wed, Feb 15, 2012 at 10:31:43AM -0200, Carlos Maiolino wrote:
> I would change a little this sentence, looks a little confusing at least for 
> me.
> How about:
> 
>               /* If we found an entry based on the device name, make sure we
>                * stat the mount point given by mtab and ensure it's accessible
>                * before using it.
>                */

Agreed, I'll redo it.

> > +                   if (S_ISLNK(sb.st_mode)) {
> > +                           struct stat64 sb2;
> 
> Particularly, I don't like variable declarations between statements, but I'm 
> not
> too experienced to say what's good or not :)

The idea is to keep variable declarations as local as possible.