On Wed, May 08, 2019 at 11:11:44AM +0800, Murphy Zhou wrote: > If the filesystem does not support STATX_ATTR, like NFS, setting > both attributes and attributes_mask to 0 seems the right thing to > do. attributes_mask can be 0 only if attributes is also 0. > This situation is covered by the "&" check in the next line. > > Signed-off-by: Murphy Zhou <xzhou@xxxxxxxxxx> Seems fine to me. Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> --D > --- > tests/generic/532 | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/tests/generic/532 b/tests/generic/532 > index ac8b7a89..3f8afa4a 100755 > --- a/tests/generic/532 > +++ b/tests/generic/532 > @@ -44,7 +44,6 @@ check_statx_attributes() > echo "MASK:$mask:ATTRS:$attrs:" >> $seqres.full > > test -z "$mask" && _notrun "xfs_io statx command does not support attributes_mask" > - test $(( mask )) -eq 0 && echo "attributes mask is zero" > test $(( attrs & ~(mask) )) -ne 0 && echo "attributes $attrs do not appear in mask $mask" > } > > -- > 2.21.0 >
