Re: [for-4.7 5/5] xen/arm: acpi: Print more error messages in acpi_map_gic_cpu_interface

[Shannon Zhao <zhaoshenglong@huawei.com>]

On 2016/4/7 18:59, Julien Grall wrote:
> It's helpful to spot any error without having to modify the hypervisor
> code.
> 
> Signed-off-by: Julien Grall <julien.grall@arm.com>

Reviewed-by: Shannon Zhao <shannon.zhao@linaro.org>
> ---
>  xen/arch/arm/acpi/boot.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/xen/arch/arm/acpi/boot.c b/xen/arch/arm/acpi/boot.c
> index 602ab39..23285f7 100644
> --- a/xen/arch/arm/acpi/boot.c
> +++ b/xen/arch/arm/acpi/boot.c
> @@ -63,7 +63,10 @@ acpi_map_gic_cpu_interface(struct acpi_madt_generic_interrupt *processor)
>  
>      total_cpus++;
>      if ( !enabled )
> +    {
> +        printk("Skipping disabled CPU entry with 0x%"PRIx64" MPIDR\n", mpidr);
>          return;
> +    }
>  
>      if ( enabled_cpus >=  NR_CPUS )
>      {
> @@ -101,7 +104,11 @@ acpi_map_gic_cpu_interface(struct acpi_madt_generic_interrupt *processor)
>      }
>  
>      if ( !acpi_psci_present() )
> +    {
> +        printk("PSCI not present, skipping CPU MPIDR 0x%"PRIx64"\n",
> +               mpidr);
>          return;
> +    }
>  
>      if ( (rc = arch_cpu_init(enabled_cpus, NULL)) < 0 )
>      {
> 

-- 
Shannon


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel