From 1afbebb7748f44f05ae76539a4d2f382a371ff00 Mon Sep 17 00:00:00 2001 From: Pavel Pisa Date: Mon, 1 Apr 2019 15:25:52 +0200 Subject: Coreview multiplexers updated and added for branch compare forward. Signed-off-by: Pavel Pisa --- qtmips_gui/coreview.cpp | 52 ++++++++++------ qtmips_gui/coreview.h | 3 + qtmips_gui/coreview/minimux.cpp | 120 ++++++++++++++++++++++++++++++++++++ qtmips_gui/coreview/minimux.h | 71 +++++++++++++++++++++ qtmips_gui/coreview/multiplexer.cpp | 18 +++--- qtmips_gui/coreview/multiplexer.h | 11 ++-- qtmips_gui/qtmips_gui.pro | 6 +- 7 files changed, 249 insertions(+), 32 deletions(-) create mode 100644 qtmips_gui/coreview/minimux.cpp create mode 100644 qtmips_gui/coreview/minimux.h (limited to 'qtmips_gui') diff --git a/qtmips_gui/coreview.cpp b/qtmips_gui/coreview.cpp index 242c416..061a3ec 100644 --- a/qtmips_gui/coreview.cpp +++ b/qtmips_gui/coreview.cpp @@ -65,6 +65,14 @@ connect(machine->core(), &machine::Core::SIG, \ VAR, &coreview::MultiText::multitext_update); \ } while(false) +#define NEW_MUX(VAR, X, Y, SIG, ...) do { \ + NEW(Multiplexer, VAR, X, Y, __VA_ARGS__); \ + connect(machine->core(), SIGNAL(SIG(std::uint32_t)), VAR, SLOT(set(std::uint32_t))); \ + } while(false) +#define NEW_MINIMUX(VAR, X, Y, SIG, ...) do { \ + NEW(MiniMux, VAR, X, Y, __VA_ARGS__); \ + connect(machine->core(), SIGNAL(SIG(std::uint32_t)), VAR, SLOT(set(std::uint32_t))); \ + } while(false) CoreViewScene::CoreViewScene(machine::QtMipsMachine *machine) : QGraphicsScene() { setSceneRect(0, 0, SC_WIDTH, SC_HEIGHT); @@ -85,7 +93,7 @@ CoreViewScene::CoreViewScene(machine::QtMipsMachine *machine) : QGraphicsScene() NEW_B(Constant, ft.adder_4, ft.adder->connector_in_b(), "4"); NEW(Junction, ft.junc_pc, 80, mem_program->connector_address()->y()); NEW(Junction, ft.junc_pc_4, 130, 380); - NEW(Multiplexer, ft.multiplex, 20, 390, 2); + NEW_MUX(ft.multiplex, 20, 390, fetch_branch_value, 2); // Decode stage NEW(LogicBlock, dc.ctl_block, 230, 90, {"Control", "unit"}); dc.ctl_block->setSize(35, 70); @@ -94,7 +102,7 @@ CoreViewScene::CoreViewScene(machine::QtMipsMachine *machine) : QGraphicsScene() NEW(Adder, dc.add, 340, 428); const coreview::Connector *dc_con_sign_ext = dc.sign_ext->new_connector(1, 0); NEW(Junction, dc.j_sign_ext, 290, dc_con_sign_ext->y()); - NEW(LogicBlock, dc.cmp, 320, 200, "="); + NEW(LogicBlock, dc.cmp, 312, 200, "="); NEW(And, dc.and_branch, 350, 190); dc.cmp->setSize(24, 12); NEW(Junction, dc.j_inst_up, 190, 126); @@ -103,8 +111,8 @@ CoreViewScene::CoreViewScene(machine::QtMipsMachine *machine) : QGraphicsScene() NEW(Junction, dc.j_jump_reg, 355, 94); // Execute stage NEW(Junction, ex.j_mux, 450, 303); - NEW(Multiplexer, ex.mux_imm, 470, 292, 2, true); - NEW(Multiplexer, ex.mux_regdest, 480, 370, 2, true); + NEW_MUX(ex.mux_imm, 470, 292, execute_alusrc_value, 2, true); + NEW_MUX(ex.mux_regdest, 480, 370, execute_regdest_value, 2, true); // Memory NEW(Junction, mm.j_addr, 570, mem_data->connector_address()->y()); static QMap excause_map = @@ -123,7 +131,7 @@ CoreViewScene::CoreViewScene(machine::QtMipsMachine *machine) : QGraphicsScene() NEW_MULTI(mm.multi_excause, 602, 447, memory_excause_value, excause_map, true); new_label("Exception", 595, 437); // WriteBack stage - NEW(Multiplexer, wb.mem_or_reg, 690, 252, 2, true); + NEW_MUX(wb.mem_or_reg, 690, 252, writeback_memtoreg_value, 2, true); NEW(Junction, wb.j_reg_write_val, 411, 510); // Connections // @@ -204,7 +212,7 @@ CoreViewScene::CoreViewScene(machine::QtMipsMachine *machine) : QGraphicsScene() NEW_V(560, 260, memory_alu_value, true); // Alu output NEW_V(560, 345, memory_rt_value, true); // rt NEW_V(650, 290, memory_mem_value, true); // Memory output - NEW_V(570, 113, execute_memtoreg_value, false, 1); + NEW_V(570, 113, memory_memtoreg_value, false, 1); NEW_V(630, 220, memory_memwrite_value, false, 1); NEW_V(620, 220, memory_memread_value, false, 1); // Write back stage @@ -377,12 +385,13 @@ CoreViewScenePipelined::CoreViewScenePipelined(machine::QtMipsMachine *machine) struct coreview::Latch::ConnectorPair lp_dc_rt = latch_id_ex->new_connector(regs->connector_read2()->y() - latch_id_ex->y()); coreview::Bus *regs_bus1 = new_bus(regs->connector_read1(), lp_dc_rs.in); coreview::Bus *regs_bus2 = new_bus(regs->connector_read2(), lp_dc_rt.in); - //if (machine->config().hazard_unit() == machine::MachineConfig::HU_NONE) { - const coreview::Connector *regs_bus_con = dc.cmp->new_connector(-0.5, 1); + const coreview::Connector *regs_bus_con; + if (machine->config().hazard_unit() != machine::MachineConfig::HU_STALL_FORWARD) { + regs_bus_con = dc.cmp->new_connector(-0.5, 1); new_bus(regs_bus1->new_connector(regs_bus_con->point(), coreview::Connector::AX_Y), regs_bus_con); regs_bus_con = dc.cmp->new_connector(0.5, 1); new_bus(regs_bus2->new_connector(regs_bus_con->point(), coreview::Connector::AX_Y), regs_bus_con); - //} // TODO else + } struct coreview::Latch::ConnectorPair lp_dc_immed = latch_id_ex->new_connector(dc.j_sign_ext->y() - latch_id_ex->y()); new_bus(dc.j_sign_ext->new_connector(coreview::Connector::AX_X), lp_dc_immed.in); struct coreview::Latch::ConnectorPair regdest_dc_rt = latch_id_ex->new_connector(ex.mux_regdest->connector_in(0)->point().y() - latch_id_ex->y()); @@ -489,8 +498,10 @@ CoreViewScenePipelined::CoreViewScenePipelined(machine::QtMipsMachine *machine) NEW_V(610, 385, memory_regw_num_value, false, 2, 0, 10); if (machine->config().hazard_unit() == machine::MachineConfig::HU_STALL_FORWARD) { - NEW(Multiplexer, hu.mux_alu_reg_a, 430, 232, 3, false); - NEW(Multiplexer, hu.mux_alu_reg_b, 430, 285, 3, false); + NEW_MUX(hu.mux_alu_reg_a, 430, 232, execute_reg1_ff_value, 3, false); + NEW_MUX(hu.mux_alu_reg_b, 430, 285, execute_reg2_ff_value, 3, false); + NEW_MINIMUX(hu.mux_branch_reg_a, 296, 228, forward_m_d_rs_value, 2, false); + NEW_MINIMUX(hu.mux_branch_reg_b, 314, 228, forward_m_d_rt_value, 2, false); NEW(Junction, hu.j_alu_out, 420, 490); con = new_bus(lp_dc_rs.out, hu.mux_alu_reg_a->connector_in(0)); @@ -507,11 +518,17 @@ CoreViewScenePipelined::CoreViewScenePipelined(machine::QtMipsMachine *machine) con = new_bus(hu.j_alu_out->new_connector(CON_AX_Y), hu.mux_alu_reg_a->connector_in(2)); con = new_bus(hu.j_alu_out->new_connector(CON_AX_Y), hu.mux_alu_reg_b->connector_in(2)); - con = new_bus(hu.j_alu_out->new_connector(CON_AX_X), dc.cmp->new_connector(-0.5, 1)); + con = new_bus(hu.j_alu_out->new_connector(CON_AX_X), hu.mux_branch_reg_a->connector_in(1)); con->setAxes({CON_AXIS_Y(380), CON_AXIS_X(330)}); - con = new_bus(hu.j_alu_out->new_connector(CON_AX_X), dc.cmp->new_connector(0.5, 1)); + con = new_bus(hu.j_alu_out->new_connector(CON_AX_X), hu.mux_branch_reg_b->connector_in(1)); con->setAxes({CON_AXIS_Y(380), CON_AXIS_X(330)}); + new_bus(regs_bus1->new_connector(hu.mux_branch_reg_a->connector_in(0)->point(), coreview::Connector::AX_Y), hu.mux_branch_reg_a->connector_in(0)); + new_bus(regs_bus2->new_connector(hu.mux_branch_reg_b->connector_in(0)->point(), coreview::Connector::AX_Y), hu.mux_branch_reg_b->connector_in(0)); + + con = new_bus(hu.mux_branch_reg_a->connector_out(), dc.cmp->new_connector(-0.75, 1)); + con = new_bus(hu.mux_branch_reg_b->connector_out(), dc.cmp->new_connector(0.75, 1)); + struct coreview::Latch::ConnectorPair regdest_dc_rs = latch_id_ex->new_connector(ex.mux_regdest->connector_in(0)->point().y() - latch_id_ex->y() - 8); new_bus(dc.instr_bus->new_connector(0, ex.mux_regdest->connector_in(0)->y() - 8), regdest_dc_rs.in, 2); new_label("RsE", 427, 364); @@ -519,12 +536,11 @@ CoreViewScenePipelined::CoreViewScenePipelined(machine::QtMipsMachine *machine) NEW(Junction, ex.j_rs_num, 442, 372); new_bus(regdest_dc_rs.out, ex.j_rs_num->new_connector(coreview::Connector::AX_X), 2); + NEW_V(434, 227, execute_reg1_ff_value, false, 1); // Register 1 forward to ALU + NEW_V(434, 280, execute_reg2_ff_value, false, 1); // Register 2 forward to ALU - NEW_V(434, 250, execute_reg1_ff_value, false, 1); // Register 1 forward to ALU - NEW_V(434, 303, execute_reg2_ff_value, false, 1); // Register 2 forward to ALU - - NEW_V(312, 290, forward_m_d_rs_value, false, 1); // Register 1 forward for bxx and jr, jalr - NEW_V(327, 290, forward_m_d_rt_value, false, 1); // Register 2 forward for beq, bne + NEW_V(291, 230, forward_m_d_rs_value, false, 1); // Register 1 forward for bxx and jr, jalr + NEW_V(333, 230, forward_m_d_rt_value, false, 1); // Register 2 forward for beq, bne } } diff --git a/qtmips_gui/coreview.h b/qtmips_gui/coreview.h index 93c0032..e788d26 100644 --- a/qtmips_gui/coreview.h +++ b/qtmips_gui/coreview.h @@ -43,6 +43,7 @@ #include "coreview/connection.h" #include "coreview/programcounter.h" #include "coreview/multiplexer.h" +#include "coreview/minimux.h" #include "coreview/latch.h" #include "coreview/alu.h" #include "coreview/adder.h" @@ -114,6 +115,8 @@ protected: coreview::Multiplexer *mux_alu_reg_b; coreview::Junction *j_alu_out; coreview::MultiText *multi_stall; + coreview::MiniMux *mux_branch_reg_a; + coreview::MiniMux *mux_branch_reg_b; } hu; coreview::Connection *new_connection(const coreview::Connector*, const coreview::Connector*); diff --git a/qtmips_gui/coreview/minimux.cpp b/qtmips_gui/coreview/minimux.cpp new file mode 100644 index 0000000..debd820 --- /dev/null +++ b/qtmips_gui/coreview/minimux.cpp @@ -0,0 +1,120 @@ +// SPDX-License-Identifier: GPL-2.0+ +/******************************************************************************* + * QtMips - MIPS 32-bit Architecture Subset Simulator + * + * Implemented to support following courses: + * + * B35APO - Computer Architectures + * https://cw.fel.cvut.cz/wiki/courses/b35apo + * + * B4M35PAP - Advanced Computer Architectures + * https://cw.fel.cvut.cz/wiki/courses/b4m35pap/start + * + * Copyright (c) 2017-2019 Karel Koci + * Copyright (c) 2019 Pavel Pisa + * + * Faculty of Electrical Engineering (http://www.fel.cvut.cz) + * Czech Technical University (http://www.cvut.cz/) + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, + * Boston, MA 02110-1301, USA. + * + ******************************************************************************/ + +#include "minimux.h" +#include + +using namespace coreview; + +////////////////////// +#define WIDTH 6 +#define HEIGHT 8 +#define GAP 8 +#define PENW 1 +////////////////////// + +MiniMux::MiniMux(unsigned size, bool ctl_up): QGraphicsObject(nullptr) { + this->size = size; + seton = -1; + ctlfrom = ctl_up; + con_ctl = new Connector(Connector::AX_X); + con_out = new Connector(Connector::AX_Y); + con_in = new Connector*[size]; + for (unsigned i = 0; i < size; i++) + con_in[i] = new Connector(Connector::AX_Y); + setPos(x(), y()); // Set connectors possitions +} + +MiniMux::~MiniMux() { + delete con_ctl; + delete con_out; + for (unsigned i = 0; i < size; i++) + delete con_in[i]; + delete[] con_in; +} + +#define C_WIDTH (WIDTH + (GAP * (size -1 ))) + +QRectF MiniMux::boundingRect() const { + return QRectF(-PENW / 2, -PENW / 2, C_WIDTH + PENW, WIDTH + PENW); +} + +void MiniMux::paint(QPainter *painter, const QStyleOptionGraphicsItem *option __attribute__((unused)), QWidget *widget __attribute__((unused))) { + painter->setPen(QPen(QColor(200, 200, 200), 2)); + if (seton >= 0 && seton < (int)size) + painter->drawLine((WIDTH / 2) + (seton * GAP), HEIGHT, C_WIDTH / 2, 0); + + painter->setPen(QColor(0, 0, 0)); + const QPointF poly[] = { + QPointF(0, HEIGHT), + QPointF(HEIGHT / 2, 0), + QPointF(C_WIDTH - (HEIGHT / 2), 0), + QPointF(C_WIDTH, HEIGHT) + }; + painter->drawPolygon(poly, sizeof(poly) / sizeof(QPointF)); +} + +void MiniMux::setPos(qreal x, qreal y) { + QGraphicsItem::setPos(x, y); + /*if (ctlfrom) + con_ctl->setPos(x + (WIDTH / 2), y + (WIDTH / 4)); + else + con_ctl->setPos(x + (WIDTH / 2), y + C_HEIGHT - (WIDTH / 4)); + */ + con_out->setPos(x + (C_WIDTH / 2), y); + for (unsigned i = 0; i < size; i++) + con_in[i]->setPos(x + (WIDTH / 2) + (i * GAP), y + HEIGHT); +} + +const Connector *MiniMux::connector_ctl() const { + return con_ctl; +} + +const Connector *MiniMux::connector_out() const { + return con_out; +} + +const Connector *MiniMux::connector_in(unsigned i) const { + SANITY_ASSERT(i < size, "MiniMux: requested out of range input connector"); + return con_in[i]; +} + +void MiniMux::set(std::uint32_t i) { + if (seton != (int)i) { + seton = (int)i; + update(); + } +} + diff --git a/qtmips_gui/coreview/minimux.h b/qtmips_gui/coreview/minimux.h new file mode 100644 index 0000000..9cc446f --- /dev/null +++ b/qtmips_gui/coreview/minimux.h @@ -0,0 +1,71 @@ +// SPDX-License-Identifier: GPL-2.0+ +/******************************************************************************* + * QtMips - MIPS 32-bit Architecture Subset Simulator + * + * Implemented to support following courses: + * + * B35APO - Computer Architectures + * https://cw.fel.cvut.cz/wiki/courses/b35apo + * + * B4M35PAP - Advanced Computer Architectures + * https://cw.fel.cvut.cz/wiki/courses/b4m35pap/start + * + * Copyright (c) 2017-2019 Karel Koci + * Copyright (c) 2019 Pavel Pisa + * + * Faculty of Electrical Engineering (http://www.fel.cvut.cz) + * Czech Technical University (http://www.cvut.cz/) + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License + * as published by the Free Software Foundation; either version 2 + * of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, + * Boston, MA 02110-1301, USA. + * + ******************************************************************************/ + +#ifndef COREVIEW_MINIMUX_H +#define COREVIEW_MINIMUX_H + +#include +#include "qtmipsexception.h" +#include "connection.h" + +namespace coreview { + +class MiniMux : public QGraphicsObject { + Q_OBJECT +public: + MiniMux(unsigned size, bool ctl_up = false); + ~MiniMux(); + + QRectF boundingRect() const; + void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget); + + void setPos(qreal x, qreal y); + const Connector *connector_ctl() const; // Control input + const Connector *connector_out() const; // Output + const Connector *connector_in(unsigned i) const; // Inputs + +public slots: + void set(std::uint32_t i); // Set what value should be set as connected (indexing from 1 where 0 is no line) + +private: + bool ctlfrom; + unsigned size; + int seton; + Connector *con_ctl, *con_out, **con_in; +}; + +} + +#endif // COREVIEW_MINIMUX_H diff --git a/qtmips_gui/coreview/multiplexer.cpp b/qtmips_gui/coreview/multiplexer.cpp index f0eebef..bd2fbd9 100644 --- a/qtmips_gui/coreview/multiplexer.cpp +++ b/qtmips_gui/coreview/multiplexer.cpp @@ -45,9 +45,9 @@ using namespace coreview; #define PENW 1 ////////////////////// -Multiplexer::Multiplexer(unsigned size, bool ctl_up) { +Multiplexer::Multiplexer(unsigned size, bool ctl_up): QGraphicsObject(nullptr) { this->size = size; - seton = 0; + seton = -1; ctlfrom = ctl_up; con_ctl = new Connector(Connector::AX_Y); con_out = new Connector(Connector::AX_X); @@ -72,9 +72,9 @@ QRectF Multiplexer::boundingRect() const { } void Multiplexer::paint(QPainter *painter, const QStyleOptionGraphicsItem *option __attribute__((unused)), QWidget *widget __attribute__((unused))) { - painter->setPen(QColor(200, 200, 200)); - if (seton > 0 && seton <= size) - painter->drawLine(0, (HEIGHT / 2) + ((seton - 1) * GAP), WIDTH, C_HEIGHT / 2); + painter->setPen(QPen(QColor(200, 200, 200), 2)); + if (seton >= 0 && seton < (int)size) + painter->drawLine(0, (HEIGHT / 2) + (seton * GAP), WIDTH, C_HEIGHT / 2); painter->setPen(QColor(0, 0, 0)); const QPointF poly[] = { @@ -110,8 +110,10 @@ const Connector *Multiplexer::connector_in(unsigned i) const { return con_in[i]; } -void Multiplexer::set(unsigned i) { - seton = i; - update(); +void Multiplexer::set(std::uint32_t i) { + if (seton != (int)i) { + seton = (int)i; + update(); + } } diff --git a/qtmips_gui/coreview/multiplexer.h b/qtmips_gui/coreview/multiplexer.h index 5de5184..0ba8cd9 100644 --- a/qtmips_gui/coreview/multiplexer.h +++ b/qtmips_gui/coreview/multiplexer.h @@ -36,13 +36,14 @@ #ifndef COREVIEW_MULTIPLEXER_H #define COREVIEW_MULTIPLEXER_H -#include +#include #include "qtmipsexception.h" #include "connection.h" namespace coreview { -class Multiplexer : public QGraphicsItem { +class Multiplexer : public QGraphicsObject { + Q_OBJECT public: Multiplexer(unsigned size, bool ctl_up = false); ~Multiplexer(); @@ -55,11 +56,13 @@ public: const Connector *connector_out() const; // Output const Connector *connector_in(unsigned i) const; // Inputs - void set(unsigned i); // Set what value should be set as connected (indexing from 1 where 0 is no line) +public slots: + void set(std::uint32_t i); // Set what value should be set as connected (indexing from 1 where 0 is no line) private: bool ctlfrom; - unsigned size, seton; + unsigned size; + int seton; Connector *con_ctl, *con_out, **con_in; }; diff --git a/qtmips_gui/qtmips_gui.pro b/qtmips_gui/qtmips_gui.pro index 905bdba..c0fbb70 100644 --- a/qtmips_gui/qtmips_gui.pro +++ b/qtmips_gui/qtmips_gui.pro @@ -67,7 +67,8 @@ SOURCES += \ fontsize.cpp \ gotosymboldialog.cpp \ cop0dock.cpp \ - hinttabledelegate.cpp + hinttabledelegate.cpp \ + coreview/minimux.cpp HEADERS += \ mainwindow.h \ @@ -107,7 +108,8 @@ HEADERS += \ fontsize.h \ gotosymboldialog.h \ cop0dock.h \ - hinttabledelegate.h + hinttabledelegate.h \ + coreview/minimux.h FORMS += \ NewDialog.ui \ -- cgit v1.2.3