Re: [PATCH] iwlwifi: Demote messages about fw flags size to info

From: Luca Coelho
Date: Wed Aug 02 2017 - 00:47:00 EST


Hi JoÃo Paulo,


On Tue, 2017-08-01 at 15:58 -0700, JoÃo Paulo Rechi Vita wrote:
> Hello Luca,
>
> On Mon, Jul 24, 2017 at 4:01 AM, Coelho, Luciano
> <luciano.coelho@xxxxxxxxx> wrote:
> > On Fri, 2017-07-21 at 07:51 -0700, JoÃo Paulo Rechi Vita wrote:
>
> (...)
>
> > > Currently these messages are presented to the user during boot if there
> > > is no bootsplash covering the console, sometimes even if the boot splash
> > > is enabled but has not started yet by the time this message is shown.
> > >
>
> I should have provided another piece of information here: all of this
> happens even when booting with the 'quiet' kernel parameter.

Oh, okay, that's annoying.


> > This specific case is harmless, but I'd rather keep this message as an
> > error, because in other situations it could lead to unexpected
> > behavioir, so I prefer to keep it very visible.
> >
> >
>
> I see your point, and I understand the purpose of these messages. I'm
> wondering if perhaps having them at the warning level would give them
> enough visibility, while still keeping a clean boot process to the end
> user. If so, I can send an updated patch.
>
> Thanks for your reply and for pointing to the fix for the root cause
> of that message.

Sure, I agree it's better to make it use IWL_WARN(), which will generate
a dev_warn() instead of a dev_err().


--
Cheers,
Luca.