Re: PCI BAR mem resource allocation "regression"

[Jesse Barnes <jbarnes@virtuousgeek.org>]

On Monday, December 15, 2008 4:25 pm Linus Torvalds wrote:> On Mon, 15 Dec 2008, Jesse Barnes wrote:> > I can put your patch in my -next branch if you think it would help (not> > that - next gets a ton of testing, but hey)...>> Better than nothing, I guess.>> We may end up back-porting it (since Chen confirms that it's not just a> warning, it actually affects functionality), but let's get it some testing> in -next first and then eventually in the merge window. So you might mark> it for attention for 'stable@kernel.org' as you commit it.
I pushed it to my -next branch, though I forgot to add the stable@ cc and fix up the comments while I was there... I think I'll have to rebase anyway, so I'll take care of both of those when I do.
Thanks,-- Jesse Barnes, Intel Open Source Technology Center
commit d4e9908a08199fe4d728c68cc7c5ca2d827ae99fAuthor: Linus Torvalds <torvalds@linux-foundation.org>Date:   Sat Dec 13 17:37:57 2008 -0800
    resource:  revert parent nesting of identical ranges        Revert the change that causes inverted resource nesting in the case of    identical resource ranges.  Initially introduced by    d33b6fba2c4350651f3f61ff2ab858a2f116e9a4, this change was provided    without a justification and seems to cause problems on some machines in    the face of PCI hot plugging.        The patch makes resources registered against an existing but identical    range nest as children of the existing resource, rather than parents.        Reported-by: Alex Chiang <achiang@hp.com>    Tested-by: Alex Chiang <achiang@hp.com>    Tested-by: Justin Chen <justin.chen@hp.com>    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>    Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
diff --git a/kernel/resource.c b/kernel/resource.cindex c251b0f..741357f 100644--- a/kernel/resource.c+++ b/kernel/resource.c@@ -381,8 +381,6 @@ static struct resource * __insert_resource(struct resource *parent, struct resou  		if ((first->start > new->start) || (first->end < new->end)) 			break;-		if ((first->start == new->start) && (first->end == new->end))-			break; 	}  	for (next = first; ; next = next->sibling) {\0ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥