[PATCH v8 0/2] s390: virtio: let arch validate VIRTIO features

From: Pierre Morel
Date: Tue Aug 18 2020 - 11:01:01 EST


Hi all,

The goal of the series is to give a chance to the architecture
to validate VIRTIO device features.

in this respin:

I use the original idea from Connie for an optional
arch_has_restricted_memory_access.

I renamed the callback accordingly, added the definition of
ARCH_HAS_RESTRICTED_MEMORY_ACCESS inside the VIRTIO Kconfig
and the selection in the PROTECTED_VIRTUALIZATION_GUEST
config entry.


Regards,
Pierre

Pierre Morel (2):
virtio: let arch validate VIRTIO features
s390: virtio: PV needs VIRTIO I/O device protection

arch/s390/Kconfig | 1 +
arch/s390/mm/init.c | 30 ++++++++++++++++++++++++++++++
drivers/virtio/Kconfig | 6 ++++++
drivers/virtio/virtio.c | 4 ++++
include/linux/virtio_config.h | 9 +++++++++
5 files changed, 50 insertions(+)

--
2.25.1

Changelog

to v8:

- refactoring by using an optional callback
(Connie)

to v7:

- typo in warning message
(Connie)
to v6:

- rewording warning messages
(Connie, Halil)

to v5:

- return directly from S390 arch_validate_virtio_features()
when the guest is not protected.
(Connie)

- Somme rewording
(Connie, Michael)

- moved back code from arch/s390/ ...kernel/uv.c to ...mm/init.c
(Christian)

to v4:

- separate virtio and arch code
(Pierre)

- moved code from arch/s390/mm/init.c to arch/s390/kernel/uv.c
(as interpreted from Heiko's comment)

- moved validation inside the arch code
(Connie)

- moved the call to arch validation before VIRTIO_F_1 test
(Michael)

to v3:

- add warning
(Connie, Christian)

- add comment
(Connie)

- change hook name
(Halil, Connie)

to v2:

- put the test in virtio_finalize_features()
(Connie)

- put the test inside VIRTIO core
(Jason)

- pass a virtio device as parameter
(Halil)