Re: [PATCH 0/9 v5] xHCI ring expansion patchset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sorry, got a typo in the mail address... I'll resend the patchset. :(

Thanks,
Andiry


On 03/05/2012 05:41 PM, Andiry Xu wrote:
> Hi Sarah & Alan,
> 
> This is the ring expansion patchset v5, sorry for the long delay.
> 
> Changelog from v4:
> 
> 1. Fix the bug in handle set TRDP command completion;
> 2. Add patch 8 and 9, allocate 2 segments for transfer ring at the beginning,
>    and expand the ring if the enqueue pointer will advance into dequeue segment.
> 
> Thank you Alan, your suggestion makes things much more easier!
> 
> I've tested with usbtest -t 10, which could hit the case with multiple control
> requests. Seems it works fine.
> 
> Hi Paul,
> 
> Can you run usbtest 10 with this patchset? Thanks.
> 
> I've some questions, not quite related to the patchset...
> 1. Does usbtest require specific compliant device? I used a USB2.0 storage
> device, and some test cases fail with unmatched transfer length and unexpected
> status.
> 2. What will happen if the host driver does not call usb_hcd_link_urb_to_ep() in
> enqueue() method? In my original design, the host driver will "suspend" the urbs
> in some cases, does not queue them immediately, just return enqueue() as
> successful and queue them later. In this case, does the host driver need to
> call usb_hcd_link_urb_to_ep() before it returns?    
> 
> Thanks,
> Andiry
> 
> ---
> 
> Andiry Xu (9):
>   xHCI: store ring's type
>   xHCI: store ring's last segment and segment numbers
>   xHCI: count free TRBs on transfer ring
>   xHCI: factor out segments allocation and free function
>   xHCI: set cycle state when allocate rings
>   xHCI: dynamic ring expansion
>   xHCI: update sg tablesize
>   xHCI: Allocate 2 segments for transfer ring
>   xHCI: check enqueue pointer advance into dequeue seg
> 
>  drivers/usb/host/xhci-mem.c  |  230 +++++++++++++++++++++++++++----------
>  drivers/usb/host/xhci-ring.c |  257 ++++++++++++++++++++++++------------------
>  drivers/usb/host/xhci.c      |    4 +-
>  drivers/usb/host/xhci.h      |   17 +++
>  4 files changed, 334 insertions(+), 174 deletions(-)
> 


--
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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux