* [PATCH net-next v2 0/4] Remove vermagic header from global include folder
@ 2020-04-19 14:18 Leon Romanovsky
2020-04-21 12:14 ` Leon Romanovsky
2020-04-21 20:28 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Leon Romanovsky @ 2020-04-19 14:18 UTC (permalink / raw)
To: David S. Miller, Jakub Kicinski
Cc: Leon Romanovsky, Andy Gospodarek, Borislav Petkov, Ion Badulescu,
Jay Vosburgh, Jessica Yu, linux-kbuild, linux-kernel, linux-pm,
Masahiro Yamada, Michal Marek, netdev, oss-drivers, Salil Mehta,
Sebastian Reichel, Shannon Nelson, Veaceslav Falico,
Yisen Zhuang
From: Leon Romanovsky <leonro@mellanox.com>
Changelog:
v2:
* Changed the implementation of patch #4 to be like Masahiro wants.
I personally don't like this implementation and changing it just to move forward
this this patchset.
v1:
https://lore.kernel.org/lkml/20200415133648.1306956-1-leon@kernel.org
* Added tags
* Updated patch #4 with test results
* Changed scripts/mod/modpost.c to create inclusion of vermagic.h
from kernel folder and not from general include/linux. This is
needed to generate *.mod.c files, while building modules.
v0:
https://lore.kernel.org/lkml/20200414155732.1236944-1-leon@kernel.org
----------------------------------------------------------------------------
Hi,
This is followup to the failure reported by Borislav [1] and suggested
fix later on [2].
The series removes all includes of linux/vermagic.h, updates hns and
nfp to use same kernel versioning scheme (exactly like we did for
other drivers in previous cycle) and removes vermagic.h from global
include folder.
[1] https://lore.kernel.org/lkml/20200411155623.GA22175@zn.tnic
[2] https://lore.kernel.org/lkml/20200413080452.GA3772@zn.tnic
------------------------------------------------------------
1. Honestly, I have no idea if it can go to net-rc, clearly not all my
patches are fixes, so I'm sending them to the net-next.
2. Still didn't get response from kbuild, but it passed my own
compilation tests.
https://git.kernel.org/pub/scm/linux/kernel/git/leon/linux-rdma.git/log/?h=vermagic
Thanks
Leon Romanovsky (4):
drivers: Remove inclusion of vermagic header
net/hns: Remove custom driver version in favour of global one
net/nfp: Update driver to use global kernel version
kernel/module: Hide vermagic header file from general use
drivers/net/bonding/bonding_priv.h | 2 +-
drivers/net/ethernet/3com/3c509.c | 1 -
drivers/net/ethernet/3com/3c515.c | 1 -
drivers/net/ethernet/adaptec/starfire.c | 1 -
drivers/net/ethernet/hisilicon/hns3/hns3_enet.c | 3 ---
drivers/net/ethernet/hisilicon/hns3/hns3_enet.h | 4 ----
drivers/net/ethernet/hisilicon/hns3/hns3_ethtool.c | 4 ----
drivers/net/ethernet/netronome/nfp/nfp_main.c | 3 ---
drivers/net/ethernet/netronome/nfp/nfp_net_ethtool.c | 2 --
drivers/net/ethernet/pensando/ionic/ionic_main.c | 2 +-
drivers/power/supply/test_power.c | 2 +-
include/linux/vermagic.h | 5 +++++
kernel/module.c | 3 +++
net/ethtool/ioctl.c | 3 +--
scripts/mod/modpost.c | 1 +
15 files changed, 13 insertions(+), 24 deletions(-)
--
2.25.2
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next v2 0/4] Remove vermagic header from global include folder
2020-04-19 14:18 [PATCH net-next v2 0/4] Remove vermagic header from global include folder Leon Romanovsky
@ 2020-04-21 12:14 ` Leon Romanovsky
2020-04-21 20:28 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Leon Romanovsky @ 2020-04-21 12:14 UTC (permalink / raw)
To: David S. Miller, Jakub Kicinski
Cc: Andy Gospodarek, Borislav Petkov, Ion Badulescu, Jay Vosburgh,
Jessica Yu, linux-kbuild, linux-kernel, linux-pm,
Masahiro Yamada, Michal Marek, netdev, oss-drivers, Salil Mehta,
Sebastian Reichel, Shannon Nelson, Veaceslav Falico,
Yisen Zhuang
On Sun, Apr 19, 2020 at 05:18:46PM +0300, Leon Romanovsky wrote:
> From: Leon Romanovsky <leonro@mellanox.com>
>
> Changelog:
> v2:
> * Changed the implementation of patch #4 to be like Masahiro wants.
> I personally don't like this implementation and changing it just to move forward
> this this patchset.
> v1:
> https://lore.kernel.org/lkml/20200415133648.1306956-1-leon@kernel.org
> * Added tags
> * Updated patch #4 with test results
> * Changed scripts/mod/modpost.c to create inclusion of vermagic.h
> from kernel folder and not from general include/linux. This is
> needed to generate *.mod.c files, while building modules.
> v0:
> https://lore.kernel.org/lkml/20200414155732.1236944-1-leon@kernel.org
> ----------------------------------------------------------------------------
Dave,
I see in the patchworks that this series is marked as "Needs Review/ACK".
Can you please help me to understand who is needed to be approached?
https://patchwork.ozlabs.org/project/netdev/list/?series=171189
Thanks
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next v2 0/4] Remove vermagic header from global include folder
2020-04-19 14:18 [PATCH net-next v2 0/4] Remove vermagic header from global include folder Leon Romanovsky
2020-04-21 12:14 ` Leon Romanovsky
@ 2020-04-21 20:28 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2020-04-21 20:28 UTC (permalink / raw)
To: leon
Cc: kuba, leonro, andy, bp, ionut, j.vosburgh, jeyu, linux-kbuild,
linux-kernel, linux-pm, masahiroy, michal.lkml, netdev,
oss-drivers, salil.mehta, sre, snelson, vfalico, yisen.zhuang
From: Leon Romanovsky <leon@kernel.org>
Date: Sun, 19 Apr 2020 17:18:46 +0300
> This is followup to the failure reported by Borislav [1] and suggested
> fix later on [2].
>
> The series removes all includes of linux/vermagic.h, updates hns and
> nfp to use same kernel versioning scheme (exactly like we did for
> other drivers in previous cycle) and removes vermagic.h from global
> include folder.
>
> [1] https://lore.kernel.org/lkml/20200411155623.GA22175@zn.tnic
> [2] https://lore.kernel.org/lkml/20200413080452.GA3772@zn.tnic
Series applied to net-next, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-04-21 20:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-19 14:18 [PATCH net-next v2 0/4] Remove vermagic header from global include folder Leon Romanovsky
2020-04-21 12:14 ` Leon Romanovsky
2020-04-21 20:28 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).