Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1435654

Re: [PATCH 3/3] ACPI: fix acpi_parse_entries_array() so it reports overflow correctly

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

Show all headers | View raw


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 | NextPrevious in thread | Find similar | Unroll thread


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