[PATCH 2/5] lib/string_kunit: add test case for strnlen
Andy Shevchenko
andriy.shevchenko at intel.com
Wed Jan 7 23:26:59 PST 2026
On Thu, Jan 08, 2026 at 02:53:58PM +0800, Feng Jiang wrote:
> On 2026/1/7 19:56, Andy Shevchenko wrote:
> > On Wed, Jan 7, 2026 at 4:35 AM Feng Jiang <jiangfeng at kylinos.cn> wrote:
...
> >> Add a KUnit test for strnlen() to verify correctness across
> >> different string lengths and memory alignments.
> >
> > Same comment as per patch 1 (it would probably require to call for
> > arch_strnlen() or something like this).
>
> Thanks, makes sense.
>
> I'll add the performance benchmarking (random filling + timing) in V2.
>
> Since string functions are typically exported directly by each architecture
> without an arch_ prefix, I'll introduce a generic_strnlen() (based on
> lib/string.c) within the test for comparison.
Probably you want to make the existing one to have that name and use it inside
the test and in the fallback wrapper. We don't want to have duplicate code, it
is bad from maintenance perspective.
--
With Best Regards,
Andy Shevchenko
More information about the linux-riscv
mailing list