Re: [PATCH v1 1/8] virtio: Force only split mode with protected guest

From: Andi Kleen
Date: Thu Jun 03 2021 - 21:55:09 EST

For most Linux drivers, a report that a misbehaving device can corrupt
host memory is a bug, not a feature. If a USB device can corrupt kernel
memory, that's a serious bug. If a USB-C device can corrupt kernel
memory, that's also a serious bug, although, sadly, we probably have
lots of these bugs. If a Firewire device can corrupt kernel memory,
news at 11. If a Bluetooth or WiFi peer can corrupt kernel memory,
people write sonnets about it and give it clever names. Why is virtio

Well for most cases it's pointless because they don't have any memory protection anyways.

Why break compatibility if it does not buy you anything?

Anyways if you want to enable the restricted mode for something else, it's easy to do. The cases where it matters seem to already work on it, like the user space virtio ring.

My changes for boundary checking are enabled unconditionally anyways, as well as the other patchkits.

This one:

int arch_has_restricted_virtio_memory_access(void)
+ return is_tdx_guest();

I'm looking at a fairly recent kernel, and I don't see anything for s390
wired up in vring_use_dma_api.

It's not using vring_use_dma_api, but enforces the DMA API at virtio ring setup time, same as SEV/TDX.