Re: [PATCH v3 9/9] asm-generic: reverse GENERIC_{STRNCPY_FROM, STRNLEN}_USER symbols
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
- Subject: Re: [PATCH v3 9/9] asm-generic: reverse GENERIC_{STRNCPY_FROM, STRNLEN}_USER symbols
- From: Arnd Bergmann <arnd@xxxxxxxxxx>
- Date: Thu, 22 Jul 2021 16:01:39 +0200
- Cc: linux-arch <linux-arch@xxxxxxxxxxxxxxx>, Arnd Bergmann <arnd@xxxxxxxx>, "James E.J. Bottomley" <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>, Anton Ivanov <anton.ivanov@xxxxxxxxxxxxxxxxxx>, Brian Cain <bcain@xxxxxxxxxxxxxx>, Chris Zankel <chris@xxxxxxxxxx>, Christian Borntraeger <borntraeger@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, Guo Ren <guoren@xxxxxxxxxx>, Heiko Carstens <hca@xxxxxxxxxxxxx>, Helge Deller <deller@xxxxxx>, Jeff Dike <jdike@xxxxxxxxxxx>, Linus Walleij <linus.walleij@xxxxxxxxxx>, Max Filippov <jcmvbkbc@xxxxxxxxx>, Michal Simek <monstr@xxxxxxxxx>, Richard Weinberger <richard@xxxxxx>, Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>, Vasily Gorbik <gor@xxxxxxxxxxxxx>, Vineet Gupta <vgupta@xxxxxxxxxxxx>, Yoshinori Sato <ysato@xxxxxxxxxxxxx>, linux-csky@xxxxxxxxxxxxxxx, "open list:QUALCOMM HEXAGON..." <linux-hexagon@xxxxxxxxxxxxxxx>, linux-ia64@xxxxxxxxxxxxxxx, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, "open list:BROADCOM NVRAM DRIVER" <linux-mips@xxxxxxxxxxxxxxx>, Parisc List <linux-parisc@xxxxxxxxxxxxxxx>, linux-s390 <linux-s390@xxxxxxxxxxxxxxx>, "open list:SYNOPSYS ARC ARCHITECTURE" <linux-snps-arc@xxxxxxxxxxxxxxxxxxx>, linux-um <linux-um@xxxxxxxxxxxxxxxxxxx>, "open list:TENSILICA XTENSA PORT (xtensa)" <linux-xtensa@xxxxxxxxxxxxxxxx>, "moderated list:H8/300 ARCHITECTURE" <uclinux-h8-devel@xxxxxxxxxxxxx>
- In-reply-to: <29adc1c164805e355b37d1d4668ebda9fb7fa872.camel@sipsolutions.net>
- References: <20210722124814.778059-1-arnd@kernel.org> <20210722124814.778059-10-arnd@kernel.org> <29adc1c164805e355b37d1d4668ebda9fb7fa872.camel@sipsolutions.net>
On Thu, Jul 22, 2021 at 3:57 PM Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:
>
> >
> > The remaining architectures at the moment are: ia64, mips, parisc,
> > s390, um and xtensa. We should probably convert these as well, but
>
> I'm not sure it makes sense to convert um, the implementation uses
> strncpy(), and that should use the libc implementation, which is tuned
> for the actual hardware that the binary is running on, so performance
> wise that might be better.
>
> OTOH, maybe this is all in the noise given the huge syscall overhead in
> um, so maybe unifying it would make more sense.
Right, makes sense. I suppose if we end up converting mips and s390,
we should just do arch/um and the others as well for consistency, even
if that adds some overhead.
Arnd
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]