Re: [Xen-devel] [PATCH 2/8] xen/arm: acpi: Rework acpi_boot_table_init error paths

2017-02-15 Thread Stefano Stabellini
On Fri, 3 Feb 2017, Julien Grall wrote: > There are multiple path disable ACPI on error. Consolidate in a single > place, this will help in a follow-up patch to add more code on the error > path. > > Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini > --- > xen/arch/arm/acpi/boot.c

Re: [Xen-devel] [PATCH 2/8] xen/arm: acpi: Rework acpi_boot_table_init error paths

2017-02-08 Thread Julien Grall
Hi Andrew, On 03/02/17 20:44, Andrew Cooper wrote: On 03/02/17 19:18, Julien Grall wrote: @@ -260,16 +257,22 @@ int __init acpi_boot_table_init(void) error = acpi_table_init(); if ( error ) { -disable_acpi(); -return error; +printk("%s: Unable to initializ

Re: [Xen-devel] [PATCH 2/8] xen/arm: acpi: Rework acpi_boot_table_init error paths

2017-02-03 Thread Andrew Cooper
On 03/02/17 19:18, Julien Grall wrote: > @@ -260,16 +257,22 @@ int __init acpi_boot_table_init(void) > error = acpi_table_init(); > if ( error ) > { > -disable_acpi(); > -return error; > +printk("%s: Unable to initialize table parser (%d)\n", > +

[Xen-devel] [PATCH 2/8] xen/arm: acpi: Rework acpi_boot_table_init error paths

2017-02-03 Thread Julien Grall
There are multiple path disable ACPI on error. Consolidate in a single place, this will help in a follow-up patch to add more code on the error path. Signed-off-by: Julien Grall --- xen/arch/arm/acpi/boot.c | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff -