On Friday 2010-04-16 10:40, [email protected] wrote:

>
>Jan Engelhardt:
>> >> >> 13:25 xensrvneu:/ # mount -t aufs none /x -o br:/root=rw
>> >> >> /sbin/mount.aufs:plink.c:223: AUFS_CTL_PLINK_MAINT: Invalid argument
>       :::
>> Yes, the command was
>>
>> strace -fo mount.log mount -t aufs none /mnt -o br:/dev/shm=rw
>
>Ah, you just changed the paths.
>Then let's confirm what value is passed from userspace to aufs.
>Apply this patch and see the value in the kernel log.
>
>
>J. R. Okajima
>
>diff --git a/fs/aufs/plink.c b/fs/aufs/plink.c
>index 80832e6..99c9e69 100644
>--- a/fs/aufs/plink.c
>+++ b/fs/aufs/plink.c
>@@ -424,6 +424,8 @@ long au_plink_ioctl(struct file *file, unsigned int cmd)
>       default:
>               /* err = -ENOTTY; */
>               err = -EINVAL;
>+              pr_err("cmd %d, (%d, %d)\n", cmd,
>+                     AUFS_CTL_PLINK_MAINT, AUFS_CTL_PLINK_CLEAN);
>       }
>  out:
>       return err;
>

The patch does nothing, because the wrong ioctl is already caught 
earlier:

[ 9505.700896] ioctl32(mount.aufs:9155): Unknown cmd fd(4) 
cmd(00004100){t:'A';sz:0} arg(0941ff88) on /mnt



Also, `auplink list /mnt` also does not trigger the printk/pr_err, 
though it does not cause an ioctl32 message either.

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first

Reply via email to