mem: Use a flag instead of address bit 63 for generic IPRs

Using address bit 63 to identify generic IPRs caused problems on
SPARC, where IPRs are heavily used. This changeset redefines how
generic IPRs are identified. Instead of using bit 63, we now use a
separate flag (GENERIC_IPR) a memory request.
This commit is contained in:
Andreas Sandberg 2013-10-15 13:24:35 +02:00
parent c753b273dc
commit 5e7738467b
4 changed files with 13 additions and 18 deletions

View file

@ -53,7 +53,7 @@ Cycles
GenericISA::handleGenericIprRead(ThreadContext *xc, Packet *pkt) GenericISA::handleGenericIprRead(ThreadContext *xc, Packet *pkt)
{ {
Addr va(pkt->getAddr()); Addr va(pkt->getAddr());
Addr cls((va & IPR_CLASS_MASK) >> IPR_CLASS_SHIFT); Addr cls(va >> IPR_CLASS_SHIFT);
switch (cls) { switch (cls) {
case IPR_CLASS_PSEUDO_INST: case IPR_CLASS_PSEUDO_INST:
@ -70,7 +70,7 @@ Cycles
GenericISA::handleGenericIprWrite(ThreadContext *xc, Packet *pkt) GenericISA::handleGenericIprWrite(ThreadContext *xc, Packet *pkt)
{ {
Addr va(pkt->getAddr()); Addr va(pkt->getAddr());
Addr cls((va & IPR_CLASS_MASK) >> IPR_CLASS_SHIFT); Addr cls(va >> IPR_CLASS_SHIFT);
switch (cls) { switch (cls) {
case IPR_CLASS_PSEUDO_INST: case IPR_CLASS_PSEUDO_INST:

View file

@ -49,23 +49,12 @@ namespace GenericISA
* Memory requests with the MMAPPED_IPR flag are generally mapped * Memory requests with the MMAPPED_IPR flag are generally mapped
* to registers. There is a class of these registers that are * to registers. There is a class of these registers that are
* internal to gem5, for example gem5 pseudo-ops in virtualized * internal to gem5, for example gem5 pseudo-ops in virtualized
* mode. * mode. Such IPRs always have the flag GENERIC_IPR set and are
* * handled by this code.
* In order to make the IPR space manageable we always set bit 63
* (IPR_GENERIC) for accesses that should be handled by the
* generic ISA code. Architectures may use the rest of the IPR
* space internally.
*/ */
/** Is this a generic IPR access? */
const Addr IPR_GENERIC = ULL(0x8000000000000000);
/** @{ */
/** Mask when extracting the class of a generic IPR */
const Addr IPR_CLASS_MASK = ULL(0x7FFF000000000000);
/** Shift amount when extracting the class of a generic IPR */ /** Shift amount when extracting the class of a generic IPR */
const int IPR_CLASS_SHIFT = 48; const int IPR_CLASS_SHIFT = 48;
/** @} */
/** Mask to extract the offset in within a generic IPR class */ /** Mask to extract the offset in within a generic IPR class */
const Addr IPR_IN_CLASS_MASK = ULL(0x0000FFFFFFFFFFFF); const Addr IPR_IN_CLASS_MASK = ULL(0x0000FFFFFFFFFFFF);
@ -94,7 +83,7 @@ namespace GenericISA
inline Addr inline Addr
iprAddressPseudoInst(uint8_t func, uint8_t subfunc) iprAddressPseudoInst(uint8_t func, uint8_t subfunc)
{ {
return IPR_GENERIC | (IPR_CLASS_PSEUDO_INST << IPR_CLASS_SHIFT) | return (IPR_CLASS_PSEUDO_INST << IPR_CLASS_SHIFT) |
(func << 8) | subfunc; (func << 8) | subfunc;
} }
@ -113,7 +102,9 @@ namespace GenericISA
inline bool inline bool
isGenericIprAccess(const Packet *pkt) isGenericIprAccess(const Packet *pkt)
{ {
return pkt->getAddr() & IPR_GENERIC; Request::Flags flags(pkt->req->getFlags());
return (flags & Request::MMAPPED_IPR) &&
(flags & Request::GENERIC_IPR);
} }
/** /**

View file

@ -257,7 +257,7 @@ TLB::finalizePhysical(RequestPtr req, ThreadContext *tc, Mode mode) const
req->setPaddr(x86LocalAPICAddress(tc->contextId(), req->setPaddr(x86LocalAPICAddress(tc->contextId(),
paddr - apicRange.start())); paddr - apicRange.start()));
} else if (m5opRange.contains(paddr)) { } else if (m5opRange.contains(paddr)) {
req->setFlags(Request::MMAPPED_IPR); req->setFlags(Request::MMAPPED_IPR | Request::GENERIC_IPR);
req->setPaddr(GenericISA::iprAddressPseudoInst( req->setPaddr(GenericISA::iprAddressPseudoInst(
(paddr >> 8) & 0xFF, (paddr >> 8) & 0xFF,
paddr & 0xFF)); paddr & 0xFF));

View file

@ -127,6 +127,10 @@ class Request
/** The request should be marked as LRU. */ /** The request should be marked as LRU. */
static const FlagsType EVICT_NEXT = 0x04000000; static const FlagsType EVICT_NEXT = 0x04000000;
/** The request should be handled by the generic IPR code (only
* valid together with MMAPPED_IPR) */
static const FlagsType GENERIC_IPR = 0x08000000;
/** These flags are *not* cleared when a Request object is reused /** These flags are *not* cleared when a Request object is reused
(assigned a new address). */ (assigned a new address). */
static const FlagsType STICKY_FLAGS = INST_FETCH; static const FlagsType STICKY_FLAGS = INST_FETCH;