On 22/09/2020 15.38, Andrey Konovalov wrote:
On Tue, Sep 22, 2020 at 1:07 PM M. Vefa Bicakci <m.v.b@xxxxxxxxxx> wrote:
Hello all,
This is the third version of the patch sets originally published in the
e-mail thread thread at [1]. As mentioned in the same e-mail thread with
the e-mail at [2], I was able to find a more acceptable solution to the
issue reported by Andrey Konovalov, where usbip takes over the
dummy_hcd-provided devices set up by the USB fuzzing instance of the
syzkaller fuzzer.
In summary, the approach involves:
* Removal of the usbip_match function.
* Fixing two bugs in the specialised USB driver selection code.
* Accommodating usbip by changing the logic in the specialised USB
driver selection code, while preserving legacy/previous behaviour.
I have tested this patch set with Greg Kroah-Hartman's usb-next tree
based on v5.9-rc6 with the base commit mentioned below in this e-mail,
and I can report that usbip works as expected, with no regressions in
the usbip_test.sh self-test suite output compared to v4.14.119. I have
also verified that the apple-mfi-fastcharge driver is correctly used
when an iPhone is plugged in to my system. Finally, I can report that
Andrey Konovalov's "keyboard" test program making use of dummy_hcd,
found at [3], also works as expected.
I would appreciate your comments.
Thank you,
Vefa
[1] https://lore.kernel.org/linux-usb/CAAeHK+zOrHnxjRFs=OE8T=O9208B9HP_oo8RZpyVOZ9AJ54pAA@xxxxxxxxxxxxxx/
[2] https://lore.kernel.org/linux-usb/9f332d7b-e33d-ebd0-3154-246fbfb69128@xxxxxxxxxx/
[3] https://github.com/xairy/raw-gadget
Cc: Bastien Nocera <hadess@xxxxxxxxxx>
Cc: Valentina Manea <valentina.manea.m@xxxxxxxxx>
Cc: Shuah Khan <shuah@xxxxxxxxxx>
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
Cc: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
Cc: <syzkaller@xxxxxxxxxxxxxxxx>
M. Vefa Bicakci (4):
Revert "usbip: Implement a match function to fix usbip"
usbcore/driver: Fix specific driver selection
usbcore/driver: Fix incorrect downcast
usbcore/driver: Accommodate usbip
drivers/usb/core/driver.c | 50 ++++++++++++++++++++++++------------
drivers/usb/usbip/stub_dev.c | 6 -----
2 files changed, 34 insertions(+), 22 deletions(-)
Hi Vefa,
This fixes the issue that I've been having.
Tested-by: Andrey Konovalov <andreyknvl@xxxxxxxxxx>
for the series.
Thank you!
Thank you as well, Andrey, for confirming that these patches
resolve the negative interaction between usbip and dummy_hcd!
I appreciate your effort in testing the patches.
Vefa