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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Wed, 4 Oct 2017 10:25:57 +0800
From:   Fengguang Wu <fengguang.wu@...el.com>
To:     Florian Fainelli <f.fainelli@...il.com>
Cc:     LKML <linux-kernel@...r.kernel.org>, LKP <lkp@...org>,
        Ye Xiaolong <xiaolong.ye@...el.com>
Subject: Re: e4dace3615 ("lib: add test module for CONFIG_DEBUG_VIRTUAL"): [
 10.185572] kernel BUG at arch/x86/mm/physaddr.c:75!

On Tue, Oct 03, 2017 at 07:09:37PM -0700, Florian Fainelli wrote:
>Hi Fengguang,
>
>On 10/03/2017 07:24 AM, Fengguang Wu wrote:
>> Hi Florian,
>>
>> FYI we see this BUG in latest RC kernel and linux-next.
>> Have it been fixed somewhere else?
>
>The purpose of the test module is to make tests for CONFIG_DEBUG_VIRTUAL
>trigger such warnings/VIRTUAL_BUG_ON, because of how the test is coded,

If so, it may help clarify things if there is a printk notice before
triggering the BUG?

>you may want to not load such module. If the module needs to be moved to
>another directory because lib/* contains mostly functional tests, let me
>know and I will cook something.

Never mind, it looks easy to disable CONFIG_TEST_DEBUG_VIRTUAL. :)

Thanks,
Fengguang

>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>>
>> commit e4dace3615526fd66c86dd535ee4bc9e8c706e37
>> Author:     Florian Fainelli <f.fainelli@...il.com>
>> AuthorDate: Fri Sep 8 16:15:31 2017 -0700
>> Commit:     Linus Torvalds <torvalds@...ux-foundation.org>
>> CommitDate: Fri Sep 8 18:26:49 2017 -0700
>>
>>      lib: add test module for CONFIG_DEBUG_VIRTUAL
>>
>>      Add a test module that allows testing that CONFIG_DEBUG_VIRTUAL works
>>      correctly, at least that it can catch invalid calls to virt_to_phys()
>>      against the non-linear kernel virtual address map.
>>
>>      Link: http://lkml.kernel.org/r/20170808164035.26725-1-f.fainelli@gmail.com
>>      Signed-off-by: Florian Fainelli <f.fainelli@...il.com>
>>      Cc: "Luis R. Rodriguez" <mcgrof@...nel.org>
>>      Cc: Kees Cook <keescook@...omium.org>
>>      Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
>>      Signed-off-by: Linus Torvalds <torvalds@...ux-foundation.org>
>>
>> 9888a588ea  lib/hexdump.c: return -EINVAL in case of error in hex2bin()
>> e4dace3615  lib: add test module for CONFIG_DEBUG_VIRTUAL
>> 9e66317d3c  Linux 4.14-rc3
>> 1418b85217  Add linux-next specific files for 20170929
>> +------------------------------------------+------------+------------+-----------+---------------+
>> |                                          | 9888a588ea | e4dace3615 | v4.14-rc3 | next-20170929 |
>> +------------------------------------------+------------+------------+-----------+---------------+
>> | boot_successes                           | 34         | 0          | 0         | 0             |
>> | boot_failures                            | 0          | 15         | 11        | 11            |
>> | kernel_BUG_at_arch/x86/mm/physaddr.c     | 0          | 15         | 11        | 11            |
>> | invalid_opcode:#[##]                     | 0          | 15         | 11        | 11            |
>> | EIP:__phys_addr                          | 0          | 15         | 11        | 11            |
>> | Kernel_panic-not_syncing:Fatal_exception | 0          | 15         | 11        | 11            |
>> +------------------------------------------+------------+------------+-----------+---------------+
>>
>> [    9.837344] test_firmware: interface ready
>> [    9.838286] test passed
>> [    9.839498] test_printf: all 260 tests passed
>> [   10.183840] test_bitmap: all 460506 tests passed
>> [   10.184882] ------------[ cut here ]------------
>> [   10.185572] kernel BUG at arch/x86/mm/physaddr.c:75!
>> [   10.186568] invalid opcode: 0000 [#1]
>> [   10.187104] CPU: 0 PID: 1 Comm: swapper Not tainted 4.13.0-09302-ge4dace3 #1
>> [   10.187547] task: 91c45000 task.stack: 91c54000
>> [   10.187547] EIP: __phys_addr+0x244/0x2a0
>> [   10.187547] EFLAGS: 00210246 CPU: 0
>> [   10.187547] EAX: 00000000 EBX: 92fe0000 ECX: 00000000 EDX: 00000001
>> [   10.187547] ESI: 12fe0000 EDI: 00000000 EBP: 8464e539 ESP: 91c55f08
>> [   10.187547]  DS: 007b ES: 007b FS: 0000 GS: 0000 SS: 0068
>> [   10.187547] CR0: 80050033 CR2: 00000000 CR3: 04792000 CR4: 00000690
>> [   10.187547] Call Trace:
>> [   10.187547]  ? vprintk_func+0x88/0x150
>> [   10.187547]  ? test_bitmap_init+0xe7/0xe7
>> [   10.187547]  ? test_debug_virtual_init+0x22/0xf0
>> [   10.187547]  ? do_one_initcall+0x127/0x3c8
>> [   10.187547]  ? kernel_init_freeable+0x217/0x3de
>> [   10.187547]  ? kernel_init_freeable+0x245/0x3de
>> [   10.187547]  ? rest_init+0x200/0x200
>> [   10.187547]  ? kernel_init+0x17/0x340
>> [   10.187547]  ? rest_init+0x200/0x200
>> [   10.187547]  ? schedule_tail_wrapper+0x6/0x8
>> [   10.187547]  ? rest_init+0x200/0x200
>> [   10.187547]  ? ret_from_fork+0x19/0x24
>> [   10.187547] Code: c9 ba 01 00 00 00 c7 04 24 00 00 00 00 b8 08 31 17 84 e8 30 58 19 00 ff 05 74 c1 1f 84 83 05 20 01 7e 84 01 83 15 24 01 7e 84 00 <0f> 0b 83 05 30 01 7e 84 01 83 15 34 01 7e 84 00 8d 74 26 00 83
>> [   10.187547] EIP: __phys_addr+0x244/0x2a0 SS:ESP: 0068:91c55f08
>> [   10.205865] ---[ end trace 2cb481e044fc1df9 ]---
>> [   10.206542] Kernel panic - not syncing: Fatal exception
>>
>>                                                            # HH:MM RESULT GOOD BAD GOOD_BUT_DIRTY DIRTY_NOT_BAD
>> git bisect start 2bd6bf03f4c1c59381d62c61d03f6cc3fe71f66e v4.13 --
>> git bisect good 3aea311c1b4002bd730a09530f80f2f2ad3bf495  # 04:45  G     10     0    0   0  genksyms: fix gperf removal conversion
>> git bisect  bad 6d8ef53e8b2fed8b0f91df0c6da7cc92747d934a  # 05:12  B      0     9   21   0  Merge tag 'f2fs-for-4.14' of git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs
>> git bisect  bad d719518d9ce9132bad8a06e8029aeead328f66a3  # 05:40  B      0     9   21   0  Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/sparc-next
>> git bisect good c054be10ffdbd5507a1fd738067d76acfb4808fd  # 19:53  G     10     0    0   0  remove gperf left-overs from build system
>> git bisect  bad 8c1d70b2de517e7b44dcac24416e60c9662db507  # 20:35  B      0     9   24   3  Merge tag 'vfio-v4.14-rc1' of git://github.com/awilliam/linux-vfio
>> git bisect  bad 126e76ffbf78d9e948b641aadb265d16c57f5a3d  # 21:17  B      0    10   26   4  Merge branch 'for-4.14/block-postmerge' of git://git.kernel.dk/linux-block
>> git bisect  bad fbf4432ff71b7a25bef993a5312906946d27f446  # 21:27  B      0     1   13   0  Merge branch 'akpm' (patches from Andrew)
>> git bisect  bad 7ed1da84b3d88ce080ee89a26e8885c9509d1d2d  # 21:56  B      0     8   20   0  autofs: drop wrong comment
>> git bisect good 3b3c4babd898715926d24ae10aa64778ace33aae  # 18:57  G     10     0    0   0  lib/string.c: add multibyte memset functions
>> git bisect good b2ac2ea6296e7dd779168eb085b09d0fab9d1294  # 09:23  G     10     0    0   0  fs/epoll: use faster rb_first_cached()
>> git bisect  bad 63b7c73ec86b8c73560d22c7552bc6f47f0b8da9  # 09:50  B      0     1   17   4  checkpatch: add --strict check for ifs with unnecessary parentheses
>> git bisect  bad 60ef690018b262ddcd0d51edf10e40710deb9c9f  # 10:13  B      0     1   17   4  bitmap: introduce BITMAP_FROM_U64()
>> git bisect good 9888a588ea96ba2804f955bbc2667346719da887  # 00:47  G     11     0    0   0  lib/hexdump.c: return -EINVAL in case of error in hex2bin()
>> git bisect  bad 0a5ce0831d04382aa9e2420e33dff958ddade542  # 01:06  B      0     1   14   1  lib/bitmap.c: make bitmap_parselist() thread-safe and much faster
>> git bisect  bad e4dace3615526fd66c86dd535ee4bc9e8c706e37  # 01:27  B      0     1   17   4  lib: add test module for CONFIG_DEBUG_VIRTUAL
>> # first bad commit: [e4dace3615526fd66c86dd535ee4bc9e8c706e37] lib: add test module for CONFIG_DEBUG_VIRTUAL
>> git bisect good 9888a588ea96ba2804f955bbc2667346719da887  # 15:53  G     31     0    0   0  lib/hexdump.c: return -EINVAL in case of error in hex2bin()
>> # extra tests with CONFIG_DEBUG_INFO_REDUCED
>> git bisect  bad e4dace3615526fd66c86dd535ee4bc9e8c706e37  # 16:10  B      0     1   15   2  lib: add test module for CONFIG_DEBUG_VIRTUAL
>> # extra tests on HEAD of linux-devel/devel-catchup-201709290308
>> git bisect  bad fb3c6c6bf2d191751baa1c92927631cd7d3480ef  # 16:16  B      0    34   49   0  0day head guard for 'devel-catchup-201709290308'
>> # extra tests on tree/branch linus/master
>> git bisect  bad 9e66317d3c92ddaab330c125dfe9d06eee268aff  # 16:52  B      0     3   15   0  Linux 4.14-rc3
>> # extra tests with first bad commit reverted
>> git bisect good e757f3a215d2d63f0d873d3985b28971669fcd90  # 07:50  G     10     0    0   0  Revert "lib: add test module for CONFIG_DEBUG_VIRTUAL"
>> # extra tests on tree/branch linux-next/master
>> git bisect  bad 1418b852174ad50b3cb4738b8801626aefdc0bd9  # 08:51  B      0    11   23   0  Add linux-next specific files for 20170929
>>
>> ---
>> 0-DAY kernel test infrastructure                Open Source Technology Center
>> https://lists.01.org/pipermail/lkp                          Intel Corporation
>>
>
>-- 
>Florian

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ