From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rene Herman Subject: Re: [patch 1/4] PNP: make pnp_{port,mem,etc}_start(), et al work for invalid resources Date: Tue, 20 May 2008 00:23:02 +0200 Message-ID: <4831FDC6.3090709@keyaccess.nl> References: <20080515220726.029505668@ldl.fc.hp.com> <20080515220754.366570486@ldl.fc.hp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from smtpq2.groni1.gr.home.nl ([213.51.130.201]:49341 "EHLO smtpq2.groni1.gr.home.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761072AbYESXVV (ORCPT ); Mon, 19 May 2008 19:21:21 -0400 In-Reply-To: <20080515220754.366570486@ldl.fc.hp.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Bjorn Helgaas Cc: Len Brown , linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Adam Belay , Adam M Belay , Li Shaohua , Matthieu Castet , Thomas Renninger , Jaroslav Kysela , Andrew Morton , Takashi Iwai On 16-05-08 00:07, Bjorn Helgaas wrote: > Some callers use pnp_port_start() and similar functions without > making sure the resource is valid. This patch makes us fall > back to returning the initial values if the resource is not > valid or not even present. > > This mostly preserves the previous behavior, where we would just > return the initial values set by pnp_init_resource_table(). The > original 2.6.25 code didn't range-check the "bar", so it would > return garbage if the bar exceeded the table size. This code > returns sensible values instead. > > Signed-off-by: Bjorn Helgaas Acked-by: Rene Herman Rene.