Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 7 Oct 2022 12:56:39 +0200
From:      Hans Petter Selasky <hselasky@freebsd.org>
To:        koobs@FreeBSD.org, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: 55a3bd000d97 - main - usb(4): Make sure the enumeration thread doesn't loop too fast.
Message-ID:  <ac432b62-e96b-7d1c-f674-7c9b9b00ce50@freebsd.org>
In-Reply-To: <42ae3a83-bb0b-315c-ddfd-919c109a11d2@FreeBSD.org>
References:  <202210051013.295ADf7P080520@gitrepo.freebsd.org> <42ae3a83-bb0b-315c-ddfd-919c109a11d2@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 10/7/22 02:27, Kubilay Kocak wrote:
> On 5/10/2022 9:13 pm, Hans Petter Selasky wrote:
>> The branch main has been updated by hselasky:
>>
>> URL: 
>> https://cgit.FreeBSD.org/src/commit/?id=55a3bd000d9799f431c207e359466484ac63c137
>>
>> commit 55a3bd000d9799f431c207e359466484ac63c137
>> Author:     Hans Petter Selasky <hselasky@FreeBSD.org>
>> AuthorDate: 2022-06-09 13:15:49 +0000
>> Commit:     Hans Petter Selasky <hselasky@FreeBSD.org>
>> CommitDate: 2022-10-05 10:12:33 +0000
>>
>>      usb(4): Make sure the enumeration thread doesn't loop too fast.
>>      MFC after:      1 week
>>      Sponsored by:   NVIDIA Networking
>> ---
>>   sys/dev/usb/controller/usb_controller.c | 3 +++
>>   1 file changed, 3 insertions(+)
>>
>> diff --git a/sys/dev/usb/controller/usb_controller.c 
>> b/sys/dev/usb/controller/usb_controller.c
>> index 0897af0492cb..959f54a4583f 100644
>> --- a/sys/dev/usb/controller/usb_controller.c
>> +++ b/sys/dev/usb/controller/usb_controller.c
>> @@ -414,6 +414,9 @@ usb_bus_explore(struct usb_proc_msg *pm)
>>   #if USB_HAVE_ROOT_MOUNT_HOLD
>>       usb_root_mount_rel(bus);
>>   #endif
>> +
>> +    /* Nice the enumeration a bit, to avoid looping too fast. */
>> +    usb_pause_mtx(&bus->bus_mtx, USB_MS_TO_TICKS(16));
>>   }
>>   
>> /*------------------------------------------------------------------------*
>>
> 
> Is it possible to sysctl (ideally runtime tunable) this setting?
> 

Sure. I'll wrap up a patch.

--HPS



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?ac432b62-e96b-7d1c-f674-7c9b9b00ce50>