diff options
author | Jeremy Cline <jcline@redhat.com> | 2018-05-15 21:45:11 -0400 |
---|---|---|
committer | Jeremy Cline <jcline@redhat.com> | 2018-05-15 21:45:22 -0400 |
commit | a15aa7f1f8be2dcd30cd0251baf7f16f9388370c (patch) | |
tree | 6eb49b534d3f41f905dbb46f4ecdede55d0f5e87 | |
parent | c3b0254aa80299519ee3fcf7635407ea48efa07b (diff) | |
download | kernel-a15aa7f1f8be2dcd30cd0251baf7f16f9388370c.tar.gz kernel-a15aa7f1f8be2dcd30cd0251baf7f16f9388370c.tar.xz kernel-a15aa7f1f8be2dcd30cd0251baf7f16f9388370c.zip |
Silence unwanted "swiotlb buffer is full" warnings (rhbz 1556797)
-rw-r--r-- | kernel.spec | 6 | ||||
-rw-r--r-- | swiotlb-silent-unwanted-warning-buffer-is-full.patch | 43 |
2 files changed, 49 insertions, 0 deletions
diff --git a/kernel.spec b/kernel.spec index 4762a17af..e97f500b0 100644 --- a/kernel.spec +++ b/kernel.spec @@ -675,6 +675,9 @@ Patch511: 0001-xfs-set-format-back-to-extents-if-xfs_bmap_extents_t.patch # rhbz 1566258 Patch512: KVM-vmx-update-sec-exec-controls-for-UMIP-iff-emulating-UMIP.patch +# rhbz 1556797 +Patch513: swiotlb-silent-unwanted-warning-buffer-is-full.patch + # END OF PATCH DEFINITIONS %endif @@ -1925,6 +1928,9 @@ fi # # %changelog +* Tue May 15 2018 Jeremy Cline <jeremy@jcline.org> +- Silence unwanted "swiotlb buffer is full" warnings (rhbz 1556797) + * Wed May 09 2018 Jeremy Cline <jeremy@jcline.org> - Workaround for m400 uart irq firmware description (rhbz 1574718) diff --git a/swiotlb-silent-unwanted-warning-buffer-is-full.patch b/swiotlb-silent-unwanted-warning-buffer-is-full.patch new file mode 100644 index 000000000..353711008 --- /dev/null +++ b/swiotlb-silent-unwanted-warning-buffer-is-full.patch @@ -0,0 +1,43 @@ +From 05e13bb57e6f181d7605f8608181c7e6fb7f591d Mon Sep 17 00:00:00 2001 +From: Jean Delvare <jdelvare@suse.de> +Date: Sat, 12 May 2018 11:57:37 +0200 +Subject: [PATCH] swiotlb: silent unwanted warning "buffer is full" +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +If DMA_ATTR_NO_WARN is passed to swiotlb_alloc_buffer(), it should be +passed further down to swiotlb_tbl_map_single(). Otherwise we escape +half of the warnings but still log the other half. + +This is one of the multiple causes of spurious warnings reported at: +https://bugs.freedesktop.org/show_bug.cgi?id=104082 + +Signed-off-by: Jean Delvare <jdelvare@suse.de> +Fixes: 0176adb00406 ("swiotlb: refactor coherent buffer allocation") +Cc: Christoph Hellwig <hch@lst.de> +Cc: Christian König <christian.koenig@amd.com> +Cc: Michel Dänzer <michel@daenzer.net> +Cc: Takashi Iwai <tiwai@suse.de> +Cc: stable@vger.kernel.org # v4.16 +Signed-off-by: Jeremy Cline <jcline@redhat.com> +--- + lib/swiotlb.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/lib/swiotlb.c b/lib/swiotlb.c +index 12fbaa445637..cc640588f145 100644 +--- a/lib/swiotlb.c ++++ b/lib/swiotlb.c +@@ -714,7 +714,7 @@ swiotlb_alloc_buffer(struct device *dev, size_t size, dma_addr_t *dma_handle, + + phys_addr = swiotlb_tbl_map_single(dev, + __phys_to_dma(dev, io_tlb_start), +- 0, size, DMA_FROM_DEVICE, 0); ++ 0, size, DMA_FROM_DEVICE, attrs); + if (phys_addr == SWIOTLB_MAP_ERROR) + goto out_warn; + +-- +2.17.0 + |