On 2013-10-01 04:21, Boaz Harrosh wrote: > On 09/29/2013 05:17 AM, Christoph Hellwig wrote: >> Seems like layout_type should just be a field in the export ops instead >> of a method. > > No! this field does not make any sense at all it should just be > removed. > > There is a pNFS inquiry a client sends that asks for an array > of all the types supported by this mount point. So these method > should be returning an array. If at all. We're not there yet. The pnfs generic implementation and none of the filesystems do that at the moment. When we go there we should revise the interface. > > The layout_type is just an input to some operations and are > no concern of NFSD. This is not a yes/no flag for pNFS the > opts vector should be the flag. We can push the check for the layout type down to file system but then it will have to encode FATTR4_WORD1_FS_LAYOUT_TYPES and FATTR4_WORD2_LAYOUT_TYPES. Benny > > Cheers > Boaz > > -- > 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 > -- 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
