linux-kernel - Re: [PATCH v1 1/6] s390/boot/uv.c: Use a constant for more-data rc

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <20241001141027.2063bf9d@p-imbrenda.boeblingen.de.ibm.com>
Date: Tue, 1 Oct 2024 14:10:27 +0200
From: Claudio Imbrenda <imbrenda@...ux.ibm.com>
To: Steffen Eiden <seiden@...ux.ibm.com>
Cc: linux-kernel@...r.kernel.org, linux-s390@...r.kernel.org,
        Ingo Franzki
 <ifranzki@...ux.ibm.com>,
        Harald Freudenberger <freude@...ux.ibm.com>,
        Christoph Schlameuss <schlameuss@...ux.ibm.com>,
        Janosch Frank
 <frankja@...ux.ibm.com>
Subject: Re: [PATCH v1 1/6] s390/boot/uv.c: Use a constant for more-data rc

On Mon, 30 Sep 2024 15:19:04 +0200
Steffen Eiden <seiden@...ux.ibm.com> wrote:

> Add a define for the UVC rc 0x0100 that indicates that a UV-call was
> successful but may serve more data if called with a larger buffer
> again.
> 
> Reviewed-by: Christoph Schlameuss <schlameuss@...ux.ibm.com>

Reviewed-by: Claudio Imbrenda <imbrenda@...ux.ibm.com>

> Signed-off-by: Steffen Eiden <seiden@...ux.ibm.com>
> ---
>  arch/s390/boot/uv.c        | 4 ++--
>  arch/s390/include/asm/uv.h | 1 +
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/s390/boot/uv.c b/arch/s390/boot/uv.c
> index 318e6ba95bfd..2a71e759dc42 100644
> --- a/arch/s390/boot/uv.c
> +++ b/arch/s390/boot/uv.c
> @@ -22,8 +22,8 @@ void uv_query_info(void)
>  	if (!test_facility(158))
>  		return;
>  
> -	/* rc==0x100 means that there is additional data we do not process */
> -	if (uv_call(0, (uint64_t)&uvcb) && uvcb.header.rc != 0x100)
> +	/* Ignore that there might be more data we do not process */
> +	if (uv_call(0, (uint64_t)&uvcb) && uvcb.header.rc != UVC_RC_MORE_DATA)
>  		return;
>  
>  	if (IS_ENABLED(CONFIG_KVM)) {
> diff --git a/arch/s390/include/asm/uv.h b/arch/s390/include/asm/uv.h
> index 153d93468b77..94ff58336e8e 100644
> --- a/arch/s390/include/asm/uv.h
> +++ b/arch/s390/include/asm/uv.h
> @@ -28,6 +28,7 @@
>  #define UVC_RC_INV_STATE	0x0003
>  #define UVC_RC_INV_LEN		0x0005
>  #define UVC_RC_NO_RESUME	0x0007
> +#define UVC_RC_MORE_DATA	0x0100
>  #define UVC_RC_NEED_DESTROY	0x8000
>  
>  #define UVC_CMD_QUI			0x0001