Re: [PATCH] umount.nfs: restore correct error status when umount fails.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 12 Jul 2012 18:44:20 +0200 Karel Zak <kzak@xxxxxxxxxx> wrote:

> > This patch restores the use of EX_FILEIO for errors from umount.
> 
>  Looks good to me.
> 
>  Reviewed-by: Karel Zak <kzak@xxxxxxxxxx>
> 
> > There is a case that I know it doesn't handle.  If you ask umount.nfs to
> > unmount a filesystem that is not nfs or nfs4, then the old code will
> > refuse
> >   umount.nfs: /dev/sda7 on /mnt2 is not an NFS filesystem
> > 
> > and exit with status '1'.
> > The new libmount code will just think that it couldn't find anything in
> > fstab and will try to do an nfs23 unmount.  This is clearly different behaviour,
> > I'm not sure that anyone would care though.
> 
>  I care, it's stupid bug to use umount.nfs for non-NFS filesytems. The
>  patch below fixes this issue.
> 
Cool - thanks for that!

NeilBrown

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux