On 1/8/2017 18:07, Christoph Hellwig wrote: > On Sat, Dec 31, 2016 at 09:18:08PM +0800, Kinglong Mee wrote: >> Commit fae5096ad217 >> "nfsd: assume writeable exportabled filesystems have f_sync" >> have remove the checking of f_sync. >> >> Christoph Hellwig suggests, >> "Warn and refuse the writable export." >> >> I think just covert to a readonly export for !fsync filesystem, >> also, for a readonly filesystem is reasonable. > > I don't like degrading the export. We should require an explicit > ro option in this case. With this patch, we can see the ro option in the proc file. # mount |grep xfs /dev/sdc on /nfs type xfs (ro,relatime,seclabel,attr2,inode64,noquota) # cat /etc/exports /nfs/ *(rw,no_subtree_check,no_root_squash,insecure,fsid=0) # cat /proc/fs/nfsd/exports # Version 1.1 # Path Client(Flags) # IPs /nfs *(ro,insecure,no_root_squash,sync,wdelay,no_subtree_check,fsid=0,uuid=a4a352bc:252a47cb:b3953193:040e050d,sec=1,rw,insecure,no_root_squash) thanks, Kinglong Mee -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
