Thread (24 messages) 24 messages, 5 authors, 2020-02-23

Re: [PATCH net-next 00/16] Clean driver, module and FW versions

From: Jakub Kicinski <kuba@kernel.org>
Date: 2020-02-21 01:17:18

On Thu, 20 Feb 2020 16:58:39 +0200 Leon Romanovsky wrote:
From: Leon Romanovsky <redacted>

Hi,

This is first patchset to netdev (already sent RDMA [1] and arch/um)
in attempt to unify the version management for in-tree kernel code.
The patches follow already accepted ethtool change [2] to set as
a default linux kernel version.

It allows us to remove driver version and present to the users unified
picture of driver version, which is similar to default MODULE_VERSION().
Thanks for doing this! The patches look good to me.

A few minor nit picks I registered, IDK how hard we want to press 
on these:

 - it seems in couple places you remove the last user of DRV_RELDATE,
   but not the define. In case of bonding maybe we can remove the date
   too. IDK what value it brings in the description, other than perhaps
   humoring people;
 - we should probably give people a heads up by CCing maintainers
   (regardless of how dumb we find not bothering to read the ML as
   a maintainer);
 - one on the FW below..
As part of this series, I deleted various creative attempts to mark
absence of FW. There is no need to set "N/A" in ethtool ->fw_version
field and it is enough to do not set it.
These seem reasonable to me, although in abundance of caution it could
be a good idea to have them as separate commits so we can revert more
easily. Worse come to worst.
1.
The code is compile tested and passes 0-day kbuild.
2.
The proposed changes are based on commit:
  2bb07f4e1d86 ("tc-testing: updated tdc tests for basic filter")
3.
WIP branch is [3].

[1] https://lore.kernel.org/linux-rdma/20200220071239.231800-1-leon@kernel.org/ (local)
[2] https://lore.kernel.org/linux-rdma/20200127072028.19123-1-leon@kernel.org/ (local)
[3] https://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git/log/?h=ethtool
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help