]> www.infradead.org Git - users/jedix/linux-maple.git/commit
s390/uv: Panic for set and remove shared access UVC errors
authorClaudio Imbrenda <imbrenda@linux.ibm.com>
Thu, 1 Aug 2024 11:25:48 +0000 (13:25 +0200)
committerJanosch Frank <frankja@linux.ibm.com>
Wed, 7 Aug 2024 11:04:43 +0000 (11:04 +0000)
commitcff59d8631e1409ffdd22d9d717e15810181b32c
tree2b034d0a1f806b5b34ee4a09cbd307ba8852679e
parent5a44bb061d04b0306f2aa8add761d86d152b9377
s390/uv: Panic for set and remove shared access UVC errors

The return value uv_set_shared() and uv_remove_shared() (which are
wrappers around the share() function) is not always checked. The system
integrity of a protected guest depends on the Share and Unshare UVCs
being successful. This means that any caller that fails to check the
return value will compromise the security of the protected guest.

No code path that would lead to such violation of the security
guarantees is currently exercised, since all the areas that are shared
never get unshared during the lifetime of the system. This might
change and become an issue in the future.

The Share and Unshare UVCs can only fail in case of hypervisor
misbehaviour (either a bug or malicious behaviour). In such cases there
is no reasonable way forward, and the system needs to panic.

This patch replaces the return at the end of the share() function with
a panic, to guarantee system integrity.

Fixes: 5abb9351dfd9 ("s390/uv: introduce guest side ultravisor code")
Signed-off-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
Reviewed-by: Christian Borntraeger <borntraeger@linux.ibm.com>
Reviewed-by: Steffen Eiden <seiden@linux.ibm.com>
Reviewed-by: Janosch Frank <frankja@linux.ibm.com>
Link: https://lore.kernel.org/r/20240801112548.85303-1-imbrenda@linux.ibm.com
Message-ID: <20240801112548.85303-1-imbrenda@linux.ibm.com>
[frankja@linux.ibm.com: Fixed up patch subject]
Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
arch/s390/include/asm/uv.h