Re: [PATCH net-next v5 13/15] virtio-net: support AF_XDP zc rx
by kernel test robot
Hi Xuan,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
url: https://github.com/0day-ci/linux/commits/Xuan-Zhuo/virtio-net-support-xdp...
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git c7654495916e109f76a67fd3ae68f8fa70ab4faa
config: x86_64-randconfig-a015-20210617 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/f5f1e60139e7c38fbb4ed58d503e89bbb...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Xuan-Zhuo/virtio-net-support-xdp-socket-zero-copy/20210617-033438
git checkout f5f1e60139e7c38fbb4ed58d503e89bbb26c1464
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/net/virtio/virtio_net.c:1304:2: warning: variable 'oom' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
if (fill_recv_xsk(vi, rq, gfp))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:28: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:58:30: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/virtio/virtio_net.c:1329:10: note: uninitialized use occurs here
return !oom;
^~~
drivers/net/virtio/virtio_net.c:1304:2: note: remove the 'if' if its condition is always false
if (fill_recv_xsk(vi, rq, gfp))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:23: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
drivers/net/virtio/virtio_net.c:1297:10: note: initialize the variable 'oom' to silence this warning
bool oom;
^
= 0
1 warning generated.
vim +1304 drivers/net/virtio/virtio_net.c
1285
1286 /*
1287 * Returns false if we couldn't fill entirely (OOM).
1288 *
1289 * Normally run in the receive path, but can also be run from ndo_open
1290 * before we're receiving packets, or from refill_work which is
1291 * careful to disable receiving (using napi_disable).
1292 */
1293 static bool try_fill_recv(struct virtnet_info *vi, struct receive_queue *rq,
1294 gfp_t gfp)
1295 {
1296 int err;
1297 bool oom;
1298
1299 /* Because virtio-net does not yet support flow direct, all rx
1300 * channels must also process other non-xsk packets. If there is
1301 * no buf available from xsk temporarily, we try to allocate
1302 * memory to the channel.
1303 */
> 1304 if (fill_recv_xsk(vi, rq, gfp))
1305 goto kick;
1306
1307 do {
1308 if (vi->mergeable_rx_bufs)
1309 err = add_recvbuf_mergeable(vi, rq, gfp);
1310 else if (vi->big_packets)
1311 err = add_recvbuf_big(vi, rq, gfp);
1312 else
1313 err = add_recvbuf_small(vi, rq, gfp);
1314
1315 oom = err == -ENOMEM;
1316 if (err)
1317 break;
1318 } while (rq->vq->num_free);
1319
1320 kick:
1321 if (virtqueue_kick_prepare(rq->vq) && virtqueue_notify(rq->vq)) {
1322 unsigned long flags;
1323
1324 flags = u64_stats_update_begin_irqsave(&rq->stats.syncp);
1325 rq->stats.kicks++;
1326 u64_stats_update_end_irqrestore(&rq->stats.syncp, flags);
1327 }
1328
1329 return !oom;
1330 }
1331
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: + mm-rename-pud_page_vaddr-to-pud_pgtable-and-make-it-return-pmd_t.patch added to -mm tree
by kernel test robot
Hi,
I love your patch! Perhaps something to improve:
[auto build test WARNING on powerpc/next]
[also build test WARNING on tip/x86/mm asm-generic/master linus/master sparc/master v5.13-rc6 next-20210615]
[cannot apply to sparc-next/master]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/akpm-linux-foundation-org/mm-ren...
base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: m68k-allmodconfig (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://github.com/0day-ci/linux/commit/2ccec57c1def84dab91722c14fd5907ed...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review akpm-linux-foundation-org/mm-rename-pud_page_vaddr-to-pud_pgtable-and-make-it-return-pmd_t-patch-added-to-mm-tree/20210616-161816
git checkout 2ccec57c1def84dab91722c14fd5907ed7423426
# 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(a)intel.com>
All warnings (new ones prefixed by >>):
arch/m68k/mm/motorola.c: In function 'kernel_ptr_table':
>> arch/m68k/mm/motorola.c:265:8: warning: assignment to 'long unsigned int' from 'pmd_t *' {aka 'struct <anonymous> *'} makes integer from pointer without a cast [-Wint-conversion]
265 | pmd = pgd_page_vaddr(kernel_pg_dir[i]);
| ^
arch/m68k/mm/motorola.c: At top level:
arch/m68k/mm/motorola.c:390:13: warning: no previous prototype for 'paging_init' [-Wmissing-prototypes]
390 | void __init paging_init(void)
| ^~~~~~~~~~~
vim +265 arch/m68k/mm/motorola.c
^1da177e4c3f41 Linus Torvalds 2005-04-16 248
^1da177e4c3f41 Linus Torvalds 2005-04-16 249 static pmd_t * __init kernel_ptr_table(void)
^1da177e4c3f41 Linus Torvalds 2005-04-16 250 {
ef9285f69f0efb Peter Zijlstra 2020-01-31 251 if (!last_pmd_table) {
^1da177e4c3f41 Linus Torvalds 2005-04-16 252 unsigned long pmd, last;
^1da177e4c3f41 Linus Torvalds 2005-04-16 253 int i;
^1da177e4c3f41 Linus Torvalds 2005-04-16 254
^1da177e4c3f41 Linus Torvalds 2005-04-16 255 /* Find the last ptr table that was used in head.S and
^1da177e4c3f41 Linus Torvalds 2005-04-16 256 * reuse the remaining space in that page for further
^1da177e4c3f41 Linus Torvalds 2005-04-16 257 * ptr tables.
^1da177e4c3f41 Linus Torvalds 2005-04-16 258 */
^1da177e4c3f41 Linus Torvalds 2005-04-16 259 last = (unsigned long)kernel_pg_dir;
^1da177e4c3f41 Linus Torvalds 2005-04-16 260 for (i = 0; i < PTRS_PER_PGD; i++) {
60e50f34b13e9e Mike Rapoport 2019-12-04 261 pud_t *pud = (pud_t *)(&kernel_pg_dir[i]);
60e50f34b13e9e Mike Rapoport 2019-12-04 262
60e50f34b13e9e Mike Rapoport 2019-12-04 263 if (!pud_present(*pud))
^1da177e4c3f41 Linus Torvalds 2005-04-16 264 continue;
60e50f34b13e9e Mike Rapoport 2019-12-04 @265 pmd = pgd_page_vaddr(kernel_pg_dir[i]);
^1da177e4c3f41 Linus Torvalds 2005-04-16 266 if (pmd > last)
^1da177e4c3f41 Linus Torvalds 2005-04-16 267 last = pmd;
^1da177e4c3f41 Linus Torvalds 2005-04-16 268 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 269
ef9285f69f0efb Peter Zijlstra 2020-01-31 270 last_pmd_table = (pmd_t *)last;
^1da177e4c3f41 Linus Torvalds 2005-04-16 271 #ifdef DEBUG
ef9285f69f0efb Peter Zijlstra 2020-01-31 272 printk("kernel_ptr_init: %p\n", last_pmd_table);
^1da177e4c3f41 Linus Torvalds 2005-04-16 273 #endif
^1da177e4c3f41 Linus Torvalds 2005-04-16 274 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 275
ef9285f69f0efb Peter Zijlstra 2020-01-31 276 last_pmd_table += PTRS_PER_PMD;
41f1bf37a63ecd Geert Uytterhoeven 2020-08-26 277 if (PAGE_ALIGNED(last_pmd_table)) {
7e158826564fbb Geert Uytterhoeven 2020-08-26 278 last_pmd_table = memblock_alloc_low(PAGE_SIZE, PAGE_SIZE);
ef9285f69f0efb Peter Zijlstra 2020-01-31 279 if (!last_pmd_table)
8a7f97b902f4fb Mike Rapoport 2019-03-11 280 panic("%s: Failed to allocate %lu bytes align=%lx\n",
8a7f97b902f4fb Mike Rapoport 2019-03-11 281 __func__, PAGE_SIZE, PAGE_SIZE);
^1da177e4c3f41 Linus Torvalds 2005-04-16 282
ef9285f69f0efb Peter Zijlstra 2020-01-31 283 clear_page(last_pmd_table);
ef9285f69f0efb Peter Zijlstra 2020-01-31 284 mmu_page_ctor(last_pmd_table);
^1da177e4c3f41 Linus Torvalds 2005-04-16 285 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 286
ef9285f69f0efb Peter Zijlstra 2020-01-31 287 return last_pmd_table;
^1da177e4c3f41 Linus Torvalds 2005-04-16 288 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 289
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
[RFC PATCH] ACPI: scan: __acpi_device_add() can be static
by kernel test robot
drivers/acpi/scan.c:660:5: warning: symbol '__acpi_device_add' was not declared. Should it be static?
Reported-by: kernel test robot <lkp(a)intel.com>
Signed-off-by: kernel test robot <lkp(a)intel.com>
---
scan.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index be8b149244220..f30ab5a55a9a4 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -657,8 +657,8 @@ static int acpi_tie_acpi_dev(struct acpi_device *adev)
return 0;
}
-int __acpi_device_add(struct acpi_device *device,
- void (*release)(struct device *))
+static int __acpi_device_add(struct acpi_device *device,
+ void (*release)(struct device *))
{
struct acpi_device_bus_id *acpi_device_bus_id;
int result;
1 year, 3 months
Re: [PATCH 1/2] memory: tegra: Add missing dependencies
by kernel test robot
Hi Thierry,
I love your patch! Yet something to improve:
[auto build test ERROR on tegra/for-next]
[also build test ERROR on next-20210616]
[cannot apply to pza/reset/next tegra-drm/drm/tegra/for-next v5.13-rc6]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Thierry-Reding/memory-tegra-Fixe...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git for-next
config: parisc-randconfig-c023-20210616 (attached as .config)
compiler: hppa-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://github.com/0day-ci/linux/commit/0d0e9cbf83822694f35eca16dce8c5406...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Thierry-Reding/memory-tegra-Fixes-for-COMPILE_TEST/20210616-154340
git checkout 0d0e9cbf83822694f35eca16dce8c5406b4f464f
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=parisc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/tty/serial/earlycon.c: In function 'of_setup_earlycon':
>> drivers/tty/serial/earlycon.c:258:9: error: implicit declaration of function 'of_flat_dt_translate_address' [-Werror=implicit-function-declaration]
258 | addr = of_flat_dt_translate_address(node);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for OF_EARLY_FLATTREE
Depends on OF
Selected by
- TEGRA210_EMC_TABLE && MEMORY && TEGRA_MC && (ARCH_TEGRA_210_SOC || COMPILE_TEST
WARNING: unmet direct dependencies detected for OF_RESERVED_MEM
Depends on OF && OF_EARLY_FLATTREE
Selected by
- TEGRA210_EMC_TABLE && MEMORY && TEGRA_MC && (ARCH_TEGRA_210_SOC || COMPILE_TEST
vim +/of_flat_dt_translate_address +258 drivers/tty/serial/earlycon.c
8477614d9f7c5c Peter Hurley 2016-01-16 245
c90fe9c0394b06 Peter Hurley 2016-01-16 246 int __init of_setup_earlycon(const struct earlycon_id *match,
088da2a17619cf Peter Hurley 2016-01-16 247 unsigned long node,
4d118c9a866590 Peter Hurley 2016-01-16 248 const char *options)
b0b6abd34c1b50 Rob Herring 2014-03-27 249 {
b0b6abd34c1b50 Rob Herring 2014-03-27 250 int err;
b0b6abd34c1b50 Rob Herring 2014-03-27 251 struct uart_port *port = &early_console_dev.port;
088da2a17619cf Peter Hurley 2016-01-16 252 const __be32 *val;
088da2a17619cf Peter Hurley 2016-01-16 253 bool big_endian;
c90fe9c0394b06 Peter Hurley 2016-01-16 254 u64 addr;
b0b6abd34c1b50 Rob Herring 2014-03-27 255
e1dd3bef6d03c9 Geert Uytterhoeven 2015-11-27 256 spin_lock_init(&port->lock);
b0b6abd34c1b50 Rob Herring 2014-03-27 257 port->iotype = UPIO_MEM;
c90fe9c0394b06 Peter Hurley 2016-01-16 @258 addr = of_flat_dt_translate_address(node);
c90fe9c0394b06 Peter Hurley 2016-01-16 259 if (addr == OF_BAD_ADDR) {
c90fe9c0394b06 Peter Hurley 2016-01-16 260 pr_warn("[%s] bad address\n", match->name);
c90fe9c0394b06 Peter Hurley 2016-01-16 261 return -ENXIO;
c90fe9c0394b06 Peter Hurley 2016-01-16 262 }
b0b6abd34c1b50 Rob Herring 2014-03-27 263 port->mapbase = addr;
b0b6abd34c1b50 Rob Herring 2014-03-27 264
088da2a17619cf Peter Hurley 2016-01-16 265 val = of_get_flat_dt_prop(node, "reg-offset", NULL);
088da2a17619cf Peter Hurley 2016-01-16 266 if (val)
088da2a17619cf Peter Hurley 2016-01-16 267 port->mapbase += be32_to_cpu(*val);
1f66dd36bb1843 Greentime Hu 2018-02-13 268 port->membase = earlycon_map(port->mapbase, SZ_4K);
1f66dd36bb1843 Greentime Hu 2018-02-13 269
088da2a17619cf Peter Hurley 2016-01-16 270 val = of_get_flat_dt_prop(node, "reg-shift", NULL);
088da2a17619cf Peter Hurley 2016-01-16 271 if (val)
088da2a17619cf Peter Hurley 2016-01-16 272 port->regshift = be32_to_cpu(*val);
088da2a17619cf Peter Hurley 2016-01-16 273 big_endian = of_get_flat_dt_prop(node, "big-endian", NULL) != NULL ||
088da2a17619cf Peter Hurley 2016-01-16 274 (IS_ENABLED(CONFIG_CPU_BIG_ENDIAN) &&
088da2a17619cf Peter Hurley 2016-01-16 275 of_get_flat_dt_prop(node, "native-endian", NULL) != NULL);
088da2a17619cf Peter Hurley 2016-01-16 276 val = of_get_flat_dt_prop(node, "reg-io-width", NULL);
088da2a17619cf Peter Hurley 2016-01-16 277 if (val) {
088da2a17619cf Peter Hurley 2016-01-16 278 switch (be32_to_cpu(*val)) {
088da2a17619cf Peter Hurley 2016-01-16 279 case 1:
088da2a17619cf Peter Hurley 2016-01-16 280 port->iotype = UPIO_MEM;
088da2a17619cf Peter Hurley 2016-01-16 281 break;
088da2a17619cf Peter Hurley 2016-01-16 282 case 2:
088da2a17619cf Peter Hurley 2016-01-16 283 port->iotype = UPIO_MEM16;
088da2a17619cf Peter Hurley 2016-01-16 284 break;
088da2a17619cf Peter Hurley 2016-01-16 285 case 4:
088da2a17619cf Peter Hurley 2016-01-16 286 port->iotype = (big_endian) ? UPIO_MEM32BE : UPIO_MEM32;
088da2a17619cf Peter Hurley 2016-01-16 287 break;
088da2a17619cf Peter Hurley 2016-01-16 288 default:
088da2a17619cf Peter Hurley 2016-01-16 289 pr_warn("[%s] unsupported reg-io-width\n", match->name);
088da2a17619cf Peter Hurley 2016-01-16 290 return -EINVAL;
088da2a17619cf Peter Hurley 2016-01-16 291 }
088da2a17619cf Peter Hurley 2016-01-16 292 }
088da2a17619cf Peter Hurley 2016-01-16 293
31cb9a8575ca04 Eugeniy Paltsev 2017-08-21 294 val = of_get_flat_dt_prop(node, "current-speed", NULL);
31cb9a8575ca04 Eugeniy Paltsev 2017-08-21 295 if (val)
31cb9a8575ca04 Eugeniy Paltsev 2017-08-21 296 early_console_dev.baud = be32_to_cpu(*val);
31cb9a8575ca04 Eugeniy Paltsev 2017-08-21 297
814453adea7d08 Michal Simek 2018-04-10 298 val = of_get_flat_dt_prop(node, "clock-frequency", NULL);
814453adea7d08 Michal Simek 2018-04-10 299 if (val)
814453adea7d08 Michal Simek 2018-04-10 300 port->uartclk = be32_to_cpu(*val);
814453adea7d08 Michal Simek 2018-04-10 301
4d118c9a866590 Peter Hurley 2016-01-16 302 if (options) {
31cb9a8575ca04 Eugeniy Paltsev 2017-08-21 303 early_console_dev.baud = simple_strtoul(options, NULL, 0);
4d118c9a866590 Peter Hurley 2016-01-16 304 strlcpy(early_console_dev.options, options,
4d118c9a866590 Peter Hurley 2016-01-16 305 sizeof(early_console_dev.options));
4d118c9a866590 Peter Hurley 2016-01-16 306 }
05d961320ba624 Peter Hurley 2016-01-16 307 earlycon_init(&early_console_dev, match->name);
4d118c9a866590 Peter Hurley 2016-01-16 308 err = match->setup(&early_console_dev, options);
f28295cc8ce14b Hsin-Yi Wang 2020-09-15 309 earlycon_print_info(&early_console_dev);
b0b6abd34c1b50 Rob Herring 2014-03-27 310 if (err < 0)
b0b6abd34c1b50 Rob Herring 2014-03-27 311 return err;
b0b6abd34c1b50 Rob Herring 2014-03-27 312 if (!early_console_dev.con->write)
b0b6abd34c1b50 Rob Herring 2014-03-27 313 return -ENODEV;
b0b6abd34c1b50 Rob Herring 2014-03-27 314
b0b6abd34c1b50 Rob Herring 2014-03-27 315
b0b6abd34c1b50 Rob Herring 2014-03-27 316 register_console(early_console_dev.con);
b0b6abd34c1b50 Rob Herring 2014-03-27 317 return 0;
b0b6abd34c1b50 Rob Herring 2014-03-27 318 }
8477614d9f7c5c Peter Hurley 2016-01-16 319
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH 5/5] ACPI: scan: Fix race related to dropping dependencies
by kernel test robot
Hi "Rafael,
I love your patch! Perhaps something to improve:
[auto build test WARNING on pm/linux-next]
[also build test WARNING on next-20210616]
[cannot apply to linux/master linus/master v5.13-rc6]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Rafael-J-Wysocki/ACPI-scan-Fixes...
base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next
config: x86_64-randconfig-b001-20210615 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/6369a007980e42b3ba7bbfb9833146f18...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Rafael-J-Wysocki/ACPI-scan-Fixes-and-cleanups-related-to-dependencies-list-handling/20210617-013528
git checkout 6369a007980e42b3ba7bbfb9833146f1867c790b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/acpi/scan.c:660:5: warning: no previous prototype for function '__acpi_device_add' [-Wmissing-prototypes]
int __acpi_device_add(struct acpi_device *device,
^
drivers/acpi/scan.c:660:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int __acpi_device_add(struct acpi_device *device,
^
static
1 warning generated.
vim +/__acpi_device_add +660 drivers/acpi/scan.c
659
> 660 int __acpi_device_add(struct acpi_device *device,
661 void (*release)(struct device *))
662 {
663 struct acpi_device_bus_id *acpi_device_bus_id;
664 int result;
665
666 /*
667 * Linkage
668 * -------
669 * Link this device to its parent and siblings.
670 */
671 INIT_LIST_HEAD(&device->children);
672 INIT_LIST_HEAD(&device->node);
673 INIT_LIST_HEAD(&device->wakeup_list);
674 INIT_LIST_HEAD(&device->physical_node_list);
675 INIT_LIST_HEAD(&device->del_list);
676 mutex_init(&device->physical_node_lock);
677
678 mutex_lock(&acpi_device_lock);
679
680 acpi_device_bus_id = acpi_device_bus_id_match(acpi_device_hid(device));
681 if (acpi_device_bus_id) {
682 result = acpi_device_set_name(device, acpi_device_bus_id);
683 if (result)
684 goto err_unlock;
685 } else {
686 acpi_device_bus_id = kzalloc(sizeof(*acpi_device_bus_id),
687 GFP_KERNEL);
688 if (!acpi_device_bus_id) {
689 result = -ENOMEM;
690 goto err_unlock;
691 }
692 acpi_device_bus_id->bus_id =
693 kstrdup_const(acpi_device_hid(device), GFP_KERNEL);
694 if (!acpi_device_bus_id->bus_id) {
695 kfree(acpi_device_bus_id);
696 result = -ENOMEM;
697 goto err_unlock;
698 }
699
700 ida_init(&acpi_device_bus_id->instance_ida);
701
702 result = acpi_device_set_name(device, acpi_device_bus_id);
703 if (result) {
704 kfree_const(acpi_device_bus_id->bus_id);
705 kfree(acpi_device_bus_id);
706 goto err_unlock;
707 }
708
709 list_add_tail(&acpi_device_bus_id->node, &acpi_bus_id_list);
710 }
711
712 if (device->parent)
713 list_add_tail(&device->node, &device->parent->children);
714
715 if (device->wakeup.flags.valid)
716 list_add_tail(&device->wakeup_list, &acpi_wakeup_device_list);
717
718 mutex_unlock(&acpi_device_lock);
719
720 if (device->parent)
721 device->dev.parent = &device->parent->dev;
722
723 device->dev.bus = &acpi_bus_type;
724 device->dev.release = release;
725 result = device_add(&device->dev);
726 if (result) {
727 dev_err(&device->dev, "Error registering device\n");
728 goto err;
729 }
730
731 result = acpi_device_setup_files(device);
732 if (result)
733 pr_err("Error creating sysfs interface for device %s\n",
734 dev_name(&device->dev));
735
736 return 0;
737
738 err:
739 mutex_lock(&acpi_device_lock);
740
741 if (device->parent)
742 list_del(&device->node);
743
744 list_del(&device->wakeup_list);
745
746 err_unlock:
747 mutex_unlock(&acpi_device_lock);
748
749 acpi_detach_data(device->handle, acpi_scan_drop_device);
750
751 return result;
752 }
753
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
[drm-drm-intel:drm-intel-gt-next 7/8] drivers/gpu/drm/i915/intel_region_ttm.c:181:6: warning: no previous prototype for 'intel_region_ttm_node_free'
by kernel test robot
tree: git://anongit.freedesktop.org/drm/drm-intel drm-intel-gt-next
head: 13c2ceb6addb6b14468e09b75832c98909eed8e7
commit: d53ec322dc7de32a59bf1c2a56b93e90fc2f1c28 [7/8] drm/i915/ttm: switch over to ttm_buddy_man
config: x86_64-rhel-8.3-kselftests (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
git remote add drm-drm-intel git://anongit.freedesktop.org/drm/drm-intel
git fetch --no-tags drm-drm-intel drm-intel-gt-next
git checkout d53ec322dc7de32a59bf1c2a56b93e90fc2f1c28
# save the attached .config to linux build tree
make W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/i915/intel_region_ttm.c:181:6: warning: no previous prototype for 'intel_region_ttm_node_free' [-Wmissing-prototypes]
181 | void intel_region_ttm_node_free(struct intel_memory_region *mem,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
vim +/intel_region_ttm_node_free +181 drivers/gpu/drm/i915/intel_region_ttm.c
180
> 181 void intel_region_ttm_node_free(struct intel_memory_region *mem,
182 struct ttm_resource *res)
183 {
184 struct ttm_resource_manager *man = mem->region_private;
185
186 man->func->free(man, res);
187 }
188
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH 1/2] ASoC: sunxi: Add Allwinner H6 Digital MIC driver
by kernel test robot
Hi Ban,
I love your patch! Yet something to improve:
[auto build test ERROR on sunxi/sunxi/for-next]
[also build test ERROR on asoc/for-next pza/reset/next linus/master v5.13-rc6 next-20210616]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Ban-Tao/ASoC-sunxi-Add-Allwinner...
base: https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git sunxi/for-next
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-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://github.com/0day-ci/linux/commit/49b9fed77fea75987ffd3230684bff94e...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Ban-Tao/ASoC-sunxi-Add-Allwinner-H6-Digital-MIC-driver/20210616-140744
git checkout 49b9fed77fea75987ffd3230684bff94e1f22138
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
sound/soc/sunxi/sun50i-dmic.c: In function 'sun50i_dmic_startup':
>> sound/soc/sunxi/sun50i-dmic.c:96:60: error: 'struct snd_soc_pcm_runtime' has no member named 'cpu_dai'
96 | struct sun50i_dmic_dev *host = snd_soc_dai_get_drvdata(rtd->cpu_dai);
| ^~
vim +96 sound/soc/sunxi/sun50i-dmic.c
91
92 static int sun50i_dmic_startup(struct snd_pcm_substream *substream,
93 struct snd_soc_dai *cpu_dai)
94 {
95 struct snd_soc_pcm_runtime *rtd = substream->private_data;
> 96 struct sun50i_dmic_dev *host = snd_soc_dai_get_drvdata(rtd->cpu_dai);
97
98 /* only support capture */
99 if (substream->stream != SNDRV_PCM_STREAM_CAPTURE)
100 return -EINVAL;
101
102 regmap_write(host->regmap, SUN50I_DMIC_INT_STA,
103 SUN50I_DMIC_INT_STA_OVERRUN_IRQ_PENDING |
104 SUN50I_DMIC_INT_STA_DATA_IRQ_PENDING);
105 regmap_update_bits(host->regmap, SUN50I_DMIC_RXFIFO_CTL,
106 SUN50I_DMIC_RXFIFO_CTL_FLUSH,
107 SUN50I_DMIC_RXFIFO_CTL_FLUSH);
108 regmap_write(host->regmap, SUN50I_DMIC_CNT, SUN50I_DMIC_CNT_N);
109
110 return 0;
111 }
112
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [RFC PATCH 3/4] ultrasoc: Add ultrasoc AXI Communicator driver
by kernel test robot
Hi Qi,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on linus/master]
[also build test ERROR on v5.13-rc6 next-20210616]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Qi-Liu/Add-support-for-Ultrasoc-...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 94f0b2d4a1d0c52035aef425da5e022bd2cb1c71
config: m68k-allmodconfig (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://github.com/0day-ci/linux/commit/34aa8589f94ed20eca533ed8463184bb3...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qi-Liu/Add-support-for-Ultrasoc-Trace-Module/20210616-151905
git checkout 34aa8589f94ed20eca533ed8463184bb3194f80d
# 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(a)intel.com>
All error/warnings (new ones prefixed by >>):
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c: In function 'axi_com_enable_hw':
>> drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:42:8: error: implicit declaration of function 'readl' [-Werror=implicit-function-declaration]
42 | val = readl(drvdata->base + AXIC_US_CTL);
| ^~~~~
>> drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:44:2: error: implicit declaration of function 'writel' [-Werror=implicit-function-declaration]
44 | writel(val, drvdata->base + AXIC_US_CTL);
| ^~~~~~
In file included from include/linux/device.h:32,
from include/linux/acpi.h:15,
from drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:30:
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c: At top level:
>> include/linux/device/driver.h:263:1: warning: data definition has no type or storage class
263 | module_init(__driver##_init); \
| ^~~~~~~~~~~
include/linux/platform_device.h:257:2: note: in expansion of macro 'module_driver'
257 | module_driver(__platform_driver, platform_driver_register, \
| ^~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: note: in expansion of macro 'module_platform_driver'
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
>> include/linux/device/driver.h:263:1: error: type defaults to 'int' in declaration of 'module_init' [-Werror=implicit-int]
263 | module_init(__driver##_init); \
| ^~~~~~~~~~~
include/linux/platform_device.h:257:2: note: in expansion of macro 'module_driver'
257 | module_driver(__platform_driver, platform_driver_register, \
| ^~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: note: in expansion of macro 'module_platform_driver'
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: warning: parameter names (without types) in function declaration
In file included from include/linux/device.h:32,
from include/linux/acpi.h:15,
from drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:30:
include/linux/device/driver.h:268:1: warning: data definition has no type or storage class
268 | module_exit(__driver##_exit);
| ^~~~~~~~~~~
include/linux/platform_device.h:257:2: note: in expansion of macro 'module_driver'
257 | module_driver(__platform_driver, platform_driver_register, \
| ^~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: note: in expansion of macro 'module_platform_driver'
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
>> include/linux/device/driver.h:268:1: error: type defaults to 'int' in declaration of 'module_exit' [-Werror=implicit-int]
268 | module_exit(__driver##_exit);
| ^~~~~~~~~~~
include/linux/platform_device.h:257:2: note: in expansion of macro 'module_driver'
257 | module_driver(__platform_driver, platform_driver_register, \
| ^~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: note: in expansion of macro 'module_platform_driver'
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: warning: parameter names (without types) in function declaration
>> drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:331:20: error: expected declaration specifiers or '...' before string constant
331 | MODULE_DESCRIPTION("Ultrasoc AXI COM driver");
| ^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:332:16: error: expected declaration specifiers or '...' before string constant
332 | MODULE_LICENSE("Dual MIT/GPL");
| ^~~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:333:15: error: expected declaration specifiers or '...' before string constant
333 | MODULE_AUTHOR("Jonathan Zhou <jonathan.zhouwen(a)huawei.com>");
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:334:15: error: expected declaration specifiers or '...' before string constant
334 | MODULE_AUTHOR("Qi Liu <liuqi115(a)huawei.com>");
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/device.h:32,
from include/linux/acpi.h:15,
from drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:30:
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:24: warning: 'axi_com_driver_exit' defined but not used [-Wunused-function]
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~
include/linux/device/driver.h:264:20: note: in definition of macro 'module_driver'
264 | static void __exit __driver##_exit(void) \
| ^~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: note: in expansion of macro 'module_platform_driver'
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:24: warning: 'axi_com_driver_init' defined but not used [-Wunused-function]
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~
include/linux/device/driver.h:259:19: note: in definition of macro 'module_driver'
259 | static int __init __driver##_init(void) \
| ^~~~~~~~
drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c:329:1: note: in expansion of macro 'module_platform_driver'
329 | module_platform_driver(axi_com_driver);
| ^~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/readl +42 drivers/hwtracing/ultrasoc/ultrasoc-axi-com.c
37
38 static void axi_com_enable_hw(struct axi_com_drv_data *drvdata)
39 {
40 u32 val;
41
> 42 val = readl(drvdata->base + AXIC_US_CTL);
43 val |= AXIC_US_CTL_EN;
> 44 writel(val, drvdata->base + AXIC_US_CTL);
45
46 val = readl(drvdata->base + AXIC_DS_CTL);
47 val |= AXIC_DS_CTL_EN;
48 writel(val, drvdata->base + AXIC_DS_CTL);
49 }
50
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months
Re: [PATCH 1/2] watchdog: introduce watchdog_dev_suspend/resume
by kernel test robot
Hi Grzegorz,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on hwmon/hwmon-next]
[also build test WARNING on soc/for-next linus/master v5.13-rc6 next-20210616]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Grzegorz-Jaszczyk/introduce-watc...
base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
config: arm64-randconfig-r022-20210615 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 64720f57bea6a6bf033feef4a5751ab9c0c3b401)
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
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/71dadcad8da1862c1205b19ddf4279d49...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Grzegorz-Jaszczyk/introduce-watchdog_dev_suspend-resume/20210616-201447
git checkout 71dadcad8da1862c1205b19ddf4279d494e57545
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/watchdog/watchdog_dev.c:1232:2: warning: variable 'ret' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
if (watchdog_worker_should_ping(wd_data))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:28: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:58:30: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/watchdog/watchdog_dev.c:1236:6: note: uninitialized use occurs here
if (ret)
^~~
include/linux/compiler.h:56:47: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~
include/linux/compiler.h:58:52: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~
drivers/watchdog/watchdog_dev.c:1232:2: note: remove the 'if' if its condition is always true
if (watchdog_worker_should_ping(wd_data))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:23: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
drivers/watchdog/watchdog_dev.c:1225:9: note: initialize the variable 'ret' to silence this warning
int ret;
^
= 0
drivers/watchdog/watchdog_dev.c:1263:2: warning: variable 'ret' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
if (watchdog_worker_should_ping(wd_data))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:28: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:58:30: note: expanded from macro '__trace_if_var'
#define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/watchdog/watchdog_dev.c:1267:9: note: uninitialized use occurs here
return ret;
^~~
drivers/watchdog/watchdog_dev.c:1263:2: note: remove the 'if' if its condition is always true
if (watchdog_worker_should_ping(wd_data))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:56:23: note: expanded from macro 'if'
#define if(cond, ...) if ( __trace_if_var( !!(cond , ## __VA_ARGS__) ) )
^
drivers/watchdog/watchdog_dev.c:1253:9: note: initialize the variable 'ret' to silence this warning
int ret;
^
= 0
2 warnings generated.
vim +1232 drivers/watchdog/watchdog_dev.c
1221
1222 int watchdog_dev_suspend(struct watchdog_device *wdd)
1223 {
1224 struct watchdog_core_data *wd_data = wdd->wd_data;
1225 int ret;
1226
1227 if (!wdd->wd_data)
1228 return -ENODEV;
1229
1230 /* ping for the last time before suspend */
1231 mutex_lock(&wd_data->lock);
> 1232 if (watchdog_worker_should_ping(wd_data))
1233 ret = __watchdog_ping(wd_data->wdd);
1234 mutex_unlock(&wd_data->lock);
1235
1236 if (ret)
1237 return ret;
1238
1239 /*
1240 * make sure that watchdog worker will not kick in when the wdog is
1241 * suspended
1242 */
1243 hrtimer_cancel(&wd_data->timer);
1244 kthread_cancel_work_sync(&wd_data->work);
1245
1246 return 0;
1247 }
1248 EXPORT_SYMBOL_GPL(watchdog_dev_suspend);
1249
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 3 months