All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Gur Stavi <gur.stavi@huawei.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC net-next v01 1/1] net: hinic3: Add a driver for Huawei 3rd gen NIC
Date: Sat, 2 Nov 2024 16:21:01 +0800	[thread overview]
Message-ID: <202411021606.s7YoVfRW-lkp@intel.com> (raw)
In-Reply-To: <ebb0fefe47c29ffed5af21d6bd39d19c2bcddd9c.1730290527.git.gur.stavi@huawei.com>

Hi Gur,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build errors:

[auto build test ERROR on b8ee7a11c75436b85fa1641aa5f970de0f8a575c]

url:    https://github.com/intel-lab-lkp/linux/commits/Gur-Stavi/net-hinic3-Add-a-driver-for-Huawei-3rd-gen-NIC/20241030-201518
base:   b8ee7a11c75436b85fa1641aa5f970de0f8a575c
patch link:    https://lore.kernel.org/r/ebb0fefe47c29ffed5af21d6bd39d19c2bcddd9c.1730290527.git.gur.stavi%40huawei.com
patch subject: [RFC net-next v01 1/1] net: hinic3: Add a driver for Huawei 3rd gen NIC
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20241102/202411021606.s7YoVfRW-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241102/202411021606.s7YoVfRW-lkp@intel.com/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 <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202411021606.s7YoVfRW-lkp@intel.com/

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

   In file included from include/linux/device.h:15,
                    from include/linux/pci.h:37,
                    from drivers/net/ethernet/huawei/hinic3/hinic3_wq.c:5:
   drivers/net/ethernet/huawei/hinic3/hinic3_wq.c: In function 'wq_init_wq_block':
>> drivers/net/ethernet/huawei/hinic3/hinic3_wq.c:34:37: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'unsigned int' [-Wformat=]
      34 |                 dev_err(hwdev->dev, "wq num_pages exceed limit: %lu\n",
         |                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/net/ethernet/huawei/hinic3/hinic3_wq.c:34:17: note: in expansion of macro 'dev_err'
      34 |                 dev_err(hwdev->dev, "wq num_pages exceed limit: %lu\n",
         |                 ^~~~~~~
   drivers/net/ethernet/huawei/hinic3/hinic3_wq.c:34:67: note: format string is defined here
      34 |                 dev_err(hwdev->dev, "wq num_pages exceed limit: %lu\n",
         |                                                                 ~~^
         |                                                                   |
         |                                                                   long unsigned int
         |                                                                 %u
--
   drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c: In function 'cmdq_set_db':
>> drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c:438:9: error: implicit declaration of function 'writeq'; did you mean 'writel'? [-Werror=implicit-function-declaration]
     438 |         writeq(*((u64 *)&db), CMDQ_DB_ADDR(db_base, prod_idx));
         |         ^~~~~~
         |         writel
   In file included from include/linux/device.h:15,
                    from drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c:6:
   drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c: In function 'create_cmdq_wq':
>> drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c:965:45: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'unsigned int' [-Wformat=]
     965 |                         dev_err(hwdev->dev, "Cmdq number of wq pages exceeds limit: %lu\n",
         |                                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c:965:25: note: in expansion of macro 'dev_err'
     965 |                         dev_err(hwdev->dev, "Cmdq number of wq pages exceeds limit: %lu\n",
         |                         ^~~~~~~
   drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c:965:87: note: format string is defined here
     965 |                         dev_err(hwdev->dev, "Cmdq number of wq pages exceeds limit: %lu\n",
         |                                                                                     ~~^
         |                                                                                       |
         |                                                                                       long unsigned int
         |                                                                                     %u
   cc1: some warnings being treated as errors
--
   In file included from drivers/net/ethernet/huawei/hinic3/hinic3_nic_dev.h:17,
                    from drivers/net/ethernet/huawei/hinic3/hinic3_nic_io.c:14:
   drivers/net/ethernet/huawei/hinic3/hinic3_nic_io.h: In function 'hinic3_write_db':
>> drivers/net/ethernet/huawei/hinic3/hinic3_nic_io.h:102:9: error: implicit declaration of function 'writeq'; did you mean 'writel'? [-Werror=implicit-function-declaration]
     102 |         writeq(*((u64 *)&db), DB_ADDR(queue, pi));
         |         ^~~~~~
         |         writel
   cc1: some warnings being treated as errors
--
   In file included from drivers/net/ethernet/huawei/hinic3/hinic3_nic_dev.h:17,
                    from drivers/net/ethernet/huawei/hinic3/hinic3_rx.c:16:
   drivers/net/ethernet/huawei/hinic3/hinic3_nic_io.h: In function 'hinic3_write_db':
>> drivers/net/ethernet/huawei/hinic3/hinic3_nic_io.h:102:9: error: implicit declaration of function 'writeq'; did you mean 'writel'? [-Werror=implicit-function-declaration]
     102 |         writeq(*((u64 *)&db), DB_ADDR(queue, pi));
         |         ^~~~~~
         |         writel
   drivers/net/ethernet/huawei/hinic3/hinic3_rx.c: In function 'rq_associate_cqes':
>> drivers/net/ethernet/huawei/hinic3/hinic3_rx.c:146:13: warning: variable 'cqe_len' set but not used [-Wunused-but-set-variable]
     146 |         int cqe_len;
         |             ^~~~~~~
   drivers/net/ethernet/huawei/hinic3/hinic3_rx.c: In function 'recv_one_pkt':
>> drivers/net/ethernet/huawei/hinic3/hinic3_rx.c:493:32: warning: variable 'nic_dev' set but not used [-Wunused-but-set-variable]
     493 |         struct hinic3_nic_dev *nic_dev;
         |                                ^~~~~~~
   cc1: some warnings being treated as errors

Kconfig warnings: (for reference only)
   WARNING: unmet direct dependencies detected for GET_FREE_REGION
   Depends on [n]: SPARSEMEM [=n]
   Selected by [m]:
   - RESOURCE_KUNIT_TEST [=m] && RUNTIME_TESTING_MENU [=y] && KUNIT [=m]


vim +438 drivers/net/ethernet/huawei/hinic3/hinic3_cmdq.c

   429	
   430	static void cmdq_set_db(struct hinic3_cmdq *cmdq,
   431				enum hinic3_cmdq_type cmdq_type, u16 prod_idx)
   432	{
   433		u8 __iomem *db_base = cmdq->hwdev->cmdqs->cmdqs_db_base;
   434		struct hinic3_cmdq_db db;
   435	
   436		cmdq_fill_db(&db, cmdq_type, prod_idx);
   437	
 > 438		writeq(*((u64 *)&db), CMDQ_DB_ADDR(db_base, prod_idx));
   439	}
   440	

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

      parent reply	other threads:[~2024-11-02  8:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-30 12:25 [RFC net-next v01 0/1] net: hinic3: Add a driver for Huawei 3rd gen NIC Gur Stavi
2024-10-30 12:25 ` [RFC net-next v01 1/1] " Gur Stavi
2024-11-01  2:35   ` Jakub Kicinski
2024-11-03 12:29     ` Gur Stavi
2024-11-03 18:54       ` Andrew Lunn
2024-11-03 20:17         ` Gur Stavi
2024-11-03 21:19           ` Andrew Lunn
2024-11-03 22:19           ` Jakub Kicinski
2024-11-02  8:00   ` kernel test robot
2024-11-02  8:21   ` kernel test robot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202411021606.s7YoVfRW-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=gur.stavi@huawei.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.