linux-kernel - RE: [PATCH 2/3] usb: gadget: f_sourcesink support for maxburst for bulk transfers

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:
 <CH3PR12MB8726A85B8D425323EDDB96D8B128A@CH3PR12MB8726.namprd12.prod.outlook.com>
Date: Mon, 11 Aug 2025 15:36:28 +0000
From: "Chary Chennoju, Srikanth" <srikanth.chary-chennoju@....com>
To: Alan Stern <stern@...land.harvard.edu>
CC: "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
	"Thinh.Nguyen@...opsys.com" <Thinh.Nguyen@...opsys.com>,
	"m.grzeschik@...gutronix.de" <m.grzeschik@...gutronix.de>,
	"Chris.Wulff@...mp.com" <Chris.Wulff@...mp.com>, "tiwai@...e.de"
	<tiwai@...e.de>, "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "Kalluri,
 Punnaiah Choudary" <punnaiah.choudary.kalluri@....com>
Subject: RE: [PATCH 2/3] usb: gadget: f_sourcesink support for maxburst for
 bulk transfers

[AMD Official Use Only - AMD Internal Distribution Only]

Hi Stern,

Please let me know if there are any further comments for this patch.

Thanks & Regards,
Srikanth

> -----Original Message-----
> From: Chary Chennoju, Srikanth
> Sent: Monday, July 7, 2025 7:31 PM
> To: Alan Stern <stern@...land.harvard.edu>
> Cc: gregkh@...uxfoundation.org; Thinh.Nguyen@...opsys.com;
> m.grzeschik@...gutronix.de; Chris.Wulff@...mp.com; tiwai@...e.de; linux-
> usb@...r.kernel.org; linux-kernel@...r.kernel.org; Kalluri, Punnaiah
> Choudary <punnaiah.choudary.kalluri@....com>
> Subject: RE: [PATCH 2/3] usb: gadget: f_sourcesink support for maxburst for
> bulk transfers
>
> Hi Stern,
>
> Agreed. I can remove the comment as I have added the check for the value
> based on user input.
>
> Regards,
> Srikanth
>
> > -----Original Message-----
> > From: Alan Stern <stern@...land.harvard.edu>
> > Sent: Monday, July 7, 2025 7:15 PM
> > To: Chary Chennoju, Srikanth <srikanth.chary-chennoju@....com>
> > Cc: gregkh@...uxfoundation.org; Thinh.Nguyen@...opsys.com;
> > m.grzeschik@...gutronix.de; Chris.Wulff@...mp.com; tiwai@...e.de;
> > linux- usb@...r.kernel.org; linux-kernel@...r.kernel.org; Kalluri,
> > Punnaiah Choudary <punnaiah.choudary.kalluri@....com>
> > Subject: Re: [PATCH 2/3] usb: gadget: f_sourcesink support for
> > maxburst for bulk transfers
> >
> > Caution: This message originated from an External Source. Use proper
> > caution when opening attachments, clicking links, or responding.
> >
> >
> > On Mon, Jul 07, 2025 at 10:49:26AM +0000, Chary Chennoju, Srikanth
> wrote:
> > > [AMD Official Use Only - AMD Internal Distribution Only]
> > >
> > > Hi Stern,
> > >
> > > Thanks for your comments. If you are mentioning to have a check for
> > > the
> > value which user is providing, then there is a check which is carried
> > out(#34 to #37 in the patch). It limits to max burst value if user
> > exceeds the supported value.
> >
> > > > > +     /*
> > > > > +      * Fill in the SS bulk descriptors from the module parameters.
> > > > > +      * We assume that the user knows what they are doing and won't
> > > > > +      * give parameters that their UDC doesn't support.
> > > > > +      */
> >
> > In that case this comment is wrong, isn't it?  You don't assume that
> > the user knows what they are doing; you do check and limit the value
> > to maxburst if the user's value is too big.
> >
> > Alan Stern