On Fri, Mar 31, 2017 at 04:51:24PM +0100, David Howells wrote: > I'm not convinced that this is right. I'm more inclined to let any flag be > passed that is available to statx() with a filename and just mask off the > bits before handing them on. And what would these flags actually do? Currently we verify that the allowed flags are passed for the case where we have a filename. We need to do the same for the non-filename case, and we should also check that we don't pass flags that don't make sense for this case. Eric's patch is doing exactly that. The only thing he could do even better is to add a testcase to xfstests to verify this :)