Messages in this thread | | | Date | Fri, 30 May 2008 10:45:12 -0700 | From | Leonid <> | Subject | Re: [GIT PATCH] USB fixes for 2.6.26-rc4 |
| |
Hi Alan,
Not to be overly pedantic, but you weren't so wrong to be confused on the issue of "regression". Here's the scoop:
1) Kernel versions <= 2.6.24: latency of 2/6/2/6/... uframes pattern observed between packets on the bus for nVIDIA HCs.
2) Kernel versions >= 2.6.26: latency of 8/8/8/8/... uframes pattern observed between packets on the bus for nVIDIA HCs.
3) Post-patch: latency of 1/1/1/1/... uframes on all kernels versions and all HC chipsets.
So basically there was a peripheral issue in the EHCI driver between 2.6.24 and 2.6.25 that exacerbated the problem. But the underlying issue was there long before 2.6.25. Dave is quite right that this isn't technically a "regression."
The issue itself was a bug (as opposed to a "feature" ;-) ) since the original comments in the code clearly had the right intention, but the code itself did not execute as intended in certain situations. In fact, thinking about it now, this issue could even cause significant performance degradation on nVIDIA HCs if one has multiple devices using the bus at any given time. Given this, it might be best to label the patch "USB: EHCI: fix performance issue" instead of using the word regression. But regardless, we are just worrying about symmantics at this point! :)
Hope that helps clarify and put to rest this titling issue once in for all.
-Leonid
On Thu, May 29, 2008 at 6:44 PM, Alan Stern <stern@rowland.harvard.edu> wrote: > On Thu, 29 May 2008, Greg KH wrote: >> Well, I was misinformed by Alan Stern about this then, sorry for missing >> that you didn't ack this one (you acked the other patches in this series >> though.) > > The notion that this was a regression came about by mistake. It was > stated that 2.6.24 worked better than 2.6.25; apparently that remark > was wrong.
| |