hw/9pfs: fix P9_STATS_GEN handling
authorKirill A. Shutemov <kirill.shutemov@linux.intel.com>
Tue, 28 Jan 2014 15:08:27 +0000 (17:08 +0200)
committerAneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
Sun, 2 Feb 2014 16:39:16 +0000 (22:09 +0530)
commitf8b7ee38b3ed4ec2da5cc0529cf0cf82c8589805
treed0bfd4a2449640fff86149e9d456062f68d14664
parent1a9978a51a17a26fd07e1b46f31ceeb9772fc191
hw/9pfs: fix P9_STATS_GEN handling

Currently we fail getattr request altogether if we can't read
P9_STATS_GEN for some reason. It breaks valid use cases:

E.g let's assume we have non-readable directory with execution bit set
on host and we export it to client over 9p On host we can chdir into
directory, but not open directory on read and list content.

But if client will try to call getattr (as part of chdir(2)) for the
directory it will fail with -EACCES. It happens because we try to open
the directory on read to call ioctl(FS_IOC_GETVERSION), it fails and we
return the error code to client.

It's excessive. The solution is to make P9_STATS_GEN failure non-fatal
for getattr request. Just don't set P9_STATS_GEN flag in result mask on
failure.

Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
hw/9pfs/cofile.c
hw/9pfs/virtio-9p.c