From: Nuno Das Neves <nunodasne...@linux.microsoft.com> Sent: Tuesday, April 1, 2025 10:32 AM > > 'output' is already a pointer to the output argument, it should be > passed directly to hv_do_hypercall() without the '&' operator. > > Signed-off-by: Nuno Das Neves <nunodasne...@linux.microsoft.com> > --- > This patch is a fixup for: > e96204e5e96e hyperv: Move hv_current_partition_id to arch-generic code > > drivers/hv/hv_common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/hv/hv_common.c b/drivers/hv/hv_common.c > index b3b11be11650..a7d7494feaca 100644 > --- a/drivers/hv/hv_common.c > +++ b/drivers/hv/hv_common.c > @@ -307,7 +307,7 @@ void __init hv_get_partition_id(void) > > local_irq_save(flags); > output = *this_cpu_ptr(hyperv_pcpu_input_arg); > - status = hv_do_hypercall(HVCALL_GET_PARTITION_ID, NULL, &output); > + status = hv_do_hypercall(HVCALL_GET_PARTITION_ID, NULL, output); > pt_id = output->partition_id; > local_irq_restore(flags); > > -- > 2.34.1
I should have caught that when I reviewed the original patch! :-( Trying again, Reviewed-by: Michael Kelley <mhkli...@outlook.com>
