RE: [PATCH 2/8] musb_host: fix endpoint_disable() method

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

 



>>>>Sergei, was this fixed ??

>>>  By me? No. I have enough work to do rather than fixing this corner
>>>case. E.g. I'm still busy fixing ISO Tx DMA which is certainly more
>>>important (and complex) task.

>>>>Where's the new version of this patch ?

>>>  What? This is hardly even connected to my patch -- especially this one.

>>What do you mean,

>>   Please read all this thread.

>>the original patch is yours.

>>   I haven't promised to fix the corner case oops that Ajay has reported
>>(besides, I have too many other issue to deal with). And it hasn't been proven
>>that it was caused by my patches. So, I'm seeing no reason for new versions --
>>this patch in particular fixed the case of an *assured* oops/lockup.

> [1] [PATCH 2/8] musb_host: fix endpoint_disable() method at
> http://marc.info/?l=linux-usb&m=122963107004966&w=2

> [2] [PATCH 1/8] musb_host: fix urb_dequeue() method at
> http://marc.info/?l=linux-usb&m=122963106904957&w=2

> [3] usb: musb: Fix for kernel panic with multiple bulk transfer at
> http://marc.info/?l=linux-usb&m=122390068417621&w=2

> Two patch ([1] and [2]) from Sergei's submitted patch-list is suppose to be a replacement for [3]
> and is expected to solve the the kernel panic which was earlier getting fixed by [3]

>    Well, it was your expectation only. I've claimed that they are partial
> replacements, not that they'll fix your corner case.
> but seems there is still some corner case left in [1] and [2].
>    Why not find a dn fix it if you have time? I certainly don't. And I'm
> certainly not going to do all the work for everybody.
> The kernel panic I reported doesn't happen if we use [3] above.
>    If your pacth wasn't such a pile of ..., I wouldn't request pulling it
> out. You haven't offered us the reworked version during all these months.

It's in our queue and the complete solution is coming soon. 
Only concern is that the final fix may require to revert [1] and [2] if they goes in
mainline so I think its better to hold on these two untill we get the real and complete fix.
What do you say Felipe ?

-Ajay


--
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

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

  Powered by Linux