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.
More information about the cvs-src