for the v3 series, applied: https://github.com/crash-utility/crash/commit/21bdb005dc582d1e1c3817ba64cc29c52cddec25
On Tue, Oct 28, 2025 at 10:21 AM Tao Liu <[email protected]> wrote: > > Hi Alexander & Mikhail, > > Thanks for the improvements, LGTM, for the series, ack. > > Thanks, > Tao Liu > > On Tue, Oct 28, 2025 at 6:11 AM Alexander Gordeev > <[email protected]> wrote: > > > > On Mon, Oct 27, 2025 at 06:04:56PM +0100, Mikhail Zaslonko wrote: > > > In s390x_kvtop() early return may take place despite the verbose > > > flag. Thus we can miss page-table walk information in the vtop output > > > for kernel virtual addresses. > > > Make sure that s390x_vtop() is always called for kernel virtual addresses > > > when the verbose flag is passed to s390x_kvtop() by do_vtop(). > > > > > > Suggested-by: Heiko Carstens <[email protected]> > > > Signed-off-by: Mikhail Zaslonko <[email protected]> > > > --- > > > s390x.c | 6 +++--- > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/s390x.c b/s390x.c > > > index 25dca5e..77d9082 100644 > > > --- a/s390x.c > > > +++ b/s390x.c > > > @@ -943,15 +943,15 @@ s390x_kvtop(struct task_context *tc, ulong vaddr, > > > physaddr_t *paddr, int verbose > > > > > > if (!IS_KVADDR(vaddr)){ > > > *paddr = 0; > > > return FALSE; > > > } > > > > > > - if (!IS_VMALLOC_ADDR(vaddr)) { > > > - *paddr = VTOP(vaddr); > > > - return TRUE; > > > + if (!verbose && !IS_VMALLOC_ADDR(vaddr)) { > > > + *paddr = VTOP(vaddr); > > > + return TRUE; > > > } > > > > > > pgd_base = (unsigned long)vt->kernel_pgd[0]; > > > return s390x_vtop(pgd_base, vaddr, paddr, verbose); > > > } > > > > Acked-by: Alexander Gordeev <[email protected]> > > -- Crash-utility mailing list -- [email protected] To unsubscribe send an email to [email protected] https://${domain_name}/admin/lists/devel.lists.crash-utility.osci.io/ Contribution Guidelines: https://github.com/crash-utility/crash/wiki
