Re: [PATCH 1/2] usbip: Delete an error message for a failed memory allocation in two functions

[Shuah Khan]

On 12/05/2017 02:49 PM, SF Markus Elfring wrote:
> From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> Date: Tue, 5 Dec 2017 22:25:38 +0100
> 
> Omit an extra message for a memory allocation failure in these functions.
> 
> This issue was detected by using the Coccinelle software.

Please include the problem and log from Coccinelle software in any
future patches for the issues detected by Coccinelle.

> 
> Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> ---
>  drivers/usb/usbip/stub_rx.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/usb/usbip/stub_rx.c b/drivers/usb/usbip/stub_rx.c
> index 536e037f541f..a70eb81823a3 100644
> --- a/drivers/usb/usbip/stub_rx.c
> +++ b/drivers/usb/usbip/stub_rx.c
> @@ -302,7 +302,6 @@ static struct stub_priv *stub_priv_alloc(struct stub_device *sdev,
>  
>  	priv = kmem_cache_zalloc(stub_priv_cache, GFP_ATOMIC);
>  	if (!priv) {
> -		dev_err(&sdev->udev->dev, "alloc stub_priv\n");
>  		spin_unlock_irqrestore(&sdev->priv_lock, flags);
>  		usbip_event_add(ud, SDEV_EVENT_ERROR_MALLOC);
>  		return NULL;
> @@ -466,7 +465,6 @@ static void stub_recv_cmd_submit(struct stub_device *sdev,
>  	priv->urb->setup_packet = kmemdup(&pdu->u.cmd_submit.setup, 8,
>  					  GFP_KERNEL);
>  	if (!priv->urb->setup_packet) {
> -		dev_err(&udev->dev, "allocate setup_packet\n");

If Coccinelle found this as an extra message, there is something
wrong with the Coccinelle script. This is not an extra message.
This message is for the second kmemdup() failure and is necessary.

>  		usbip_event_add(ud, SDEV_EVENT_ERROR_MALLOC);
>  		return;
>  	}
> 

thanks,
-- Shuah