[Xen-changelog] [xen-4.1-testing] ACPI: acpi_table_parse() should return handler's error code

[Xen patchbot-4.1-testing <patchbot@xxxxxxx>]

# HG changeset patch
# User Boris Ostrovsky <boris.ostrovsky@xxxxxxx>
# Date 1360074895 -3600
# Node ID dd6694df1a31608cffc1ff0f89b1f76adab87b68
# Parent  546308d166834ae45113e44e5ba09ba4e76e98e6
ACPI: acpi_table_parse() should return handler's error code

Currently, the error code returned by acpi_table_parse()'s handler
is ignored. This patch will propagate handler's return value to
acpi_table_parse()'s caller.

Signed-off-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxx>
Committed-by: Jan Beulich <jbeulich@xxxxxxxx>
xen-unstable changeset: 26516:32d4516a97f0
xen-unstable date: Tue Feb  5 14:18:18 UTC 2013
---


diff -r 546308d16683 -r dd6694df1a31 xen/drivers/acpi/tables.c
--- a/xen/drivers/acpi/tables.c Thu Jan 17 16:11:02 2013 +0000
+++ b/xen/drivers/acpi/tables.c Tue Feb 05 15:34:55 2013 +0100
@@ -267,7 +267,7 @@ acpi_table_parse_madt(enum acpi_madt_typ
  * @handler: handler to run
  *
  * Scan the ACPI System Descriptor Table (STD) for a table matching @id,
- * run @handler on it.  Return 0 if table found, return on if not.
+ * run @handler on it.
  */
 int acpi_table_parse(char *id, acpi_table_handler handler)
 {
@@ -282,8 +282,7 @@ int acpi_table_parse(char *id, acpi_tabl
                acpi_get_table(id, 0, &table);
 
        if (table) {
-               handler(table);
-               return 0;
+               return handler(table);
        } else
                return 1;
 }

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog