On 10/21/20 2:15 PM, Brendan Higgins wrote: > On Tue, Oct 20, 2020 at 12:37 AM Geert Uytterhoeven > <geert@xxxxxxxxxxxxxx> wrote: >> >> EXT4_KUNIT_TESTS selects EXT4_FS, thus enabling an optional feature the >> user may not want to enable. Fix this by making the test depend on >> EXT4_FS instead. >> >> Fixes: 1cbeab1b242d16fd ("ext4: add kunit test for decoding extended timestamps") >> Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > If I remember correctly, having EXT4_KUNIT_TESTS select EXT4_FS was > something that Ted specifically requested, but I don't have any strong > feelings on it either way. omg, please No. depends on is the right fix here. -- ~Randy