On Thu, Jan 31, 2019 at 05:33:58PM +0100, Joerg Roedel wrote:> Hi, > > here is the next version of this patch-set. Previous > versions can be found here: > > V1: https://lore.kernel.org/lkml/20190110134433.15672-1-joro at 8bytes.org/ > > V2: https://lore.kernel.org/lkml/20190115132257.6426-1-joro at 8bytes.org/ > > V3: https://lore.kernel.org/lkml/20190123163049.24863-1-joro at 8bytes.org/ > > V4: https://lore.kernel.org/lkml/20190129084342.26030-1-joro at 8bytes.org/ > > V5: https://lore.kernel.org/lkml/20190130164007.26497-1-joro at 8bytes.org/ > > The problem solved here is a limitation of the SWIOTLB implementation, > which does not support allocations larger than 256kb. When the > virtio-blk driver tries to read/write a block larger than that, the > allocation of the dma-handle fails and an IO error is reported. > > Changes to v5 are: > > - Changed patch 3 to uninline dma_max_mapping_size()And this lead to problems reported by kbuild :( BTW when you repost, can I ask you to pls include the version in all patches? Both --subject-prefix and -v flags to git format-patch will do this for you.> Please review. > > Thanks, > > Joerg > > Joerg Roedel (5): > swiotlb: Introduce swiotlb_max_mapping_size() > swiotlb: Add is_swiotlb_active() function > dma: Introduce dma_max_mapping_size() > virtio: Introduce virtio_max_dma_size() > virtio-blk: Consider virtio_max_dma_size() for maximum segment size > > Documentation/DMA-API.txt | 8 ++++++++ > drivers/block/virtio_blk.c | 10 ++++++---- > drivers/virtio/virtio_ring.c | 11 +++++++++++ > include/linux/dma-mapping.h | 8 ++++++++ > include/linux/swiotlb.h | 11 +++++++++++ > include/linux/virtio.h | 2 ++ > kernel/dma/direct.c | 11 +++++++++++ > kernel/dma/mapping.c | 14 ++++++++++++++ > kernel/dma/swiotlb.c | 14 ++++++++++++++ > 9 files changed, 85 insertions(+), 4 deletions(-) > > -- > 2.17.1
Hi Michael, On Tue, Feb 05, 2019 at 03:52:38PM -0500, Michael S. Tsirkin wrote:> On Thu, Jan 31, 2019 at 05:33:58PM +0100, Joerg Roedel wrote: > > Changes to v5 are: > > > > - Changed patch 3 to uninline dma_max_mapping_size() > > And this lead to problems reported by kbuild :(Hmm, I didn't get any kbuild emails for this series. Can you please forward it me so that I can look into it?> BTW when you repost, can I ask you to pls include > the version in all patches? Both --subject-prefix > and -v flags to git format-patch will do this for you.Will do, thanks. Regards, Joerg
On Thu, Feb 07, 2019 at 09:46:13AM +0100, Joerg Roedel wrote:> Hmm, I didn't get any kbuild emails for this series. Can you please > forward it me so that I can look into it?Nevermind, just found them in another inbox. Joerg