From: Gabe Black Date: Wed, 28 Aug 2019 00:20:30 +0000 (-0700) Subject: fastmodel: Pull out and simplify the interrupt mechanism in the GIC. X-Git-Tag: v19.0.0.0~493 X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=54cf33fbf392f4dc47c8c3559e6f6f10ae3288dc;p=gem5.git fastmodel: Pull out and simplify the interrupt mechanism in the GIC. This change pulls out the SPI and PPI command structures and replaces them with a custom protocol which can deliver a SPI or PPI without having to bundle their parameters into a structure. Change-Id: I8f15c8b3182bd6560bf5ef0345b0bc64173def85 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/21042 Reviewed-by: Giacomo Travaglini Maintainer: Giacomo Travaglini Tested-by: kokoro --- diff --git a/src/arch/arm/fastmodel/GIC/GIC.lisa b/src/arch/arm/fastmodel/GIC/GIC.lisa index b15c9ef5e..bb1e5649c 100644 --- a/src/arch/arm/fastmodel/GIC/GIC.lisa +++ b/src/arch/arm/fastmodel/GIC/GIC.lisa @@ -28,50 +28,6 @@ * Chun-Chen TK Hsu */ -protocol AMBAPVPPICommand -{ - properties - { - sc_master_port_class_name = - "amba_pv::signal_master_port"; - sc_slave_base_class_name = - "amba_pv::signal_slave_base"; - sc_slave_export_class_name = - "amba_pv::signal_slave_export"; - } - - includes - { - #include - #include "arch/arm/fastmodel/GIC/commands.hh" - } - - slave behavior set_state( - int export_id, const FastModel::PPICommand &value); -} - -protocol AMBAPVSPICommand -{ - properties - { - sc_master_port_class_name = - "amba_pv::signal_master_port"; - sc_slave_base_class_name = - "amba_pv::signal_slave_base"; - sc_slave_export_class_name = - "amba_pv::signal_slave_export"; - } - - includes - { - #include - #include "arch/arm/fastmodel/GIC/commands.hh" - } - - slave behavior set_state( - int export_id, const FastModel::SPICommand &value); -} - component GIC { composition @@ -410,13 +366,12 @@ component GIC #define setPPI(C) \ case C: ppi_##C[num].setValue(state); \ break - slave port ppi_command + slave port signal_interrupt { - behavior set_state(int export_id, const FastModel::PPICommand &value) + behavior ppi(uint8_t cpu, uint32_t num, bool state_val) { - sg::Signal::State state = value.state(); - uint32_t num = value.num(); - uint8_t cpu = value.cpu(); + sg::Signal::State state = + state_val ? sg::Signal::Set : sg::Signal::Clear; switch (cpu) { setPPI(0); setPPI(1); setPPI(2); setPPI(3); setPPI(4); @@ -475,7 +430,15 @@ component GIC sc_assert(false); } } + + behavior spi(uint32_t num, bool state_val) + { + sg::Signal::State state = + state_val ? sg::Signal::Set : sg::Signal::Clear; + spi[num].setValue(state); + } } + // CPU-side connections slave port cnthpirq; slave port cnthvirq; @@ -487,14 +450,6 @@ component GIC slave port pmuirq; slave port vcpumntirq; - slave port spi_command - { - behavior set_state(int export_id, const FastModel::SPICommand &value) - { - spi[value.num()].setValue(value.state()); - } - } - internal slave port spi[988]; internal slave port ppi_0[16]; diff --git a/src/arch/arm/fastmodel/GIC/GIC.sgproj b/src/arch/arm/fastmodel/GIC/GIC.sgproj index 2213bff1a..c01670587 100644 --- a/src/arch/arm/fastmodel/GIC/GIC.sgproj +++ b/src/arch/arm/fastmodel/GIC/GIC.sgproj @@ -21,5 +21,6 @@ files { path = "GIC.lisa"; path = "${PVLIB_HOME}/etc/sglib.sgrepo"; + path = "../protocol/SignalInterruptProtocol.lisa"; } } diff --git a/src/arch/arm/fastmodel/GIC/SConscript b/src/arch/arm/fastmodel/GIC/SConscript index c52304d2d..be5cbbded 100644 --- a/src/arch/arm/fastmodel/GIC/SConscript +++ b/src/arch/arm/fastmodel/GIC/SConscript @@ -30,7 +30,10 @@ Import('*') if not env['USE_ARM_FASTMODEL'] or env['TARGET_ISA'] != 'arm': Return() -ArmFastModelComponent(File('GIC.sgproj'), File('commands.hh'), - File('GIC.lisa')).prepare_env(env) +protocol_dir = Dir('..').Dir('protocol') + +ArmFastModelComponent(File('GIC.sgproj'), File('GIC.lisa'), + protocol_dir.File('SignalInterruptProtocol.lisa') + ).prepare_env(env) SimObject('FastModelGIC.py') Source('gic.cc') diff --git a/src/arch/arm/fastmodel/GIC/commands.hh b/src/arch/arm/fastmodel/GIC/commands.hh deleted file mode 100644 index 70f69c86c..000000000 --- a/src/arch/arm/fastmodel/GIC/commands.hh +++ /dev/null @@ -1,84 +0,0 @@ -/* - * Copyright 2019 Google, Inc. - * - * Redistribution and use in source and binary forms, with or without - * modification, are permitted provided that the following conditions are - * met: redistributions of source code must retain the above copyright - * notice, this list of conditions and the following disclaimer; - * redistributions in binary form must reproduce the above copyright - * notice, this list of conditions and the following disclaimer in the - * documentation and/or other materials provided with the distribution; - * neither the name of the copyright holders nor the names of its - * contributors may be used to endorse or promote products derived from - * this software without specific prior written permission. - * - * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS - * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT - * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR - * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT - * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, - * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT - * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, - * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY - * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT - * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE - * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - * - * Authors: Gabe Black - */ - -#ifndef __ARCH_ARM_FASTMODEL_GIC_COMMANDS_HH__ -#define __ARCH_ARM_FASTMODEL_GIC_COMMANDS_HH__ - -#include - -#include "base/logging.hh" -#include "sg/SGSignal.h" - -namespace FastModel -{ - -class PPICommand -{ - private: - uint8_t _cpu; - uint32_t _num; - sg::Signal::State _state; - - public: - PPICommand(uint8_t c, uint32_t n, bool set) : - _cpu(c), _num(n), _state(set ? sg::Signal::Set : sg::Signal::Clear) - { - panic_if(_cpu > 255, "PPICommand CPU out of bounds"); - panic_if(_num > 15, "PPICommand number out of bounds"); - } - - PPICommand() : PPICommand(0, 0, false) {} - - uint8_t cpu() const { return _cpu; } - uint32_t num() const { return _num; } - sg::Signal::State state() const { return _state; } -}; - -class SPICommand -{ - private: - uint32_t _num; - sg::Signal::State _state; - - public: - SPICommand(uint32_t n, bool set) : - _num(n), _state(set ? sg::Signal::Set : sg::Signal::Clear) - { - panic_if(_num > 987, "SPICommand number out of bounds"); - } - - SPICommand() : SPICommand(0, false) {} - - uint32_t num() const { return _num; } - sg::Signal::State state() const { return _state; } -}; - -} // namespace FastModel - -#endif // __ARCH_ARM_FASTMODEL_GIC_COMMANDS_HH__ diff --git a/src/arch/arm/fastmodel/GIC/gic.cc b/src/arch/arm/fastmodel/GIC/gic.cc index 2041e4186..cd332c62b 100644 --- a/src/arch/arm/fastmodel/GIC/gic.cc +++ b/src/arch/arm/fastmodel/GIC/gic.cc @@ -48,8 +48,7 @@ SCGIC::SCGIC(const SCFastModelGICParams ¶ms, pmuirqWrapper(pmuirq, params.name + ".pmuirq", -1), vcpumntirqWrapper(vcpumntirq, params.name + ".vcpumntirq", -1) { - ppiCommand.bind(ppi_command); - spiCommand.bind(spi_command); + signalInterrupt.bind(signal_interrupt); set_parameter("gic.enabled", params.enabled); set_parameter("gic.has-gicv3", params.has_gicv3); @@ -315,25 +314,25 @@ GIC::getPort(const std::string &if_name, PortID idx) void GIC::sendInt(uint32_t num) { - scGIC->spiCommand.set_state(0, SPICommand(num - 32, true)); + scGIC->signalInterrupt->spi(num - 32, true); } void GIC::clearInt(uint32_t num) { - scGIC->spiCommand.set_state(0, SPICommand(num - 32, false)); + scGIC->signalInterrupt->spi(num - 32, false); } void GIC::sendPPInt(uint32_t num, uint32_t cpu) { - scGIC->ppiCommand.set_state(0, PPICommand(cpu, num, true)); + scGIC->signalInterrupt->ppi(cpu, num, true); } void GIC::clearPPInt(uint32_t num, uint32_t cpu) { - scGIC->ppiCommand.set_state(0, PPICommand(cpu, num, false)); + scGIC->signalInterrupt->ppi(cpu, num, false); } } // namespace FastModel diff --git a/src/arch/arm/fastmodel/GIC/gic.hh b/src/arch/arm/fastmodel/GIC/gic.hh index 7910686cd..b5d37c42d 100644 --- a/src/arch/arm/fastmodel/GIC/gic.hh +++ b/src/arch/arm/fastmodel/GIC/gic.hh @@ -30,7 +30,8 @@ #ifndef __ARCH_ARM_FASTMODEL_GIC_GIC_HH__ #define __ARCH_ARM_FASTMODEL_GIC_GIC_HH__ -#include "arch/arm/fastmodel/GIC/commands.hh" +#include + #include "arch/arm/fastmodel/amba_ports.hh" #include "dev/arm/base_gic.hh" #include "params/FastModelGIC.hh" @@ -57,8 +58,7 @@ class SCGIC : public scx_evs_GIC public: SCGIC(const SCFastModelGICParams ¶ms, sc_core::sc_module_name _name); - amba_pv::signal_master_port ppiCommand; - amba_pv::signal_master_port spiCommand; + SignalInterruptInitiatorSocket signalInterrupt; sc_gem5::ScInterfaceWrapper cnthpirqWrapper; sc_gem5::ScInterfaceWrapper cnthvirqWrapper; diff --git a/src/arch/arm/fastmodel/protocol/SConscript b/src/arch/arm/fastmodel/protocol/SConscript index b0171f136..f82e04c2a 100644 --- a/src/arch/arm/fastmodel/protocol/SConscript +++ b/src/arch/arm/fastmodel/protocol/SConscript @@ -27,3 +27,5 @@ Depends('ExportedClockRateControlProtocol.lisa', 'exported_clock_rate_control.hh') +Depends('SignalInterruptProtocol.lisa', + 'signal_interrupt.hh') diff --git a/src/arch/arm/fastmodel/protocol/SignalInterruptProtocol.lisa b/src/arch/arm/fastmodel/protocol/SignalInterruptProtocol.lisa new file mode 100644 index 000000000..d8b9a6182 --- /dev/null +++ b/src/arch/arm/fastmodel/protocol/SignalInterruptProtocol.lisa @@ -0,0 +1,47 @@ +/* + * Copyright 2019 Google Inc. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions are + * met: redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer; + * redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution; + * neither the name of the copyright holders nor the names of its + * contributors may be used to endorse or promote products derived from + * this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + * + * Authors: Gabe Black + */ + +protocol SignalInterrupt +{ + includes + { + #include "arch/arm/fastmodel/protocol/signal_interrupt.hh" + } + properties + { + description = "Signal an interrupt from gem5."; + version = "1.0"; + dso_safe = 0; + sc_slave_base_class_name = "SignalInterruptSlaveBase"; + sc_slave_socket_class_name = "SignalInterruptTargetSocket"; + } + + slave behavior ppi(uint8_t cpu, uint32_t num, bool state); + slave behavior spi(uint32_t num, bool state); +} diff --git a/src/arch/arm/fastmodel/protocol/signal_interrupt.hh b/src/arch/arm/fastmodel/protocol/signal_interrupt.hh new file mode 100644 index 000000000..4cf047158 --- /dev/null +++ b/src/arch/arm/fastmodel/protocol/signal_interrupt.hh @@ -0,0 +1,121 @@ +/* + * Copyright 2019 Google, Inc. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions are + * met: redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer; + * redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution; + * neither the name of the copyright holders nor the names of its + * contributors may be used to endorse or promote products derived from + * this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + * + * Authors: Gabe Black + */ + +#ifndef __ARCH_ARM_FASTMODEL_PROTOCOL_SIGNAL_INTERRUPT_HH__ +#define __ARCH_ARM_FASTMODEL_PROTOCOL_SIGNAL_INTERRUPT_HH__ + +#include +#include +#include + +struct SignalInterruptDummyProtocolType {}; + +class SignalInterruptFwIf : public virtual sc_core::sc_interface +{ + public: + virtual ~SignalInterruptFwIf() {} + virtual void ppi(uint8_t cpu, uint32_t num, bool state) = 0; + virtual void spi(uint32_t num, bool state) = 0; +}; + +class SignalInterruptBwIf : public virtual sc_core::sc_interface +{ + public: + virtual ~SignalInterruptBwIf() {} +}; + +class SignalInterruptSlaveBase : public SignalInterruptFwIf +{ + public: + SignalInterruptSlaveBase(const std::string &name) {} +}; + +class SignalInterruptInitiatorSocket : + public tlm::tlm_base_initiator_socket<64, SignalInterruptFwIf, + SignalInterruptBwIf> +{ + private: + SignalInterruptBwIf dummyBwIf; + + public: + typedef tlm::tlm_base_initiator_socket<64, SignalInterruptFwIf, + SignalInterruptBwIf> Base; + + using Base::bind; + using Base::operator(); + + SignalInterruptInitiatorSocket() : Base() + { + get_base_export().bind(dummyBwIf); + } + SignalInterruptInitiatorSocket(const char *name) : Base(name) + { + get_base_export().bind(dummyBwIf); + } + + const char * + kind() const override + { + return "SignalInterruptInitiatorSocket"; + } + + std::type_index + get_protocol_types() const override + { + return typeid(SignalInterruptDummyProtocolType); + } +}; + +class SignalInterruptTargetSocket : + public tlm::tlm_base_target_socket<64, SignalInterruptFwIf, + SignalInterruptBwIf> +{ + public: + typedef tlm::tlm_base_target_socket<64, SignalInterruptFwIf, + SignalInterruptBwIf> Base; + + using Base::bind; + using Base::operator(); + + using Base::Base; + + const char * + kind() const override + { + return "SignalInterruptInitiatorSocket"; + } + + std::type_index + get_protocol_types() const override + { + return typeid(SignalInterruptDummyProtocolType); + } +}; + +#endif // __ARCH_ARM_FASTMODEL_PROTOCOL_SIGNAL_INTERRUPT_HH__