]> git.baikalelectronics.ru Git - kernel.git/commit
s390/cio: fix virtio-ccw DMA without PV
authorHalil Pasic <pasic@linux.ibm.com>
Mon, 30 Sep 2019 15:38:02 +0000 (17:38 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Thu, 10 Oct 2019 08:49:46 +0000 (10:49 +0200)
commit5598eda14067e0b49278d789a07fb538f85456fb
treeab5f00d623767c76ff5b5291955bf1d17054044a
parent52b7759f73b03afc7cba59d3686ec69261291235
s390/cio: fix virtio-ccw DMA without PV

Commit 3c88acc85395 ("s390/cio: add basic protected virtualization
support") breaks virtio-ccw devices with VIRTIO_F_IOMMU_PLATFORM for non
Protected Virtualization (PV) guests. The problem is that the dma_mask
of the ccw device, which is used by virtio core, gets changed from 64 to
31 bit, because some of the DMA allocations do require 31 bit
addressable memory. For PV the only drawback is that some of the virtio
structures must end up in ZONE_DMA because we have the bounce the
buffers mapped via DMA API anyway.

But for non PV guests we have a problem: because of the 31 bit mask
guests bigger than 2G are likely to try bouncing buffers. The swiotlb
however is only initialized for PV guests, because we don't want to
bounce anything for non PV guests. The first such map kills the guest.

Since the DMA API won't allow us to specify for each allocation whether
we need memory from ZONE_DMA (31 bit addressable) or any DMA capable
memory will do, let us use coherent_dma_mask (which is used for
allocations) to force allocating form ZONE_DMA while changing dma_mask
to DMA_BIT_MASK(64) so that at least the streaming API will regard
the whole memory DMA capable.

Signed-off-by: Halil Pasic <pasic@linux.ibm.com>
Reported-by: Marc Hartmayer <mhartmay@linux.ibm.com>
Suggested-by: Robin Murphy <robin.murphy@arm.com>
Fixes: 3c88acc85395 ("s390/cio: add basic protected virtualization support")
Link: https://lore.kernel.org/lkml/20190930153803.7958-1-pasic@linux.ibm.com
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
drivers/s390/cio/cio.h
drivers/s390/cio/css.c
drivers/s390/cio/device.c