[LTP] [PATCH 1/1] lib: Rename array used for .all_filesystems flag

Cyril Hrubis chrubis@suse.cz
Tue Feb 8 14:16:21 CET 2022


Hi!
> > Although fs_type_whitelist[] is also used for whitelisting with
> > .skip_filesystems, the main purpose is to be used for .all_filesystems.
> 
> > Thus rename it to all_filesystems[].
> 
> NOTE: the main purpose of this change is to increase code readability.
> https://lore.kernel.org/ltp/CAEemH2fNfFes-eUtiQKX9JJxqEQUQ+O5nWQM8G-yNyTo8sxviw@mail.gmail.com/
> 
> That's why I added doc.

I guess that fs_type_whitelist[] may be confusing but all_filesystems[]
is IMHO not that much better since we use that a as base for the
all_filesystem before we filter out these that are not supported. Maybe
we should call it try_filesystems[] instead?

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list