Re: [PATCH v9 1/1] smb/client: introduce KUnit test to check search result of smb2_error_map_table

[ChenXiaoSong <chenxiaosong.chenxiaosong@linux.dev>]

Hi Geert,

Thank you for reviewing this patch.

The KUnit test cases are only executed when the CONFIG_SMB_KUNIT_TESTS 
is enabled.

Making it a separate test module would require exporting local variables 
and functions so that the test code can access them. However, exporting 
local variables and functions would likely make the code much uglier, as 
it would require adding "#if" conditionals into the production code to 
isolate the test code.

Geert, please let me know if you have a better idea.

I am also discussing this with the ext4 community, and we all hope to 
find a way to make the tests a separate module.

Thanks,
ChenXiaoSong <chenxiaosong@kylinos.cn>

On 2/10/26 4:10 PM, Geert Uytterhoeven wrote:
>   	Hi ChenXiaoSong,
> 
> Thanks for your patch, which is now commit 480afcb19b61385d
> ("smb/client: introduce KUnit test to check search result of
> smb2_error_map_table") in linus/master
> 
>> The KUnit test are executed when cifs.ko is loaded.
> This means the tests are_always_ executed when cifs.ko is loaded,
> which is different from how most other test modules work.
> Please make it a separate test module, so it can be loaded independently
> of the main cifs module.  That way people can enable all tests in
> production kernels, without affecting the system unless a test module
> is actually loaded.