From: Tiezhu Yang <yangtiezhu@loongson.cn>
To: Jinyang He <hejinyang@loongson.cn>
Cc: Huacai Chen <chenhuacai@kernel.org>,
Xi Zhang <zhangxi@kylinos.cn>,
Xianglai Li <lixianglai@loongson.cn>,
loongarch@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1] LoongArch: Handle special PC in unwind_next_frame()
Date: Wed, 26 Nov 2025 14:08:27 +0800 [thread overview]
Message-ID: <6cc59312-7cdc-19a5-4d18-e120ea9c7563@loongson.cn> (raw)
In-Reply-To: <658bb477-b3f1-97c1-ab37-5dafcb6215f5@loongson.cn>
On 2025/11/25 下午6:52, Jinyang He wrote:
> On 2025-11-25 14:33, Tiezhu Yang wrote:
>
>> When running virtual machine before testing the kernel live patching with
>> "modprobe livepatch-sample", there is a timeout over 15 seconds, the
>> dmesg
>> command shows "unreliable stack" for user tasks in debug mode.
>>
>> The "unreliable stack" is because it can not unwind from
>> kvm_handle_exit()
>> to its previous frame kvm_exc_entry() due to the PC is not a valid kernel
>> address, the root cause is that the code of kvm_exc_entry() was copied to
>> the DMW area in kvm_loongarch_env_init(), so it should check the PC range
>> and then finish unwinding for this special case.
>
> Hi, Tiezhu,
>
> Since kvm_loongarch_env_init copies kvm_exc_entry which similar to how
> trap_init copies the exception handler. We should apply the same
> offset-based adjustment to compute the shadow PC address:
> shadow_pc = kvm_exc_entry + (cur_pc − copied_base_address)
> This allows the ORC unwinder to correctly locate the corresponding
> unwind info.
Thank you. I did some tests, it can unwind from kvm_handle_exit()
to its previous frame kvm_exc_entry() only when CONFIG_KVM=y,
but it can not unwind from kvm_handle_exit() to its previous frame
kvm_exc_entry() when CONFIG_KVM=m because we can not get the two
kvm address (unsigned long)kvm_exc_entry and
(unsigned long)kvm_loongarch_ops->exc_entry
in arch/loongarch/kernel/unwind_orc.c.
The KVM team will provide a patch to do something, then I will
test it again.
Thanks,
Tiezhu
next prev parent reply other threads:[~2025-11-26 6:08 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-25 6:33 Tiezhu Yang
2025-11-25 8:08 ` Huacai Chen
2025-11-25 8:54 ` Tiezhu Yang
2025-11-25 9:05 ` Huacai Chen
2025-11-26 5:36 ` Tiezhu Yang
2025-11-25 10:52 ` Jinyang He
2025-11-26 6:08 ` Tiezhu Yang [this message]
2025-11-26 6:28 ` Tiezhu Yang
2025-11-26 9:09 ` Tiezhu Yang
2025-11-26 9:27 ` Huacai Chen
2025-11-27 1:32 ` Tiezhu Yang
2025-11-26 9:56 ` Bibo Mao
2025-11-27 1:39 ` Tiezhu Yang
2025-11-27 1:53 ` Bibo Mao
2025-11-27 2:53 ` Jinyang He
2025-11-27 2:58 ` Bibo Mao
2025-12-09 2:08 ` Tiezhu Yang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6cc59312-7cdc-19a5-4d18-e120ea9c7563@loongson.cn \
--to=yangtiezhu@loongson.cn \
--cc=chenhuacai@kernel.org \
--cc=hejinyang@loongson.cn \
--cc=linux-kernel@vger.kernel.org \
--cc=lixianglai@loongson.cn \
--cc=loongarch@lists.linux.dev \
--cc=zhangxi@kylinos.cn \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®