linux-kernel - Re: [PATCH 12/27] tty: hvc: convert to u8 and size

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 6 Dec 2023 09:30:46 +0000
From:   Christophe Leroy <christophe.leroy@...roup.eu>
To:     "Jiri Slaby (SUSE)" <jirislaby@...nel.org>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
CC:     "linux-serial@...r.kernel.org" <linux-serial@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Michael Ellerman <mpe@...erman.id.au>,
        Nicholas Piggin <npiggin@...il.com>,
        Amit Shah <amit@...nel.org>, Arnd Bergmann <arnd@...db.de>,
        Paul Walmsley <paul.walmsley@...ive.com>,
        Palmer Dabbelt <palmer@...belt.com>,
        Albert Ou <aou@...s.berkeley.edu>,
        "linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>,
        "virtualization@...ts.linux.dev" <virtualization@...ts.linux.dev>,
        "linux-riscv@...ts.infradead.org" <linux-riscv@...ts.infradead.org>
Subject: Re: [PATCH 12/27] tty: hvc: convert to u8 and size_t



Le 06/12/2023 à 08:36, Jiri Slaby (SUSE) a écrit :
> Switch character types to u8 and sizes to size_t. To conform to
> characters/sizes in the rest of the tty layer.
> 
> Signed-off-by: Jiri Slaby (SUSE) <jirislaby@...nel.org>
> Cc: Michael Ellerman <mpe@...erman.id.au>
> Cc: Nicholas Piggin <npiggin@...il.com>
> Cc: Christophe Leroy <christophe.leroy@...roup.eu>
> Cc: Amit Shah <amit@...nel.org>
> Cc: Arnd Bergmann <arnd@...db.de>
> Cc: Paul Walmsley <paul.walmsley@...ive.com>
> Cc: Palmer Dabbelt <palmer@...belt.com>
> Cc: Albert Ou <aou@...s.berkeley.edu>
> Cc: linuxppc-dev@...ts.ozlabs.org
> Cc: virtualization@...ts.linux.dev
> Cc: linux-riscv@...ts.infradead.org
> ---
>   arch/powerpc/include/asm/hvconsole.h       |  4 ++--
>   arch/powerpc/include/asm/hvsi.h            | 18 ++++++++--------
>   arch/powerpc/include/asm/opal.h            |  8 +++++---
>   arch/powerpc/platforms/powernv/opal.c      | 14 +++++++------
>   arch/powerpc/platforms/pseries/hvconsole.c |  4 ++--
>   drivers/char/virtio_console.c              | 10 ++++-----
>   drivers/tty/hvc/hvc_console.h              |  4 ++--
>   drivers/tty/hvc/hvc_dcc.c                  | 24 +++++++++++-----------
>   drivers/tty/hvc/hvc_iucv.c                 | 18 ++++++++--------
>   drivers/tty/hvc/hvc_opal.c                 |  5 +++--
>   drivers/tty/hvc/hvc_riscv_sbi.c            |  9 ++++----
>   drivers/tty/hvc/hvc_rtas.c                 | 11 +++++-----
>   drivers/tty/hvc/hvc_udbg.c                 |  9 ++++----
>   drivers/tty/hvc/hvc_vio.c                  | 18 ++++++++--------
>   drivers/tty/hvc/hvc_xen.c                  | 23 +++++++++++----------
>   drivers/tty/hvc/hvsi_lib.c                 | 20 ++++++++++--------
>   16 files changed, 107 insertions(+), 92 deletions(-)
> 
> diff --git a/arch/powerpc/include/asm/hvconsole.h b/arch/powerpc/include/asm/hvconsole.h
> index ccb2034506f0..d841a97010a0 100644
> --- a/arch/powerpc/include/asm/hvconsole.h
> +++ b/arch/powerpc/include/asm/hvconsole.h
> @@ -21,8 +21,8 @@
>    * Vio firmware always attempts to fetch MAX_VIO_GET_CHARS chars.  The 'count'
>    * parm is included to conform to put_chars() function pointer template
>    */
> -extern int hvc_get_chars(uint32_t vtermno, char *buf, int count);
> -extern int hvc_put_chars(uint32_t vtermno, const char *buf, int count);
> +extern ssize_t hvc_get_chars(uint32_t vtermno, u8 *buf, size_t count);
> +extern ssize_t hvc_put_chars(uint32_t vtermno, const u8 *buf, size_t count);

Would be a good opportunity to drop this pointless deprecated 'extern' 
keyword on all function prototypes you are changing.

Christophe