Re: [PATCH v2] fat: Add KUnit tests for checksums and timestamps

David Gow davidgow@google.com writes:

...

Hm, can this export only if FAT_KUNIT_TEST is builtin or module (maybe #if IS_ENABLED(...))? And #if will also be worked as the comment too.

That's possible, but I'd prefer to export it unconditionally for two reasons:

1. It'd make it possible to build the fat_test module without having

to rebuild the whole kernel/fat. 2. It'd be consistent with fat_time_unix2fat(), which is exported for use in vfat/msdos anyway.

Neither of those are dealbreakers, though, so if you'd still prefer this to be behind an ifdef, I'll change it.

OK. If nobody complain, let's export. However, then, can you add the comment instead of ifdef to mark for kunit?

Thanks.