Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1435634
| From | Al Stone <ahs3@redhat.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables |
| Date | 2016-07-01 23:30 +0200 |
| Message-ID | <rQeaB-3g8-5@gated-at.bofh.it> (permalink) |
| References | <rQeaB-3g8-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
Without this patch, the acpi_parse_entries_array() function will return
the very first time there is any error found in either the array of
callback functions or if one of the callbacks returns an non-zero value.
However, the array of callbacks could still have valid entries further
on in the array, or the callbacks may be able to process subsequent
subtables without error. The change here makes the function consistent
with its description so that it will properly return the sum of all
matching entries for all proc handlers, instead of stopping abruptly
as it does today.
Signed-off-by: Al Stone <ahs3@redhat.com>
Cc: Rafael J. Wysocki <rjw@rjwysocki.net>
Cc: Len Brown <lenb@kernel.org>
---
drivers/acpi/tables.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
index 3e167b4..76c07ed 100644
--- a/drivers/acpi/tables.c
+++ b/drivers/acpi/tables.c
@@ -246,6 +246,7 @@ acpi_parse_entries_array(char *id, unsigned long table_size,
struct acpi_subtable_header *entry;
unsigned long table_end;
int count = 0;
+ int errs_found = 0;
int i;
if (acpi_disabled)
@@ -278,8 +279,10 @@ acpi_parse_entries_array(char *id, unsigned long table_size,
if (entry->type != proc[i].id)
continue;
if (!proc[i].handler ||
- proc[i].handler(entry, table_end))
- return -EINVAL;
+ proc[i].handler(entry, table_end)) {
+ errs_found++;
+ continue;
+ }
proc[i].count++;
break;
@@ -305,7 +308,7 @@ acpi_parse_entries_array(char *id, unsigned long table_size,
id, proc->id, count - max_entries, count);
}
- return count;
+ return (errs_found) ? -EINVAL : count;
}
int __init
--
2.7.4
Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread
[PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables Al Stone <ahs3@redhat.com> - 2016-07-01 23:30 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-01 23:40 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables Al Stone <ahs3@redhat.com> - 2016-07-01 23:50 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-01 23:50 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables Al Stone <ahs3@redhat.com> - 2016-07-02 00:00 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-02 00:10 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables Al Stone <ahs3@redhat.com> - 2016-07-02 01:10 +0200
Re: [PATCH 2/3] ACPI: fix acpi_parse_entries_array() so it traverses all subtables "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-02 01:30 +0200
csiph-web