Subject: Re: kern/10318 -- CVS commit: syssrc
To: HAYAKAWA Koichi <haya@netbsd.org>
From: John Hawkinson <jhawk@MIT.EDU>
List: source-changes
Date: 06/08/2000 16:33:41
| Module Name:	syssrc
| Committed By:	haya
| Date:		Tue Jun  6 08:17:38 UTC 2000
| 
| Modified Files:
| 	  syssrc/sys/arch/i386/i386: rbus_machdep.c
| 
| Log Message:
| Use entire extent region when PCIBIOS_ADDR_FIXUP is set.  PCIBIOS
| provides PCI-dedicated memory and io extent.
| 
| To generate a diff of this commit:
| cvs rdiff -r1.5 -r1.6 syssrc/sys/arch/i386/i386/rbus_machdep.c
|
Hayakawa-san:

This breaks cardbus on my Sony VAIO (in weird ways), where
PCIBIOS_ADDR_FIXUP is set. Please see PR 10318 (copy sent to you under
seperate cover).

What's the right way to investigate this and should we back it
out in the meantime?

it seems like this sort of change is not at all clearly described by
the summary of PCIBIOS_ADDR_FIXUP, either in GENERIC or in
pcibios(4). I'm reluctant to turn it off because I remember having
more prolems without it than with it.


Somewhat unrelatedly, I think that pcibios(4) needs to be a lot more
clear about whether users should turn on the *FIXUPs by default, or
whether they should only do so in special circumstances.

Thanks!

--jhawk