Re: [PATCH net] mlxsw: pci: Quiesce EQ tasklet and CQ NAPI before teardown

Hi Myeonghun,

kernel test robot noticed the following build errors:

[auto build test ERROR on net/main]

url:    https://github.com/intel-lab-lkp/linux/commits/Myeonghun-Pak/mlxsw-pci-Quiesce-EQ-tasklet-and-CQ-NAPI-before-teardown/20260721-142527
base:   net/main
patch link:    https://lore.kernel.org/r/20260721062105.55014-1-mhun512%40gmail.com
patch subject: [PATCH net] mlxsw: pci: Quiesce EQ tasklet and CQ NAPI before teardown
config: x86_64-rhel-9.4-bpf (https://download.01.org/0day-ci/archive/20260721/[email protected]/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260721/[email protected]/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All errors (new ones prefixed by >>):

   drivers/net/ethernet/mellanox/mlxsw/pci.c: In function 'mlxsw_pci_cq_init':
>> drivers/net/ethernet/mellanox/mlxsw/pci.c:1077:32: error: 'truea' undeclared (first use in this function); did you mean 'true'?
    1077 |         q->u.cq.napi_enabled = truea
         |                                ^~~~~
         |                                true
   drivers/net/ethernet/mellanox/mlxsw/pci.c:1077:32: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/net/ethernet/mellanox/mlxsw/pci.c:1077:37: error: expected ';' before 'mlxsw_pci_queue_doorbell_consumer_ring'
    1077 |         q->u.cq.napi_enabled = truea
         |                                     ^
         |                                     ;
    1078 |         mlxsw_pci_queue_doorbell_consumer_ring(mlxsw_pci, q);
         |         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


vim +1077 drivers/net/ethernet/mellanox/mlxsw/pci.c

  1036	
  1037	static int mlxsw_pci_cq_init(struct mlxsw_pci *mlxsw_pci, char *mbox,
  1038				     struct mlxsw_pci_queue *q)
  1039	{
  1040		enum mlxsw_pci_cq_type cq_type = mlxsw_pci_cq_type(mlxsw_pci, q);
  1041		int i;
  1042		int err;
  1043	
  1044		q->consumer_counter = 0;
  1045	
  1046		for (i = 0; i < q->count; i++) {
  1047			char *elem = mlxsw_pci_queue_elem_get(q, i);
  1048	
  1049			mlxsw_pci_cqe_owner_set(q->u.cq.v, elem, 1);
  1050		}
  1051	
  1052		if (q->u.cq.v == MLXSW_PCI_CQE_V1)
  1053			mlxsw_cmd_mbox_sw2hw_cq_cqe_ver_set(mbox,
  1054					MLXSW_CMD_MBOX_SW2HW_CQ_CQE_VER_1);
  1055		else if (q->u.cq.v == MLXSW_PCI_CQE_V2)
  1056			mlxsw_cmd_mbox_sw2hw_cq_cqe_ver_set(mbox,
  1057					MLXSW_CMD_MBOX_SW2HW_CQ_CQE_VER_2);
  1058	
  1059		mlxsw_cmd_mbox_sw2hw_cq_c_eqn_set(mbox, MLXSW_PCI_EQ_COMP_NUM);
  1060		mlxsw_cmd_mbox_sw2hw_cq_st_set(mbox, 0);
  1061		mlxsw_cmd_mbox_sw2hw_cq_log_cq_size_set(mbox, ilog2(q->count));
  1062		for (i = 0; i < MLXSW_PCI_AQ_PAGES; i++) {
  1063			dma_addr_t mapaddr = __mlxsw_pci_queue_page_get(q, i);
  1064	
  1065			mlxsw_cmd_mbox_sw2hw_cq_pa_set(mbox, i, mapaddr);
  1066		}
  1067		err = mlxsw_cmd_sw2hw_cq(mlxsw_pci->core, mbox, q->num);
  1068		if (err)
  1069			return err;
  1070		mlxsw_pci_cq_napi_setup(q, cq_type);
  1071	
  1072		err = mlxsw_pci_cq_page_pool_init(q, cq_type);
  1073		if (err)
  1074			goto err_page_pool_init;
  1075	
  1076		napi_enable(&q->u.cq.napi);
> 1077		q->u.cq.napi_enabled = truea
  1078		mlxsw_pci_queue_doorbell_consumer_ring(mlxsw_pci, q);
  1079		mlxsw_pci_queue_doorbell_arm_consumer_ring(mlxsw_pci, q);
  1080		return 0;
  1081	
  1082	err_page_pool_init:
  1083		mlxsw_pci_cq_napi_teardown(q);
  1084		return err;
  1085	}
  1086	

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki