Re: [PATCH 3/6] [v2] hexagon: use generic strncpy/strnlen from_user

From: Christoph Hellwig
Date: Mon May 17 2021 - 02:16:47 EST


On Sat, May 15, 2021 at 12:18:00PM +0200, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@xxxxxxxx>
>
> Most per-architecture versions of these functions are broken in some form,
> and they are almost certainly slower than the generic code as well.
>
> Remove the ones for hexagon and instead use the generic version.
> This custom version reads the data twice for strncpy() by doing an extra
> strnlen(), and it apparently lacks a check for user_addr_max().

I'd be tempted to just remove the first paragraph and reword the second
as:

Remove the hexagon implementation of strncpy/strnlen and instead use the
generic versions. The hexago version of strncpy reads the data twice by
doing an extra strnlen(), and it apparently lacks a check for
user_addr_max().