Added Benjamin. On Mon, Jul 15, 2013 at 8:37 PM, Andrew de los Reyes <a...@chromium.org> wrote: > I believe this commit was an optimization, so it should be okay to revert > it. > > Nestor, can you confirm?
5962640 is not an optimization, is a workaround for the dropped packets during enumeration. After a9dd22b73085734 it should not be necessary as the issue should be resolved. Nevertheless, it turns out that the workaround not only helps in the case that some input reports get dropped (which no longer happens), but also in the case that the underlying usb controllers fail to send the report that request the enumeration of the Unifying receiver. We've seen this ocassionally depending on the usb controllers and some time dependent behaviour. Concluding I'd say that this patch being otherwise harmless, albeit ugly, could be put back in. I'd suggest as well investigating why the usb controllers fails to send the enumeration request with -EPIPE, as this my be hiding a race inside those usb drivers. If we re-add 596264082f10dd4. We should fix setting querying_devices flag to true. This was not done in the original patch. As the impact was not seen and the patch was to be removed anyway it was never corrected. > > > On Mon, Jul 15, 2013 at 10:26 AM, Joseph Salisbury > <joseph.salisb...@canonical.com> wrote: >> >> Hi Andrew, >> >> A bug was opened against the Ubuntu kernel[0]. It was found that >> reverting the following commit resolved this bug: >> >> commit 8af6c08830b1ae114d1a8b548b1f8b056e068887 >> Author: Andrew de los Reyes <a...@chromium.org> >> Date: Mon Feb 18 09:20:23 2013 -0800 >> >> Revert "HID: Fix logitech-dj: missing Unifying device issue" >> >> >> The regression was introduced as of v3.10-rc1. Commit 8af6c08 reverts >> commit 5962640, so it seems commit 5962640 may still be needed. >> >> I see that you are the author of this patch, so I wanted to run this by >> you. I was thinking of requesting a revert for v3.11, but I wanted to >> get your feedback first. >> >> >> Thanks, >> >> Joe >> >> [0] http://pad.lv/1194649 > > Cheers, Nestor -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/