swiotlb: fix unexpected swiotlb_alloc_coherent failures
authorTakashi Iwai <tiwai@suse.de>
Tue, 10 Apr 2018 17:05:13 +0000 (19:05 +0200)
committerChristoph Hellwig <hch@lst.de>
Tue, 10 Apr 2018 20:30:43 +0000 (22:30 +0200)
commit9e7f06c8beee304ee21b791653fefcd713f48b9a
treeaaccea45fd03820b33eee35b77e3bb154e10ea46
parentb284d4d5a6785f8cd07eda2646a95782373cd01e
swiotlb: fix unexpected swiotlb_alloc_coherent failures

The code refactoring by commit 0176adb00406 ("swiotlb: refactor coherent
buffer allocation") made swiotlb_alloc_buffer almost always failing due
to a thinko: namely, the function evaluates the dma_coherent_ok call
incorrectly and dealing as if it's invalid. This ends up with weird
errors like iwlwifi probe failure or amdgpu screen flickering.

This patch corrects the logic error.

Bugzilla: https://bugzilla.suse.com/show_bug.cgi?id=1088658
Bugzilla: https://bugzilla.suse.com/show_bug.cgi?id=1088902
Fixes: 0176adb00406 ("swiotlb: refactor coherent buffer allocation")
Cc: <stable@vger.kernel.org> # v4.16+
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Christoph Hellwig <hch@lst.de>
lib/swiotlb.c