Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion drivers/8.3/source/atlantic-module-alt

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why #if 0 ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is in the commit description:

c368c4defa94 ("net: create netdev->dev_addr assignment helpers") in
v4.19.291 made the definition of eth_hw_addr_set redundant and leads to
build errors like:

  In file included from /home/builder/rpmbuild/BUILD/atlantic-module-alt-2.5.12/aq_common.h:22:0,
		   from /home/builder/rpmbuild/BUILD/atlantic-module-alt-2.5.12/aq_nic.h:13,
		   from /home/builder/rpmbuild/BUILD/atlantic-module-alt-2.5.12/aq_nic.c:24:
  /home/builder/rpmbuild/BUILD/atlantic-module-alt-2.5.12/aq_compat.h:306:20: error: redefinition of 'eth_hw_addr_set'
   static inline void eth_hw_addr_set(struct net_device *dev, const u8 *addr)
		      ^
  In file included from /home/builder/rpmbuild/BUILD/atlantic-module-alt-2.5.12/aq_nic.c:12:0:
  ./include/linux/etherdevice.h:301:20: note: previous definition of 'eth_hw_addr_set' was here
   static inline void eth_hw_addr_set(struct net_device *dev, const u8 *addr)

Sadly we can't just add another linux version check to enforce the
definition in the header file when < 4.19.291 because the SUBLEVEL version
is always zero on our XS-derived kernels, and the SUBLEVEL has been moved
to an non-exported STABLE_SUBLEVEL variable only visible from the top-level
linux kernel Makefile.

Did you miss it or is there anything not clear?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yes, sorry, I missed the commit, didn't see it.
Thanks it makes sense now.
It's really too bad we do the .0 thing with the sublevel value :(
Do you know why we/XS do this?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is to have only one kernel installed and accompanying modules in /lib/modules/4.19.0+/ as opposed to having to install kernel modules in multiple directories, given lack of dkms support.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah yes, and since kABI is supposed to be stable, you don't need a set of module per kernel installed.
Just one set of modules is enough

2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-e1000e-alt
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-i40e
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-i40e-alt
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-ice
Submodule intel-ice updated 1 files
+1 −1 src/kcompat_gcc.h
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-ice-alt
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-igb-alt
Submodule intel-igb-alt updated 1 files
+1 −1 src/kcompat.h
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-ixgbe
Submodule intel-ixgbe updated 1 files
+1 −1 src/kcompat_gcc.h
2 changes: 1 addition & 1 deletion drivers/8.3/source/intel-ixgbe-alt
Submodule intel-ixgbe-alt updated 1 files
+2 −1 src/kcompat.h
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/aqc111u-module
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/avago-megaraid-sas
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/avago-mpt3sas
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/broadcom-bnxt-en
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/chelsio-cxgb4
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/cisco-enic
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/cisco-enic-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/cisco-fnic
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/cisco-fnic-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/coretemp-module-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/dell-rbu-module
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/emulex-lpfc
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/emulex-lpfc-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/intel-e1000e
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/intel-fm10k
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/intel-igb
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/intel-igc
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/mellanox-mlnxen
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/mellanox-mlnxen-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/microsemi-aacraid
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/microsemi-smartpqi
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/mlx4-modules-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/mpi3mr-module
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/qlogic-fastlinq
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/qlogic-fastlinq-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/qlogic-qla2xxx
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/qlogic-qla2xxx-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/r8125-module
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/r8169-module-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/sfc-module-alt
2 changes: 1 addition & 1 deletion drivers/8.3/srpm/tg3-module-alt
Loading
Loading