lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 18 Jan 2021 22:01:15 +0000 From: Grant Grundler <grundler@...omium.org> To: Grant Grundler <grundler@...omium.org>, nic_swsd <nic_swsd@...ltek.com> Cc: Oliver Neukum <oliver@...kum.org>, "David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-usb@...r.kernel.org, netdev <netdev@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org> Subject: Re: [PATCH 1/3] net: usb: cdc_ncm: emit dev_err on error paths +nic_swsd [adding per Realtek developer team request] On Sat, Jan 16, 2021 at 5:26 AM Grant Grundler <grundler@...omium.org> wrote: > > Several error paths in bind/probe code will only emit > output using dev_dbg. But if we are going to fail the > bind/probe, emit related output with "err" priority. > > Signed-off-by: Grant Grundler <grundler@...omium.org> > --- > drivers/net/usb/cdc_ncm.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/net/usb/cdc_ncm.c b/drivers/net/usb/cdc_ncm.c > index 5a78848db93f..25498c311551 100644 > --- a/drivers/net/usb/cdc_ncm.c > +++ b/drivers/net/usb/cdc_ncm.c > @@ -849,17 +849,17 @@ int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_ > > /* check if we got everything */ > if (!ctx->data) { > - dev_dbg(&intf->dev, "CDC Union missing and no IAD found\n"); > + dev_err(&intf->dev, "CDC Union missing and no IAD found\n"); > goto error; > } > if (cdc_ncm_comm_intf_is_mbim(intf->cur_altsetting)) { > if (!ctx->mbim_desc) { > - dev_dbg(&intf->dev, "MBIM functional descriptor missing\n"); > + dev_err(&intf->dev, "MBIM functional descriptor missing\n"); > goto error; > } > } else { > if (!ctx->ether_desc || !ctx->func_desc) { > - dev_dbg(&intf->dev, "NCM or ECM functional descriptors missing\n"); > + dev_err(&intf->dev, "NCM or ECM functional descriptors missing\n"); > goto error; > } > } > @@ -868,7 +868,7 @@ int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_ > if (ctx->data != ctx->control) { > temp = usb_driver_claim_interface(driver, ctx->data, dev); > if (temp) { > - dev_dbg(&intf->dev, "failed to claim data intf\n"); > + dev_err(&intf->dev, "failed to claim data intf\n"); > goto error; > } > } > @@ -924,7 +924,7 @@ int cdc_ncm_bind_common(struct usbnet *dev, struct usb_interface *intf, u8 data_ > if (ctx->ether_desc) { > temp = usbnet_get_ethernet_addr(dev, ctx->ether_desc->iMACAddress); > if (temp) { > - dev_dbg(&intf->dev, "failed to get mac address\n"); > + dev_err(&intf->dev, "failed to get mac address\n"); > goto error2; > } > dev_info(&intf->dev, "MAC-Address: %pM\n", dev->net->dev_addr); > -- > 2.29.2 >
Powered by blists - more mailing lists