netdev - Re: [net-next v3 4/9] igc: Use netdev log helpers in igc

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: <158956116965.37785.11864549120054924700@akiranx-mobl.amr.corp.intel.com>
Date:   Fri, 15 May 2020 09:46:09 -0700
From:   Andre Guedes <andre.guedes@...el.com>
To:     Jeff Kirsher <jeffrey.t.kirsher@...el.com>, davem@...emloft.net
Cc:     netdev@...r.kernel.org, nhorman@...hat.com, sassmann@...hat.com,
        Aaron Brown <aaron.f.brown@...el.com>,
        Jeff Kirsher <jeffrey.t.kirsher@...el.com>
Subject: Re: [net-next v3 4/9] igc: Use netdev log helpers in igc_ethtool.c

Hi Jeff,

Quoting Jeff Kirsher (2020-05-14 21:21:34)
> From: Andre Guedes <andre.guedes@...el.com>
> 
> In igc_ethtool.c we print log messages using dev_* helpers, generating
> inconsistent output with the rest of the driver. Since this is a network
> device driver, we should preferably use netdev_* helpers because they
> append the interface name to the message, helping making sense the of
> the logs.
> 
> This patch converts all dev_* calls to netdev_*. It also takes this
> opportunity to remove the '\n' character at the end of messages since it
> is automatically added by netdev_* log helpers.

It seems you missed removing the statement about '\n' from the commit message.
It doesn't apply anymore, as mentioned in the cover letter.

The commit message from the other patches in this series needs to be updated as
well.

Thank you,

Andre