| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 28 Mar 2022 18:30:16 +0200
From: Hans de Goede <hdegoede@...hat.com>
To: Werner Sembach <wse@...edocomputers.com>,
kernel test robot <lkp@...el.com>, dmitry.torokhov@...il.com,
tiwai@...e.de, mpdesouza@...e.com, arnd@...db.de, samuel@...oj.net,
linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Cc: kbuild-all@...ts.01.org
Subject: Re: [PATCH v4 3/4] input/i8042: Add debug output for quirks
Hi,
On 3/28/22 18:28, Werner Sembach wrote:
>
> Am 28.03.22 um 14:08 schrieb Hans de Goede:
>> Hi,
>>
>> On 3/22/22 21:50, kernel test robot wrote:
>>> Hi Werner,
>>>
>>> Thank you for the patch! Yet something to improve:
>>>
>>> [auto build test ERROR on dtor-input/next]
>>> [also build test ERROR on v5.17 next-20220322]
>>> [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/Werner-Sembach/input-i8042-Move-__initconst-to-fix-code-styling-warning/20220323-012205
>>> base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next
>>> config: ia64-defconfig (https://download.01.org/0day-ci/archive/20220323/202203230435.z5FH2fWa-lkp@intel.com/config)
>>> compiler: ia64-linux-gcc (GCC) 11.2.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/d92be72641d5753be3aa94fe5961e2dbe2b5bdc9
>>> git remote add linux-review https://github.com/0day-ci/linux
>>> git fetch --no-tags linux-review Werner-Sembach/input-i8042-Move-__initconst-to-fix-code-styling-warning/20220323-012205
>>> git checkout d92be72641d5753be3aa94fe5961e2dbe2b5bdc9
>>> # save the config file to linux build tree
>>> mkdir build_dir
>>> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=ia64 SHELL=/bin/bash
>>>
>>> If you fix the issue, kindly add following tag as appropriate
>>> Reported-by: kernel test robot <lkp@...el.com>
>> i8042_dritek is only defined when building on x86, I would solve this
>> by moving the new pr_debug to inside i8042_check_quirks();
>
> Thanks I missed that. But moving the debug function in i8042_check_quirks would make the output x86 exclusive, while the
> quirks in general are not (on non x86 they must be set via command line but still have effects on the code if i see that
> correctly)
True.
>> The i8042_nopnp not being defined order is slightly tricker, you
>> can more or less cleanly fix this by doing this at the end of
>> i8042_check_quirks() and then put the new pr_debug after this:
>>
>> #ifdef CONFIG_PNP
>> if (quirks & SERIO_QUIRK_NOPNP)
>> i8042_nopnp = true;
>> #else
>> #define i8042_nopnp 0
>> #endif
>
> As an alternative i would suggest putting defines in the pr_debug call. Otherwise it might not be immediately that the
> #define i8042_nopnp 0 is there specifically for this one statement only.
>
> pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
> i8042_nokbd ? " nokbd" : "",
> i8042_noaux ? " noaux" : "",
> i8042_nomux ? " nomux" : "",
> i8042_unlock ? " unlock" : "",
> i8042_probe_defer ? "probe_defer" : "",
> i8042_reset == I8042_RESET_DEFAULT ?
> "" : i8042_reset == I8042_RESET_ALWAYS ?
> " reset_always" : " reset_never",
> i8042_direct ? " direct" : "",
> i8042_dumbkbd ? " dumbkbd" : "",
> i8042_noloop ? " noloop" : "",
> i8042_notimeout ? " notimeout" : "",
> i8042_kbdreset ? " kbdreset" : "",
> #ifdef CONFIG_X86
> i8042_dritek ? " dritek" : "",
> #else
> "",
> #endif
> #ifdef CONFIG_PNP
> i8042_nopnp ? " nopnp" : "");
> #else
> "");
> #endif
>
> As a side effect: like this the code snipped also stays completely position independed.
Ok, the above solution is fine by me.
It also has the advantage of not defining i8042_nopnp which may
lead to accidentally getting used when not set later.
Regards,
Hans
>>> All errors (new ones prefixed by >>):
>>>
>>> In file included from include/asm-generic/bug.h:22,
>>> from arch/ia64/include/asm/bug.h:17,
>>> from include/linux/bug.h:5,
>>> from include/linux/thread_info.h:13,
>>> from include/asm-generic/preempt.h:5,
>>> from ./arch/ia64/include/generated/asm/preempt.h:1,
>>> from include/linux/preempt.h:78,
>>> from include/linux/rcupdate.h:27,
>>> from include/linux/rculist.h:11,
>>> from include/linux/pid.h:5,
>>> from include/linux/sched.h:14,
>>> from include/linux/delay.h:23,
>>> from drivers/input/serio/i8042.c:12:
>>> drivers/input/serio/i8042-x86ia64io.h: In function 'i8042_platform_init':
>>>>> drivers/input/serio/i8042-x86ia64io.h:1511:17: error: 'i8042_dritek' undeclared (first use in this function); did you mean 'i8042_direct'?
>>> 1511 | i8042_dritek ? " dritek" : "",
>>> | ^~~~~~~~~~~~
>>> include/linux/printk.h:418:33: note: in definition of macro 'printk_index_wrap'
>>> 418 | _p_func(_fmt, ##__VA_ARGS__); \
>>> | ^~~~~~~~~~~
>>> include/linux/printk.h:132:17: note: in expansion of macro 'printk'
>>> 132 | printk(fmt, ##__VA_ARGS__); \
>>> | ^~~~~~
>>> include/linux/printk.h:576:9: note: in expansion of macro 'no_printk'
>>> 576 | no_printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__)
>>> | ^~~~~~~~~
>>> drivers/input/serio/i8042-x86ia64io.h:1497:9: note: in expansion of macro 'pr_debug'
>>> 1497 | pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
>>> | ^~~~~~~~
>>> drivers/input/serio/i8042-x86ia64io.h:1511:17: note: each undeclared identifier is reported only once for each function it appears in
>>> 1511 | i8042_dritek ? " dritek" : "",
>>> | ^~~~~~~~~~~~
>>> include/linux/printk.h:418:33: note: in definition of macro 'printk_index_wrap'
>>> 418 | _p_func(_fmt, ##__VA_ARGS__); \
>>> | ^~~~~~~~~~~
>>> include/linux/printk.h:132:17: note: in expansion of macro 'printk'
>>> 132 | printk(fmt, ##__VA_ARGS__); \
>>> | ^~~~~~
>>> include/linux/printk.h:576:9: note: in expansion of macro 'no_printk'
>>> 576 | no_printk(KERN_DEBUG pr_fmt(fmt), ##__VA_ARGS__)
>>> | ^~~~~~~~~
>>> drivers/input/serio/i8042-x86ia64io.h:1497:9: note: in expansion of macro 'pr_debug'
>>> 1497 | pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
>>> | ^~~~~~~~
>>>
>>>
>>> vim +1511 drivers/input/serio/i8042-x86ia64io.h
>>>
>>> 1494
>>> 1495 i8042_check_quirks();
>>> 1496
>>> 1497 pr_debug("Active quirks (empty means none):%s%s%s%s%s%s%s%s%s%s%s%s%s\n",
>>> 1498 i8042_nokbd ? " nokbd" : "",
>>> 1499 i8042_noaux ? " noaux" : "",
>>> 1500 i8042_nomux ? " nomux" : "",
>>> 1501 i8042_unlock ? " unlock" : "",
>>> 1502 i8042_probe_defer ? "probe_defer" : "",
>>> 1503 i8042_reset == I8042_RESET_DEFAULT ?
>>> 1504 "" : i8042_reset == I8042_RESET_ALWAYS ?
>>> 1505 " reset_always" : " reset_never",
>>> 1506 i8042_direct ? " direct" : "",
>>> 1507 i8042_dumbkbd ? " dumbkbd" : "",
>>> 1508 i8042_noloop ? " noloop" : "",
>>> 1509 i8042_notimeout ? " notimeout" : "",
>>> 1510 i8042_kbdreset ? " kbdreset" : "",
>>>> 1511 i8042_dritek ? " dritek" : "",
>>> 1512 i8042_nopnp ? " nopnp" : "");
>>> 1513
>>> 1514 retval = i8042_pnp_init();
>>> 1515 if (retval)
>>> 1516 return retval;
>>> 1517
>>>
>
Powered by blists - more mailing lists