Re: [PATCH v3] lib/math: Add int_sqrt test suite

[Shuah Khan <skhan@linuxfoundation.org>]

On 10/31/24 18:39, Andrew Morton wrote:
> On Thu, 31 Oct 2024 11:50:16 -0600 Shuah Khan <skhan@linuxfoundation.org> wrote:
> 
>> On 10/30/24 07:43, Luis Felipe Hernandez wrote:
>>> Adds test suite for integer based square root function.
>>>
>>> The test suite is designed to verify the correctness of the int_sqrt()
>>> math library function.
>>>
>>> Signed-off-by: Luis Felipe Hernandez <luis.hernandez093@gmail.com>
>>> ---
>>> Changes in v2
>>>     - Add new line at the end of int_sqrt_kunit.c
>>>     - Add explicit header includes for MODULE_* macros, strscpy, and ULONG_MAX
>>>
>>> Changes in v3
>>>     - Remove unnecesary new line after Kconfig entry for INT_SQRT_KUNIT_TEST
>>>     - Correct int_sqrt instances with int_sqrt() in commit message and kconfig
>>> entry desc
>>>     - Fix limits.h header include path
>>
>> Adding Andrew to the thread.
> 
> Thanks.
> 
>> I think this depends on the other lib kunit
>> content that is already in next.
> 
> Actually the patch applies cleanly to 6.12-rc5.
> 
>>> --- a/lib/math/Makefile
>>> +++ b/lib/math/Makefile
>>> @@ -9,3 +9,4 @@ obj-$(CONFIG_INT_POW_TEST)  += tests/int_pow_kunit.o
>>>    obj-$(CONFIG_TEST_DIV64)	+= test_div64.o
>>>    obj-$(CONFIG_TEST_MULDIV64)	+= test_mul_u64_u64_div_u64.o
>>>    obj-$(CONFIG_RATIONAL_KUNIT_TEST) += rational-test.o
>>> +obj-y  += tests/
> 
> What's this change about?  It seems somewhat unrelated to adding a
> single test.  I mean, there's an unrelated test listed in
> lib/math/tests/Makefile so what change does this patch have upon that
> one?
> 

I think this is a new test. I can take this for 6.13-rc1

I thought the conflicts might persist in case you have any 6.13 material
already in linux-next. Doesn't look like it if you could apply to 6.12-rc5

Sorry for the confusion.  I can take this for 6.13-rc1

thanks,
-- Shuah