#13793 closed enhancement (fixed)

USB-ECM: support for QEMU USB network devices

Reported by: korli Owned by: mmlr
Priority: normal Milestone: Unscheduled
Component: Drivers/Network/usb_ecm Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

QEMU can configure USB network devices with for instance: -netdev user,id=mynet0 -device usb-net,netdev=mynet0

This patch adds support for it. It needs to be tested for possible regressions.

Attachments (1)

0001-usb_ecm-support-for-QEmu-CDCE-network-device.patch (5.0 KB) - added by korli 20 months ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 20 months ago by korli

Has a Patch: set

comment:2 Changed 20 months ago by korli

Owner: changed from nobody to mmlr
Status: newassigned

Michael, please kindly review!

comment:3 Changed 20 months ago by mmlr

Looks good. Not a big fan of the goto though.

comment:4 Changed 20 months ago by korli

I can eventually break each loop at the end. I thought the changes might break things for ECM devices (like phones), may I "as is" commit and request testing afterwards?

comment:5 Changed 20 months ago by korli

Component: Drivers/NetworkDrivers/Network/usb_ecm

comment:6 Changed 20 months ago by mmlr

I don't see why the change would break existing hardware, as presumably there simply wouldn't be multiple configurations?

To avoid the goto I'd set a bool found to true and break out of the outer loop when set (or add it to the loop condition). But you can also go ahead and push as is.

comment:7 Changed 20 months ago by korli

Resolution: fixed
Status: assignedclosed

Thanks for the feedback, applied in hrev51606.

Note: See TracTickets for help on using tickets.