public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* "x86: allocate space within a region top-down" causes bar0 access issue
@ 2010-12-17 19:44 Jon Mason
  2010-12-17 19:47 ` Jesse Barnes
  2010-12-17 20:16 ` Bjorn Helgaas
  0 siblings, 2 replies; 7+ messages in thread
From: Jon Mason @ 2010-12-17 19:44 UTC (permalink / raw)
  To: Bjorn Helgaas; +Cc: linux-kernel, netdev, Ramkrishna Vepa

The following patch is causing problem with the vxge driver/adapter on
HP x86-64 systems. Reads to bar0 to return 0xffffffffffffffff instead
of their intended value.  This prevents the vxge module from loading
by failing sanity checks in the driver for certain values in bar0.  We
are not seeing any issues with this patch on non-HP systems in our
lab.

Can this patch be removed from 2.6.37 until a better solution can be
found?

Thanks,
Jon

commit 1af3c2e45e7a641e774bbb84fa428f2f0bf2d9c9
Author: Bjorn Helgaas <bjorn.helgaas@hp.com>
Date:   Tue Oct 26 15:41:54 2010 -0600

    x86: allocate space within a region top-down
    
    Request that allocate_resource() use available space from high addresses
    first, rather than the default of using low addresses first.
    
    The most common place this makes a difference is when we move or assign
    new PCI device resources.  Low addresses are generally scarce, so it's
    better to use high addresses when possible.  This follows Windows practice
    for PCI allocation.
    
    Reference: https://bugzilla.kernel.org/show_bug.cgi?id=16228#c42
    Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com>
    Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

diff --git a/arch/x86/kernel/setup.c b/arch/x86/kernel/setup.c
index 922b5a1..0fe76df 100644
--- a/arch/x86/kernel/setup.c
+++ b/arch/x86/kernel/setup.c
@@ -788,6 +788,7 @@ void __init setup_arch(char **cmdline_p)
 
        x86_init.oem.arch_setup();
 
+       resource_alloc_from_bottom = 0;
        iomem_resource.end = (1ULL << boot_cpu_data.x86_phys_bits) - 1;
        setup_memory_map();
        parse_setup_data();


^ permalink raw reply related	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2010-12-18 22:01 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-17 19:44 "x86: allocate space within a region top-down" causes bar0 access issue Jon Mason
2010-12-17 19:47 ` Jesse Barnes
2010-12-18 22:01   ` Jon Mason
2010-12-17 20:16 ` Bjorn Helgaas
2010-12-17 23:12   ` Jon Mason
2010-12-18  0:17     ` Bjorn Helgaas
2010-12-18 22:00       ` Jon Mason

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox