linux-kernel - RE: [LINUX PATCH] usb: gadget: Configure bulk maxburst through module parameter in gadget zero.

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 for Android: free password hash cracker in your pocket

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

Date:   Fri, 30 Sep 2016 11:12:18 +0000
From:   Manish Narani <manish.narani@...inx.com>
To:     Greg KH <gregkh@...uxfoundation.org>
CC:     "balbi@...nel.org" <balbi@...nel.org>,
        "k.opasiak@...sung.com" <k.opasiak@...sung.com>,
        "r.baldyga@...sung.com" <r.baldyga@...sung.com>,
        "peter.chen@...escale.com" <peter.chen@...escale.com>,
        "John.Youn@...opsys.com" <John.Youn@...opsys.com>,
        "eu@...ipetonello.com" <eu@...ipetonello.com>,
        "i.kotrasinsk@...sung.com" <i.kotrasinsk@...sung.com>,
        "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Anurag Kumar Vulisha <anuragku@...inx.com>,
        Punnaiah Choudary Kalluri <punnaia@...inx.com>
Subject: RE: [LINUX PATCH] usb: gadget: Configure bulk maxburst through
 module parameter in gadget zero.

Hi Greg,

Thanks for the suggestion. I will fix it and send you in next version. :)

Regards,
Manish

-----Original Message-----
From: Greg KH [mailto:gregkh@...uxfoundation.org]
Sent: Thursday, September 29, 2016 2:24 PM
To: Manish Narani <MNARANI@...inx.com>
Cc: balbi@...nel.org; k.opasiak@...sung.com; r.baldyga@...sung.com; peter.chen@...escale.com; Manish Narani <MNARANI@...inx.com>; John.Youn@...opsys.com; eu@...ipetonello.com; i.kotrasinsk@...sung.com; linux-usb@...r.kernel.org; linux-kernel@...r.kernel.org; Anurag Kumar Vulisha <anuragku@...inx.com>; Punnaiah Choudary Kalluri <punnaia@...inx.com>
Subject: Re: [LINUX PATCH] usb: gadget: Configure bulk maxburst through module parameter in gadget zero.

On Thu, Sep 29, 2016 at 02:16:44PM +0530, Manish Narani wrote:
> This patch adds support to configure bulk maxburst through module
> parameter. This parameter can be used to modify bulk maxburst in case
> if one wants to measure peak Bulk/Isoc-IN/OUT performance.

Eeek, this isn't the 1990's, please don't add new module parameters :)

We have much better ways of handling configuration options for a device, why not use them (configfs, sysfs files, etc.)?

Ugh, it looks like iso_maxburst is handled this way as well.

Felipe, why is this?

thanks,

greg k-h


This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.