I found that this test succeeds on UBIFS: stat_test /mnt/foo attr=+compressed attr=-compressed Since stx_attributes can hold many flags, check the flags correctly with the & operator. Signed-off-by: Richard Weinberger <richard@xxxxxx> --- src/stat_test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/stat_test.c b/src/stat_test.c index 3dc44bbdb71d..fc87ed5c6e8f 100644 --- a/src/stat_test.c +++ b/src/stat_test.c @@ -621,10 +621,10 @@ static void check_attribute(const struct statx *stx, char *arg) attr = p->attr_flag; if (set) { - check(stx->stx_attributes && attr, + check((stx->stx_attributes & attr) == attr, "Attribute %s should be set\n", arg); } else { - check(~stx->stx_attributes && attr, + check((stx->stx_attributes & attr) == 0, "Attribute %s should be unset\n", arg); } } -- 2.7.3 -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html