Re: [PATCH net] net/sonic: use dma_mapping_error() for error check

From: Zhang Changzhong
Date: Wed Nov 02 2022 - 04:57:29 EST

Next message: Naresh Solanki: "[PATCH v5 0/2] mfd: max597x: Add support for max597x"
Previous message: ypodemsk: "Re: [PATCH] x86/aperfmperf: Fix arch_scale_freq_tick() on tickless systems"
In reply to: kernel test robot: "Re: [PATCH net] net/sonic: use dma_mapping_error() for error check"
Next in thread: kernel test robot: "Re: [PATCH net] net/sonic: use dma_mapping_error() for error check"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On 2022/11/2 7:04, kernel test robot wrote:
> Hi Zhang,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on net/master]
>

Sorry, my fault! I forgot to check my kernel config when building.

Thanks,
Changzhong

> url: https://github.com/intel-lab-lkp/linux/commits/Zhang-Changzhong/net-sonic-use-dma_mapping_error-for-error-check/20221031-204340
> patch link: https://lore.kernel.org/r/1667221322-8317-1-git-send-email-zhangchangzhong%40huawei.com
> patch subject: [PATCH net] net/sonic: use dma_mapping_error() for error check
> config: mips-jazz_defconfig
> compiler: mipsel-linux-gcc (GCC) 12.1.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://github.com/intel-lab-lkp/linux/commit/837273d17ba29bbc661c5c82432e83ce4198200a
> git remote add linux-review https://github.com/intel-lab-lkp/linux
> git fetch --no-tags linux-review Zhang-Changzhong/net-sonic-use-dma_mapping_error-for-error-check/20221031-204340
> git checkout 837273d17ba29bbc661c5c82432e83ce4198200a
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=mips SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp@xxxxxxxxx>
>
> All errors (new ones prefixed by >>):
>
> In file included from drivers/net/ethernet/natsemi/jazzsonic.c:228:
> drivers/net/ethernet/natsemi/sonic.c: In function 'sonic_send_packet':
> drivers/net/ethernet/natsemi/sonic.c:295:9: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
> 295 | if (dma_mapping_error(lp->device, laddr))
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:297:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
> 297 | dev_kfree_skb_any(skb);
> | ^~~~~~~~~~~~~~~~~
> drivers/net/ethernet/natsemi/sonic.c:279:23: warning: unused variable 'flags' [-Wunused-variable]
> 279 | unsigned long flags;
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:278:13: warning: unused variable 'entry' [-Wunused-variable]
> 278 | int entry;
> | ^~~~~
> In file included from include/linux/kref.h:16,
> from include/linux/mm_types.h:8,
> from include/linux/buildid.h:5,
> from include/linux/module.h:14,
> from drivers/net/ethernet/natsemi/jazzsonic.c:23:
> drivers/net/ethernet/natsemi/sonic.c: At top level:
> include/linux/spinlock.h:379:1: error: expected identifier or '(' before 'do'
> 379 | do { \
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:301:9: note: in expansion of macro 'spin_lock_irqsave'
> 301 | spin_lock_irqsave(&lp->lock, flags);
> | ^~~~~~~~~~~~~~~~~
> include/linux/spinlock.h:381:3: error: expected identifier or '(' before 'while'
> 381 | } while (0)
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:301:9: note: in expansion of macro 'spin_lock_irqsave'
> 301 | spin_lock_irqsave(&lp->lock, flags);
> | ^~~~~~~~~~~~~~~~~
> drivers/net/ethernet/natsemi/sonic.c:303:9: warning: data definition has no type or storage class
> 303 | entry = (lp->eol_tx + 1) & SONIC_TDS_MASK;
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:303:9: error: type defaults to 'int' in declaration of 'entry' [-Werror=implicit-int]
> drivers/net/ethernet/natsemi/sonic.c:303:18: error: 'lp' undeclared here (not in a function); did you mean 'up'?
> 303 | entry = (lp->eol_tx + 1) & SONIC_TDS_MASK;
> | ^~
> | up
> drivers/net/ethernet/natsemi/sonic.c:305:34: error: expected ')' before numeric constant
> 305 | sonic_tda_put(dev, entry, SONIC_TD_STATUS, 0); /* clear status */
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:306:34: error: expected ')' before numeric constant
> 306 | sonic_tda_put(dev, entry, SONIC_TD_FRAG_COUNT, 1); /* single fragment */
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:307:34: error: expected ')' before numeric constant
> 307 | sonic_tda_put(dev, entry, SONIC_TD_PKTSIZE, length); /* length of packet */
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:308:34: error: expected ')' before numeric constant
> 308 | sonic_tda_put(dev, entry, SONIC_TD_FRAG_PTR_L, laddr & 0xffff);
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:309:34: error: expected ')' before numeric constant
> 309 | sonic_tda_put(dev, entry, SONIC_TD_FRAG_PTR_H, laddr >> 16);
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:310:34: error: expected ')' before numeric constant
> 310 | sonic_tda_put(dev, entry, SONIC_TD_FRAG_SIZE, length);
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:311:34: error: expected ')' before numeric constant
> 311 | sonic_tda_put(dev, entry, SONIC_TD_LINK,
> | ^
> | )
> drivers/net/ethernet/natsemi/sonic.c:314:30: error: expected ')' before '->' token
> 314 | sonic_tda_put(dev, lp->eol_tx, SONIC_TD_LINK, ~SONIC_EOL &
> | ^~
> | )
> In file included from include/linux/skbuff.h:45,
> from include/net/net_namespace.h:43,
> from include/linux/netdevice.h:38,
> from drivers/net/ethernet/natsemi/jazzsonic.c:33:
> include/net/net_debug.h:123:2: error: expected identifier or '(' before '{' token
> 123 | ({ \
> | ^
> drivers/net/ethernet/natsemi/sonic.c:317:9: note: in expansion of macro 'netif_dbg'
> 317 | netif_dbg(lp, tx_queued, dev, "%s: issuing Tx command\n", __func__);
> | ^~~~~~~~~
>>> drivers/net/ethernet/natsemi/jazzsonic.c:59:1: error: expected identifier or '(' before 'do'
> 59 | do { \
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:319:9: note: in expansion of macro 'SONIC_WRITE'
> 319 | SONIC_WRITE(SONIC_CMD, SONIC_CR_TXP);
> | ^~~~~~~~~~~
>>> drivers/net/ethernet/natsemi/jazzsonic.c:61:3: error: expected identifier or '(' before 'while'
> 61 | } while (0)
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:319:9: note: in expansion of macro 'SONIC_WRITE'
> 319 | SONIC_WRITE(SONIC_CMD, SONIC_CR_TXP);
> | ^~~~~~~~~~~
> drivers/net/ethernet/natsemi/sonic.c:321:11: error: expected '=', ',', ';', 'asm' or '__attribute__' before '->' token
> 321 | lp->tx_len[entry] = length;
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:322:11: error: expected '=', ',', ';', 'asm' or '__attribute__' before '->' token
> 322 | lp->tx_laddr[entry] = laddr;
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:323:11: error: expected '=', ',', ';', 'asm' or '__attribute__' before '->' token
> 323 | lp->tx_skb[entry] = skb;
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:325:11: error: expected '=', ',', ';', 'asm' or '__attribute__' before '->' token
> 325 | lp->eol_tx = entry;
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:327:9: warning: data definition has no type or storage class
> 327 | entry = (entry + 1) & SONIC_TDS_MASK;
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:327:9: error: type defaults to 'int' in declaration of 'entry' [-Werror=implicit-int]
> drivers/net/ethernet/natsemi/sonic.c:327:9: error: redefinition of 'entry'
> drivers/net/ethernet/natsemi/sonic.c:303:9: note: previous definition of 'entry' with type 'int'
> 303 | entry = (lp->eol_tx + 1) & SONIC_TDS_MASK;
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:327:17: error: initializer element is not constant
> 327 | entry = (entry + 1) & SONIC_TDS_MASK;
> | ^
> drivers/net/ethernet/natsemi/sonic.c:328:9: error: expected identifier or '(' before 'if'
> 328 | if (lp->tx_skb[entry]) {
> | ^~
> drivers/net/ethernet/natsemi/sonic.c:335:32: error: expected declaration specifiers or '...' before '&' token
> 335 | spin_unlock_irqrestore(&lp->lock, flags);
> | ^
> drivers/net/ethernet/natsemi/sonic.c:335:43: error: unknown type name 'flags'
> 335 | spin_unlock_irqrestore(&lp->lock, flags);
> | ^~~~~
> drivers/net/ethernet/natsemi/sonic.c:337:9: error: expected identifier or '(' before 'return'
> 337 | return NETDEV_TX_OK;
> | ^~~~~~
> drivers/net/ethernet/natsemi/sonic.c:338:1: error: expected identifier or '(' before '}' token
> 338 | }
> | ^
> cc1: some warnings being treated as errors
>
>
> vim +59 drivers/net/ethernet/natsemi/jazzsonic.c
>
> ^1da177e4c3f41 drivers/net/jazzsonic.c Linus Torvalds 2005-04-16 57
> ^1da177e4c3f41 drivers/net/jazzsonic.c Linus Torvalds 2005-04-16 58 #define SONIC_WRITE(reg,val) \
> ^1da177e4c3f41 drivers/net/jazzsonic.c Linus Torvalds 2005-04-16 @59 do { \
> efcce839360fb3 drivers/net/jazzsonic.c Finn Thain 2005-08-20 60 *((volatile unsigned int *)dev->base_addr+(reg)) = (val); \
> ^1da177e4c3f41 drivers/net/jazzsonic.c Linus Torvalds 2005-04-16 @61 } while (0)
> ^1da177e4c3f41 drivers/net/jazzsonic.c Linus Torvalds 2005-04-16 62
>

Next message: Naresh Solanki: "[PATCH v5 0/2] mfd: max597x: Add support for max597x"
Previous message: ypodemsk: "Re: [PATCH] x86/aperfmperf: Fix arch_scale_freq_tick() on tickless systems"
In reply to: kernel test robot: "Re: [PATCH net] net/sonic: use dma_mapping_error() for error check"
Next in thread: kernel test robot: "Re: [PATCH net] net/sonic: use dma_mapping_error() for error check"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]