cvs commit: src/sys/dev/bce if_bcereg.h

M. Warner Losh imp at bsdimp.com
Wed Apr 26 16:06:12 UTC 2006


In message: <20060425165324.M64286 at ns1.feral.com>
            Matthew Jacob <mj at feral.com> writes:
: 
: > The problem is that PAE's bus_size_t is a 32-bit quantity, when it
: > should be a 64-bit quantity:
: >
: > #ifdef PAE
: > typedef uint64_t bus_addr_t;
: > #else
: > typedef uint32_t bus_addr_t;
: > #endif
: > typedef uint32_t bus_size_t;
: >
: > For bus addresses, we should use bus_addr_t, of course, but the above
: > is wrong.  I don't have a PAE machine, or I'd commit my local changes
: > that fix this...
: >
: 
: Why do you believe that it should be a 64 bit quantity under PAE?
: 
: Strictly speaking, I don't believe it is. PAE allows you to *address* 
: 36 physical address bits of memory using Dual Address cycles, but the 
: underlying bus is still a 32 bit bus.

Because it expands the address space > 32 bits.

Warner


More information about the cvs-src mailing list