A gentle reminder: please take a look.

On Fri, Aug 7, 2026 at 2:11 PM Gaurav Yadav <[email protected]>
wrote:

> Replace generic printk() calls in kvmppc_dump_vcpu() with vcpu_err()
> and remove the corresponding TODO comment.
>
> Using vcpu_err() provides vCPU-aware logging context, making it easier
> to associate register dumps with specific vCPU instances in logs.
>
> Signed-off-by: Gaurav Yadav <[email protected]>
> ---
>  arch/powerpc/kvm/booke.c | 25 ++++++++++++-------------
>  1 file changed, 12 insertions(+), 13 deletions(-)
>
> diff --git a/arch/powerpc/kvm/booke.c b/arch/powerpc/kvm/booke.c
> index f3ddb24ece74..23159eb5f5af 100644
> --- a/arch/powerpc/kvm/booke.c
> +++ b/arch/powerpc/kvm/booke.c
> @@ -86,26 +86,25 @@ const struct kvm_stats_header kvm_vcpu_stats_header = {
>                        sizeof(kvm_vcpu_stats_desc),
>  };
>
> -/* TODO: use vcpu_printf() */
>  void kvmppc_dump_vcpu(struct kvm_vcpu *vcpu)
>  {
>         int i;
>
> -       printk("pc:   %08lx msr:  %08llx\n", vcpu->arch.regs.nip,
> -                       vcpu->arch.shared->msr);
> -       printk("lr:   %08lx ctr:  %08lx\n", vcpu->arch.regs.link,
> -                       vcpu->arch.regs.ctr);
> -       printk("srr0: %08llx srr1: %08llx\n", vcpu->arch.shared->srr0,
> -                                           vcpu->arch.shared->srr1);
> +       vcpu_err(vcpu, "pc:   %08lx msr:  %08llx\n", vcpu->arch.regs.nip,
> +                vcpu->arch.shared->msr);
> +       vcpu_err(vcpu, "lr:   %08lx ctr:  %08lx\n", vcpu->arch.regs.link,
> +                vcpu->arch.regs.ctr);
> +       vcpu_err(vcpu, "srr0: %08llx srr1: %08llx\n",
> vcpu->arch.shared->srr0,
> +                vcpu->arch.shared->srr1);
>
> -       printk("exceptions: %08lx\n", vcpu->arch.pending_exceptions);
> +       vcpu_err(vcpu, "exceptions: %08lx\n",
> vcpu->arch.pending_exceptions);
>
>         for (i = 0; i < 32; i += 4) {
> -               printk("gpr%02d: %08lx %08lx %08lx %08lx\n", i,
> -                      kvmppc_get_gpr(vcpu, i),
> -                      kvmppc_get_gpr(vcpu, i+1),
> -                      kvmppc_get_gpr(vcpu, i+2),
> -                      kvmppc_get_gpr(vcpu, i+3));
> +               vcpu_err(vcpu, "gpr%02d: %08lx %08lx %08lx %08lx\n", i,
> +                        kvmppc_get_gpr(vcpu, i),
> +                        kvmppc_get_gpr(vcpu, i+1),
> +                        kvmppc_get_gpr(vcpu, i+2),
> +                        kvmppc_get_gpr(vcpu, i+3));
>         }
>  }
>
> --
> 2.54.0
>
>

-- 
With Regards
Gaurav Yadav
+919310150558

Reply via email to