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

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

 



Hello.

Subbrathnam, Swaminathan wrote:
I believe it is time to stop the exchange. Let Ajay update the
[3] usb: musb: Fix for kernel panic with multiple bulk transfer at  http://marc.info/?l=linux-usb&m=122390068417621&w=2

patch as he has indicated.

  When this will be done? I'm afraid I can't wait several more weeks...
I won't resubmit musb_urb_dequeue() fix due to the identified issue with the current approach to handling qh but I'm certainly not going to hold up the musb_h_disable() fix...

Let us review the same and decide on the future course of action.  For now let us park the discussion here.

  Yes, I've said everything I had to say already. Let's see the code.

regards
swami
________________________________________
From: davinci-linux-open-source-bounces@xxxxxxxxxxxxxxxxxxxx [davinci-linux-open-source-bounces@xxxxxxxxxxxxxxxxxxxx] On Behalf Of Sergei Shtylyov [sshtylyov@xxxxxxxxxxxxx]
Sent: Sunday, January 25, 2009 1:37 AM
To: Gupta, Ajay Kumar
Cc: davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; felipe.balbi@xxxxxxxxx; gregkh@xxxxxxx
Subject: Re: [PATCH 2/8] musb_host: fix endpoint_disable() method

Hello, I wrote:

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.

   To which you agreed. Moreover, you said back then that I seemed to
have found the cause of the issue (which you never described) your patch
was dealing with.

You haven't offered us the reworked version during all these months.

   I have found severa; mistakes in your patch back then. Please, do me
the same favor. Don't just complain that it haven't fixed you problem
which I've never claimed it would. We've tackled the driver issues in
different ways -- you were trying to fix your corner case failure, I was
trying to fix the obviously (or not so obviously) wrong code.
   As to your promised new patch (I still haven't gotten your reply, and
saw it only on http://marc.info), we need to see it yet... and I don't
get why you said that my patches *may* need to be reveted. You're not
sure or have not seen the actual fix? Stop intriguing me already. :-)

-Ajay

WBR, Sergei


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