NACK: [PATCH][SRU BIONIC] UBUNTU: [Config]: set CONFIG_EDAC_DEBUG=y for ARM64

[Seth Forshee]

On Thu, Jul 19, 2018 at 07:54:44AM -0500, Manoj Iyer wrote:
> On Thu, 19 Jul 2018, Seth Forshee wrote:
> 
> > On Wed, Jun 20, 2018 at 03:14:12PM -0500, Manoj Iyer wrote:
> > > Testing EDAC requires EDAC_DEBUG to be enabled. ARM64 server customers
> > > currently need to rebuild Ubuntu kernels with this config option to test
> > > EDAC support. Please consider this SRU to Bionic to enable EDAC_DEBUG
> > > config option in the kernel.
> > > 
> > > A patch to enable EDAC_DEBUG was acked and applied to cosmic recently.
> > > https://lists.ubuntu.com/archives/kernel-team/2018-May/092595.html
> > 
> > Based on the testing in cosmic this is producing a lot of dmesg spam
> > that isn't useful. I think something needs to be done to at least reduce
> > the spam, or eliminate it entirely if the debugfs files is all that's
> > required.
> 
> It produces 'spam' output only on ThunderX hardware, on ThunderX2 and
> QDF2400 it is relatively quiet.

However it is trivial to eliminate most or all of the output by
defaulting the debug level to 0, or it looks like even -1 would probably
work to eliminate any debug output by default. Or if the 700+ debug
messages aren't needed at all it shouldn't be too difficult to split out
the debugfs stuff under a separate config option so the messages aren't
consuming space in the kernel image.