s390/ism: don't pass bogus GFP_ flags to dma_alloc_coherent
authorChristoph Hellwig <hch@lst.de>
Wed, 9 Nov 2022 07:23:05 +0000 (08:23 +0100)
committerChristoph Hellwig <hch@lst.de>
Mon, 21 Nov 2022 08:36:09 +0000 (09:36 +0100)
dma_alloc_coherent is an opaque allocator that only uses the GFP_ flags
for allocation context control.  Don't pass __GFP_COMP which makes no
sense for an allocation that can't in any way be converted to a page
pointer.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Acked-by: Wenjia Zhang <wenjia@linux.ibm.com>
drivers/s390/net/ism_drv.c

index d34bb6e..dfd401d 100644 (file)
@@ -243,7 +243,8 @@ static int ism_alloc_dmb(struct ism_dev *ism, struct smcd_dmb *dmb)
 
        dmb->cpu_addr = dma_alloc_coherent(&ism->pdev->dev, dmb->dmb_len,
                                           &dmb->dma_addr,
-                                          GFP_KERNEL | __GFP_NOWARN | __GFP_NOMEMALLOC | __GFP_COMP | __GFP_NORETRY);
+                                          GFP_KERNEL | __GFP_NOWARN |
+                                          __GFP_NOMEMALLOC | __GFP_NORETRY);
        if (!dmb->cpu_addr)
                clear_bit(dmb->sba_idx, ism->sba_bitmap);