Re: [PATCH v4 1/3] Revert "Revert "usb: gadget: udc: core: Invoke usb_gadget_connect only when started""

[Alan Stern]

On Mon, May 29, 2023 at 11:48:14PM +0000, Badhri Jagan Sridharan wrote:
> This reverts commit f22e9b67f19ccc73de1ae04375d4b30684e261f8.
> 
> The regression reported in
> https://lore.kernel.org/all/ZF4bMptC3Lf2Hnee@xxxxxxxxxxx/ is being
> fixed in
> commit 7d7863db7cc0 ("usb: gadget: udc: core: Offload usb_udc_vbus_handler processing").

What commit is that?  It doesn't exist yet, at least, not in the 
mainline kernel.

> Hence reverting the revert.
> 
> Signed-off-by: Badhri Jagan Sridharan <badhri@xxxxxxxxxx>

No!  Do not do this.  If you do, there will again be a version of the 
kernel that has the bug that caused the revert in the first place.  Even 
if it's only temporary, it could still affect people who are (for 
example) trying to run bisections.

Instead, reorder the patches.  First fix the underlying problem that 
led to the deadlocks.  Once that's in good shape then you can safely 
make this change.

Alan Stern