From 9ef2928aefd0983c7b3b04023c0b958b86dd77ff Mon Sep 17 00:00:00 2001 From: bd Date: Thu, 24 Apr 2025 19:33:06 -0400 Subject: Fix presumed bug with illegal types --- src/id.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/id.cc b/src/id.cc index 0135c21..d8368c8 100644 --- a/src/id.cc +++ b/src/id.cc @@ -98,6 +98,7 @@ void ID::get_instr_fields( this->decode_J_type(s1, s2, s3, m); break; case 0b11: + m = NOP; this->status = OK; } } -- cgit v1.2.3 From a3528b83dd10fa9cdf7ef5635f615c1b295f3f4c Mon Sep 17 00:00:00 2001 From: bd Date: Fri, 25 Apr 2025 21:12:58 -0400 Subject: Pass full DTO to GUI --- gui/gui.cc | 59 +++++++++++++++++++++++++++++------------------------------ gui/gui.h | 10 +++++----- gui/worker.cc | 10 +++++----- gui/worker.h | 10 +++++----- inc/id.h | 2 -- inc/if.h | 2 -- inc/stage.h | 15 ++++++++------- src/id.cc | 10 ---------- src/if.cc | 9 --------- src/stage.cc | 13 +------------ 10 files changed, 53 insertions(+), 87 deletions(-) diff --git a/gui/gui.cc b/gui/gui.cc index 496a443..bf9e6cf 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -169,36 +169,36 @@ void GUI::on_worker_refresh_gui(int cycles, int pc) ui->cycle_counter->set_value(cycles); } -void GUI::onWorkerFetchInfo(const std::vector info) +void GUI::onWorkerFetchInfo(const InstrDTO *i) { - if (!info.empty()) { - ui->fetch_squashed->setText(QString::number(info[0])); - ui->fetch_bits->set_value(info[1]); + if (i) { + ui->fetch_squashed->setText(QString::number(i->is_squashed)); + ui->fetch_bits->set_value(i->slot_A); } else { ui->fetch_squashed->clear(); ui->fetch_bits->clear(); } } -void GUI::onWorkerDecodeInfo(const std::vector info) +void GUI::onWorkerDecodeInfo(const InstrDTO *i) { - if (!info.empty()) { - ui->decode_squashed->setText(QString::number(info[0])); - ui->decode_bits->set_value(info[1]); + if (i) { + ui->decode_squashed->setText(QString::number(i->is_squashed)); + ui->decode_bits->set_value(i->slot_A); } else { ui->decode_squashed->clear(); ui->decode_bits->clear(); } } -void GUI::onWorkerExecuteInfo(const std::vector info) +void GUI::onWorkerExecuteInfo(const InstrDTO *i) { - if (!info.empty()) { - ui->execute_mnemonic->setText(mnemonicToString((Mnemonic)info[0])); - ui->execute_squashed->setText(QString::number(info[1])); - ui->execute_s1->set_value(info[2]); - ui->execute_s2->set_value(info[3]); - ui->execute_s3->set_value(info[4]); + if (i) { + ui->execute_mnemonic->setText(mnemonicToString(i->mnemonic)); + ui->execute_squashed->setText(QString::number(i->is_squashed)); + ui->execute_s1->set_value(i->operands.integer.slot_one); + ui->execute_s2->set_value(i->operands.integer.slot_two); + ui->execute_s3->set_value(i->operands.integer.slot_three); } else { ui->execute_mnemonic->clear(); ui->execute_squashed->clear(); @@ -208,15 +208,14 @@ void GUI::onWorkerExecuteInfo(const std::vector info) } } -void GUI::onWorkerMemoryInfo(const std::vector info) +void GUI::onWorkerMemoryInfo(const InstrDTO *i) { - if (!info.empty()) { - std::cout << "this " << info[3] << std::endl; - ui->memory_mnemonic->setText(mnemonicToString((Mnemonic)info[0])); - ui->memory_squashed->setText(QString::number(info[1])); - ui->memory_s1->set_value(info[2]); - ui->memory_s2->set_value(info[3]); - ui->memory_s3->set_value(info[4]); + if (i) { + ui->memory_mnemonic->setText(mnemonicToString(i->mnemonic)); + ui->memory_squashed->setText(QString::number(i->is_squashed)); + ui->memory_s1->set_value(i->operands.integer.slot_one); + ui->memory_s2->set_value(i->operands.integer.slot_two); + ui->memory_s3->set_value(i->operands.integer.slot_three); } else { ui->memory_mnemonic->clear(); ui->memory_squashed->clear(); @@ -226,14 +225,14 @@ void GUI::onWorkerMemoryInfo(const std::vector info) } } -void GUI::onWorkerWriteBackInfo(const std::vector info) +void GUI::onWorkerWriteBackInfo(const InstrDTO *i) { - if (!info.empty()) { - ui->write_mnemonic->setText(mnemonicToString((Mnemonic)info[0])); - ui->write_squashed->setText(QString::number(info[1])); - ui->write_s1->set_value(info[2]); - ui->write_s2->set_value(info[3]); - ui->write_s3->set_value(info[4]); + if (i) { + ui->write_mnemonic->setText(mnemonicToString(i->mnemonic)); + ui->write_squashed->setText(QString::number(i->is_squashed)); + ui->write_s1->set_value(i->operands.integer.slot_one); + ui->write_s2->set_value(i->operands.integer.slot_two); + ui->write_s3->set_value(i->operands.integer.slot_three); } else { ui->write_mnemonic->clear(); ui->write_squashed->clear(); diff --git a/gui/gui.h b/gui/gui.h index 7b03f75..b3a3110 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -57,15 +57,15 @@ class GUI : public QMainWindow private slots: void on_worker_refresh_gui(int value, int pc); - void onWorkerFetchInfo(const std::vector info); + void onWorkerFetchInfo(const InstrDTO *); - void onWorkerDecodeInfo(const std::vector info); + void onWorkerDecodeInfo(const InstrDTO *); - void onWorkerExecuteInfo(const std::vector info); + void onWorkerExecuteInfo(const InstrDTO *); - void onWorkerMemoryInfo(const std::vector info); + void onWorkerMemoryInfo(const InstrDTO *); - void onWorkerWriteBackInfo(const std::vector info); + void onWorkerWriteBackInfo(const InstrDTO *); void onWorkerShowStorage( const std::vector> data, int i); diff --git a/gui/worker.cc b/gui/worker.cc index 2652fce..203f907 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -89,10 +89,10 @@ void Worker::runSteps(int steps) emit storage(this->s.at(i - 1)->view(0, 1 << this->size_inc * i), i + 1); emit clock_cycles(this->ct->get_clock_cycle(), this->ct->get_pc()); - emit if_info(this->if_stage->stage_info()); - emit id_info(this->id_stage->stage_info()); - emit ex_info(this->ex_stage->stage_info()); - emit mm_info(this->mm_stage->stage_info()); - emit wb_info(this->wb_stage->stage_info()); + emit if_info(this->if_stage->get_instr()); + emit id_info(this->id_stage->get_instr()); + emit ex_info(this->ex_stage->get_instr()); + emit mm_info(this->mm_stage->get_instr()); + emit wb_info(this->wb_stage->get_instr()); this->ct_mutex.unlock(); } diff --git a/gui/worker.h b/gui/worker.h index 95c81d5..072263a 100644 --- a/gui/worker.h +++ b/gui/worker.h @@ -72,11 +72,11 @@ class Worker : public QObject void storage(const std::vector> data, int i); void register_storage(const std::array data); - void if_info(const std::vector info); - void id_info(const std::vector info); - void ex_info(const std::vector info); - void mm_info(const std::vector info); - void wb_info(const std::vector info); + void if_info(const InstrDTO *); + void id_info(const InstrDTO *); + void ex_info(const InstrDTO *); + void mm_info(const InstrDTO *); + void wb_info(const InstrDTO *); void finished(); }; diff --git a/inc/id.h b/inc/id.h index 39485b6..aafe2e5 100644 --- a/inc/id.h +++ b/inc/id.h @@ -50,8 +50,6 @@ class ID : public Stage */ void write_guard(signed int &r); - std::vector stage_info() override; - private: /** * Helper for `get_instr_fields` diff --git a/inc/if.h b/inc/if.h index c374145..f0eb6e2 100644 --- a/inc/if.h +++ b/inc/if.h @@ -28,8 +28,6 @@ class IF : public Stage InstrDTO *advance(Response p) override; - std::vector stage_info() override; - private: void advance_helper() override; }; diff --git a/inc/stage.h b/inc/stage.h index 7dcb7b4..16f1235 100644 --- a/inc/stage.h +++ b/inc/stage.h @@ -53,8 +53,14 @@ class Stage * Must set the status to READY when the current instruction is evicted.. */ virtual InstrDTO *advance(Response p); - - virtual std::vector stage_info(); + /** + * @return the current instruction. + */ + InstrDTO *get_instr(); + /** + * Squashes the pipeline. + */ + void squash(); /* The following methods are made public so that they may be tested, and are * not to be called from outside classes during standard execution. @@ -72,11 +78,6 @@ class Stage */ void set_condition(CC c, bool v); - /** - * Squashes the pipeline. - */ - void squash(); - /** * The set of registers currently checked out. */ diff --git a/src/id.cc b/src/id.cc index d8368c8..d2a8f02 100644 --- a/src/id.cc +++ b/src/id.cc @@ -211,13 +211,3 @@ void ID::decode_J_type( } } - -std::vector ID::stage_info() -{ - std::vector info; - if (this->curr_instr) { - info.push_back(this->curr_instr->is_squashed); - info.push_back(this->curr_instr->slot_A); - } - return info; -} diff --git a/src/if.cc b/src/if.cc index 5f07ee2..0f622d4 100644 --- a/src/if.cc +++ b/src/if.cc @@ -37,15 +37,6 @@ InstrDTO *IF::advance(Response p) return r; } -std::vector IF::stage_info() { - std::vector info; - if(this->curr_instr){ - info.push_back(this->curr_instr->is_squashed); - info.push_back(this->curr_instr->slot_A); - } - return info; -} - void IF::advance_helper() { Response r; diff --git a/src/stage.cc b/src/stage.cc index 55756b6..4efe2fe 100644 --- a/src/stage.cc +++ b/src/stage.cc @@ -65,18 +65,7 @@ InstrDTO *Stage::advance(Response p) return r; } -std::vector Stage::stage_info() -{ - std::vector info; - if (this->curr_instr) { - info.push_back(this->curr_instr->mnemonic); - info.push_back(this->curr_instr->is_squashed); - info.push_back(this->curr_instr->operands.integer.slot_one); - info.push_back(this->curr_instr->operands.integer.slot_two); - info.push_back(this->curr_instr->operands.integer.slot_three); - } - return info; -} +InstrDTO *Stage::get_instr() { return this->curr_instr; } void Stage::set_condition(CC c, bool v) { -- cgit v1.2.3 From f3e6d5ed8acdeb3f5eccffb5aeeab2e7b040908c Mon Sep 17 00:00:00 2001 From: bd Date: Fri, 25 Apr 2025 21:19:10 -0400 Subject: Separate update method for worker into its own method --- gui/worker.cc | 16 ++++++++++------ gui/worker.h | 7 +++++++ 2 files changed, 17 insertions(+), 6 deletions(-) diff --git a/gui/worker.cc b/gui/worker.cc index 203f907..22f8738 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -69,24 +69,28 @@ void Worker::configure( delete old; this->ct_mutex.unlock(); - emit clock_cycles(this->ct->get_clock_cycle(), this->ct->get_pc()); + this->update(); } void Worker::runSteps(int steps) { - unsigned long i; - - this->ct_mutex.lock(); qDebug() << "Running for " << steps << "steps"; this->ct->run_for(steps); + this->update(); +} - // TODO move these to separate functions +void Worker::update() +{ + unsigned long i; + + this->ct_mutex.lock(); emit register_storage(this->ct->get_gprs()); emit storage(this->s.at(0)->view(0, 255), 1); for (i = 1; i < s.size(); ++i) - emit storage(this->s.at(i - 1)->view(0, 1 << this->size_inc * i), i + 1); + emit storage( + this->s.at(i - 1)->view(0, 1 << this->size_inc * i), i + 1); emit clock_cycles(this->ct->get_clock_cycle(), this->ct->get_pc()); emit if_info(this->if_stage->get_instr()); diff --git a/gui/worker.h b/gui/worker.h index 072263a..06e4a16 100644 --- a/gui/worker.h +++ b/gui/worker.h @@ -78,6 +78,13 @@ class Worker : public QObject void mm_info(const InstrDTO *); void wb_info(const InstrDTO *); void finished(); + + private: + /** + * Sets the GUI signals to update the storage, clock cycle, and stage + * displays. + */ + void update(); }; #endif // WORKER_H -- cgit v1.2.3 From 55a5a757dea26efad1e57fa7805c62ed19206ad3 Mon Sep 17 00:00:00 2001 From: bd Date: Fri, 25 Apr 2025 21:23:02 -0400 Subject: Remove onWorkerFinished --- gui/gui.cc | 6 ------ gui/gui.h | 2 -- gui/worker.cc | 3 --- 3 files changed, 11 deletions(-) diff --git a/gui/gui.cc b/gui/gui.cc index bf9e6cf..905b018 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -92,7 +92,6 @@ GUI::GUI(QWidget *parent) : QMainWindow(parent), ui(new Ui::GUI) }); // Proper cleanup when worker finishes - connect(worker, &Worker::finished, this, &GUI::onWorkerFinished); connect(worker, &Worker::finished, &workerThread, &QThread::quit); connect(&workerThread, &QThread::finished, worker, &QObject::deleteLater); @@ -254,12 +253,8 @@ void GUI::onWorkerShowRegisters(const std::array &data) displayArrayHTML(this->tab_text_boxes.at(0), data); } -void GUI::onWorkerFinished() { qDebug() << "Worker has finished processing."; } - void GUI::on_upload_intructions_btn_clicked() { - qDebug() << "Upload intructions button clicked."; - // why ui->register_table, or now ui->storage QString filePath = QFileDialog::getOpenFileName( ui->storage, "Open Binary File", QDir::homePath(), @@ -311,7 +306,6 @@ void GUI::on_base_toggle_checkbox_checkStateChanged(const Qt::CheckState &state) void GUI::on_step_btn_clicked() { - qDebug() << "Run step button clicked."; // try to configure first if (!this->ready) this->on_config_clicked(); diff --git a/gui/gui.h b/gui/gui.h index b3a3110..26e9b03 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -72,8 +72,6 @@ class GUI : public QMainWindow void onWorkerShowRegisters(const std::array &data); - void onWorkerFinished(); - void on_upload_intructions_btn_clicked(); void on_upload_program_state_btn_clicked(); diff --git a/gui/worker.cc b/gui/worker.cc index 22f8738..a0e21f6 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -23,8 +23,6 @@ Worker::Worker(QObject *parent) : QObject(parent) {} Worker::~Worker() { emit finished(); - qDebug() << "Worker destructor called in thread:" - << QThread::currentThread(); delete this->ct; } @@ -74,7 +72,6 @@ void Worker::configure( void Worker::runSteps(int steps) { - qDebug() << "Running for " << steps << "steps"; this->ct->run_for(steps); this->update(); } -- cgit v1.2.3 From 858a682d11cef5b7695050967b1c6b184eda3c6a Mon Sep 17 00:00:00 2001 From: bd Date: Fri, 25 Apr 2025 23:05:00 -0400 Subject: Add DigitLabelHelper, which centralizes integer display formatting --- gui/digitlabel.cc | 10 +++------- gui/digitlabelhelper.h | 43 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 46 insertions(+), 7 deletions(-) create mode 100644 gui/digitlabelhelper.h diff --git a/gui/digitlabel.cc b/gui/digitlabel.cc index 8314943..a24a1e1 100644 --- a/gui/digitlabel.cc +++ b/gui/digitlabel.cc @@ -16,6 +16,7 @@ // along with this program. If not, see . #include "digitlabel.h" +#include "digitlabelhelper.h" #include "gui.h" DigitLabel::DigitLabel(QWidget *parent) : QLabel(parent) { setText(QString()); } @@ -42,11 +43,6 @@ void DigitLabel::on_hex_toggle(bool is_hex) void DigitLabel::update_display() { QString t; - if (this->is_cleared) { - setText(QString()); - } else { - t = (this->is_hex) ? QString::number(this->v, 16).toUpper() - : QString::number(this->v); - setText(t); - } + t = DigitLabelHelper::format_value(this->v, this->is_hex, this->is_cleared); + setText(t); } diff --git a/gui/digitlabelhelper.h b/gui/digitlabelhelper.h new file mode 100644 index 0000000..a0f845f --- /dev/null +++ b/gui/digitlabelhelper.h @@ -0,0 +1,43 @@ +// Simulator for the RISC-V[ECTOR] mini-ISA +// Copyright (C) 2025 Siddarth Suresh +// Copyright (C) 2025 bdunahu + +// 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 3 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, see . + +#ifndef DIGITLABELHELPER_H +#define DIGITLABELHELPER_H + +#include + +class DigitLabelHelper +{ + public: + /** + * Contains the main formatting logic used to format integers. + * @param the value to be formated + * @param if the value should be displayed in hex. If false, displays in + * decimal. + * @param if the value should display. + * @return a string respecting the above parameters. + */ + static QString format_value(int value, bool is_hex, bool is_cleared = false) + { + if (is_cleared) + return QString(); + return is_hex ? QString::number(value, 16).toUpper() + : QString::number(value); + } +}; + +#endif // DIGITLABELHELPER_H -- cgit v1.2.3 From c98a0c26c4ccb5c4ae0e9f5810be910a7b299037 Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 03:06:14 -0400 Subject: Add proper tables display for storage devices --- gui/gui.cc | 71 ++++++++++++++------------------------- gui/gui.h | 6 ++-- gui/resources/styles.qss | 30 ++++++++++++----- gui/storageview.cc | 70 +++++++++++++++++++++++++++++++++++++++ gui/storageview.h | 86 ++++++++++++++++++++++++++++++++++++++++++++++++ gui/worker.cc | 20 +++++++++-- gui/worker.h | 9 ++++- 7 files changed, 229 insertions(+), 63 deletions(-) create mode 100644 gui/storageview.cc create mode 100644 gui/storageview.h diff --git a/gui/gui.cc b/gui/gui.cc index 905b018..9baed38 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -19,8 +19,11 @@ #include "./ui_gui.h" #include "dynamicwaysentry.h" #include "messages.h" +#include "storageview.h" #include #include +#include +#include GUI::GUI(QWidget *parent) : QMainWindow(parent), ui(new Ui::GUI) { @@ -131,37 +134,6 @@ void displayArrayHTML(QTextEdit *textEdit, const std::array &data) textEdit->setReadOnly(true); } -void displayTableHTML( - QTextEdit *textEdit, - const std::vector> &data) -{ - textEdit->setReadOnly(false); - QString tableText = ""; - - int index = 0; - for (const auto &row : data) { - tableText += ""; - for (signed int value : row) { - tableText += QString("") - .arg(QString::asprintf("%04X", value)) - .arg(index); - index++; - } - tableText += ""; - } - - tableText += "
" - "%1 %2" - "
"; - - textEdit->setHtml(tableText); - textEdit->setReadOnly(true); -} - void GUI::on_worker_refresh_gui(int cycles, int pc) { ui->p_counter->set_value(pc); @@ -241,16 +213,15 @@ void GUI::onWorkerWriteBackInfo(const InstrDTO *i) } } -void GUI::onWorkerShowStorage( - const std::vector> data, int i) +void GUI::onWorkerShowStorage(const QVector> &data, int i) { - std::cout << this->tab_text_boxes.size() << std::endl; - displayTableHTML(this->tab_text_boxes.at(i), data); + this->tab_boxes.at(i)->set_data(data); } void GUI::onWorkerShowRegisters(const std::array &data) { - displayArrayHTML(this->tab_text_boxes.at(0), data); + ; + // displayArrayHTML(this->tab_boxes.at(0), data); } void GUI::on_upload_intructions_btn_clicked() @@ -368,27 +339,33 @@ void GUI::on_config_clicked() void GUI::make_tabs(int num) { int i; - QStringList names; - QTextEdit *e; + QStringList xTra; + StorageView *e; + QTableView *t; QString n; - names = {"Registers", "DRAM"}; + xTra = {"Registers", "DRAM"}; ui->storage->clear(); - this->tab_text_boxes.clear(); + + this->tab_boxes.clear(); + qDeleteAll(this->tab_boxes); for (i = 0; i < num; ++i) { - e = new QTextEdit(); - e->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding); + e = new StorageView(10, this); + + t = new QTableView; + t->setModel(e); + t->horizontalHeader()->setSectionResizeMode(QHeaderView::Stretch); // make the name - if (i < names.size()) - n = names[i]; + if (i < xTra.size()) + n = xTra[i]; else - n = QString("Level %1").arg(i - 1); + n = QString("L%1").arg(i - 1); - ui->storage->addTab(e, n); - this->tab_text_boxes.push_back(e); + ui->storage->addTab(t, n); + this->tab_boxes.push_back(e); } } diff --git a/gui/gui.h b/gui/gui.h index 26e9b03..f723729 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -18,6 +18,7 @@ #ifndef GUI_H #define GUI_H +#include "storageview.h" #include "worker.h" #include #include @@ -67,8 +68,7 @@ class GUI : public QMainWindow void onWorkerWriteBackInfo(const InstrDTO *); - void onWorkerShowStorage( - const std::vector> data, int i); + void onWorkerShowStorage(const QVector> &data, int i); void onWorkerShowRegisters(const std::array &data); @@ -104,7 +104,7 @@ class GUI : public QMainWindow /** * The list of storage displays. */ - std::vector tab_text_boxes; + std::vector tab_boxes; /** * Whether or not numerical values are currently displaying in hex. diff --git a/gui/resources/styles.qss b/gui/resources/styles.qss index 76d0311..a61035e 100644 --- a/gui/resources/styles.qss +++ b/gui/resources/styles.qss @@ -50,15 +50,27 @@ QGroupBox::title { QLabel { } -/* text entry */ -QLineEdit { - font-size: 18pt; - border-radius: 0px; - padding: 0 4px; - selection-background-color: "#00cc00"; +QTableView { + border: 0px; + selection-background-color: transparent; + selection-color: black; + outline: none; +} + +QTableView::item { + border: none; + padding: 4px; +} + +QHeaderView::section { + color: "#00cc00"; + background-color: "#000200"; + border: none; } -QTextEdit, QListView { +QTableView QTableCornerButton::section { + background-color: "#000200"; + border: none; } QPushButton { @@ -71,8 +83,8 @@ QPushButton { QPushButton:pressed { color: "#00cc00"; - background-color: "#000200"; -} + background-color: "#000200";} + QPushButton:flat { border: none; /* no border for a flat push button */ diff --git a/gui/storageview.cc b/gui/storageview.cc new file mode 100644 index 0000000..f6f9736 --- /dev/null +++ b/gui/storageview.cc @@ -0,0 +1,70 @@ +// Simulator for the RISC-V[ECTOR] mini-ISA +// Copyright (C) 2025 Siddarth Suresh +// Copyright (C) 2025 bdunahu + +// 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 3 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, see . + +#include "storageview.h" +#include "definitions.h" +#include +#include + +StorageView::StorageView(int rows, QObject *parent) + : QAbstractTableModel(parent) +{ + this->r = rows; + this->d.resize(rows); + for (auto &row : this->d) + row.resize(LINE_SIZE, 0); +} + +int StorageView::rowCount(const QModelIndex &) const { return this->r; } + +int StorageView::columnCount(const QModelIndex &) const { return LINE_SIZE; } + +QVariant StorageView::data(const QModelIndex &i, int role) const +{ + Qt::Alignment a; + + if (role == Qt::TextAlignmentRole) { + a = Qt::AlignRight | Qt::AlignVCenter; + return QVariant(static_cast(a)); + } + if (!i.isValid() || role != Qt::DisplayRole) + return QVariant(); + return this->d[i.row()][i.column()]; +} + +QVariant StorageView::headerData(int section, Qt::Orientation o, int role) const +{ + if (role != Qt::DisplayRole) + return QVariant(); + + if (o == Qt::Vertical) { + return section * 4; + } + return QVariant(); +} + +Qt::ItemFlags StorageView::flags(const QModelIndex &i) const { + (void)i; + return Qt::ItemIsEnabled; +} + +void StorageView::set_data(const QVector> &data) +{ + beginResetModel(); + this->d = data; + endResetModel(); +} diff --git a/gui/storageview.h b/gui/storageview.h new file mode 100644 index 0000000..4956f23 --- /dev/null +++ b/gui/storageview.h @@ -0,0 +1,86 @@ +// Simulator for the RISC-V[ECTOR] mini-ISA +// Copyright (C) 2025 Siddarth Suresh +// Copyright (C) 2025 bdunahu + +// 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 3 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, see . + +#ifndef STORAGEVIEW_H +#define STORAGEVIEW_H + +#include +#include + +// see https://doc.qt.io/qt-6/qabstracttablemodel.html +class StorageView : public QAbstractTableModel +{ + Q_OBJECT + public: + /** + * Constructor. Initializes a clean StorageView object with + * `rows' rows. + * @param the number of rows + */ + StorageView(int rows, QObject *parent = nullptr); + + /** + * Returns the number of rows in this table. + * @param the parent + * @return the number of rows under the given parent. + */ + int rowCount(const QModelIndex &) const override; + /** + * Returns the number of columns in this table. + * @param the parent + * @return the number of columns under the given parent (hint: it's + * LINE_SIZE) + */ + int columnCount(const QModelIndex &) const override; + + /** + * Returns a properly formatted cell, including alignment.This function is + * specific to the implementation details of QAbstractTableModel. + */ + QVariant + data(const QModelIndex &index, int role = Qt::DisplayRole) const override; + + /** + * Adds custom formatting options for row and column headers. + */ + QVariant headerData( + int section, + Qt::Orientation o, + int role = Qt::DisplayRole) const override; + + /** + * Ensures the table is enabled, but not selectable. + */ + Qt::ItemFlags flags(const QModelIndex &i) const override; + + /** + * @param field to assign to `this->d' + */ + void set_data(const QVector> &data); + + private: + /** + * The number of rows in this table. + */ + int r; + /** + * The data this table displays. + */ + QVector> d; +}; + +#endif // STORAGEVIEW_H diff --git a/gui/worker.cc b/gui/worker.cc index a0e21f6..6419b73 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -83,11 +83,10 @@ void Worker::update() this->ct_mutex.lock(); emit register_storage(this->ct->get_gprs()); - emit storage(this->s.at(0)->view(0, 255), 1); + emit storage(this->data_to_QT(this->s.at(0)->get_data()), 1); for (i = 1; i < s.size(); ++i) - emit storage( - this->s.at(i - 1)->view(0, 1 << this->size_inc * i), i + 1); + emit storage(this->data_to_QT(this->s.at(i - 1)->get_data()), i + 1); emit clock_cycles(this->ct->get_clock_cycle(), this->ct->get_pc()); emit if_info(this->if_stage->get_instr()); @@ -97,3 +96,18 @@ void Worker::update() emit wb_info(this->wb_stage->get_instr()); this->ct_mutex.unlock(); } + +QVector> +Worker::data_to_QT(std::vector> data) +{ + QVector> r; + QVector tmp; + + r.reserve(static_cast(data.size())); + + for (const auto &line : data) { + tmp = QVector(line.begin(), line.end()); + r.append(tmp); + } + return r; +} diff --git a/gui/worker.h b/gui/worker.h index 06e4a16..c0e72d3 100644 --- a/gui/worker.h +++ b/gui/worker.h @@ -70,7 +70,7 @@ class Worker : public QObject signals: void clock_cycles(int value, int pc); void - storage(const std::vector> data, int i); + storage(QVector> data, int i); void register_storage(const std::array data); void if_info(const InstrDTO *); void id_info(const InstrDTO *); @@ -80,6 +80,13 @@ class Worker : public QObject void finished(); private: + /** + * Converts a vector of arrays into a QVector of QVectors. + * @param the original data + * @return a less universal version of the same thing + */ + QVector> + data_to_QT(std::vector> data); /** * Sets the GUI signals to update the storage, clock cycle, and stage * displays. -- cgit v1.2.3 From a78163745b43a0c420ae4ea5792def30a94420eb Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 03:39:28 -0400 Subject: Partial cache size generation, full cache display --- gui/gui.cc | 26 +++++++++++++++----------- gui/gui.h | 5 +++++ gui/util.cc | 16 ++++++++++++++++ gui/util.h | 8 ++++++++ gui/worker.cc | 8 +++----- gui/worker.h | 5 ----- 6 files changed, 47 insertions(+), 21 deletions(-) create mode 100644 gui/util.cc create mode 100644 gui/util.h diff --git a/gui/gui.cc b/gui/gui.cc index 9baed38..ecdf9d3 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -20,10 +20,11 @@ #include "dynamicwaysentry.h" #include "messages.h" #include "storageview.h" +#include "util.h" +#include #include #include #include -#include GUI::GUI(QWidget *parent) : QMainWindow(parent), ui(new Ui::GUI) { @@ -332,6 +333,8 @@ void GUI::on_config_clicked() else this->set_status(get_initialize, "happy"); + this->curr_cache_levels = ways.size(); + emit sendConfigure(ways, this->p, is_pipelined); make_tabs(2 + ways.size()); } @@ -339,31 +342,32 @@ void GUI::on_config_clicked() void GUI::make_tabs(int num) { int i; - QStringList xTra; StorageView *e; QTableView *t; QString n; - xTra = {"Registers", "DRAM"}; - ui->storage->clear(); this->tab_boxes.clear(); qDeleteAll(this->tab_boxes); for (i = 0; i < num; ++i) { - e = new StorageView(10, this); + // make the name + if (i == 0) { + n = "Registers"; + e = new StorageView(0, this); + } else if (i == 1) { + n = "DRAM"; + e = new StorageView(MEM_LINES, this); + } else { + n = QString("L%1").arg(i - 1); + e = new StorageView(cache_size_mapper(this->curr_cache_levels, i), this); + } t = new QTableView; t->setModel(e); t->horizontalHeader()->setSectionResizeMode(QHeaderView::Stretch); - // make the name - if (i < xTra.size()) - n = xTra[i]; - else - n = QString("L%1").arg(i - 1); - ui->storage->addTab(t, n); this->tab_boxes.push_back(e); } diff --git a/gui/gui.h b/gui/gui.h index f723729..10c8f67 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -101,6 +101,11 @@ class GUI : public QMainWindow */ bool ready; + /** + * The current number of cache levels. + */ + int curr_cache_levels; + /** * The list of storage displays. */ diff --git a/gui/util.cc b/gui/util.cc new file mode 100644 index 0000000..ee75e56 --- /dev/null +++ b/gui/util.cc @@ -0,0 +1,16 @@ +#include "util.h" + +int cache_size_mapper(int total_levels, int level) +{ + const int y_min = 4; + const int y_max = 12; + double f, r; + + if (total_levels <= 1) + return 8; + + f = level / total_levels; + r = y_min + f * (y_max - y_min); + + return r; +} diff --git a/gui/util.h b/gui/util.h new file mode 100644 index 0000000..87c33f6 --- /dev/null +++ b/gui/util.h @@ -0,0 +1,8 @@ +/** + * Given `total_levels', returns an integer between 4 and 12 which is a linear map of `level' onto `total_levels'. + * This is used for generating cache sizes given a number of levels. + * @param the total number of cache levels, zero-indexed. + * @param a numberedcache level, zero-indexed. + * @return an integer between 4-12, linearly scaled with level. + */ +int cache_size_mapper(int total_levels, int level); diff --git a/gui/worker.cc b/gui/worker.cc index 6419b73..93ccbea 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -17,6 +17,7 @@ #include "worker.h" #include "storage.h" +#include "util.h" Worker::Worker(QObject *parent) : QObject(parent) {} @@ -39,10 +40,6 @@ void Worker::configure( this->s.clear(); this->ct_mutex.lock(); - if (ways.size() != 0) { - // TODO optimal proper sizes - this->size_inc = ((MEM_LINE_SPEC * 0.75) / ways.size()); - } d = new Dram(DRAM_DELAY); s = static_cast(d); @@ -51,7 +48,8 @@ void Worker::configure( for (i = ways.size(); i > 0; --i) { s = static_cast(new Cache( - s, this->size_inc * (i), ways.at(i - 1), CACHE_DELAY + i)); + s, cache_size_mapper(ways.size() - 1, i), ways.at(i - 1), + CACHE_DELAY + i)); this->s.push_front(s); } diff --git a/gui/worker.h b/gui/worker.h index c0e72d3..c62f4ed 100644 --- a/gui/worker.h +++ b/gui/worker.h @@ -50,11 +50,6 @@ class Worker : public QObject Controller *ct = nullptr; QMutex ct_mutex; - /** - * The size each progressive cache level increases by. - */ - unsigned int size_inc; - public: explicit Worker(QObject *parent = nullptr); ~Worker(); -- cgit v1.2.3 From 94c0d7b5045244f20dfa13f7d31e0e06901908c2 Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 12:32:34 -0400 Subject: Add a digit delegate to toggle table base display --- gui/digitlabeldelegate.cc | 57 +++++++++++++++++++++++++++++++++++++++++++++++ gui/digitlabeldelegate.h | 47 ++++++++++++++++++++++++++++++++++++++ gui/gui.cc | 12 ++++++++-- 3 files changed, 114 insertions(+), 2 deletions(-) create mode 100644 gui/digitlabeldelegate.cc create mode 100644 gui/digitlabeldelegate.h diff --git a/gui/digitlabeldelegate.cc b/gui/digitlabeldelegate.cc new file mode 100644 index 0000000..641b5a0 --- /dev/null +++ b/gui/digitlabeldelegate.cc @@ -0,0 +1,57 @@ +// Simulator for the RISC-V[ECTOR] mini-ISA +// Copyright (C) 2025 Siddarth Suresh +// Copyright (C) 2025 bdunahu + +// 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 3 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, see . + +#include "digitlabeldelegate.h" +#include "digitlabelhelper.h" +#include + +DigitLabelDelegate::DigitLabelDelegate(QObject *parent) + : QStyledItemDelegate(parent) +{ + ; +} + +void DigitLabelDelegate::set_hex_display(bool hex) +{ + if (this->is_hex != hex) { + this->is_hex = hex; + if (auto v = qobject_cast(parent())) + v->viewport()->update(); + } +} + +void DigitLabelDelegate::paint( + QPainter *painter, + const QStyleOptionViewItem &option, + const QModelIndex &index) const +{ + int v; + QString t; + QStyleOptionViewItem o; + QStyle *s; + + v = index.data(Qt::DisplayRole).toInt(); + t = DigitLabelHelper::format_value(v, this->is_hex); + + o = option; + initStyleOption(&o, index); + o.text = t; + + const QWidget *w = option.widget; + s = w ? w->style() : QApplication::style(); + s->drawControl(QStyle::CE_ItemViewItem, &o, painter, w); +} diff --git a/gui/digitlabeldelegate.h b/gui/digitlabeldelegate.h new file mode 100644 index 0000000..a823154 --- /dev/null +++ b/gui/digitlabeldelegate.h @@ -0,0 +1,47 @@ +// Simulator for the RISC-V[ECTOR] mini-ISA +// Copyright (C) 2025 Siddarth Suresh +// Copyright (C) 2025 bdunahu + +// 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 3 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, see . + +#ifndef DIGITLABELDELEGATE_H +#define DIGITLABELDELEGATE_H + +#include +#include +#include +#include +#include +#include + +class DigitLabelDelegate : public QStyledItemDelegate +{ + Q_OBJECT + + public: + explicit DigitLabelDelegate(QObject *parent = nullptr); + + public slots: + void set_hex_display(bool hex); + + private: + bool is_hex = true; + + void paint( + QPainter *painter, + const QStyleOptionViewItem &option, + const QModelIndex &index) const override; +}; + +#endif // DIGITLABELDELEGATE_H diff --git a/gui/gui.cc b/gui/gui.cc index ecdf9d3..80d4a2a 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -17,6 +17,7 @@ #include "gui.h" #include "./ui_gui.h" +#include "digitlabeldelegate.h" #include "dynamicwaysentry.h" #include "messages.h" #include "storageview.h" @@ -57,7 +58,6 @@ GUI::GUI(QWidget *parent) : QMainWindow(parent), ui(new Ui::GUI) for (DigitLabel *label : labels) { connect(this, &GUI::hex_toggled, label, &DigitLabel::on_hex_toggle); } - emit this->hex_toggled(this->is_hex); // display clock cycles and PC connect(worker, &Worker::clock_cycles, this, &GUI::on_worker_refresh_gui); @@ -345,6 +345,7 @@ void GUI::make_tabs(int num) StorageView *e; QTableView *t; QString n; + DigitLabelDelegate *d; ui->storage->clear(); @@ -361,11 +362,18 @@ void GUI::make_tabs(int num) e = new StorageView(MEM_LINES, this); } else { n = QString("L%1").arg(i - 1); - e = new StorageView(cache_size_mapper(this->curr_cache_levels, i), this); + e = new StorageView( + cache_size_mapper(this->curr_cache_levels, i), this); } t = new QTableView; t->setModel(e); + d = new DigitLabelDelegate(t); + + connect( + this, &GUI::hex_toggled, d, &DigitLabelDelegate::set_hex_display); + + t->setItemDelegate(d); t->horizontalHeader()->setSectionResizeMode(QHeaderView::Stretch); ui->storage->addTab(t, n); -- cgit v1.2.3 From 1c174cc103cb1b00befed6e52b40bb3e76cc7797 Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 14:29:48 -0400 Subject: Use two's complement in number display --- gui/digitlabel.h | 2 +- gui/digitlabelhelper.h | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/gui/digitlabel.h b/gui/digitlabel.h index 68b01cb..bffdd0a 100644 --- a/gui/digitlabel.h +++ b/gui/digitlabel.h @@ -59,7 +59,7 @@ class DigitLabel : public QLabel /** * If this digit should display in hexidecinmal. */ - int is_hex; + int is_hex = true; /** * If this digit should not display. */ diff --git a/gui/digitlabelhelper.h b/gui/digitlabelhelper.h index a0f845f..715089a 100644 --- a/gui/digitlabelhelper.h +++ b/gui/digitlabelhelper.h @@ -24,7 +24,8 @@ class DigitLabelHelper { public: /** - * Contains the main formatting logic used to format integers. + * Contains the main formatting logic used to format integers. Uses 2's + * complement for hexadecimal numbers. * @param the value to be formated * @param if the value should be displayed in hex. If false, displays in * decimal. @@ -35,7 +36,7 @@ class DigitLabelHelper { if (is_cleared) return QString(); - return is_hex ? QString::number(value, 16).toUpper() + return is_hex ? QString::asprintf("%04X", value) : QString::number(value); } }; -- cgit v1.2.3 From b32bc409c18ceb4cd8147f11021e2c4b2746184b Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 14:41:02 -0400 Subject: right-align colum headers on table objects --- gui/digitlabeldelegate.cc | 6 ------ gui/digitlabeldelegate.h | 3 +-- gui/digitlabelhelper.h | 2 +- gui/storageview.cc | 7 +++++++ 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/gui/digitlabeldelegate.cc b/gui/digitlabeldelegate.cc index 641b5a0..bf4b175 100644 --- a/gui/digitlabeldelegate.cc +++ b/gui/digitlabeldelegate.cc @@ -19,12 +19,6 @@ #include "digitlabelhelper.h" #include -DigitLabelDelegate::DigitLabelDelegate(QObject *parent) - : QStyledItemDelegate(parent) -{ - ; -} - void DigitLabelDelegate::set_hex_display(bool hex) { if (this->is_hex != hex) { diff --git a/gui/digitlabeldelegate.h b/gui/digitlabeldelegate.h index a823154..7714d25 100644 --- a/gui/digitlabeldelegate.h +++ b/gui/digitlabeldelegate.h @@ -28,9 +28,8 @@ class DigitLabelDelegate : public QStyledItemDelegate { Q_OBJECT - public: - explicit DigitLabelDelegate(QObject *parent = nullptr); + using QStyledItemDelegate::QStyledItemDelegate; public slots: void set_hex_display(bool hex); diff --git a/gui/digitlabelhelper.h b/gui/digitlabelhelper.h index 715089a..26e4637 100644 --- a/gui/digitlabelhelper.h +++ b/gui/digitlabelhelper.h @@ -36,7 +36,7 @@ class DigitLabelHelper { if (is_cleared) return QString(); - return is_hex ? QString::asprintf("%04X", value) + return is_hex ? QString::asprintf("%X", value) : QString::number(value); } }; diff --git a/gui/storageview.cc b/gui/storageview.cc index f6f9736..43837a9 100644 --- a/gui/storageview.cc +++ b/gui/storageview.cc @@ -48,6 +48,13 @@ QVariant StorageView::data(const QModelIndex &i, int role) const QVariant StorageView::headerData(int section, Qt::Orientation o, int role) const { + Qt::Alignment a; + + if (role == Qt::TextAlignmentRole) { + a = Qt::AlignRight | Qt::AlignVCenter; + return QVariant(static_cast(a)); + } + if (role != Qt::DisplayRole) return QVariant(); -- cgit v1.2.3 From ae2aaecfc1b2402a55e99cf674eff7b6175b0b6d Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 14:57:56 -0400 Subject: Basic hex formatting on row headers for storage tables --- gui/digitlabelhelper.h | 3 +-- gui/gui.cc | 3 ++- gui/storageview.cc | 15 +++++++++++++-- gui/storageview.h | 7 +++++++ 4 files changed, 23 insertions(+), 5 deletions(-) diff --git a/gui/digitlabelhelper.h b/gui/digitlabelhelper.h index 26e4637..4d45c68 100644 --- a/gui/digitlabelhelper.h +++ b/gui/digitlabelhelper.h @@ -36,8 +36,7 @@ class DigitLabelHelper { if (is_cleared) return QString(); - return is_hex ? QString::asprintf("%X", value) - : QString::number(value); + return is_hex ? QString::asprintf("%X", value) : QString::number(value); } }; diff --git a/gui/gui.cc b/gui/gui.cc index 80d4a2a..2555435 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -353,7 +353,6 @@ void GUI::make_tabs(int num) qDeleteAll(this->tab_boxes); for (i = 0; i < num; ++i) { - // make the name if (i == 0) { n = "Registers"; e = new StorageView(0, this); @@ -370,6 +369,8 @@ void GUI::make_tabs(int num) t->setModel(e); d = new DigitLabelDelegate(t); + connect( + this, &GUI::hex_toggled, e, &StorageView::set_hex_display); connect( this, &GUI::hex_toggled, d, &DigitLabelDelegate::set_hex_display); diff --git a/gui/storageview.cc b/gui/storageview.cc index 43837a9..22baca0 100644 --- a/gui/storageview.cc +++ b/gui/storageview.cc @@ -17,6 +17,7 @@ #include "storageview.h" #include "definitions.h" +#include "digitlabelhelper.h" #include #include @@ -59,12 +60,13 @@ QVariant StorageView::headerData(int section, Qt::Orientation o, int role) const return QVariant(); if (o == Qt::Vertical) { - return section * 4; + return DigitLabelHelper::format_value(section * 4, this->is_hex); } return QVariant(); } -Qt::ItemFlags StorageView::flags(const QModelIndex &i) const { +Qt::ItemFlags StorageView::flags(const QModelIndex &i) const +{ (void)i; return Qt::ItemIsEnabled; } @@ -75,3 +77,12 @@ void StorageView::set_data(const QVector> &data) this->d = data; endResetModel(); } + +void StorageView::set_hex_display(bool hex) +{ + if (this->is_hex != hex) { + beginResetModel(); + this->is_hex = hex; + endResetModel(); + } +} diff --git a/gui/storageview.h b/gui/storageview.h index 4956f23..0518d8f 100644 --- a/gui/storageview.h +++ b/gui/storageview.h @@ -72,11 +72,18 @@ class StorageView : public QAbstractTableModel */ void set_data(const QVector> &data); + public slots: + void set_hex_display(bool hex); + private: /** * The number of rows in this table. */ int r; + /** + * Whether or not the headers should be displayed in hex. + */ + bool is_hex = true; /** * The data this table displays. */ -- cgit v1.2.3 From 118427baa510419041c3f8e9c7a4616b93f01672 Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 15:02:09 -0400 Subject: Decide to zero pad all hexadecimal numbers --- gui/digitlabelhelper.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gui/digitlabelhelper.h b/gui/digitlabelhelper.h index 4d45c68..744bf0f 100644 --- a/gui/digitlabelhelper.h +++ b/gui/digitlabelhelper.h @@ -36,7 +36,7 @@ class DigitLabelHelper { if (is_cleared) return QString(); - return is_hex ? QString::asprintf("%X", value) : QString::number(value); + return is_hex ? QString::asprintf("%08X", value) : QString::number(value); } }; -- cgit v1.2.3 From 6382d595cf947eb54249ff5fea20d8eb073ef3c1 Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 15:11:19 -0400 Subject: Move digit formatter to util.h --- gui/digitlabel.cc | 4 ++-- gui/digitlabeldelegate.cc | 4 ++-- gui/digitlabelhelper.h | 43 ------------------------------------------- gui/storageview.cc | 4 ++-- gui/util.cc | 8 ++++++++ gui/util.h | 23 +++++++++++++++++++++-- 6 files changed, 35 insertions(+), 51 deletions(-) delete mode 100644 gui/digitlabelhelper.h diff --git a/gui/digitlabel.cc b/gui/digitlabel.cc index a24a1e1..f77c1fa 100644 --- a/gui/digitlabel.cc +++ b/gui/digitlabel.cc @@ -16,7 +16,7 @@ // along with this program. If not, see . #include "digitlabel.h" -#include "digitlabelhelper.h" +#include "util.h" #include "gui.h" DigitLabel::DigitLabel(QWidget *parent) : QLabel(parent) { setText(QString()); } @@ -43,6 +43,6 @@ void DigitLabel::on_hex_toggle(bool is_hex) void DigitLabel::update_display() { QString t; - t = DigitLabelHelper::format_value(this->v, this->is_hex, this->is_cleared); + t = format_toggled_value(this->v, this->is_hex, this->is_cleared); setText(t); } diff --git a/gui/digitlabeldelegate.cc b/gui/digitlabeldelegate.cc index bf4b175..430946c 100644 --- a/gui/digitlabeldelegate.cc +++ b/gui/digitlabeldelegate.cc @@ -16,7 +16,7 @@ // along with this program. If not, see . #include "digitlabeldelegate.h" -#include "digitlabelhelper.h" +#include "util.h" #include void DigitLabelDelegate::set_hex_display(bool hex) @@ -39,7 +39,7 @@ void DigitLabelDelegate::paint( QStyle *s; v = index.data(Qt::DisplayRole).toInt(); - t = DigitLabelHelper::format_value(v, this->is_hex); + t = format_toggled_value(v, this->is_hex); o = option; initStyleOption(&o, index); diff --git a/gui/digitlabelhelper.h b/gui/digitlabelhelper.h deleted file mode 100644 index 744bf0f..0000000 --- a/gui/digitlabelhelper.h +++ /dev/null @@ -1,43 +0,0 @@ -// Simulator for the RISC-V[ECTOR] mini-ISA -// Copyright (C) 2025 Siddarth Suresh -// Copyright (C) 2025 bdunahu - -// 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 3 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, see . - -#ifndef DIGITLABELHELPER_H -#define DIGITLABELHELPER_H - -#include - -class DigitLabelHelper -{ - public: - /** - * Contains the main formatting logic used to format integers. Uses 2's - * complement for hexadecimal numbers. - * @param the value to be formated - * @param if the value should be displayed in hex. If false, displays in - * decimal. - * @param if the value should display. - * @return a string respecting the above parameters. - */ - static QString format_value(int value, bool is_hex, bool is_cleared = false) - { - if (is_cleared) - return QString(); - return is_hex ? QString::asprintf("%08X", value) : QString::number(value); - } -}; - -#endif // DIGITLABELHELPER_H diff --git a/gui/storageview.cc b/gui/storageview.cc index 22baca0..2f444a9 100644 --- a/gui/storageview.cc +++ b/gui/storageview.cc @@ -17,7 +17,7 @@ #include "storageview.h" #include "definitions.h" -#include "digitlabelhelper.h" +#include "util.h" #include #include @@ -60,7 +60,7 @@ QVariant StorageView::headerData(int section, Qt::Orientation o, int role) const return QVariant(); if (o == Qt::Vertical) { - return DigitLabelHelper::format_value(section * 4, this->is_hex); + return format_toggled_value(section * 4, this->is_hex); } return QVariant(); } diff --git a/gui/util.cc b/gui/util.cc index ee75e56..72c0d87 100644 --- a/gui/util.cc +++ b/gui/util.cc @@ -1,4 +1,5 @@ #include "util.h" +#include int cache_size_mapper(int total_levels, int level) { @@ -14,3 +15,10 @@ int cache_size_mapper(int total_levels, int level) return r; } + +QString format_toggled_value(int value, bool is_hex, bool is_cleared) +{ + if (is_cleared) + return QString(); + return is_hex ? QString::asprintf("%X", value) : QString::number(value); +} diff --git a/gui/util.h b/gui/util.h index 87c33f6..8e9d308 100644 --- a/gui/util.h +++ b/gui/util.h @@ -1,8 +1,27 @@ +#ifndef UTIL_H +#define UTIL_H + +#include + /** - * Given `total_levels', returns an integer between 4 and 12 which is a linear map of `level' onto `total_levels'. - * This is used for generating cache sizes given a number of levels. + * Given `total_levels', returns an integer between 4 and 12 which is a linear + * map of `level' onto `total_levels'. This is used for generating cache sizes + * given a number of levels. * @param the total number of cache levels, zero-indexed. * @param a numberedcache level, zero-indexed. * @return an integer between 4-12, linearly scaled with level. */ int cache_size_mapper(int total_levels, int level); + +/** + * Contains the main formatting logic used to format integers. Uses 2's + * complement for hexadecimal numbers. + * @param the value to be formated + * @param if the value should be displayed in hex. If false, displays in + * decimal. + * @param if the value should display. + * @return a string respecting the above parameters. + */ +QString format_toggled_value(int value, bool is_hex, bool is_cleared = false); + +#endif // UTIL_H -- cgit v1.2.3 From d449750f789076459de8d47c2960a1279e543c32 Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 16:38:07 -0400 Subject: Fix some issues in GUI looking for memory leak --- gui/gui.cc | 19 +++++++++++-------- gui/gui.h | 4 ++-- gui/util.cc | 5 +++-- 3 files changed, 16 insertions(+), 12 deletions(-) diff --git a/gui/gui.cc b/gui/gui.cc index 2555435..7df1bfc 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -349,28 +349,31 @@ void GUI::make_tabs(int num) ui->storage->clear(); - this->tab_boxes.clear(); qDeleteAll(this->tab_boxes); + this->tab_boxes.clear(); for (i = 0; i < num; ++i) { if (i == 0) { n = "Registers"; e = new StorageView(0, this); - } else if (i == 1) { + } else if (i == num - 1) { n = "DRAM"; - e = new StorageView(MEM_LINES, this); + e = new StorageView(4, this); } else { - n = QString("L%1").arg(i - 1); + n = QString("L%1").arg(i); e = new StorageView( - cache_size_mapper(this->curr_cache_levels, i), this); + // cache_size_mapper(this->curr_cache_levels-1, i-1) + 4, this); } + std::cout << "total levels: " << num << ":" + << this->curr_cache_levels - 1 << " level: " << i + << std::endl; - t = new QTableView; + t = new QTableView(ui->storage); t->setModel(e); d = new DigitLabelDelegate(t); - connect( - this, &GUI::hex_toggled, e, &StorageView::set_hex_display); + connect(this, &GUI::hex_toggled, e, &StorageView::set_hex_display); connect( this, &GUI::hex_toggled, d, &DigitLabelDelegate::set_hex_display); diff --git a/gui/gui.h b/gui/gui.h index 10c8f67..3db88ff 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -99,12 +99,12 @@ class GUI : public QMainWindow /** * Indicates if the program has been initialized. */ - bool ready; + bool ready = false; /** * The current number of cache levels. */ - int curr_cache_levels; + int curr_cache_levels = 0; /** * The list of storage displays. diff --git a/gui/util.cc b/gui/util.cc index 72c0d87..21bf0be 100644 --- a/gui/util.cc +++ b/gui/util.cc @@ -1,16 +1,17 @@ #include "util.h" +#include "definitions.h" #include int cache_size_mapper(int total_levels, int level) { const int y_min = 4; - const int y_max = 12; + const int y_max = MEM_LINES - 2; double f, r; if (total_levels <= 1) return 8; - f = level / total_levels; + f = level / (double)total_levels; r = y_min + f * (y_max - y_min); return r; -- cgit v1.2.3 From 270d5c95c46385b7f8c3ad5ffd1adb4ac5a43acb Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 23:49:35 -0400 Subject: Fix bug which caused a very large allocation --- gui/gui.cc | 9 +++------ gui/util.cc | 2 +- gui/worker.cc | 2 +- 3 files changed, 5 insertions(+), 8 deletions(-) diff --git a/gui/gui.cc b/gui/gui.cc index 7df1bfc..6e7da5f 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -358,16 +358,13 @@ void GUI::make_tabs(int num) e = new StorageView(0, this); } else if (i == num - 1) { n = "DRAM"; - e = new StorageView(4, this); + e = new StorageView(MEM_LINES, this); } else { n = QString("L%1").arg(i); e = new StorageView( - // cache_size_mapper(this->curr_cache_levels-1, i-1) - 4, this); + (1 << cache_size_mapper(this->curr_cache_levels - 1, i - 1)), + this); } - std::cout << "total levels: " << num << ":" - << this->curr_cache_levels - 1 << " level: " << i - << std::endl; t = new QTableView(ui->storage); t->setModel(e); diff --git a/gui/util.cc b/gui/util.cc index 21bf0be..f3486fb 100644 --- a/gui/util.cc +++ b/gui/util.cc @@ -5,7 +5,7 @@ int cache_size_mapper(int total_levels, int level) { const int y_min = 4; - const int y_max = MEM_LINES - 2; + const int y_max = MEM_LINE_SPEC - 2; double f, r; if (total_levels <= 1) diff --git a/gui/worker.cc b/gui/worker.cc index 93ccbea..f490fb4 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -48,7 +48,7 @@ void Worker::configure( for (i = ways.size(); i > 0; --i) { s = static_cast(new Cache( - s, cache_size_mapper(ways.size() - 1, i), ways.at(i - 1), + s, cache_size_mapper(ways.size() - 1, i - 1), ways.at(i - 1), CACHE_DELAY + i)); this->s.push_front(s); } -- cgit v1.2.3 From b6a4bcd0b9ea23d4c760127e77112aaaa30cef0d Mon Sep 17 00:00:00 2001 From: bd Date: Sat, 26 Apr 2025 23:58:06 -0400 Subject: Ensure each tab is linked to correct cache object --- gui/worker.cc | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/gui/worker.cc b/gui/worker.cc index f490fb4..0ba364b 100644 --- a/gui/worker.cc +++ b/gui/worker.cc @@ -81,10 +81,8 @@ void Worker::update() this->ct_mutex.lock(); emit register_storage(this->ct->get_gprs()); - emit storage(this->data_to_QT(this->s.at(0)->get_data()), 1); - - for (i = 1; i < s.size(); ++i) - emit storage(this->data_to_QT(this->s.at(i - 1)->get_data()), i + 1); + for (i = 0; i < s.size(); ++i) + emit storage(this->data_to_QT(this->s.at(i)->get_data()), i + 1); emit clock_cycles(this->ct->get_clock_cycle(), this->ct->get_pc()); emit if_info(this->if_stage->get_instr()); -- cgit v1.2.3 From af643762a1627b85f779ea9d864393f7d7035e6a Mon Sep 17 00:00:00 2001 From: bd Date: Sun, 27 Apr 2025 00:04:56 -0400 Subject: Adjust cache size generation algorithm --- gui/util.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gui/util.cc b/gui/util.cc index f3486fb..b62ed81 100644 --- a/gui/util.cc +++ b/gui/util.cc @@ -5,11 +5,11 @@ int cache_size_mapper(int total_levels, int level) { const int y_min = 4; - const int y_max = MEM_LINE_SPEC - 2; + const int y_max = MEM_LINE_SPEC - 4; double f, r; - if (total_levels <= 1) - return 8; + if (total_levels <= 0) + return 7; f = level / (double)total_levels; r = y_min + f * (y_max - y_min); -- cgit v1.2.3 From 2c99a2eab1919af938c03418c551a1f035b99a5c Mon Sep 17 00:00:00 2001 From: bd Date: Sun, 27 Apr 2025 02:42:40 -0400 Subject: Add two new high-value step options in the step slider --- gui/gui.h | 2 +- gui/gui.ui | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/gui/gui.h b/gui/gui.h index 3db88ff..5c95db5 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -138,7 +138,7 @@ class GUI : public QMainWindow /** * The possible step slider values. */ - QVector step_values = {1, 5, 20, 50, 250, 1000, 10000}; + QVector step_values = {1, 5, 20, 50, 250, 1000, 10000, 100000, 500000}; QThread workerThread; diff --git a/gui/gui.ui b/gui/gui.ui index a21a200..ec1c23b 100644 --- a/gui/gui.ui +++ b/gui/gui.ui @@ -630,7 +630,7 @@ 0 - 6 + 8 1 -- cgit v1.2.3 From a4dd1f00a5d0108058fb3bfbd5f399a507792859 Mon Sep 17 00:00:00 2001 From: bd Date: Sun, 27 Apr 2025 03:02:37 -0400 Subject: Add a slider value for 100 million cycles --- gui/gui.cc | 2 +- gui/gui.h | 2 +- gui/gui.ui | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/gui/gui.cc b/gui/gui.cc index 6e7da5f..b4feda6 100644 --- a/gui/gui.cc +++ b/gui/gui.cc @@ -139,6 +139,7 @@ void GUI::on_worker_refresh_gui(int cycles, int pc) { ui->p_counter->set_value(pc); ui->cycle_counter->set_value(cycles); + this->set_status(get_waiting, "idle"); } void GUI::onWorkerFetchInfo(const InstrDTO *i) @@ -288,7 +289,6 @@ void GUI::on_step_btn_clicked() this->set_status(get_running, "busy"); int steps = step_values[ui->step_slider->value()]; emit sendRunSteps(steps); - this->set_status(get_waiting, "idle"); } void GUI::on_save_program_state_btn_clicked() diff --git a/gui/gui.h b/gui/gui.h index 5c95db5..830d852 100644 --- a/gui/gui.h +++ b/gui/gui.h @@ -138,7 +138,7 @@ class GUI : public QMainWindow /** * The possible step slider values. */ - QVector step_values = {1, 5, 20, 50, 250, 1000, 10000, 100000, 500000}; + QVector step_values = {1, 5, 20, 50, 250, 1000, 10000, 100000, 500000, 100000000}; QThread workerThread; diff --git a/gui/gui.ui b/gui/gui.ui index ec1c23b..67cca60 100644 --- a/gui/gui.ui +++ b/gui/gui.ui @@ -630,7 +630,7 @@ 0 - 8 + 9 1 -- cgit v1.2.3