diff options
| author | Steve Reinhardt <stever@gmail.com> | 2013-07-11 21:57:04 -0500 |
|---|---|---|
| committer | Steve Reinhardt <stever@gmail.com> | 2013-07-11 21:57:04 -0500 |
| commit | 1f43e244bda0c78844720609455fd77f6f275e51 (patch) | |
| tree | 65eb7262e58a61992368dfe10f2152dd85eb6129 /src/arch/x86 | |
| parent | 502ad1e6757116867e0e0529c0c080320a2b440b (diff) | |
| download | gem5-1f43e244bda0c78844720609455fd77f6f275e51.tar.xz | |
dev: make BasicPioDevice take size in constructor
Instead of relying on derived classes explicitly assigning
to the BasicPioDevice pioSize field, require them to pass
a size value in to the constructor.
Committed by: Nilay Vaish <nilay@cs.wisc.edu>
Diffstat (limited to 'src/arch/x86')
| -rw-r--r-- | src/arch/x86/interrupts.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/arch/x86/interrupts.cc b/src/arch/x86/interrupts.cc index eba4b0010..afed9c132 100644 --- a/src/arch/x86/interrupts.cc +++ b/src/arch/x86/interrupts.cc @@ -608,7 +608,7 @@ X86ISA::Interrupts::setReg(ApicRegIndex reg, uint32_t val) X86ISA::Interrupts::Interrupts(Params * p) - : BasicPioDevice(p), IntDevice(this, p->int_latency), + : BasicPioDevice(p, PageBytes), IntDevice(this, p->int_latency), apicTimerEvent(this), pendingSmi(false), smiVector(0), pendingNmi(false), nmiVector(0), @@ -619,7 +619,6 @@ X86ISA::Interrupts::Interrupts(Params * p) pendingIPIs(0), cpu(NULL), intSlavePort(name() + ".int_slave", this, this) { - pioSize = PageBytes; memset(regs, 0, sizeof(regs)); //Set the local apic DFR to the flat model. regs[APIC_DESTINATION_FORMAT] = (uint32_t)(-1); |