John and Felipe, On Tue, Jun 23, 2015 at 10:56 AM, John Youn <John.Youn@xxxxxxxxxxxx> wrote: > On 6/22/2015 4:00 PM, Doug Anderson wrote: >> Hi, >> >> On Thu, Jun 4, 2015 at 6:12 AM, Kaukab, Yousaf <yousaf.kaukab@xxxxxxxxx> wrote: >>>>>>>>> Tested-by: Heiko Stuebner <heiko@xxxxxxxxx> >>>>>>>>> >>>>>>>>> ---------- 8< ------------------ >>>>>>>>> [ 19.799200] BUG: sleeping function called from invalid context at >>>>>>>> mm/slab.c:2863 >>>>>>>> >>>>>>>> Will I see a patch for fixing this ? >>>>>>> >>>>>>> I am currently on a business trip and can't look into this for at >>>>>>> least a couple of weeks. >>>>>> >>>>>> John, since this is your driver, could you fix it up ? >>>>>> >>>>> >>>>> Hi Felipe, >>>>> >>>>> I've been out of the office for the past 2 weeks and am catching up on >>>>> stuff now. >>>>> >>>>> I'll take a look at it later this week if I don't hear anything from >>>>> Yousaf. >>> >>> I have patches to fix this issue (and another one). I will send them out hopefully tomorrow. >> >> I noticed that (33ad261 usb: dwc2: host: spinlock urb_enqueue) is now >> in linuxnext, so I pulled it (and many other) changes into the >> chromeos-3.14 kernel for testing. I still see the problem that Heiko >> reported. Note that aside from the warning, this causes all sorts of >> "spinlock recursion" issues and an eventual freeze on my system. >> >> Until Yousaf's patch has landed it seems as if we should do a revert >> of (33ad261 usb: dwc2: host: spinlock urb_enqueue) to avoid regressing >> existing systems. >> >> -Doug >> > > Hi Doug, > > Hopefully the fixes will go into the next rc. > > > Hi Felipe, > > If you haven't already, could you queue these patches for the next rc? They fix issues for patches applied for 4.2. > > http://marc.info/?l=linux-usb&m=143409924626643&w=2 > http://marc.info/?l=linux-usb&m=143409925026644&w=2 > http://marc.info/?l=linux-usb&m=143409925026645&w=2 OK, I tried those changes and they fix my problems too (which is not a terrible surprise). You can add my Tested-by if it helps. We need either these fixes or a revert of the patch. As it stands right now ToT linuxnext is pretty broken for me. -Doug -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
