include/linux/avf/virtchnl.h:833:33: error: enumerator value for 'virtchnl_static_assert_virtchnl_proto_hdrs' is not an integer constant

From: kernel test robot
Date: Tue May 18 2021 - 15:41:23 EST

Next message: Vincent Knecht: "Re: Testing wanted for Linux NFC subsystem"
Previous message: Fabien Parent: "[PATCH 2/2] soc: mediatek: pm-domains: Add support for MT8365"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Qi,

FYI, the error/warning still remains.

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 8ac91e6c6033ebc12c5c1e4aa171b81a662bd70f
commit: 1f7ea1cd6a3748427512ccc9582e18cd9efea966 ice: Enable FDIR Configure for AVF
date: 8 weeks ago
config: m68k-randconfig-r025-20210519 (attached as .config)
compiler: m68k-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1f7ea1cd6a3748427512ccc9582e18cd9efea966
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 1f7ea1cd6a3748427512ccc9582e18cd9efea966
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All error/warnings (new ones prefixed by >>):

In file included from drivers/net/ethernet/intel/i40e/i40e_prototype.h:9,
from drivers/net/ethernet/intel/i40e/i40e.h:41,
from drivers/net/ethernet/intel/i40e/i40e_main.c:11:
>> include/linux/avf/virtchnl.h:150:36: warning: division by zero [-Wdiv-by-zero]
150 | { virtchnl_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
| ^
include/linux/avf/virtchnl.h:833:1: note: in expansion of macro 'VIRTCHNL_CHECK_STRUCT_LEN'
833 | VIRTCHNL_CHECK_STRUCT_LEN(2312, virtchnl_proto_hdrs);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/avf/virtchnl.h:833:33: error: enumerator value for 'virtchnl_static_assert_virtchnl_proto_hdrs' is not an integer constant
833 | VIRTCHNL_CHECK_STRUCT_LEN(2312, virtchnl_proto_hdrs);
| ^~~~~~~~~~~~~~~~~~~
include/linux/avf/virtchnl.h:150:53: note: in definition of macro 'VIRTCHNL_CHECK_STRUCT_LEN'
150 | { virtchnl_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
| ^
>> include/linux/avf/virtchnl.h:150:36: warning: division by zero [-Wdiv-by-zero]
150 | { virtchnl_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
| ^
include/linux/avf/virtchnl.h:874:1: note: in expansion of macro 'VIRTCHNL_CHECK_STRUCT_LEN'
874 | VIRTCHNL_CHECK_STRUCT_LEN(2604, virtchnl_fdir_rule);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/avf/virtchnl.h:874:33: error: enumerator value for 'virtchnl_static_assert_virtchnl_fdir_rule' is not an integer constant
874 | VIRTCHNL_CHECK_STRUCT_LEN(2604, virtchnl_fdir_rule);
| ^~~~~~~~~~~~~~~~~~
include/linux/avf/virtchnl.h:150:53: note: in definition of macro 'VIRTCHNL_CHECK_STRUCT_LEN'
150 | { virtchnl_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
| ^
>> include/linux/avf/virtchnl.h:150:36: warning: division by zero [-Wdiv-by-zero]
150 | { virtchnl_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
| ^
include/linux/avf/virtchnl.h:928:1: note: in expansion of macro 'VIRTCHNL_CHECK_STRUCT_LEN'
928 | VIRTCHNL_CHECK_STRUCT_LEN(2616, virtchnl_fdir_add);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
>> include/linux/avf/virtchnl.h:928:33: error: enumerator value for 'virtchnl_static_assert_virtchnl_fdir_add' is not an integer constant
928 | VIRTCHNL_CHECK_STRUCT_LEN(2616, virtchnl_fdir_add);
| ^~~~~~~~~~~~~~~~~
include/linux/avf/virtchnl.h:150:53: note: in definition of macro 'VIRTCHNL_CHECK_STRUCT_LEN'
150 | { virtchnl_static_assert_##X = (n)/((sizeof(struct X) == (n)) ? 1 : 0) }
| ^

vim +/virtchnl_static_assert_virtchnl_proto_hdrs +833 include/linux/avf/virtchnl.h

832
> 833 VIRTCHNL_CHECK_STRUCT_LEN(2312, virtchnl_proto_hdrs);
834
835 /* action configuration for FDIR */
836 struct virtchnl_filter_action {
837 enum virtchnl_action type;
838 union {
839 /* used for queue and qgroup action */
840 struct {
841 u16 index;
842 u8 region;
843 } queue;
844 /* used for count action */
845 struct {
846 /* share counter ID with other flow rules */
847 u8 shared;
848 u32 id; /* counter ID */
849 } count;
850 /* used for mark action */
851 u32 mark_id;
852 u8 reserve[32];
853 } act_conf;
854 };
855
856 VIRTCHNL_CHECK_STRUCT_LEN(36, virtchnl_filter_action);
857
858 #define VIRTCHNL_MAX_NUM_ACTIONS 8
859
860 struct virtchnl_filter_action_set {
861 /* action number must be less then VIRTCHNL_MAX_NUM_ACTIONS */
862 int count;
863 struct virtchnl_filter_action actions[VIRTCHNL_MAX_NUM_ACTIONS];
864 };
865
866 VIRTCHNL_CHECK_STRUCT_LEN(292, virtchnl_filter_action_set);
867
868 /* pattern and action for FDIR rule */
869 struct virtchnl_fdir_rule {
870 struct virtchnl_proto_hdrs proto_hdrs;
871 struct virtchnl_filter_action_set action_set;
872 };
873
> 874 VIRTCHNL_CHECK_STRUCT_LEN(2604, virtchnl_fdir_rule);
875
876 /* Status returned to VF after VF requests FDIR commands
877 * VIRTCHNL_FDIR_SUCCESS
878 * VF FDIR related request is successfully done by PF
879 * The request can be OP_ADD/DEL.
880 *
881 * VIRTCHNL_FDIR_FAILURE_RULE_NORESOURCE
882 * OP_ADD_FDIR_FILTER request is failed due to no Hardware resource.
883 *
884 * VIRTCHNL_FDIR_FAILURE_RULE_EXIST
885 * OP_ADD_FDIR_FILTER request is failed due to the rule is already existed.
886 *
887 * VIRTCHNL_FDIR_FAILURE_RULE_CONFLICT
888 * OP_ADD_FDIR_FILTER request is failed due to conflict with existing rule.
889 *
890 * VIRTCHNL_FDIR_FAILURE_RULE_NONEXIST
891 * OP_DEL_FDIR_FILTER request is failed due to this rule doesn't exist.
892 *
893 * VIRTCHNL_FDIR_FAILURE_RULE_INVALID
894 * OP_ADD_FDIR_FILTER request is failed due to parameters validation
895 * or HW doesn't support.
896 *
897 * VIRTCHNL_FDIR_FAILURE_RULE_TIMEOUT
898 * OP_ADD/DEL_FDIR_FILTER request is failed due to timing out
899 * for programming.
900 */
901 enum virtchnl_fdir_prgm_status {
902 VIRTCHNL_FDIR_SUCCESS = 0,
903 VIRTCHNL_FDIR_FAILURE_RULE_NORESOURCE,
904 VIRTCHNL_FDIR_FAILURE_RULE_EXIST,
905 VIRTCHNL_FDIR_FAILURE_RULE_CONFLICT,
906 VIRTCHNL_FDIR_FAILURE_RULE_NONEXIST,
907 VIRTCHNL_FDIR_FAILURE_RULE_INVALID,
908 VIRTCHNL_FDIR_FAILURE_RULE_TIMEOUT,
909 };
910
911 /* VIRTCHNL_OP_ADD_FDIR_FILTER
912 * VF sends this request to PF by filling out vsi_id,
913 * validate_only and rule_cfg. PF will return flow_id
914 * if the request is successfully done and return add_status to VF.
915 */
916 struct virtchnl_fdir_add {
917 u16 vsi_id; /* INPUT */
918 /*
919 * 1 for validating a fdir rule, 0 for creating a fdir rule.
920 * Validate and create share one ops: VIRTCHNL_OP_ADD_FDIR_FILTER.
921 */
922 u16 validate_only; /* INPUT */
923 u32 flow_id; /* OUTPUT */
924 struct virtchnl_fdir_rule rule_cfg; /* INPUT */
925 enum virtchnl_fdir_prgm_status status; /* OUTPUT */
926 };
927
> 928 VIRTCHNL_CHECK_STRUCT_LEN(2616, virtchnl_fdir_add);
929

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

Next message: Vincent Knecht: "Re: Testing wanted for Linux NFC subsystem"
Previous message: Fabien Parent: "[PATCH 2/2] soc: mediatek: pm-domains: Add support for MT8365"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]