linux-kernel - Re: vmlinux.lds.h: Bug report: unable to handle page fault when start the virtual machine with qemu

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <b714ad78-4689-ad0b-9316-efcc1665f6bf@huawei.com>
Date:   Tue, 8 Nov 2022 15:46:32 +0800
From:   xiafukun <xiafukun@...wei.com>
To:     Ard Biesheuvel <ardb@...nel.org>
CC:     <arnd@...db.de>, <keescook@...omium.org>, <nathan@...nel.org>,
        <linux-arch@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <yusongping@...wei.com>, <zhaowenhui8@...wei.com>
Subject: Re: vmlinux.lds.h: Bug report: unable to handle page fault when start
 the virtual machine with qemu

Thank you for your reply.
We tested your changes to this patch and did fix the issue. Following the
solution you provided, we recompile the kernel and successfully start the
virtual machine.

在 2022/11/8 0:00, Ard Biesheuvel 写道:
> 
> That patch looks incorrect to me. Without CONFIG_SMP, the PERCPU
> sections are not instantiated, and the only copy of those variables is
> created in the ordinary .data/.bss sections
> 
> Does the change below fix the issue for you?
> 
> --- a/include/asm-generic/vmlinux.lds.h
> +++ b/include/asm-generic/vmlinux.lds.h
> @@ -347,6 +347,7 @@
>  #define DATA_DATA                                                      \
>         *(.xiptext)                                                     \
>         *(DATA_MAIN)                                                    \
> +       *(.data..decrypted)                                             \
>         *(.ref.data)                                                    \
>         *(.data..shared_aligned) /* percpu related */                   \
>         MEM_KEEP(init.data*)                                            \
> @@ -995,7 +996,6 @@
>  #ifdef CONFIG_AMD_MEM_ENCRYPT
>  #define PERCPU_DECRYPTED_SECTION                                       \
>         . = ALIGN(PAGE_SIZE);                                           \
> -       *(.data..decrypted)                                             \
>         *(.data..percpu..decrypted)                                     \
>         . = ALIGN(PAGE_SIZE);
>  #else