linux-kernel - Re: [PATCH] usb: devio: update max count of DPs per interval for ISOC

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]

Message-ID: <20181228105148.GA22073@kroah.com>
Date:   Fri, 28 Dec 2018 11:51:48 +0100
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Chunfeng Yun <chunfeng.yun@...iatek.com>
Cc:     Felipe Balbi <felipe.balbi@...ux.intel.com>,
        Matthias Brugger <matthias.bgg@...il.com>,
        linux-usb@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH] usb: devio: update max count of DPs per interval for ISOC

On Fri, Dec 28, 2018 at 06:27:44PM +0800, Chunfeng Yun wrote:
> The host shall be able to accept and send up to 96 DPs for
> devices operating at Gen 2 speed.

Why?  What is this keeping from working properly today?

I need a much better changelog text here in order to be able to accept
this patch.

> 
> Signed-off-by: Chunfeng Yun <chunfeng.yun@...iatek.com>
> ---
>  drivers/usb/core/devio.c | 8 +++-----
>  1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/usb/core/devio.c b/drivers/usb/core/devio.c
> index a75bc0b8a50f..82c16210e34c 100644
> --- a/drivers/usb/core/devio.c
> +++ b/drivers/usb/core/devio.c
> @@ -1564,12 +1564,10 @@ static int proc_do_submiturb(struct usb_dev_state *ps, struct usbdevfs_urb *uurb
>  		}
>  		for (totlen = u = 0; u < number_of_packets; u++) {
>  			/*
> -			 * arbitrary limit need for USB 3.0
> -			 * bMaxBurst (0~15 allowed, 1~16 packets)
> -			 * bmAttributes (bit 1:0, mult 0~2, 1~3 packets)
> -			 * sizemax: 1024 * 16 * 3 = 49152
> +			 * arbitrary limit need for USB 3.1 Gen2
> +			 * sizemax: 96 DPs at SSP, 96 * 1024 = 98304
>  			 */
> -			if (isopkt[u].length > 49152) {
> +			if (isopkt[u].length > 98304) {

Are we going to have to keep bumping this up as speeds get faster and
faster?

thanks,

greg k-h