linux-kernel - Re: [PATCH v5 0/8] usb: dwc3: Fix broken BULK stream support to dwc3 gadget driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 21 Sep 2018 02:53:18 +0000
From:   Tejas Joglekar <Tejas.Joglekar@...opsys.com>
To:     Anurag Kumar Vulisha <anurag.kumar.vulisha@...inx.com>,
        "balbi@...nel.org" <balbi@...nel.org>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
CC:     "v.anuragkumar@...il.com" <v.anuragkumar@...il.com>,
        "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Thinh.Nguyen@...opsys.com" <Thinh.Nguyen@...opsys.com>,
        "apandey@...inx.com" <apandey@...inx.com>
Subject: Re: [PATCH v5 0/8] usb: dwc3: Fix broken BULK stream support to
 dwc3 gadget driver

Hello Anurag,
On 9/15/2018 8:00 PM, Anurag Kumar Vulisha wrote:
> These patch series fixes the broken BULK streaming support in
> dwc3 gadget driver.
>
> Changes in v5:
> 	1. Removed the dev_dbg prints as suggested bt "Thinh Nguyen"
>
> Changes in v4:
> 	1. Corrected the commit messgae and stream timeout description
> 	   as suggested by "Thinh Nguyen"
>
> Changes in v3:
> 	1. Added the changes suggested by "Thinh Nguyen"
>
> Changes in v2:
> 	1. Added "usb: dwc3:" in subject heading
>
> Anurag Kumar Vulisha (8):
>   usb: dwc3: Correct the logic for checking TRB full in
>     __dwc3_prepare_one_trb()
>   usb: dwc3: update stream id in depcmd
>   usb: dwc3: make controller clear transfer resources after complete
>   usb: dwc3: implement stream transfer timeout
>   usb: dwc3: don't issue no-op trb for stream capable endpoints
>   usb: dwc3: check for requests in started list for stream capable
>     endpoints
>   usb: dwc3: Check for IOC/LST bit in both event->status and TRB->ctrl
>     fields
>   usb: dwc3: Check MISSED ISOC bit only for ISOC endpoints
>
>  drivers/usb/dwc3/core.h   |  7 ++++
>  drivers/usb/dwc3/gadget.c | 85 ++++++++++++++++++++++++++++++++++++++++++-----
>  2 files changed, 84 insertions(+), 8 deletions(-)
>
Tested-By: Tejas Joglekar <tejas.joglekar@...opsys.com>

I have tested this patch series except the stream transfer timeout patch on HAPS-DX platform. I am not aware of exact scenarios where I can test the timeout patch, and don't have test to perform timeout testing.

Thanks & Regards,
 Tejas Joglekar