Commit 690ea7fe authored by Selvin Xavier's avatar Selvin Xavier Committed by Jason Gunthorpe
Browse files

RDMA/bnxt_re: Use GFP_KERNEL in non atomic context 

Use GFP_KERNEL instead of GFP_ATOMIC while allocating control path
structures which will be only called from non atomic context

Link: https://lore.kernel.org/r/1631709163-2287-10-git-send-email-selvin.xavier@broadcom.com


Signed-off-by: default avatarSelvin Xavier <selvin.xavier@broadcom.com>
Reviewed-by: default avatarLeon Romanovsky <leonro@nvidia.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
parent 2b4ccce6

drivers/infiniband/hw/bnxt_re/main.c

+1 −1
Original line number Diff line number Diff line
@@ -1730,7 +1730,7 @@ static int bnxt_re_netdev_event(struct notifier_block *notifier, 
	}

	if (sch_work) {

		/* Allocate for the deferred task */

		re_work = kzalloc(sizeof(*re_work), GFP_ATOMIC);

		re_work = kzalloc(sizeof(*re_work), GFP_KERNEL);

		if (re_work) {

			get_device(&rdev->ibdev.dev);

			re_work->rdev = rdev;

drivers/infiniband/hw/bnxt_re/qplib_rcfw.c

+2 −2
Original line number Diff line number Diff line
@@ -848,13 +848,13 @@ struct bnxt_qplib_rcfw_sbuf *bnxt_qplib_rcfw_alloc_sbuf( 
{

	struct bnxt_qplib_rcfw_sbuf *sbuf;



	sbuf = kzalloc(sizeof(*sbuf), GFP_ATOMIC);

	sbuf = kzalloc(sizeof(*sbuf), GFP_KERNEL);

	if (!sbuf)

		return NULL;



	sbuf->size = size;

	sbuf->sb = dma_alloc_coherent(&rcfw->pdev->dev, sbuf->size,

				      &sbuf->dma_addr, GFP_ATOMIC);

				      &sbuf->dma_addr, GFP_KERNEL);

	if (!sbuf->sb)

		goto bail;