[LTP] [PATCH 0/2] swapon01: Test on all filesystems, cleanup
Marius Kittler
mkittler@suse.de
Thu Oct 12 11:28:51 CEST 2023
I guess the code would work and looks generally good to merge. In fact, I
tested `.all_filesystems = 1` myself yesterday. It is just the question
whether we actually want it. Is there really coverage to be gained (or does
the kernel just the same under the hood anyway regardless of the filesystem)?
> Test on all filesystems to increase coverage. btrfs and tmpfs
> currently does not support swap file, but keep it in case this get
> changed in the future.
Considering btrfs does not support it I guess that means the kernel does
indeed different things under the hood so the coverage might be beneficial. (If
it was just about tmpfs then I'd say it makes no sense to put a swapfile there
in any case and it will therefore likely never be supported.)
Reviewed-by: Marius Kittler <mkittler@suse.de>
More information about the ltp
mailing list