lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 15 Dec 2008 16:53:58 -0800 From: Jesse Barnes <jbarnes@...tuousgeek.org> To: Linus Torvalds <torvalds@...ux-foundation.org> Cc: Alex Chiang <achiang@...com>, Matthew Wilcox <matthew@....cx>, Justin Chen <justin.chen@...com>, Linux Kernel Mailing List <linux-kernel@...r.kernel.org> Subject: Re: PCI BAR mem resource allocation "regression" 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@...nel.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 d4e9908a08199fe4d728c68cc7c5ca2d827ae99f Author: Linus Torvalds <torvalds@...ux-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@...com> Tested-by: Alex Chiang <achiang@...com> Tested-by: Justin Chen <justin.chen@...com> Signed-off-by: Linus Torvalds <torvalds@...ux-foundation.org> Signed-off-by: Jesse Barnes <jbarnes@...tuousgeek.org> diff --git a/kernel/resource.c b/kernel/resource.c index 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) { .
Powered by blists - more mailing lists