> -----Original Message----- > From: SF Markus Elfring [mailto:elfr...@users.sourceforge.net] > Sent: Sunday, January 7, 2018 3:10 PM > To: de...@linuxdriverproject.org; netdev@vger.kernel.org; Haiyang Zhang > <haiya...@microsoft.com>; KY Srinivasan <k...@microsoft.com>; Stephen > Hemminger <sthem...@microsoft.com> > Cc: LKML <linux-ker...@vger.kernel.org>; kernel-janit...@vger.kernel.org > Subject: [PATCH] hyperv/netvsc: Delete two error messages for a failed > memory allocation in netvsc_init_buf() > > From: Markus Elfring <elfr...@users.sourceforge.net> > Date: Sun, 7 Jan 2018 21:03:26 +0100 > > Omit extra messages for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <elfr...@users.sourceforge.net> > --- > drivers/net/hyperv/netvsc.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c index > 17e529af79dc..c1ec02f801f6 100644 > --- a/drivers/net/hyperv/netvsc.c > +++ b/drivers/net/hyperv/netvsc.c > @@ -275,9 +275,6 @@ static int netvsc_init_buf(struct hv_device *device, > > net_device->recv_buf = vzalloc(buf_size); > if (!net_device->recv_buf) { > - netdev_err(ndev, > - "unable to allocate receive buffer of size %u\n", > - buf_size); > ret = -ENOMEM; > goto cleanup; > } > @@ -357,8 +354,6 @@ static int netvsc_init_buf(struct hv_device *device, > > net_device->send_buf = vzalloc(buf_size); > if (!net_device->send_buf) { > - netdev_err(ndev, "unable to allocate send buffer of > size %u\n", > - buf_size); > ret = -ENOMEM; > goto cleanup; > }
These messages are not displayed anywhere else: "unable to allocate receive buffer of size %u\n" "unable to allocate send buffer of size %u\n", After set ret = -ENOMEM; and cleanup, we won't know which buffer allocation failed without the error message. So please do not remove these messages. Thanks, - Haiyang
