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().