[PATCH] usbip: Reduce CONNRESET message noise in dmesg from stub

Shuah Khan posted 1 patch 2 weeks, 3 days ago
There is a newer version of this series
drivers/usb/usbip/stub_tx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] usbip: Reduce CONNRESET message noise in dmesg from stub
Posted by Shuah Khan 2 weeks, 3 days ago
stub_complete() prints informational messages for each urb unlink
filling dmesg. Change the message to dev_dbg() similar to vhci
reports the CONNRESET condition.

Reported-by: Ignacio Hernandez-Ros <ignacio@hernandez-ros.com>
Closes: https://lore.kernel.org/all/0101019b92e81c20-09906fb4-d5e8-40a6-9192-ab693eef4179-000000@us-west-2.amazonses.com/
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
---
 drivers/usb/usbip/stub_tx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/usbip/stub_tx.c b/drivers/usb/usbip/stub_tx.c
index 55919c3762ba..4cae452bfadb 100644
--- a/drivers/usb/usbip/stub_tx.c
+++ b/drivers/usb/usbip/stub_tx.c
@@ -55,7 +55,7 @@ void stub_complete(struct urb *urb)
 			 "stopped by a call to usb_kill_urb() because of cleaning up a virtual connection\n");
 		return;
 	case -ECONNRESET:
-		dev_info(&urb->dev->dev,
+		dev_dbg(&urb->dev->dev,
 			 "unlinked by a call to usb_unlink_urb()\n");
 		break;
 	case -EPIPE:
-- 
2.51.0
Re: [PATCH] usbip: Reduce CONNRESET message noise in dmesg from stub
Posted by Ignacio Hernández-Ros 2 weeks, 2 days ago
Hi,

I need a couple of days to set up a new RPi hardware and compile kernel with USBIP debug set on. 
I don't want to put my infrastructure on risk if anything goes wrong. But I’ll definitively send you the debug information required.

Regards
Ignacio

> El 21 ene 2026, a las 17:37, Shuah Khan <skhan@linuxfoundation.org> escribió:
> 
> stub_complete() prints informational messages for each urb unlink
> filling dmesg. Change the message to dev_dbg() similar to vhci
> reports the CONNRESET condition.
> 
> Reported-by: Ignacio Hernandez-Ros <ignacio@hernandez-ros.com>
> Closes: https://lore.kernel.org/all/0101019b92e81c20-09906fb4-d5e8-40a6-9192-ab693eef4179-000000@us-west-2.amazonses.com/
> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
> ---
> drivers/usb/usbip/stub_tx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/usbip/stub_tx.c b/drivers/usb/usbip/stub_tx.c
> index 55919c3762ba..4cae452bfadb 100644
> --- a/drivers/usb/usbip/stub_tx.c
> +++ b/drivers/usb/usbip/stub_tx.c
> @@ -55,7 +55,7 @@ void stub_complete(struct urb *urb)
> 			 "stopped by a call to usb_kill_urb() because of cleaning up a virtual connection\n");
> 		return;
> 	case -ECONNRESET:
> -		dev_info(&urb->dev->dev,
> +		dev_dbg(&urb->dev->dev,
> 			 "unlinked by a call to usb_unlink_urb()\n");
> 		break;
> 	case -EPIPE:
> -- 
> 2.51.0
> 
Re: [PATCH] usbip: Reduce CONNRESET message noise in dmesg from stub
Posted by Sergey Shtylyov 2 weeks, 3 days ago
On 1/21/26 7:37 PM, Shuah Khan wrote:

> stub_complete() prints informational messages for each urb unlink
> filling dmesg. Change the message to dev_dbg() similar to vhci
> reports the CONNRESET condition.
> 
> Reported-by: Ignacio Hernandez-Ros <ignacio@hernandez-ros.com>
> Closes: https://lore.kernel.org/all/0101019b92e81c20-09906fb4-d5e8-40a6-9192-ab693eef4179-000000@us-west-2.amazonses.com/
> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
> ---
>  drivers/usb/usbip/stub_tx.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/usbip/stub_tx.c b/drivers/usb/usbip/stub_tx.c
> index 55919c3762ba..4cae452bfadb 100644
> --- a/drivers/usb/usbip/stub_tx.c
> +++ b/drivers/usb/usbip/stub_tx.c
> @@ -55,7 +55,7 @@ void stub_complete(struct urb *urb)
>  			 "stopped by a call to usb_kill_urb() because of cleaning up a virtual connection\n");
>  		return;
>  	case -ECONNRESET:
> -		dev_info(&urb->dev->dev,
> +		dev_dbg(&urb->dev->dev,
>  			 "unlinked by a call to usb_unlink_urb()\n");

   Please re-align this line for it to start under & again.

[...]

MBR, Sergey
Re: [PATCH] usbip: Reduce CONNRESET message noise in dmesg from stub
Posted by Shuah 2 weeks, 3 days ago
On 1/21/26 13:38, Sergey Shtylyov wrote:
> On 1/21/26 7:37 PM, Shuah Khan wrote:
> 
>> stub_complete() prints informational messages for each urb unlink
>> filling dmesg. Change the message to dev_dbg() similar to vhci
>> reports the CONNRESET condition.
>>
>> Reported-by: Ignacio Hernandez-Ros <ignacio@hernandez-ros.com>
>> Closes: https://lore.kernel.org/all/0101019b92e81c20-09906fb4-d5e8-40a6-9192-ab693eef4179-000000@us-west-2.amazonses.com/
>> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
>> ---
>>   drivers/usb/usbip/stub_tx.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/usb/usbip/stub_tx.c b/drivers/usb/usbip/stub_tx.c
>> index 55919c3762ba..4cae452bfadb 100644
>> --- a/drivers/usb/usbip/stub_tx.c
>> +++ b/drivers/usb/usbip/stub_tx.c
>> @@ -55,7 +55,7 @@ void stub_complete(struct urb *urb)
>>   			 "stopped by a call to usb_kill_urb() because of cleaning up a virtual connection\n");
>>   		return;
>>   	case -ECONNRESET:
>> -		dev_info(&urb->dev->dev,
>> +		dev_dbg(&urb->dev->dev,
>>   			 "unlinked by a call to usb_unlink_urb()\n");
> 
>     Please re-align this line for it to start under & again.

I missed that - thanks - will do that in v2.

thanks,
-- Shuah