2020-07-07 08:45:39

by Pierre Morel

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

Hi all,

I changed the patch subject to reflect the content, becoming more
general.

1) I removed the ack from Christian and Jason even far as
I understand they gave it for the functionality more than for the
implementation.
@Jason, @Christian, please can I get back your acked-by with these changes?

2) previous patch had another name:
[PATCH v3 0/1] s390: virtio: let arch choose to accept devices without IOMMU feature
id: Message-Id: <[email protected]>

3) The new version generalize the validation of the features by the
architecture, making it not IOMMU_PLATFORM specific anymore inside
virtio.c

The architecture specific code for s390 is now testing the virtio
features.

4) Since I reworked the patch I also moved the arch specific code
from arch/s390/mm/init.c to arch/s390/kernel/to uv.c

5) Finaly, I splitted the patch into generic virtio and arch
specific code.

Regards,
Pierre

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

arch/s390/kernel/uv.c | 25 +++++++++++++++++++++++++
drivers/virtio/virtio.c | 19 +++++++++++++++++++
include/linux/virtio_config.h | 1 +
3 files changed, 45 insertions(+)

--
2.25.1

Changelog

to v4:

- separate virtio and arch code
(Pierre)

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

- 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)