Re: [PATCH] usb: chipidea: udc: assign interrupt number to USB gadget structure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 04, 2021 at 11:34:53AM +0000, Jun Li wrote:
> 
> 
> > -----Original Message-----
> > From: Jun Li
> > Sent: Friday, June 4, 2021 7:10 PM
> > To: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
> > Cc: peter.chen@xxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; dl-linux-imx
> > <linux-imx@xxxxxxx>; Faqiang Zhu <faqiang.zhu@xxxxxxx>
> > Subject: RE: [PATCH] usb: chipidea: udc: assign interrupt number to USB gadget
> > structure
> > 
> > > -----Original Message-----
> > > From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
> > > Sent: Friday, June 4, 2021 6:52 PM
> > > To: Jun Li <jun.li@xxxxxxx>
> > > Cc: peter.chen@xxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; dl-linux-imx
> > > <linux-imx@xxxxxxx>; Faqiang Zhu <faqiang.zhu@xxxxxxx>
> > > Subject: Re: [PATCH] usb: chipidea: udc: assign interrupt number to
> > > USB gadget structure
> > >
> > > On Fri, May 14, 2021 at 06:59:44PM +0800, Li Jun wrote:
> > > > Chipidea also need sync interrupt before unbind the udc while gadget
> > > > remove driver, otherwise setup irq handling may happen while unbind,
> > > > see below dump generated from android function switch stress test:
> > > >
> > ...
> > > >
> > >
> > > What commit does this fix?  Should it go to stable kernels, and if so,
> > > how far back?
> > 
> > I will resend the patch with fix tag added.
> 
> Hi Greg,
> 
> I assume this problem may exist all along or very far, but this patch
> need base one commit[1] to fix this problem, so I think that commit is
> the point to know how far this patch can apply back for stable kernels,
> in this case, please let me know if adding fix tag pointing to that commit
> still the right way for this.
> 
> [1] 3c73bc52195d ("usb: gadget: core: sync interrupt before unbind the udc")

Yes, that's fine and the correct way to do it.

thanks,

greg k-h



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux