Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1435654
| From | "Rafael J. Wysocki" <rafael@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly |
| Date | 2016-07-01 23:50 +0200 |
| Message-ID | <rQetY-3mA-15@gated-at.bofh.it> (permalink) |
| References | <rQeaB-3g8-7@gated-at.bofh.it> <rQeaB-3g8-15@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Fri, Jul 1, 2016 at 11:21 PM, Al Stone <ahs3@redhat.com> wrote: > The function acpi_parse_entries_array() has a limiting parameter, > max_entries, which tells the function to stop looking at subtables > once that limit has been reached. Further, if the limit is reached, > it is reported. However, the logic is incorrect in that the loop > to examine all subtables will always stop when exactly max_entries > have been found, regardless of whether or not there are still subtables > to examine, and it will always report that zero subtables have been > ignored. This change allows the loop to continue to look at all > subtables and count all the ones of interest; if we have already > reached the number of max_entries, though, we will not invoke the > callback functions. If the max_entries limit has been exceeded, > report on that, as before, but more accurately, listing how many > subtables of interest there are in total (as was meant), and how > many entries each subtable type occupied. The problem appears to be that, if max_entries has been reached, it prints "ignored 0", although it should count all of the entries in that case too in principle. Do I think correctly?
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly Al Stone <ahs3@redhat.com> - 2016-07-01 23:30 +0200
Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly Al Stone <ahs3@redhat.com> - 2016-07-01 23:50 +0200
Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-02 00:00 +0200
Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly Al Stone <ahs3@redhat.com> - 2016-07-02 00:40 +0200
Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-02 00:50 +0200
Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly "Rafael J. Wysocki" <rafael@kernel.org> - 2016-07-01 23:50 +0200
csiph-web