On 10/31/24 18:39, Andrew Morton wrote:
On Thu, 31 Oct 2024 11:50:16 -0600 Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> 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@xxxxxxxxx>
---
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