linux-kernel - Re: [PATCH] usb: core: sysfs: constify attribute

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] [day] [month] [year] [list]

Message-ID: <20170624075536.GA9346@kroah.com>
Date:   Sat, 24 Jun 2017 09:55:36 +0200
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Arvind Yadav <arvind.yadav.cs@...il.com>
Cc:     linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] usb: core: sysfs: constify attribute_group

On Fri, Jun 23, 2017 at 11:28:10AM +0530, Arvind Yadav wrote:
> File size before:
>    text	   data	    bss	    dec	    hex	filename
>    6798	   2848	      4	   9650	   25b2	drivers/usb/core/sysfs.o
> 
> File size After adding 'const':
>    text	   data	    bss	    dec	    hex	filename
>    7230	   2400	      4	   9634	   25a2	drivers/usb/core/sysfs.o
> 
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@...il.com>

You have sent a lot of these types of patches out, and really, this is a
horrible changelog entry.  You are assuming that I know what this is,
and why it is a good thing.  Now I might, but maybe not...

Please actually explain why you are making this change.  Why it is
needed.  And big hint, it's not only about the output of the file
size...

Please fix this up for all of the patches you have sent me in this
format, as I'm deleting them all from my queue right now.

thanks,

greg k-h