lkml.org 
[lkml]   [2009]   [Jun]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] x86/pci: don't use crs for root if we only have one root bus

Should we not just revert 9e9f46c44e487af0a82eb61b624553e2f7118f5b?

The thing says:

"At this point, it seems to solve more problems than it causes, so let's
try using it by default. It's an easy revert if it ends up causing
trouble."

and it clearly does _not_ solve more problems than it causes, and the
whole message in that commit implies we should revert it.

I'm happy to apply various patches to fix it up, but regardless, I thinkwe
should revert that commit as bogus. We can try making it the default again
next round, when maybe it will be true that it doesn't cause issues.

What did it even ever help with?

Linus

On Wed, 24 Jun 2009, Yinghai Lu wrote:
>
> for AMD system, when only one PCI root, just set PCI_NO_ROOT_CRS for it
>
> Signed-off-by: Yinghai Lu <yinghai@kernel.org>
>
> ---
> arch/x86/pci/amd_bus.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> Index: linux-2.6/arch/x86/pci/amd_bus.c
> ===================================================================
> --- linux-2.6.orig/arch/x86/pci/amd_bus.c
> +++ linux-2.6/arch/x86/pci/amd_bus.c
> @@ -561,6 +561,10 @@ static int __init early_fill_mp_bus_info
> }
> }
>
> + /* don't use _CRS if we only have one root */
> + if (pci_root_num <= 1)
> + pci_probe |= PCI_NO_ROOT_CRS;
> +
> return 0;
> }
>
>


\
 
 \ /
  Last update: 2009-10-18 23:28    [W:0.095 / U:5.228 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site