From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sudeep Holla Subject: Re: [PATCH v2 1/2] ACPI / tables: simplify acpi_parse_entries Date: Mon, 28 Sep 2015 11:11:11 +0100 Message-ID: <5609123F.7020307@arm.com> References: <1442243686-15845-1-git-send-email-sudeep.holla@arm.com> <1442408287-10410-1-git-send-email-sudeep.holla@arm.com> <2184252.JT4i34L2sb@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from foss.arm.com ([217.140.101.70]:50150 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932497AbbI1KLO (ORCPT ); Mon, 28 Sep 2015 06:11:14 -0400 In-Reply-To: <2184252.JT4i34L2sb@vostro.rjw.lan> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "Rafael J. Wysocki" Cc: Sudeep Holla , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Al Stone , Lorenzo Pieralisi On 26/09/15 01:27, Rafael J. Wysocki wrote: > On Wednesday, September 16, 2015 01:58:06 PM Sudeep Holla wrote: >> acpi_parse_entries passes the table end pointer to the sub-table entry >> handler. acpi_parse_entries itself could validate the end of an entry >> against the table end using the length in the sub-table entry. >> >> This patch adds the validation of the sub-table entry end using the >> length field.This will help to eliminate the need to pass the table end >> to the handlers. >> >> It also moves the check for zero length entry early so that execution of >> the handler can be avoided. >> >> Cc: "Rafael J. Wysocki" >> Signed-off-by: Sudeep Holla >> >> --- >> drivers/acpi/tables.c | 31 +++++++++++++++---------------- >> 1 file changed, 15 insertions(+), 16 deletions(-) >> >> Hi Rafael, >> >> As I mentioned earlier, this needs to be applied after Al's MADT changes >> are merged. You might get simple conflicts in acpi_parse_entries. > > This needs to be rebased on top of some patches in my linux-next branch. > > It probably is better to rebase it on top of my bleeding-edge branch that > contains the Al's patches already, though. > I don't see Al's patches in your linux-next or bleeding-edge Regards, Sudeep