On Tue, Apr 26, 2016 at 07:08:25PM +0800, Shannon Zhao wrote: > > > On 2016/4/25 21:49, Shannon Zhao wrote: > > On 2016年04月25日 20:42, Andrew Jones wrote: > >> > On Mon, Apr 25, 2016 at 03:11:46PM +0800, Shannon Zhao wrote: > >>>> >> > From: Shannon Zhao <shannon.z...@linaro.org> > >>>> >> > > >>>> >> > Add PMU IRQ number in ACPI table, then we can use PMU in guest > >>>> >> > through > >>>> >> > ACPI. > >>>> >> > > >>>> >> > Signed-off-by: Shannon Zhao <shannon.z...@linaro.org> > >>>> >> > Reviewed-by: Andrew Jones <drjo...@redhat.com> > >>>> >> > --- > >>>> >> > hw/arm/virt-acpi-build.c | 3 +++ > >>>> >> > 1 file changed, 3 insertions(+) > >> > Question: for testing of this, did you use > >> > https://lkml.org/lkml/2016/4/12/755 ? > > Actually I didn't test PMU with ACPI since I didn't notice the patch > > series you point out and it's very obvious for QEMU changes(adding irq > > flag and irq number in GICC). I will have a test later. > > I have test this patch series using ACPI and guest with the patches you > point out. Guest prints below logs and perf works well in guest. > > [ 0.000000] ACPI-PMU: Assign CPU 0 girq 23 level 0 > [ 0.000000] ACPI-PMU: Assign CPU 1 girq 23 level 0 > [ 0.000000] ACPI-PMU: Assign CPU 2 girq 23 level 0 > [ 0.000000] ACPI-PMU: Assign CPU 3 girq 23 level 0 > [...] > [ 0.094782] ACPI-PMU: Setting up 4 PMUs for CPU type D07
Thanks for the additional testing Shannon! drew
