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
<agordeev(a)linux.ibm.com> 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 <hca(a)linux.ibm.com>
> Signed-off-by: Mikhail Zaslonko <zaslonko(a)linux.ibm.com>
> ---
> 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 <agordeev(a)linux.ibm.com>