[linux-next:master 2190/5053] drivers/tty/serial/pch_uart.c:1815:9: error: use of undeclared identifier 'port_regs_ops'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: fa903833ae344e4f4d798a8b85ba3ef0c5ce96c9
commit: 4dec5f1af694526db760dfbb47211236e556e27c [2190/5053] tty: serial: pch_uart.c: remove debugfs dentry pointer
config: powerpc64-randconfig-r024-20210317 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8ef111222a3dd12a9175f69c3bff598c46e8bdf7)
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 powerpc64 cross compiling tool for clang build
# apt-get install binutils-powerpc64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commi...
git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
git fetch --no-tags linux-next master
git checkout 4dec5f1af694526db760dfbb47211236e556e27c
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc64
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/pch_uart.c:1815:9: error: use of undeclared identifier 'port_regs_ops'
&port_regs_ops);
^
1 error generated.
vim +/port_regs_ops +1815 drivers/tty/serial/pch_uart.c
1725
1726 static struct eg20t_port *pch_uart_init_port(struct pci_dev *pdev,
1727 const struct pci_device_id *id)
1728 {
1729 struct eg20t_port *priv;
1730 int ret;
1731 unsigned int iobase;
1732 unsigned int mapbase;
1733 unsigned char *rxbuf;
1734 int fifosize;
1735 int port_type;
1736 struct pch_uart_driver_data *board;
1737 char name[32];
1738
1739 board = &drv_dat[id->driver_data];
1740 port_type = board->port_type;
1741
1742 priv = kzalloc(sizeof(struct eg20t_port), GFP_KERNEL);
1743 if (priv == NULL)
1744 goto init_port_alloc_err;
1745
1746 rxbuf = (unsigned char *)__get_free_page(GFP_KERNEL);
1747 if (!rxbuf)
1748 goto init_port_free_txbuf;
1749
1750 switch (port_type) {
1751 case PORT_PCH_8LINE:
1752 fifosize = 256; /* EG20T/ML7213: UART0 */
1753 break;
1754 case PORT_PCH_2LINE:
1755 fifosize = 64; /* EG20T:UART1~3 ML7213: UART1~2*/
1756 break;
1757 default:
1758 dev_err(&pdev->dev, "Invalid Port Type(=%d)\n", port_type);
1759 goto init_port_hal_free;
1760 }
1761
1762 pci_enable_msi(pdev);
1763 pci_set_master(pdev);
1764
1765 spin_lock_init(&priv->lock);
1766
1767 iobase = pci_resource_start(pdev, 0);
1768 mapbase = pci_resource_start(pdev, 1);
1769 priv->mapbase = mapbase;
1770 priv->iobase = iobase;
1771 priv->pdev = pdev;
1772 priv->tx_empty = 1;
1773 priv->rxbuf.buf = rxbuf;
1774 priv->rxbuf.size = PAGE_SIZE;
1775
1776 priv->fifo_size = fifosize;
1777 priv->uartclk = pch_uart_get_uartclk();
1778 priv->port_type = port_type;
1779 priv->port.dev = &pdev->dev;
1780 priv->port.iobase = iobase;
1781 priv->port.membase = NULL;
1782 priv->port.mapbase = mapbase;
1783 priv->port.irq = pdev->irq;
1784 priv->port.iotype = UPIO_PORT;
1785 priv->port.ops = &pch_uart_ops;
1786 priv->port.flags = UPF_BOOT_AUTOCONF;
1787 priv->port.fifosize = fifosize;
1788 priv->port.line = board->line_no;
1789 priv->port.has_sysrq = IS_ENABLED(CONFIG_SERIAL_PCH_UART_CONSOLE);
1790 priv->trigger = PCH_UART_HAL_TRIGGER_M;
1791
1792 snprintf(priv->irq_name, IRQ_NAME_SIZE,
1793 KBUILD_MODNAME ":" PCH_UART_DRIVER_DEVICE "%d",
1794 priv->port.line);
1795
1796 spin_lock_init(&priv->port.lock);
1797
1798 pci_set_drvdata(pdev, priv);
1799 priv->trigger_level = 1;
1800 priv->fcr = 0;
1801
1802 if (pdev->dev.of_node)
1803 of_property_read_u32(pdev->dev.of_node, "clock-frequency"
1804 , &user_uartclk);
1805
1806 #ifdef CONFIG_SERIAL_PCH_UART_CONSOLE
1807 pch_uart_ports[board->line_no] = priv;
1808 #endif
1809 ret = uart_add_one_port(&pch_uart_driver, &priv->port);
1810 if (ret < 0)
1811 goto init_port_hal_free;
1812
1813 snprintf(name, sizeof(name), "uart%d_regs", priv->port.line);
1814 debugfs_create_file(name, S_IFREG | S_IRUGO, NULL, priv,
> 1815 &port_regs_ops);
1816
1817 return priv;
1818
1819 init_port_hal_free:
1820 #ifdef CONFIG_SERIAL_PCH_UART_CONSOLE
1821 pch_uart_ports[board->line_no] = NULL;
1822 #endif
1823 free_page((unsigned long)rxbuf);
1824 init_port_free_txbuf:
1825 kfree(priv);
1826 init_port_alloc_err:
1827
1828 return NULL;
1829 }
1830
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 6 months
Re: [PATCH v2] Increase page and bit waitqueue hash size
by kernel test robot
Hi Nicholas,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linux/master]
[also build test WARNING on linus/master hnaz-linux-mm/master v5.12-rc3 next-20210317]
[cannot apply to tip/sched/core]
[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/Nicholas-Piggin/Increase-page-an...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a74e6a014c9d4d4161061f770c9b4f98372ac778
config: x86_64-randconfig-a015-20210317 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8ef111222a3dd12a9175f69c3bff598c46e8bdf7)
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/d741b0903849631440ae34fb070178e97...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Nicholas-Piggin/Increase-page-and-bit-waitqueue-hash-size/20210317-155619
git checkout d741b0903849631440ae34fb070178e9743c6928
# 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 >>):
>> mm/filemap.c:997:42: warning: variable length arrays are a C99 feature [-Wvla-extension]
static wait_queue_head_t page_wait_table[PAGE_WAIT_TABLE_SIZE] __cacheline_aligned;
^~~~~~~~~~~~~~~~~~~~
mm/filemap.c:994:30: note: expanded from macro 'PAGE_WAIT_TABLE_SIZE'
#define PAGE_WAIT_TABLE_SIZE (1 << page_wait_table_bits)
^~~~~~~~~~~~~~~~~~~~~~~~~~~
mm/filemap.c:1018:8: error: passing 'const unsigned int *' to parameter of type 'unsigned int *' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers]
&page_wait_table_bits,
^~~~~~~~~~~~~~~~~~~~~
include/linux/memblock.h:579:24: note: passing argument to parameter '_hash_shift' here
unsigned int *_hash_shift,
^
mm/filemap.c:1013:19: error: array type 'wait_queue_head_t [16]' is not assignable
page_wait_table = alloc_large_system_hash("page waitqueue hash",
~~~~~~~~~~~~~~~ ^
1 warning and 2 errors generated.
--
>> kernel/sched/wait_bit.c:11:41: warning: variable length arrays are a C99 feature [-Wvla-extension]
static wait_queue_head_t bit_wait_table[BIT_WAIT_TABLE_SIZE] __cacheline_aligned;
^~~~~~~~~~~~~~~~~~~
kernel/sched/wait_bit.c:8:29: note: expanded from macro 'BIT_WAIT_TABLE_SIZE'
#define BIT_WAIT_TABLE_SIZE (1 << bit_wait_table_bits)
^~~~~~~~~~~~~~~~~~~~~~~~~~
kernel/sched/wait_bit.c:260:8: error: passing 'const unsigned int *' to parameter of type 'unsigned int *' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers]
&bit_wait_table_bits,
^~~~~~~~~~~~~~~~~~~~
include/linux/memblock.h:579:24: note: passing argument to parameter '_hash_shift' here
unsigned int *_hash_shift,
^
kernel/sched/wait_bit.c:255:18: error: array type 'wait_queue_head_t [8]' is not assignable
bit_wait_table = alloc_large_system_hash("bit waitqueue hash",
~~~~~~~~~~~~~~ ^
1 warning and 2 errors generated.
vim +997 mm/filemap.c
44110fe385af23 Paul Jackson 2006-03-24 983
^1da177e4c3f41 Linus Torvalds 2005-04-16 984 /*
^1da177e4c3f41 Linus Torvalds 2005-04-16 985 * In order to wait for pages to become available there must be
^1da177e4c3f41 Linus Torvalds 2005-04-16 986 * waitqueues associated with pages. By using a hash table of
^1da177e4c3f41 Linus Torvalds 2005-04-16 987 * waitqueues where the bucket discipline is to maintain all
^1da177e4c3f41 Linus Torvalds 2005-04-16 988 * waiters on the same queue and wake all when any of the pages
^1da177e4c3f41 Linus Torvalds 2005-04-16 989 * become available, and for the woken contexts to check to be
^1da177e4c3f41 Linus Torvalds 2005-04-16 990 * sure the appropriate page became available, this saves space
^1da177e4c3f41 Linus Torvalds 2005-04-16 991 * at a cost of "thundering herd" phenomena during rare hash
^1da177e4c3f41 Linus Torvalds 2005-04-16 992 * collisions.
^1da177e4c3f41 Linus Torvalds 2005-04-16 993 */
d741b090384963 Nicholas Piggin 2021-03-17 994 #define PAGE_WAIT_TABLE_SIZE (1 << page_wait_table_bits)
d741b090384963 Nicholas Piggin 2021-03-17 995 #if CONFIG_BASE_SMALL
d741b090384963 Nicholas Piggin 2021-03-17 996 static const unsigned int page_wait_table_bits = 4;
62906027091f1d Nicholas Piggin 2016-12-25 @997 static wait_queue_head_t page_wait_table[PAGE_WAIT_TABLE_SIZE] __cacheline_aligned;
d741b090384963 Nicholas Piggin 2021-03-17 998 #else
d741b090384963 Nicholas Piggin 2021-03-17 999 static unsigned int page_wait_table_bits __ro_after_init;
d741b090384963 Nicholas Piggin 2021-03-17 1000 static wait_queue_head_t *page_wait_table __ro_after_init;
d741b090384963 Nicholas Piggin 2021-03-17 1001 #endif
62906027091f1d Nicholas Piggin 2016-12-25 1002
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 6 months
Re: [PATCH v2] Increase page and bit waitqueue hash size
by kernel test robot
Hi Nicholas,
I love your patch! Yet something to improve:
[auto build test ERROR on linux/master]
[also build test ERROR on tip/sched/core linus/master v5.12-rc3 next-20210317]
[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/Nicholas-Piggin/Increase-page-an...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a74e6a014c9d4d4161061f770c9b4f98372ac778
config: nds32-randconfig-r016-20210317 (attached as .config)
compiler: nds32le-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/d741b0903849631440ae34fb070178e97...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Nicholas-Piggin/Increase-page-and-bit-waitqueue-hash-size/20210317-155619
git checkout d741b0903849631440ae34fb070178e9743c6928
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=nds32
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 >>):
>> mm/filemap.c:997:26: error: variably modified 'page_wait_table' at file scope
997 | static wait_queue_head_t page_wait_table[PAGE_WAIT_TABLE_SIZE] __cacheline_aligned;
| ^~~~~~~~~~~~~~~
mm/filemap.c: In function 'pagecache_init':
>> mm/filemap.c:1018:8: warning: passing argument 6 of 'alloc_large_system_hash' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
1018 | &page_wait_table_bits,
| ^~~~~~~~~~~~~~~~~~~~~
In file included from mm/filemap.c:37:
include/linux/memblock.h:574:14: note: expected 'unsigned int *' but argument is of type 'const unsigned int *'
574 | extern void *alloc_large_system_hash(const char *tablename,
| ^~~~~~~~~~~~~~~~~~~~~~~
>> mm/filemap.c:1013:19: error: assignment to expression with array type
1013 | page_wait_table = alloc_large_system_hash("page waitqueue hash",
| ^
--
>> kernel/sched/wait_bit.c:11:26: error: variably modified 'bit_wait_table' at file scope
11 | static wait_queue_head_t bit_wait_table[BIT_WAIT_TABLE_SIZE] __cacheline_aligned;
| ^~~~~~~~~~~~~~
kernel/sched/wait_bit.c: In function 'wait_bit_init':
>> kernel/sched/wait_bit.c:260:8: warning: passing argument 6 of 'alloc_large_system_hash' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
260 | &bit_wait_table_bits,
| ^~~~~~~~~~~~~~~~~~~~
In file included from kernel/sched/wait_bit.c:5:
include/linux/memblock.h:574:14: note: expected 'unsigned int *' but argument is of type 'const unsigned int *'
574 | extern void *alloc_large_system_hash(const char *tablename,
| ^~~~~~~~~~~~~~~~~~~~~~~
>> kernel/sched/wait_bit.c:255:18: error: assignment to expression with array type
255 | bit_wait_table = alloc_large_system_hash("bit waitqueue hash",
| ^
vim +/page_wait_table +997 mm/filemap.c
44110fe385af23 Paul Jackson 2006-03-24 983
^1da177e4c3f41 Linus Torvalds 2005-04-16 984 /*
^1da177e4c3f41 Linus Torvalds 2005-04-16 985 * In order to wait for pages to become available there must be
^1da177e4c3f41 Linus Torvalds 2005-04-16 986 * waitqueues associated with pages. By using a hash table of
^1da177e4c3f41 Linus Torvalds 2005-04-16 987 * waitqueues where the bucket discipline is to maintain all
^1da177e4c3f41 Linus Torvalds 2005-04-16 988 * waiters on the same queue and wake all when any of the pages
^1da177e4c3f41 Linus Torvalds 2005-04-16 989 * become available, and for the woken contexts to check to be
^1da177e4c3f41 Linus Torvalds 2005-04-16 990 * sure the appropriate page became available, this saves space
^1da177e4c3f41 Linus Torvalds 2005-04-16 991 * at a cost of "thundering herd" phenomena during rare hash
^1da177e4c3f41 Linus Torvalds 2005-04-16 992 * collisions.
^1da177e4c3f41 Linus Torvalds 2005-04-16 993 */
d741b090384963 Nicholas Piggin 2021-03-17 994 #define PAGE_WAIT_TABLE_SIZE (1 << page_wait_table_bits)
d741b090384963 Nicholas Piggin 2021-03-17 995 #if CONFIG_BASE_SMALL
d741b090384963 Nicholas Piggin 2021-03-17 996 static const unsigned int page_wait_table_bits = 4;
62906027091f1d Nicholas Piggin 2016-12-25 @997 static wait_queue_head_t page_wait_table[PAGE_WAIT_TABLE_SIZE] __cacheline_aligned;
d741b090384963 Nicholas Piggin 2021-03-17 998 #else
d741b090384963 Nicholas Piggin 2021-03-17 999 static unsigned int page_wait_table_bits __ro_after_init;
d741b090384963 Nicholas Piggin 2021-03-17 1000 static wait_queue_head_t *page_wait_table __ro_after_init;
d741b090384963 Nicholas Piggin 2021-03-17 1001 #endif
62906027091f1d Nicholas Piggin 2016-12-25 1002
62906027091f1d Nicholas Piggin 2016-12-25 1003 static wait_queue_head_t *page_waitqueue(struct page *page)
^1da177e4c3f41 Linus Torvalds 2005-04-16 1004 {
d741b090384963 Nicholas Piggin 2021-03-17 1005 return &page_wait_table[hash_ptr(page, page_wait_table_bits)];
^1da177e4c3f41 Linus Torvalds 2005-04-16 1006 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 1007
62906027091f1d Nicholas Piggin 2016-12-25 1008 void __init pagecache_init(void)
^1da177e4c3f41 Linus Torvalds 2005-04-16 1009 {
62906027091f1d Nicholas Piggin 2016-12-25 1010 int i;
62906027091f1d Nicholas Piggin 2016-12-25 1011
d741b090384963 Nicholas Piggin 2021-03-17 1012 if (!CONFIG_BASE_SMALL) {
d741b090384963 Nicholas Piggin 2021-03-17 @1013 page_wait_table = alloc_large_system_hash("page waitqueue hash",
d741b090384963 Nicholas Piggin 2021-03-17 1014 sizeof(wait_queue_head_t),
d741b090384963 Nicholas Piggin 2021-03-17 1015 0,
d741b090384963 Nicholas Piggin 2021-03-17 1016 21,
d741b090384963 Nicholas Piggin 2021-03-17 1017 0,
d741b090384963 Nicholas Piggin 2021-03-17 @1018 &page_wait_table_bits,
d741b090384963 Nicholas Piggin 2021-03-17 1019 NULL,
d741b090384963 Nicholas Piggin 2021-03-17 1020 0,
d741b090384963 Nicholas Piggin 2021-03-17 1021 0);
d741b090384963 Nicholas Piggin 2021-03-17 1022 }
d741b090384963 Nicholas Piggin 2021-03-17 1023
62906027091f1d Nicholas Piggin 2016-12-25 1024 for (i = 0; i < PAGE_WAIT_TABLE_SIZE; i++)
62906027091f1d Nicholas Piggin 2016-12-25 1025 init_waitqueue_head(&page_wait_table[i]);
^1da177e4c3f41 Linus Torvalds 2005-04-16 1026
62906027091f1d Nicholas Piggin 2016-12-25 1027 page_writeback_init();
^1da177e4c3f41 Linus Torvalds 2005-04-16 1028 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 1029
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 6 months
Re: [RFC PATCH v4 17/19] dyndbg: RFC - DECLARE/DEFINE_DYNAMIC_DEBUG_TABLE
by kernel test robot
Hi Jim,
[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on tip/x86/core]
[also build test ERROR on tip/x86/vdso efi/next drm-intel/for-linux-next kbuild/for-next linus/master v5.12-rc3 next-20210316]
[cannot apply to asm-generic/master jeyu/modules-next]
[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/Jim-Cromie/dyndbg-split-struct-_...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git f935178b5c1c32ff803b15892a8ba85a1280cb01
config: h8300-randconfig-r011-20210317 (attached as .config)
compiler: h8300-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/9a01f28486c78ab95308a4a70f2877016...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Jim-Cromie/dyndbg-split-struct-_ddebug-move-display-fields-to-new-_ddebug_site/20210317-155358
git checkout 9a01f28486c78ab95308a4a70f2877016d805ff0
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=h8300
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 >>):
h8300-linux-ld: section .init.text LMA [00000000005b2d00,000000000061007d] overlaps section .text LMA [0000000000000260,000000000118af9f]
h8300-linux-ld: section .data VMA [0000000000400000,00000000005b2cff] overlaps section .text VMA [0000000000000260,000000000118af9f]
h8300-linux-ld: lib/dynamic_debug.o: in function `.L14':
dynamic_debug.c:(.text+0x18c): undefined reference to `__start___dyndbg'
>> h8300-linux-ld: dynamic_debug.c:(.text+0x194): undefined reference to `__stop___dyndbg'
>> h8300-linux-ld: dynamic_debug.c:(.text+0x19e): undefined reference to `__start___dyndbg'
h8300-linux-ld: lib/dynamic_debug.o: in function `.L16':
>> dynamic_debug.c:(.text+0x1d4): undefined reference to `__start___dyndbg_sites'
h8300-linux-ld: lib/dynamic_debug.o: in function `.L15':
dynamic_debug.c:(.text+0x1e6): undefined reference to `__start___dyndbg'
h8300-linux-ld: lib/dynamic_debug.o: in function `dynamic_debug_init':
dynamic_debug.c:(.init.text+0xe2): undefined reference to `__start___dyndbg'
>> h8300-linux-ld: dynamic_debug.c:(.init.text+0xe8): undefined reference to `__stop___dyndbg'
>> h8300-linux-ld: dynamic_debug.c:(.init.text+0x110): undefined reference to `__start___dyndbg_sites'
h8300-linux-ld: dynamic_debug.c:(.init.text+0x2ec): undefined reference to `__stop___dyndbg'
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 6 months
[intel-linux-intel-lts:4.19/android_r 1/1] sound/soc/intel/atom/sst/sst_drv_interface.c:381:11: warning: format specifies type 'unsigned long long' but the argument has type '__u32' (aka 'unsigned int')
by kernel test robot
tree: https://github.com/intel/linux-intel-lts.git 4.19/android_r
head: f5400e4310f25e3d70ebbdd07834d5794e49d838
commit: f5400e4310f25e3d70ebbdd07834d5794e49d838 [1/1] Revert "ANDROID: GKI: ASoC: msm: fix integer overflow for long duration offload playback"
config: x86_64-randconfig-a011-20210317 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8ef111222a3dd12a9175f69c3bff598c46e8bdf7)
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/intel/linux-intel-lts/commit/f5400e4310f25e3d70ebbdd07...
git remote add intel-linux-intel-lts https://github.com/intel/linux-intel-lts.git
git fetch --no-tags intel-linux-intel-lts 4.19/android_r
git checkout f5400e4310f25e3d70ebbdd07834d5794e49d838
# 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 >>):
>> sound/soc/intel/atom/sst/sst_drv_interface.c:381:11: warning: format specifies type 'unsigned long long' but the argument has type '__u32' (aka 'unsigned int') [-Wformat]
str_id, tstamp->copied_total, tstamp->pcm_frames);
^~~~~~~~~~~~~~~~~~~~
include/linux/device.h:1562:47: note: expanded from macro 'dev_dbg'
dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \
~~~ ^~~~~~~~~~~
1 warning generated.
vim +381 sound/soc/intel/atom/sst/sst_drv_interface.c
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 354
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 355 static int sst_cdev_tstamp(struct device *dev, unsigned int str_id,
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 356 struct snd_compr_tstamp *tstamp)
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 357 {
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 358 struct snd_sst_tstamp fw_tstamp = {0,};
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 359 struct stream_info *stream;
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 360 struct intel_sst_drv *ctx = dev_get_drvdata(dev);
ce1cfe295abaa7 sound/soc/intel/atom/sst/sst_drv_interface.c Pierre-Louis Bossart 2018-07-24 361 void __iomem *addr;
ce1cfe295abaa7 sound/soc/intel/atom/sst/sst_drv_interface.c Pierre-Louis Bossart 2018-07-24 362
ce1cfe295abaa7 sound/soc/intel/atom/sst/sst_drv_interface.c Pierre-Louis Bossart 2018-07-24 363 addr = (void __iomem *)(ctx->mailbox + ctx->tstamp) +
ce1cfe295abaa7 sound/soc/intel/atom/sst/sst_drv_interface.c Pierre-Louis Bossart 2018-07-24 364 (str_id * sizeof(fw_tstamp));
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 365
ce1cfe295abaa7 sound/soc/intel/atom/sst/sst_drv_interface.c Pierre-Louis Bossart 2018-07-24 366 memcpy_fromio(&fw_tstamp, addr, sizeof(fw_tstamp));
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 367
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 368 stream = get_stream_info(ctx, str_id);
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 369 if (!stream)
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 370 return -EINVAL;
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 371 dev_dbg(dev, "rb_counter %llu in bytes\n", fw_tstamp.ring_buffer_counter);
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 372
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 373 tstamp->copied_total = fw_tstamp.ring_buffer_counter;
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 374 tstamp->pcm_frames = fw_tstamp.frames_decoded;
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 375 tstamp->pcm_io_frames = div_u64(fw_tstamp.hardware_counter,
75afbd052b3675 sound/soc/intel/atom/sst/sst_drv_interface.c Dan Carpenter 2015-04-09 376 (u64)stream->num_ch * SST_GET_BYTES_PER_SAMPLE(24));
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 377 tstamp->sampling_rate = fw_tstamp.sampling_frequency;
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 378
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 379 dev_dbg(dev, "PCM = %u\n", tstamp->pcm_io_frames);
2190ece2f516de sound/soc/intel/atom/sst/sst_drv_interface.c Chih-Wei Huang 2020-05-07 380 dev_dbg(dev, "Ptr Query on strid = %d copied_total %llu, decodec %d\n",
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 @381 str_id, tstamp->copied_total, tstamp->pcm_frames);
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 382 dev_dbg(dev, "rendered %d\n", tstamp->pcm_io_frames);
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 383
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 384 return 0;
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 385 }
7adab122a57c5a sound/soc/intel/sst/sst_drv_interface.c Vinod Koul 2014-10-30 386
:::::: The code at line 381 was first introduced by commit
:::::: 7adab122a57c5ade8efc2e4de67c72b084c31cda ASoC: Intel: sst - add compressed ops handling
:::::: TO: Vinod Koul <vinod.koul(a)intel.com>
:::::: CC: Mark Brown <broonie(a)kernel.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 6 months
[jarkko-linux-tpmdd:master 10/12] security/keys/trusted-keys/trusted_tee.c:288:21: error: a parameter list without types is only allowed in a function definition
by kernel test robot
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jarkko/linux-tpmdd.git master
head: 8a3fa8ade8a35d8f7c178f5680f07f223da41b87
commit: 2199c5e531b9630fc0f58da0d58bbd2a50f27787 [10/12] KEYS: trusted: Introduce TEE based Trusted Keys
config: mips-randconfig-r016-20210316 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 50c7504a93fdb90c26870db8c8ea7add895c7725)
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 mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/jarkko/linux-tpmdd.git/co...
git remote add jarkko-linux-tpmdd git://git.kernel.org/pub/scm/linux/kernel/git/jarkko/linux-tpmdd.git
git fetch --no-tags jarkko-linux-tpmdd master
git checkout 2199c5e531b9630fc0f58da0d58bbd2a50f27787
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
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 >>):
>> security/keys/trusted-keys/trusted_tee.c:288:1: warning: declaration specifier missing, defaulting to 'int'
MODULE_DEVICE_TABLE(tee, trusted_key_id_table);
^
int
>> security/keys/trusted-keys/trusted_tee.c:288:21: error: a parameter list without types is only allowed in a function definition
MODULE_DEVICE_TABLE(tee, trusted_key_id_table);
^
1 warning and 1 error generated.
vim +288 security/keys/trusted-keys/trusted_tee.c
282
283 static const struct tee_client_device_id trusted_key_id_table[] = {
284 {UUID_INIT(0xf04a0fe7, 0x1f5d, 0x4b9b,
285 0xab, 0xf7, 0x61, 0x9b, 0x85, 0xb4, 0xce, 0x8c)},
286 {}
287 };
> 288 MODULE_DEVICE_TABLE(tee, trusted_key_id_table);
289
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 6 months
