linux-kernel - Re: [PATCH 0/3] selftests: cachestat: fix build and run on older kernels

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 15 Aug 2023 11:07:03 -0700
From:   Nhat Pham <nphamcs@...il.com>
To:     Andre Przywara <andre.przywara@....com>
Cc:     Shuah Khan <shuah@...nel.org>,
        Johannes Weiner <hannes@...xchg.org>,
        linux-kselftest@...r.kernel.org, linux-mm@...ck.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] selftests: cachestat: fix build and run on older kernels

On Tue, Aug 15, 2023 at 8:56 AM Andre Przywara <andre.przywara@....com> wrote:
>
> I ran all kernel selftests on some test machine, and stumbled upon
> cachestat failing (among others).
> Those patches fix the cachestat test compilation and run on older
> kernels.
>
> Also I found that the but-last test (on a normal file) fails when run on
> a tmpfs mounted directory, as it happens on an initramfs-only system, or
> when the current directory happens to be /dev/shm or /tmp:
> # Create/open tmpfilecachestat
> # Cachestat call returned 0
> # Using cachestat: Cached: 4, Dirty: 4, Writeback: 0, Evicted: 0, Recently Evicted: 0
> # Cachestat call (after fsync) returned 0
> # Using cachestat: Cached: 4, Dirty: 4, Writeback: 0, Evicted: 0, Recently Evicted: 0
> # Number of dirty should be zero after fsync.
> not ok 6 cachestat fails with normal file
>
> That same test binary succeeds on the same machine right afterwards if
> the current directory is changed to an ext4 filesystem.

Ah, if I recall correctly, these kinds of fs have no-op fsync, correct?
Something along the line of:
https://github.com/torvalds/linux/blob/91aa6c4/mm/shmem.c#L4111
The fsync logic would fail indeed. Thanks for pointing that out!

>
> I don't really know if this is expected, and whether we should try to
> figure out if the test file lives on a tmpfs filesystem, or whether the

This would be nice. I think there's a userspace method to check
this, right? There's a TMPFS_MAGIC here - not sure if relevant:

https://man7.org/linux/man-pages/man2/statfs.2.html

> test itself is not strict enough, and requires more "flushing"
> (drop_caches?) to cover tmpfs directories as well.
>
> Any ideas how to fix this would be appreciated.
>
> Cheers,
> Andre
>
> Andre Przywara (3):
>   selftests: cachestat: properly link in librt
>   selftests: cachestat: use proper syscall number macro
>   selftests: cachestat: test for cachestat availability
>
>  tools/testing/selftests/cachestat/Makefile    |  2 +-
>  .../selftests/cachestat/test_cachestat.c      | 29 +++++++++++++++----
>  2 files changed, 25 insertions(+), 6 deletions(-)
>
> --
> 2.25.1
>