Re: [Xen-devel] [PATCH v2 10/10] x86/SVM: Append AMD AVIC related data to IRQ keyhandler 'i'

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 07.05.18 at 23:07, <Janakarajan.Natarajan@xxxxxxx> wrote:
> @@ -2351,6 +2352,7 @@ static void dump_irqs(unsigned char key)
>              printk("   %#02x -> %ps()\n", i, direct_apic_vector[i]);
>  
>      dump_ioapic_irq_info();
> +    dump_avic_info();
>  }

While this is better than a separate key, I still don't like it. These 
statistics
are unrelated to the purpose of 'i'. Why can't this be connected to e.g. 'v',
provided these statistics are all that relevant in the first place?

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel