Messages in this thread | | | From | Alok Prasad <> | Subject | RE: [EXT] [PATCH] qed: Fix missing error code in qed_slowpath_start() | Date | Mon, 11 Oct 2021 05:17:32 +0000 |
| |
> -----Original Message----- > From: Jiapeng Chong <jiapeng.chong@linux.alibaba.com> > Sent: 09 October 2021 13:39 > To: Ariel Elior <aelior@marvell.com> > Cc: GR-everest-linux-l2 <GR-everest-linux-l2@marvell.com>; davem@davemloft.net; > kuba@kernel.org; linux@armlinux.org.uk; ast@kernel.org; daniel@iogearbox.net; > hawk@kernel.org; john.fastabend@gmail.com; netdev@vger.kernel.org; linux- > kernel@vger.kernel.org; bpf@vger.kernel.org; chongjiapeng > <jiapeng.chong@linux.alibaba.com> > Subject: [EXT] [PATCH] qed: Fix missing error code in qed_slowpath_start() > > External Email > > ---------------------------------------------------------------------- > From: chongjiapeng <jiapeng.chong@linux.alibaba.com> > > The error code is missing in this code scenario, add the error code > '-EINVAL' to the return value 'rc'. > > Eliminate the follow smatch warning: > > drivers/net/ethernet/qlogic/qed/qed_main.c:1298 qed_slowpath_start() > warn: missing error code 'rc'. > > Reported-by: Abaci Robot <abaci@linux.alibaba.com> > Fixes: d51e4af5c209 ("qed: aRFS infrastructure support") > Signed-off-by: chongjiapeng <jiapeng.chong@linux.alibaba.com> > --- > drivers/net/ethernet/qlogic/qed/qed_main.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/ethernet/qlogic/qed/qed_main.c > b/drivers/net/ethernet/qlogic/qed/qed_main.c > index 5e7242304ee2..359ad859ae18 100644 > --- a/drivers/net/ethernet/qlogic/qed/qed_main.c > +++ b/drivers/net/ethernet/qlogic/qed/qed_main.c > @@ -1295,6 +1295,7 @@ static int qed_slowpath_start(struct qed_dev *cdev, > } else { > DP_NOTICE(cdev, > "Failed to acquire PTT for aRFS\n"); > + rc = -EINVAL; > goto err; > } > } > -- > 2.19.1.6.gb485710b
Thanks!
Acked-by: Alok Prasad <palok@marvell.com>
| |