linux-kernel - Re: [RESEND PATCH] sparc: defconfig: Cleanup from old Kconfig options

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]

Date:   Thu, 20 Jul 2017 12:38:04 -0700 (PDT)
From:   David Miller <davem@...emloft.net>
To:     krzk@...nel.org
Cc:     sam@...nborg.org, linux-kernel@...r.kernel.org,
        sparclinux@...r.kernel.org
Subject: Re: [RESEND PATCH] sparc: defconfig: Cleanup from old Kconfig
 options

From: Krzysztof Kozlowski <krzk@...nel.org>
Date: Thu, 20 Jul 2017 20:11:24 +0200

> On Thu, Jul 20, 2017 at 02:36:33PM +0200, Sam Ravnborg wrote:
>> On Thu, Jul 20, 2017 at 07:00:55AM +0200, Krzysztof Kozlowski wrote:
>> > Remove old, dead Kconfig options (in order appearing in this commit):
>> >  - EXPERIMENTAL is gone since v3.9;
>> >  - INET_LRO: commit 7bbf3cae65b6 ("ipv4: Remove inet_lro library");
>> >  - AUTOFS_FS: commit 561c5cf9236a ("staging: Remove autofs3");
>> >  - RCU_CPU_STALL_DETECTOR: commit a00e0d714fbd ("rcu: Remove conditional
>> >    compilation for RCU CPU stall warnings");
>> >  - USB_DEVICE_CLASS: commit 007bab91324e ("USB: remove
>> >    CONFIG_USB_DEVICE_CLASS");
>> >  - SYSCTL_SYSCALL_CHECK: commit 7c60c48f58a7 ("sysctl: Improve the
>> >    sysctl sanity checks");
>> > 
>> > Signed-off-by: Krzysztof Kozlowski <krzk@...nel.org>
>> > Acked-by: David S. Miller <davem@...emloft.net>
>> 
>> Hi Krzysztof.
>> 
>> This looks like a manually edited defconfig.
>> A better approch would be to do:
>> $ make ARCH=sparc64 sparc64_defconfig
>> $ make ARCH=sparc64 savedefconfig
>> 
>> And then to check the generated defconfig.
>> Same for sparc32.
> 
> Actually this was generated with sed instead of defconfig on purpose -
> as part of my bigger cleanup starting from ARM [1]. In case of
> defconfigs for my platform we wanted to preserve the explanation why
> things are disappearing, not only to savedefconfig. Indeed this brought
> up that few items from savedefconfig would be gone but should be
> replaced with different (new) items.
> 
> Anyway this way you have a full control and explanation why things are
> being changed. After this you can always run defconfig to reorganize and
> clean up.
> 
> [1] https://www.linux-mips.org/archives/linux-mips/2017-06/msg00255.html

Yeah I guess this is OK for now.  But we should do a tool based regen at
some point soon.

Applied, thanks.