2022-07-07 01:51:53

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: build failure after merge of the net-next tree

Hi all,

After merging the net-next tree, today's linux-next build (x86_64
allmodconfig) failed like this:

In file included from drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:14:
drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:15120:28: error: 'npc_mkex_default' defined but not used [-Werror=unused-variable]
15120 | static struct npc_mcam_kex npc_mkex_default = {
| ^~~~~~~~~~~~~~~~
drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:15000:30: error: 'npc_lt_defaults' defined but not used [-Werror=unused-variable]
15000 | static struct npc_lt_def_cfg npc_lt_defaults = {
| ^~~~~~~~~~~~~~~
drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:14901:31: error: 'npc_kpu_profiles' defined but not used [-Werror=unused-variable]
14901 | static struct npc_kpu_profile npc_kpu_profiles[] = {
| ^~~~~~~~~~~~~~~~
drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:483:38: error: 'ikpu_action_entries' defined but not used [-Werror=unused-variable]
483 | static struct npc_kpu_profile_action ikpu_action_entries[] = {
| ^~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

Caused by commit

c6238bc0614d ("octeontx2-af: Drop rules for NPC MCAM")

I do wonder why static structs are declared in a header file ...

I have used the net-next tree from next-20220706 for today.

--
Cheers,
Stephen Rothwell

Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2022-07-07 05:14:23

by Ratheesh Kannoth

[permalink] [raw]
Subject: RE: [EXT] linux-next: build failure after merge of the net-next tree

-----Original Message-----
From: Stephen Rothwell <[email protected]>
Sent: Thursday, July 7, 2022 6:30 AM
To: David Miller <[email protected]>; Networking <[email protected]>
Cc: Ratheesh Kannoth <[email protected]>; Linux Kernel Mailing List <[email protected]>; Linux Next Mailing List <[email protected]>
Subject: [EXT] linux-next: build failure after merge of the net-next tree

External Email

----------------------------------------------------------------------
>Hi all,

>After merging the net-next tree, today's linux-next build (x86_64
>allmodconfig) failed like this:

>In file included from drivers/net/ethernet/marvell/octeontx2/af/rvu_npc_fs.c:14:
>drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:15120:28: error: 'npc_mkex_default' defined but not used [-Werror=unused-variable]
>15120 | static struct npc_mcam_kex npc_mkex_default = {
> | ^~~~~~~~~~~~~~~~
>drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:15000:30: error: 'npc_lt_defaults' defined but not used [-Werror=unused-variable]
>15000 | static struct npc_lt_def_cfg npc_lt_defaults = {
| ^~~~~~~~~~~~~~~
>drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:14901:31: error: 'npc_kpu_profiles' defined but not used [-Werror=unused-variable]
>14901 | static struct npc_kpu_profile npc_kpu_profiles[] = {
| ^~~~~~~~~~~~~~~~
>drivers/net/ethernet/marvell/octeontx2/af/npc_profile.h:483:38: error: 'ikpu_action_entries' defined but not used [-Werror=unused-variable]
> 483 | static struct npc_kpu_profile_action ikpu_action_entries[] = {
> | ^~~~~~~~~~~~~~~~~~~
>cc1: all warnings being treated as errors

>Caused by commit

>c6238bc0614d ("octeontx2-af: Drop rules for NPC MCAM")

>I do wonder why static structs are declared in a header file ...

>I have used the net-next tree from next-20220706 for today.

I have fixed compilation errors and reposted V2 patch set. Apologies for the trouble.

--
Cheers,
Stephen Rothwell