[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4E7B04F8.3090500@logix.net.nz>
Date: Thu, 22 Sep 2011 21:50:48 +1200
From: Michal Ludvig <mludvig@...ix.net.nz>
To: Ram Pai <linuxram@...ibm.com>
CC: linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org,
Bjorn Helgaas <bhelgaas@...gle.com>,
Jesse Barnes <jbarnes@...tuousgeek.org>,
weiyang@...ux.vnet.ibm.com, wangyun@...ux.vnet.ibm.com,
shangw@...ux.vnet.ibm.com
Subject: Re: [PATCH] Resource: wrong resource window calculation
On 09/22/2011 07:48 PM, Ram Pai wrote:
> Resource: wrong resource window calculation
>
> __find_resource() incorrectly returns a resource window which overlaps an
> existing allocated window. This happens when the parent's resource-window spans
> 0x00000000 to 0xffffffff and is entirely allocated to all its children
> resource-windows.
>
> __find_resource() looks for gaps in resource allocation among the children
> resource windows. When it encounters the last child window it blindly tries
> the range next to one allocated to the last child. Since the last child's
> window ends at 0xffffffff the calculation overflows, leading the algorithm to
> believe that any window in the range 0x0000000 to 0xfffffff is available for
> allocation. This leads to a conflicting window allocation.
>
> Michal Ludvig reported this issue seen on his platform. The following patch
> fixes the problem and has been verified by Michal. I believe this bug has been
> there for ages. It got exposed by git commit
> 2bbc6942273b5b3097bd265d82227bdd84b351b2
> [PATCH] PCI : ability to relocate assigned pci-resources
>
> Signed-off-by: Ram Pai<linuxram@...ibm.com>
Tested-by: Michal Ludvig <mludvig@...ix.net.nz>
Works fine on top of 3.1.0-rc7 and solves the problem with my Jetway board.
Thanks Ram!
M.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists