I noticed that ixl(4) is currently enabled in GENERIC64 on powerpc64 for 12.2-BETA1. This is a great change, however currently missing is a MFC of r358698 to fix runtime on powerpc64. Could that be also pushed to stable/12 and then releng/12.2? Since ixl(4) is enabled anyway, could you also MFC (and then MFS) endianness fix? BTW, I'm not sure how relevant that is, since stable/12 uses old GCC, but there are some warnings during building: root@talos:$/usr/src/sys/modules/ixl$ make -s In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/ixl_pf.h:41, from /usr/src/sys/dev/ixl/ixl_pf_main.c:36: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here /usr/src/sys/dev/ixl/ixl_pf_main.c:50: warning: 'ixl_add_maddr' declared 'static' but never defined /usr/src/sys/dev/ixl/ixl_pf_main.c:51: warning: 'ixl_match_maddr' declared 'static' but never defined In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/ixl_pf.h:41, from /usr/src/sys/dev/ixl/ixl_pf_qmgr.h:36, from /usr/src/sys/dev/ixl/ixl_pf_qmgr.c:36: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/ixl_txrx.c:47: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/ixl_pf.h:41, from /usr/src/sys/dev/ixl/ixl_pf_i2c.c:35: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/i40e_osdep.c:38: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/if_ixl.c:35: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/ixl.h:108, from /usr/src/sys/dev/ixl/ixl_pf.h:41, from /usr/src/sys/dev/ixl/ixl_pf_iflib.c:35: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/i40e_common.c:37: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/i40e_nvm.c:35: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/i40e_adminq.c:39: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/i40e_lan_hmc.c:40: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here In file included from /usr/src/sys/dev/ixl/i40e_dcb.c:36: /usr/src/sys/dev/ixl/i40e_prototype.h:631: warning: redundant redeclaration of 'i40e_blink_phy_link_led' [-Wredundant-decls] /usr/src/sys/dev/ixl/i40e_prototype.h:102: warning: previous declaration of 'i40e_blink_phy_link_led' was here
(In reply to Piotr Kubaj from comment #0) Ok, I'll look into it. The redeclaration error will probably be fixed by MFCing r364241 as well.
A commit references this bug: Author: erj Date: Sat Sep 12 00:22:02 UTC 2020 New revision: 365647 URL: https://svnweb.freebsd.org/changeset/base/365647 Log: MFC r358698 and r364241 r358698: ixl: Add missing conversions from/to LE16 r364241: Remove redeclaration found by gcc build Together, these should allow ixl(4) to work properly on powerpc64 and prevent build warnings there. PR: 249254 Reported by: pkubaj@ MFC after: 3 days Changes: _U stable/12/ stable/12/sys/dev/ixl/i40e_prototype.h stable/12/sys/dev/ixl/ixl_pf_main.c
(In reply to Piotr Kubaj from comment #0) Did my MFC commit fix this for you?
(In reply to Eric Joyner from comment #3) No, it also needs to be MFS'd to releng/12.2.
A commit references this bug: Author: erj Date: Mon Sep 14 23:49:17 UTC 2020 New revision: 365736 URL: https://svnweb.freebsd.org/changeset/base/365736 Log: MFS r365647: ixl powerpc64 fixes PR: 249254 Approved by: re (gjb@) Changes: _U releng/12.2/ releng/12.2/sys/dev/ixl/i40e_prototype.h releng/12.2/sys/dev/ixl/ixl_pf_main.c
Now that it's in releng/12.2, it should be fixed now?
(In reply to Eric Joyner from comment #6) Yes, thank you!