From: "Jason A. Donenfeld" Subject: Re: Re: [PATCH] random: silence compiler warnings and fix race Date: Tue, 20 Jun 2017 22:09:06 +0200 Message-ID: References: <20170614192838.3jz4sxpcuhxygx4z@breakpoint.cc> <20170614224526.29076-1-Jason@zx2c4.com> <20170620060344.ngbnzg2mz5hvq4kw@thunk.org> <20170620093642.3ri6dct5qkf7vhuc@thunk.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Sandy Harris , Jeffrey Walton , "Theodore Ts'o" , tglx@breakpoint.cc, David Miller , Linus Torvalds , Eric Biggers , LKML , Greg Kroah-Hartman , "kernel-hardening@lists.openwall.com" , Linux Crypto Mailing List , Michael Ellerman To: Kees Cook Return-path: List-Post: List-Help: List-Unsubscribe: List-Subscribe: In-Reply-To: List-Id: linux-crypto.vger.kernel.org On Tue, Jun 20, 2017 at 8:14 PM, Kees Cook wrote: > How about doing this: > > default DEBUG_KERNEL > > Most distro kernel select DEBUG_KERNEL because it unhides a bunch of > other useful configs. Since it doesn't strictly _depend_ on > DEBUG_KERNEL, I think it's probably a mistake to enforce a false > dependency. Using it as a hint for the default seems maybe like a good > middle ground. (And if people can't agree on that, then I guess > "default n"...) I didn't know you could do that with Kconfig. Great idea. I'll make this change and submit a new patch to Ted. Jason