2020-07-09 08:42:54

by Pierre Morel

[permalink] [raw]
Subject: [PATCH v5 0/2] s390: virtio: let arch validate VIRTIO features

Hi all,

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

in this respin:

1) I kept removed the ack from Jason as I reworked the patch
@Jason, the nature and goal of the patch did not really changed
please can I get back your acked-by with these changes?

2) I suppressed the unnecessary verbosity of the architecture
specific patch

3) put back the arch specific code inside arch/s390/mm/init.c
after offline discussion with Christian.

Regards,
Pierre

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

arch/s390/mm/init.c | 27 +++++++++++++++++++++++++++
drivers/virtio/virtio.c | 19 +++++++++++++++++++
include/linux/virtio_config.h | 1 +
3 files changed, 47 insertions(+)

--
2.25.1

Changelog

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)