summaryrefslogtreecommitdiff
path: root/src/dev/arm/kmi.cc
diff options
context:
space:
mode:
authorSudhanshu Jha <sudhanshu.jha@arm.com>2017-02-27 10:29:56 +0000
committerAndreas Sandberg <andreas.sandberg@arm.com>2017-03-07 11:14:28 +0000
commit746e2f3c27ad83c36b7bc3b8bd3c92004fcf995b (patch)
tree3db0d446673d300a037516b8c8abebb8511dd296 /src/dev/arm/kmi.cc
parent46bfc14312f9b3b2f4e356f5d31405ae4ad394a8 (diff)
downloadgem5-746e2f3c27ad83c36b7bc3b8bd3c92004fcf995b.tar.xz
arm, kmi: Clear interrupts in KMI devices
Added functionality to check and clear interrupts for KMI devices. This fixes a boot bug when using KVM and in-kernel GIC emulation. Change-Id: Ia3e91d07567b7faf3f82b0adfda4a165a502a339 Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com> Reviewed-on: https://gem5-review.googlesource.com/2225 Reviewed-by: Rahul Thakur <rjthakur@google.com>
Diffstat (limited to 'src/dev/arm/kmi.cc')
-rw-r--r--src/dev/arm/kmi.cc34
1 files changed, 14 insertions, 20 deletions
diff --git a/src/dev/arm/kmi.cc b/src/dev/arm/kmi.cc
index 941546743..9230c15d2 100644
--- a/src/dev/arm/kmi.cc
+++ b/src/dev/arm/kmi.cc
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2010 ARM Limited
+ * Copyright (c) 2010, 2017 ARM Limited
* All rights reserved
*
* The license below extends only to copyright in the software and shall
@@ -53,7 +53,7 @@
Pl050::Pl050(const Params *p)
: AmbaIntDevice(p, 0xfff), control(0), status(0x43), clkdiv(0),
- interrupts(0), rawInterrupts(0), ackNext(false), shiftDown(false),
+ rawInterrupts(0), ackNext(false), shiftDown(false),
vnc(p->vnc), driverInitialized(false), intEvent(this)
{
if (vnc) {
@@ -101,8 +101,8 @@ Pl050::read(PacketPtr pkt)
data = clkdiv;
break;
case kmiISR:
- data = interrupts;
- DPRINTF(Pl050, "Read Interrupts: %#x\n", (uint32_t)interrupts);
+ data = getInterrupt();
+ DPRINTF(Pl050, "Read Interrupts: %#x\n", getInterrupt());
break;
default:
if (readId(pkt, ambaId, pioAddr)) {
@@ -238,28 +238,29 @@ Pl050::processCommand(uint8_t byte)
void
Pl050::updateIntStatus()
{
+ const bool old_interrupt(getInterrupt());
+
if (!rxQueue.empty())
rawInterrupts.rx = 1;
else
rawInterrupts.rx = 0;
- interrupts.tx = rawInterrupts.tx & control.txint_enable;
- interrupts.rx = rawInterrupts.rx & control.rxint_enable;
-
- DPRINTF(Pl050, "rawInterupts=%#x control=%#x interrupts=%#x\n",
- (uint32_t)rawInterrupts, (uint32_t)control, (uint32_t)interrupts);
-
- if (interrupts && !intEvent.scheduled())
+ if ((!old_interrupt && getInterrupt()) && !intEvent.scheduled()) {
schedule(intEvent, curTick() + intDelay);
+ } else if (old_interrupt && !(getInterrupt())) {
+ gic->clearInt(intNum);
+ }
}
void
Pl050::generateInterrupt()
{
+ DPRINTF(Pl050, "Generate Interrupt: rawInt=%#x ctrl=%#x int=%#x\n",
+ rawInterrupts, control, getInterrupt());
- if (interrupts) {
+ if (getInterrupt()) {
gic->sendInt(intNum);
- DPRINTF(Pl050, "Generated interrupt\n");
+ DPRINTF(Pl050, " -- Generated\n");
}
}
@@ -318,9 +319,6 @@ Pl050::serialize(CheckpointOut &cp) const
SERIALIZE_SCALAR(stsreg);
SERIALIZE_SCALAR(clkdiv);
- uint8_t ints = interrupts;
- SERIALIZE_SCALAR(ints);
-
uint8_t raw_ints = rawInterrupts;
SERIALIZE_SCALAR(raw_ints);
@@ -344,10 +342,6 @@ Pl050::unserialize(CheckpointIn &cp)
UNSERIALIZE_SCALAR(clkdiv);
- uint8_t ints;
- UNSERIALIZE_SCALAR(ints);
- interrupts = ints;
-
uint8_t raw_ints;
UNSERIALIZE_SCALAR(raw_ints);
rawInterrupts = raw_ints;