From: kernel test robot <lkp@intel•com>
To: FUJITA Tomonori <fujita.tomonori@gmail•com>, netdev@vger•kernel.org
Cc: oe-kbuild-all@lists•linux.dev, andrew@lunn•ch
Subject: Re: [PATCH net-next v1 4/5] net: tn40xx: add basic Rx handling
Date: Tue, 16 Apr 2024 08:38:53 +0800 [thread overview]
Message-ID: <202404160840.Me9Gpj7a-lkp@intel.com> (raw)
In-Reply-To: <20240415104352.4685-5-fujita.tomonori@gmail.com>
Hi FUJITA,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 32affa5578f0e6b9abef3623d3976395afbd265c]
url: https://github.com/intel-lab-lkp/linux/commits/FUJITA-Tomonori/net-tn40xx-add-pci-driver-for-Tehuti-Networks-TN40xx-chips/20240415-185416
base: 32affa5578f0e6b9abef3623d3976395afbd265c
patch link: https://lore.kernel.org/r/20240415104352.4685-5-fujita.tomonori%40gmail.com
patch subject: [PATCH net-next v1 4/5] net: tn40xx: add basic Rx handling
config: microblaze-allmodconfig (https://download.01.org/0day-ci/archive/20240416/202404160840.Me9Gpj7a-lkp@intel.com/config)
compiler: microblaze-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240416/202404160840.Me9Gpj7a-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/202404160840.Me9Gpj7a-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from include/linux/printk.h:566,
from include/asm-generic/bug.h:22,
from ./arch/microblaze/include/generated/asm/bug.h:1,
from include/linux/bug.h:5,
from include/linux/thread_info.h:13,
from include/linux/sched.h:14,
from include/linux/delay.h:23,
from drivers/net/ethernet/tehuti/tn40.h:8,
from drivers/net/ethernet/tehuti/tn40.c:4:
drivers/net/ethernet/tehuti/tn40.c: In function 'bdx_rx_reuse_page':
>> drivers/net/ethernet/tehuti/tn40.c:154:20: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
154 | (void *)dm->dma);
| ^
include/linux/dynamic_debug.h:224:29: note: in definition of macro '__dynamic_func_call_cls'
224 | func(&id, ##__VA_ARGS__); \
| ^~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:277:9: note: in expansion of macro '_dynamic_func_call'
277 | _dynamic_func_call(fmt, __dynamic_netdev_dbg, \
| ^~~~~~~~~~~~~~~~~~
include/net/net_debug.h:57:9: note: in expansion of macro 'dynamic_netdev_dbg'
57 | dynamic_netdev_dbg(__dev, format, ##args); \
| ^~~~~~~~~~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c:153:9: note: in expansion of macro 'netdev_dbg'
153 | netdev_dbg(priv->ndev, "dm size %d off %d dma %p\n", dm->size, dm->off,
| ^~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c:156:67: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
156 | netdev_dbg(priv->ndev, "unmap page %p size %d\n", (void *)dm->dma, dm->size);
| ^
include/linux/dynamic_debug.h:224:29: note: in definition of macro '__dynamic_func_call_cls'
224 | func(&id, ##__VA_ARGS__); \
| ^~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:277:9: note: in expansion of macro '_dynamic_func_call'
277 | _dynamic_func_call(fmt, __dynamic_netdev_dbg, \
| ^~~~~~~~~~~~~~~~~~
include/net/net_debug.h:57:9: note: in expansion of macro 'dynamic_netdev_dbg'
57 | dynamic_netdev_dbg(__dev, format, ##args); \
| ^~~~~~~~~~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c:156:17: note: in expansion of macro 'netdev_dbg'
156 | netdev_dbg(priv->ndev, "unmap page %p size %d\n", (void *)dm->dma, dm->size);
| ^~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c: In function 'bdx_rx_alloc_buffers':
drivers/net/ethernet/tehuti/tn40.c:324:28: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
324 | (void *)dm->dma);
| ^
include/linux/dynamic_debug.h:224:29: note: in definition of macro '__dynamic_func_call_cls'
224 | func(&id, ##__VA_ARGS__); \
| ^~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:277:9: note: in expansion of macro '_dynamic_func_call'
277 | _dynamic_func_call(fmt, __dynamic_netdev_dbg, \
| ^~~~~~~~~~~~~~~~~~
include/net/net_debug.h:57:9: note: in expansion of macro 'dynamic_netdev_dbg'
57 | dynamic_netdev_dbg(__dev, format, ##args); \
| ^~~~~~~~~~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c:323:17: note: in expansion of macro 'netdev_dbg'
323 | netdev_dbg(priv->ndev, "dm size %d off %d dma %p\n", dm->size, dm->off,
| ^~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c: In function 'bdx_rx_receive':
>> drivers/net/ethernet/tehuti/tn40.c:465:26: warning: variable 'rxf_fifo' set but not used [-Wunused-but-set-variable]
465 | struct rxf_fifo *rxf_fifo;
| ^~~~~~~~
drivers/net/ethernet/tehuti/tn40.c: In function 'bdx_start_xmit':
drivers/net/ethernet/tehuti/tn40.c:923:29: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
923 | txdd->va_lo = (u32)((u64)skb);
| ^
drivers/net/ethernet/tehuti/tn40.c: In function 'bdx_tx_cleanup':
>> drivers/net/ethernet/tehuti/tn40.c:1007:48: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
1007 | netdev_dbg(priv->ndev, "pci_unmap_page 0x%llx len %d\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1008 | db->rptr->addr.dma, db->rptr->len);
| ~~~~~~~~~~~~~~~~~~
| |
| dma_addr_t {aka unsigned int}
include/linux/dynamic_debug.h:224:29: note: in definition of macro '__dynamic_func_call_cls'
224 | func(&id, ##__VA_ARGS__); \
| ^~~~~~~~~~~
include/linux/dynamic_debug.h:250:9: note: in expansion of macro '_dynamic_func_call_cls'
250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:277:9: note: in expansion of macro '_dynamic_func_call'
277 | _dynamic_func_call(fmt, __dynamic_netdev_dbg, \
| ^~~~~~~~~~~~~~~~~~
include/net/net_debug.h:57:9: note: in expansion of macro 'dynamic_netdev_dbg'
57 | dynamic_netdev_dbg(__dev, format, ##args); \
| ^~~~~~~~~~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c:1007:25: note: in expansion of macro 'netdev_dbg'
1007 | netdev_dbg(priv->ndev, "pci_unmap_page 0x%llx len %d\n",
| ^~~~~~~~~~
drivers/net/ethernet/tehuti/tn40.c:1007:69: note: format string is defined here
1007 | netdev_dbg(priv->ndev, "pci_unmap_page 0x%llx len %d\n",
| ~~~^
| |
| long long unsigned int
| %x
vim +154 drivers/net/ethernet/tehuti/tn40.c
150
151 static void bdx_rx_reuse_page(struct bdx_priv *priv, struct rx_map *dm)
152 {
153 netdev_dbg(priv->ndev, "dm size %d off %d dma %p\n", dm->size, dm->off,
> 154 (void *)dm->dma);
155 if (dm->off == 0) {
156 netdev_dbg(priv->ndev, "unmap page %p size %d\n", (void *)dm->dma, dm->size);
157 dma_unmap_page(&priv->pdev->dev, dm->dma, dm->size,
158 DMA_FROM_DEVICE);
159 }
160 }
161
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2024-04-16 0:39 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-15 10:43 [PATCH net-next v1 0/5] add ethernet driver for Tehuti Networks TN40xx chips FUJITA Tomonori
2024-04-15 10:43 ` [PATCH net-next v1 1/5] net: tn40xx: add pci " FUJITA Tomonori
2024-04-15 12:39 ` kernel test robot
2024-04-15 14:24 ` Andrew Lunn
2024-04-21 2:28 ` FUJITA Tomonori
2024-04-15 10:43 ` [PATCH net-next v1 2/5] net: tn40xx: add register defines FUJITA Tomonori
2024-04-15 10:43 ` [PATCH net-next v1 3/5] net: tn40xx: add basic Tx handling FUJITA Tomonori
2024-04-15 22:29 ` kernel test robot
2024-04-18 17:23 ` Simon Horman
2024-04-18 17:24 ` Simon Horman
2024-04-22 7:29 ` FUJITA Tomonori
2024-04-22 10:38 ` Simon Horman
2024-04-15 10:43 ` [PATCH net-next v1 4/5] net: tn40xx: add basic Rx handling FUJITA Tomonori
2024-04-16 0:38 ` kernel test robot [this message]
2024-04-15 10:43 ` [PATCH net-next v1 5/5] net: tn40xx: add PHYLIB support FUJITA Tomonori
2024-04-15 14:44 ` Andrew Lunn
2024-04-16 12:19 ` FUJITA Tomonori
2024-04-16 12:57 ` Andrew Lunn
2024-04-25 1:24 ` FUJITA Tomonori
2024-04-25 14:37 ` Andrew Lunn
2024-04-15 23:21 ` [PATCH net-next v1 0/5] add ethernet driver for Tehuti Networks TN40xx chips Florian Fainelli
2024-04-16 10:59 ` FUJITA Tomonori
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=202404160840.Me9Gpj7a-lkp@intel.com \
--to=lkp@intel$(echo .)com \
--cc=andrew@lunn$(echo .)ch \
--cc=fujita.tomonori@gmail$(echo .)com \
--cc=netdev@vger$(echo .)kernel.org \
--cc=oe-kbuild-all@lists$(echo .)linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox