On Thu, 22 May 2014, Pratyush Anand wrote:
> OTG3 and EH Compliance Plan 1.0 talks about Super Speed OTG Verification
> system (SS-OVS) which consists of an excersizer and analyzer.
>
> USB Compliance Suite from Lecroy or Ellisys can act as such SS-OVS for
> Link Layer Validation (LVS).
>
> Some modifications are needed for an embedded Linux USB host to pass all
> these tests. Most of these tests require just Link to be in U0. They do
> not work with default Linux USB stack since, default stack does port
> reset and then starts sending setup packet, which is not expected by
> Link Layer Validation (LVS) device of Lecroy Compliance Suit. Then,
> There are many Link Layer Tests which need host to generate specific
> traffic.
>
> This patch supports specific traffic generation cases. As of now all the
> host Lecroy Link Layer-USBIF tests (except TD7.26) passes
> with this patch for single run using Lecroy USB Compliance Suite
> Version 1.98 Build 239 and Lecroy USB Protocol Analyzer version 4.80
> Build 1603. Therefore patch seems to be a good candidate for inclusion.
> Further modification can be done on top of it.
>
> lvstest driver will not bind to any device by default. It can bind
> manually to a super speed USB host controller root hub. Therefore, regular
> hub driver must be unbound before this driver is bound. For example, if
> 2-0:1.0 is the xhci root hub, then execute following to unbind hub driver.
>
> echo 2-0:1.0 > /sys/bus/usb/drivers/hub/unbind
>
> Then to bind lvs driver write Linux Foundation's vendor ID and SS root
> hub's device ID into new_id file.
>
> echo "1D6B 3" > /sys/bus/usb/drivers/lvs/new_id
>
> Now connect LVS device with root hub port.
You forgot to include an example here showing how to do this:
echo 2-0\:1.0 > /sys/bus/usb/drivers/lvs/bind
> Test case specific traffic can be generated as follows whenever needed:
>
> 1. To issue "Get Device descriptor" command for TD.7.06:
> echo 1 > /sys/bus/usb/devices/2-0\:1.0/get_dev_desc
>
> 2. To set U1 timeout to 127 for TD.7.18
> echo 127 > /sys/bus/usb/devices/2-0\:1.0/u1_timeout
>
> 3. To set U2 timeout to 0 for TD.7.18
> echo 0 > /sys/bus/usb/devices/2-0\:1.0/u2_timeout
>
> 4. To issue "Hot Reset" for TD.7.29
> echo 1 > /sys/bus/usb/devices/2-0\:1.0/hot_reset
>
> 5. To issue "U3 Entry" for TD.7.35
> echo 1 > /sys/bus/usb/devices/2-0\:1.0/u3_entry
>
> 6. To issue "U3 Exit" for TD.7.36
> echo 1 > /sys/bus/usb/devices/2-0\:1.0/u3_exit
None of those example need to contain the "1". For instance, the last
one should be written:
echo > /sys/bus/usb/devices/2-0\:1.0/u3_exit
> diff --git a/Documentation/ABI/testing/sysfs-bus-usb-lvstest b/Documentation/ABI/testing/sysfs-bus-usb-lvstest
> new file mode 100644
> index 0000000..f47e8a3
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-bus-usb-lvstest
> @@ -0,0 +1,47 @@
> +Link Layer Validation Device is a standard device for testing of Super
> +Speed Link Layer tests. These nodes are available in sysfs only when lvs
> +driver is bound with root hub device.
> +
> +What: /sys/bus/usb/devices/.../get_dev_desc
> +Date: March 2014
> +Contact: Pratyush Anand <pratyush.anand@xxxxxx>
> +Description:
> + Write to this node to issue "Get Device Descriptor"
> + for Link Layer Validation device. It is needed for TD.7.06.
> +
> +What: /sys/bus/usb/devices/.../u1_timeout
> +Date: March 2014
> +Contact: Pratyush Anand <pratyush.anand@xxxxxx>
> +Description:
> + Set "U1 timeout" for the downstream port where Link Layer
> + Validation device is connected. It is needed for TD.7.18,
> + TD.7.19, TD.7.20 and TD.7.21.
This doesn't say that the value must be between 0 and 127.
> +
> +What: /sys/bus/usb/devices/.../u2_timeout
> +Date: March 2014
> +Contact: Pratyush Anand <pratyush.anand@xxxxxx>
> +Description:
> + Set "U2 timeout" for the downstream port where Link Layer
> + Validation device is connected. It is needed for TD.7.18,
> + TD.7.19, TD.7.20 and TD.7.21.
Same here.
> +static void lvs_rh_work(struct work_struct *work)
> +{
> + struct lvs_rh *lvs = container_of(work, struct lvs_rh, rh_work);
> + struct usb_interface *intf = lvs->intf;
> + struct usb_device *hdev = interface_to_usbdev(intf);
> + struct usb_hcd *hcd = bus_to_hcd(hdev->bus);
> + struct usb_hub_descriptor *descriptor = &lvs->descriptor;
> + struct usb_port_status port_status;
> + int i, ret = 0;
> +
> + /* Examine each root port */
> + for (i = 1; i <= descriptor->bNbrPorts; i++) {
> + ret = usb_control_msg(hdev, usb_rcvctrlpipe(hdev, 0),
> + USB_REQ_GET_STATUS, USB_DIR_IN | USB_RT_PORT, 0, i,
> + &port_status, sizeof(port_status), 1000);
You must not use I/O buffers allocated on the stack. The buffer must
be allocated by kmalloc or something equivalent.
> + if (ret < 4)
> + continue;
> + /* handle only connection change notification */
> + if (!(port_status.wPortChange & USB_PORT_STAT_C_CONNECTION))
> + continue;
> + usb_control_msg(hdev, usb_sndctrlpipe(hdev, 0),
> + USB_REQ_CLEAR_FEATURE, USB_RT_PORT,
> + USB_PORT_FEAT_C_CONNECTION, i, NULL, 0, 1000);
> +
> + if (port_status.wPortStatus & USB_PORT_STAT_CONNECTION) {
> + lvs->present = true;
> + lvs->portnum = i;
> + if (hcd->phy)
> + usb_phy_notify_connect(hcd->phy,
> + USB_SPEED_SUPER);
> + } else {
> + lvs->present = false;
> + if (hcd->phy)
> + usb_phy_notify_disconnect(hcd->phy,
> + USB_SPEED_SUPER);
> + }
> + break;
> + }
> +
> + ret = usb_submit_urb(lvs->urb, GFP_ATOMIC);
This doesn't need to be GFP_ATOMIC; you can use GFP_KERNEL.
> +static int lvs_rh_probe(struct usb_interface *intf,
> + const struct usb_device_id *id)
> +{
...
> + ret = usb_submit_urb(lvs->urb, GFP_NOIO);
And this doesn't need to be GFP_NOIO.
Alan Stern
--
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
