From 0b52c24da66fd614e04ac1555470ccd68bfbe49d Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Mon, 25 Nov 2019 03:00:59 -0800 Subject: arch,sim: Stop decoding the pseudo inst subfunc value. This isn't used by anything any more. The func field is left in place to ensure compatability, but there's no reason to decode a value nobody is going to use. Jira Issue: https://gem5.atlassian.net/browse/GEM5-187 Change-Id: I85fcd0e4a362551c29af6bff350d99af86050415 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/23179 Tested-by: kokoro Reviewed-by: Giacomo Travaglini Maintainer: Gabe Black --- src/arch/arm/kvm/arm_cpu.cc | 3 +-- src/arch/generic/mmapped_ipr.cc | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) (limited to 'src/arch') diff --git a/src/arch/arm/kvm/arm_cpu.cc b/src/arch/arm/kvm/arm_cpu.cc index 0a77ee0d6..80576a25a 100644 --- a/src/arch/arm/kvm/arm_cpu.cc +++ b/src/arch/arm/kvm/arm_cpu.cc @@ -318,11 +318,10 @@ ArmKvmCPU::onKvmExitHypercall() ThreadContext *tc(getContext(0)); const uint32_t reg_ip(tc->readIntRegFlat(INTREG_R12)); const uint8_t func((reg_ip >> 8) & 0xFF); - const uint8_t subfunc(reg_ip & 0xFF); DPRINTF(Kvm, "KVM Hypercall: %#x/%#x\n", func, subfunc); const uint64_t ret = - PseudoInst::pseudoInst(getContext(0), func, subfunc); + PseudoInst::pseudoInst(getContext(0), func); // Just set the return value using the KVM API instead of messing // with the context. We could have used the context, but that diff --git a/src/arch/generic/mmapped_ipr.cc b/src/arch/generic/mmapped_ipr.cc index 8bdd18016..754ee026b 100644 --- a/src/arch/generic/mmapped_ipr.cc +++ b/src/arch/generic/mmapped_ipr.cc @@ -43,11 +43,10 @@ handlePseudoInst(ThreadContext *xc, Packet *pkt) { const Addr offset(pkt->getAddr() & IPR_IN_CLASS_MASK); const uint8_t func((offset >> 8) & 0xFF); - const uint8_t subfunc(offset & 0xFF); uint64_t ret; assert((offset >> 16) == 0); - ret = PseudoInst::pseudoInst(xc, func, subfunc); + ret = PseudoInst::pseudoInst(xc, func); if (pkt->isRead()) pkt->set(ret, TheISA::GuestByteOrder); } -- cgit v1.2.3