cvs commit: src/sys/dev/bce if_bcereg.h
Matthew Jacob
mj at feral.com
Tue Apr 25 23:56:01 UTC 2006
> 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.
More information about the cvs-src
mailing list