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