On Mon, May 17, 2021 at 8:16 AM Christoph Hellwig <hch@xxxxxx> wrote: > > 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(). Changed to your version now, thanks! Arnd