mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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


  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®