From 486d18df5ca93e043fdd14fac1d22b5fe40fb6f6 Mon Sep 17 00:00:00 2001 From: bd Date: Mon, 10 Mar 2025 14:53:59 -0400 Subject: Update cli method signatures, add some getters to cache and storage --- inc/cli.h | 171 +++++++++++++++++++++++++++++++------------------------------- 1 file changed, 86 insertions(+), 85 deletions(-) (limited to 'inc/cli.h') diff --git a/inc/cli.h b/inc/cli.h index 69bd3a9..4a5726c 100644 --- a/inc/cli.h +++ b/inc/cli.h @@ -1,95 +1,96 @@ #ifndef CLI_H #define CLI_H -#include +#include "cache.h" #include #include +#include + +class Cli +{ + public: + /** + * Constructor. + * @return A newly allocated CLI object. + */ + Cli(); + ~Cli(); + + /** + * Prints all available commands to the console. + */ + void help(); + + /** + * Loads data from memory from the specified memory address. + * @param memory_address address of the memory where data needs to be loaded + * from + */ + void load(int memory_address); + + /** + * Stores data into memory at the specified address. + * @param memory_address address of the memory where data needs to be stored + * @param data data value to be written to the memory + */ + void store(int memory_address, int data); + + /** + * Resets the memory configuration and cycles to their initial state. + * This function provides a side door reset interface to the memory system, + * allowing the user to reset the memory configuration directly. + */ + void reset(); + + /** + * Updates the memory at the specified address with the given data. + * This function provides a side door modification interface to the memory + * system, allowing the user to modify the memory configuration directly. + * @param memory_address address of the memory to be updated + * @param data data value to be written to the memory + */ + void update(int memory_address, int data); -class Cli { - public: - - Cli(); - - ~Cli(); - - /** - * Prints all available commands to the console. - */ - void help(); - - /** - * Loads data from memory from the specified memory address. - * @param memory_address address of the memory where data needs to be loaded from - * @param pipeline_stage pipeline stage to be served by memory subsystem - */ - void load(int memory_address, int pipeline_stage); - - /** - * Stores data into memory at the specified address. - * @param memory_address address of the memory where data needs to be stored - * @param pipeline_stage pipeline stage to be served by memory subsystem - * @param data data value to be written to the memory - */ - void store(int memory_address, int pipeline_stage, int data); - - /** - * Loads a memory image from a file and configures memory to the image. - * This function provides a side door memory image loading interface to the memory system, - * allowing the user to load a memory image from a file and configure the memory subsystem to the image. - * @param filename name of file containing memory image - */ - void load_memory_image(const std::string& filename); - - /** - * Resets the memory configuration and cycles to their initial state. - * This function provides a side door reset interface to the memory system, - * allowing the user to reset the memory configuration directly. - */ - void reset(); - - /** - * Updates the memory at the specified address with the given data. - * This function provides a side door modification interface to the memory system, - * allowing the user to modify the memory configuration directly. - * @param memory_address address of the memory to be updated - * @param data data value to be written to the memory - */ - void update_memory(int memory_address, int data); - - /** - * Displays the current status of the entire memory subsystem. - * This function provides a side door view into the memory system, - * showing its current state and configuration. - */ - void view_memory(); - - /** - * Displays the data at the specified memory address. - * This function provides a side door view into the memory system, - * showing the data at the specified memory address. - * @param memory_address address of the memory to be viewed - */ - void view_memory_address(int memory_address); - - /** - * Updates the controls using a configuration file. - * This function provides a side door modification interface to the control system, - * allowing the user to update the controls directly. - * @param config_file name of file containing control configuration - */ - void update_controls(const std::string& config_file); + /** + * Advance the clock one cycle, refreshing the storage devices. + */ + void clock(); - /** - * Runs the command line interface - * This function is the main entry point for the command line interface. - */ - void run(); + /** + * Displays `lines` lines of the data in `level`, starting from `base`. + * + * + * This function provides a side door view into the storage system, showing + * its current state and configuration. + * @param level the level specifying the storage device. The first level + * one cache is level zero, with descending levels incrementing by a factor + * of one. + * @param base the first index to be printed + * @param the number of lines to be printed + */ + void view(int level, int base, int lines); - private: + /** + * Runs the command line interface + * This function is the main entry point for the command line interface. + */ + void run(); - /** Map of commands and their corresponding functions. - * This map is used to store the commands and their corresponding functions. - */ - std::unordered_map)>> commands; + private: + /** + * Initializes the cache object. + */ + void initialize(); + /** Map of commands and their corresponding functions. + * This map is used to store the commands and their corresponding functions. + */ + std::unordered_map< + std::string, + std::function)>> + commands; + /** + * The cache object to interface with. + */ + Cache *cache; }; -#endif /* CLI_H_INCLUDED */ \ No newline at end of file +#endif /* CLI_H_INCLUDED */ -- cgit v1.2.3 From 3f07846136767cbed13fd83baaba416f19c61667 Mon Sep 17 00:00:00 2001 From: bd Date: Mon, 10 Mar 2025 19:42:01 -0400 Subject: CLI view, clock, store, program banner --- inc/accessor.h | 15 ++++++++++++ inc/cli.h | 9 ++++--- inc/response.h | 3 +++ inc/risc_vector.h | 12 ++++++++++ inc/storage.h | 15 ++++-------- src/cli/cli.cc | 65 ++++++++++++++++++++++++++++++-------------------- src/main.cc | 34 ++++++++++++++++++++++---- src/storage/cache.cc | 14 +++++------ src/storage/storage.cc | 2 ++ src/utils/accessor.cc | 8 +++++++ src/utils/response.cc | 8 +++++++ tests/cache.cc | 15 ------------ 12 files changed, 132 insertions(+), 68 deletions(-) create mode 100644 inc/accessor.h create mode 100644 inc/risc_vector.h create mode 100644 src/utils/accessor.cc create mode 100644 src/utils/response.cc (limited to 'inc/cli.h') diff --git a/inc/accessor.h b/inc/accessor.h new file mode 100644 index 0000000..fb4999d --- /dev/null +++ b/inc/accessor.h @@ -0,0 +1,15 @@ +#ifndef ACCESSOR_H +#define ACCESSOR_H +#include + +enum Accessor { + IDLE, + MEM, + FETCH, + L1CACHE, + SIDE, +}; + +std::ostream &operator<<(std::ostream &os, Accessor a); + +#endif /* ACCESSOR_H_INCLUDED */ diff --git a/inc/cli.h b/inc/cli.h index 4a5726c..2194228 100644 --- a/inc/cli.h +++ b/inc/cli.h @@ -29,10 +29,11 @@ class Cli /** * Stores data into memory at the specified address. - * @param memory_address address of the memory where data needs to be stored + * @param accessor the pipline stage that is making this request * @param data data value to be written to the memory + * @param address address of the memory where data needs to be stored */ - void store(int memory_address, int data); + void store(Accessor accessor, int data, int address); /** * Resets the memory configuration and cycles to their initial state. @@ -64,10 +65,8 @@ class Cli * @param level the level specifying the storage device. The first level * one cache is level zero, with descending levels incrementing by a factor * of one. - * @param base the first index to be printed - * @param the number of lines to be printed */ - void view(int level, int base, int lines); + void peek(int level); /** * Runs the command line interface diff --git a/inc/response.h b/inc/response.h index d945e0f..6cd6678 100644 --- a/inc/response.h +++ b/inc/response.h @@ -1,5 +1,6 @@ #ifndef RESPONSE_H #define RESPONSE_H +#include enum Response { OK, @@ -7,4 +8,6 @@ enum Response { BLOCKED, }; +std::ostream &operator<<(std::ostream &os, Response r); + #endif /* RESPONSE_H_INCLUDED */ diff --git a/inc/risc_vector.h b/inc/risc_vector.h new file mode 100644 index 0000000..244adb6 --- /dev/null +++ b/inc/risc_vector.h @@ -0,0 +1,12 @@ +#ifndef RISC_VECTOR_H +#define RISC_VECTOR_H +#include + + + +/** + * Outputs the program version and banner. + */ +void print_version_number(); + +#endif /* RISC_VECTOR_H_INCLUDED */ diff --git a/inc/storage.h b/inc/storage.h index 3f113d3..a30e74d 100644 --- a/inc/storage.h +++ b/inc/storage.h @@ -1,19 +1,13 @@ #ifndef STORAGE_H #define STORAGE_H +#include "accessor.h" #include "definitions.h" #include "response.h" #include #include +#include #include -enum Accessor { - IDLE, - MEM, - FETCH, - L1CACHE, - SIDE, -}; - class Storage { public: @@ -45,9 +39,10 @@ class Storage * @return A matrix of data values, where each row is a line and each column * is a word. */ - std::vector> view(int base, int lines) const; + std::vector> + view(int base, int lines) const; /** - * Advances to the next job if the current job is completed. + * Refreshes the state of this storage device and lower. */ void resolve(); diff --git a/src/cli/cli.cc b/src/cli/cli.cc index d1631b4..a5706fb 100644 --- a/src/cli/cli.cc +++ b/src/cli/cli.cc @@ -2,20 +2,20 @@ #include "cache.h" #include "definitions.h" #include "dram.h" +#include "response.h" #include #include #include Cli::Cli() { - Dram *d = new Dram(MEM_SIZE, MEM_DELAY); - this->cache = new Cache(d, L1_CACHE_DELAY); + this->initialize(); commands["l"] = [this](std::vector args) { if (args.size() >= 1) { try { load(std::stoi(args[0])); - } catch (const std::exception &e) { + } catch (const std::invalid_argument &e) { std::cerr << "Invalid input: " << e.what() << std::endl; } } else { @@ -25,10 +25,17 @@ Cli::Cli() }; commands["s"] = [this](std::vector args) { - if (args.size() >= 2) { + Accessor a; + if (args.size() >= 3) { try { - store(std::stoi(args[0]), std::stoi(args[1])); - } catch (const std::exception &e) { + if (tolower(args[0][0]) == 'f') + a = FETCH; + else if (tolower(args[0][0]) == 'm') + a = MEM; + else + throw std::invalid_argument(args[0]); + store(a, std::stoi(args[1]), std::stoi(args[2])); + } catch (const std::invalid_argument &e) { std::cerr << "Invalid input: " << e.what() << std::endl; } } else { @@ -42,12 +49,11 @@ Cli::Cli() return; }; - commands["v"] = [this](std::vector args) { - if (args.size() >= 3) { + commands["p"] = [this](std::vector args) { + if (args.size() >= 1) { try { - view( - std::stoi(args[0]), std::stoi(args[1]), std::stoi(args[2])); - } catch (const std::exception &e) { + peek(std::stoi(args[0])); + } catch (const std::invalid_argument &e) { std::cerr << "Invalid input: " << e.what() << std::endl; } } else { @@ -56,6 +62,11 @@ Cli::Cli() return; }; + commands["c"] = [this](std::vector args) { + clock(); + return; + }; + commands["h"] = [this](std::vector args) { help(); return; @@ -68,16 +79,17 @@ void Cli::help() { std::cout << "Available commands:\n" - << " l - Load data from memory at the specified " + << " l
- Load data from memory at the specified " "address\n" - << " s - Stores data into " - "memory at specified address\n" + << " s
- Stores data into memory at " + "specified address. Accessor must be one of: \"MEM\", \"FETCH\", " + "\"L1CACHE\".\n" << " c - manually advances the clock\n" << " r - side door function that resets the memory configuration and " "cycles\n" - << " u - side door function that updates " + << " u
- side door function that updates " "the memory at the specified address with data provided\n" - << " v - side door function that views " + << " p - side door function that peeks " "the current status of the entire memory subsystem\n" << " h - Prints this help text\n" << " q - Quits the program\n"; @@ -88,10 +100,10 @@ void Cli::load(int memory_address) std::cout << "Loading data from memory address " << memory_address; } -void Cli::store(int memory_address, int data) +void Cli::store(Accessor accessor, int data, int address) { - std::cout << "Storing " << data << " into memory address " - << memory_address; + Response r = this->cache->write(accessor, data, address); + std::cout << r << " to " << accessor << " storing " << data << '\n'; } void Cli::update(int memory_address, int data) @@ -99,17 +111,15 @@ void Cli::update(int memory_address, int data) std::cout << "Resetting memory configuration and cycles.\n"; }; -void Cli::clock() -{ - std::cout << "Resetting memory configuration and cycles.\n"; -} +void Cli::clock() { this->cache->resolve(); } void Cli::reset() { - std::cout << "Resetting memory configuration and cycles.\n"; + this->initialize(); + std::cout << "Done.\n"; } -void Cli::view(int level, int base, int lines) +void Cli::peek(int level) { Storage *curr = this->cache; for (int i = 0; i < level; ++i) { @@ -166,6 +176,9 @@ void Cli::run() void Cli::initialize() { - if (this->cache) + std::cout << "Resetting memory configuration.\n"; + if (this->cache == nullptr) delete this->cache; + Dram *d = new Dram(MEM_SIZE, MEM_DELAY); + this->cache = new Cache(d, L1_CACHE_DELAY); } diff --git a/src/main.cc b/src/main.cc index 08cb89f..cc5fbb3 100644 --- a/src/main.cc +++ b/src/main.cc @@ -3,25 +3,47 @@ #include #include -void err() +static std::string version_number = "v0.1"; + +static std::string banner = + " _/_/_/ _/_/_/ _/_/_/ _/_/_/ \n" + " _/ _/ _/ _/ _/ \n" + " _/_/_/ _/ _/_/ _/ \n" + " _/ _/ _/ _/ _/ \n" + "_/ _/ _/_/_/ _/_/_/ _/_/_/ \n" + " \n" + " \n" + " _/_/ _/_/ \n" + " _/ _/ _/ _/_/_/_/ _/_/_/ _/_/_/_/_/ _/_/ _/_/_/ _/ \n" + " _/ _/ _/ _/ _/ _/ _/ _/ _/ _/ _/ \n" + "_/ _/ _/ _/_/_/ _/ _/ _/ _/ _/_/_/ _/ \n" + " _/ _/ _/ _/ _/ _/ _/ _/ _/ _/ _/ \n" + " _/ _/ _/_/_/_/ _/_/_/ _/ _/_/ _/ _/ _/ \n" + " _/_/ _/_/ \n"; + +static void print_version_number() { std::cout << banner << version_number << '\n'; } + +static void err() { std::cerr << "Usage:\n\trisc_vector [OPTIONS]\nOptions:\n\t--debug,\t-d: " - "turn on verbose output\n\t--no-python,\t-p: run without GUI\n" + "turn on verbose output\n\t--no-python,\t-p: run without " + "GUI\n\t--version,\t-v: print the version information and exit\n" << std::endl; } -void parseArguments(int argc, char **argv, Logger &logger, bool &python) +static void parseArguments(int argc, char **argv, Logger &logger, bool &python) { struct option long_options[] = { {"debug", no_argument, 0, 'd'}, {"no-python", no_argument, 0, 'p'}, + {"version", no_argument, 0, 'v'}, {0, 0, 0, 0}}; python = true; int opt; - while ((opt = getopt_long(argc, argv, "d:p", long_options, NULL)) != -1) { + while ((opt = getopt_long(argc, argv, "d:p:v", long_options, NULL)) != -1) { switch (opt) { case 'd': logger.setLevel(DEBUG); @@ -31,6 +53,9 @@ void parseArguments(int argc, char **argv, Logger &logger, bool &python) logger.log(INFO, "Python will NOT be started!"); python = false; break; + case 'v': + print_version_number(); + exit(EXIT_SUCCESS); default: err(); exit(EXIT_FAILURE); @@ -40,6 +65,7 @@ void parseArguments(int argc, char **argv, Logger &logger, bool &python) int main(int argc, char **argv) { + print_version_number(); Logger logger("vector.log"); Cli cli; logger.log(INFO, "Initializing..."); diff --git a/src/storage/cache.cc b/src/storage/cache.cc index 45ef0ab..d382c3d 100644 --- a/src/storage/cache.cc +++ b/src/storage/cache.cc @@ -100,13 +100,11 @@ std::ostream &operator<<(std::ostream &os, const Cache &c) c.view(0, L1_CACHE_SIZE); std::array, L1_CACHE_SIZE> meta = c.get_meta(); - os << std::setfill(' ') << std::setw(L1_CACHE_SPEC + 1) << " ADDRESS |" - << std::setfill(' ') << std::setw(11) << "0" << std::setfill(' ') - << std::setw(11) << "1" << std::setfill(' ') << std::setw(11) << "2" - << std::setfill(' ') << std::setw(11) << "3" - << " |" << std::setfill(' ') - << std::setw(MEM_SPEC - LINE_SPEC - L1_CACHE_SPEC + 4) << "TAG " - << "| D \n"; + os << " " << std::setfill(' ') << std::setw(L1_CACHE_SPEC + 2) << "INDEX" + << " | " << std::setfill(' ') << std::setw((8 + 3) * 4 - 1) << "DATA" + << " | " << std::setfill(' ') + << std::setw(MEM_SPEC - LINE_SPEC - L1_CACHE_SPEC + 2) << "TAG" + << " | D\n"; for (int i = 0; i < L1_CACHE_SIZE; ++i) { os << " 0b" << std::setw(L1_CACHE_SPEC) << std::bitset(i) << " | "; @@ -116,7 +114,7 @@ std::ostream &operator<<(std::ostream &os, const Cache &c) } os << "| 0x" << std::setfill(' ') << std::bitset(meta.at(i)[0]) - << " | " << (int)(meta.at(i)[0] < 0) << '\n'; + << " | " << (int)(meta.at(i)[0] >= 0) << '\n'; } std::cout.flags(default_flags); diff --git a/src/storage/storage.cc b/src/storage/storage.cc index 62f3699..8e2e461 100644 --- a/src/storage/storage.cc +++ b/src/storage/storage.cc @@ -17,6 +17,8 @@ Storage *Storage::get_lower() { return this->lower; } void Storage::resolve() { + if (this->lower) + this->lower->resolve(); if (this->wait_time == 0) { this->requester = IDLE; this->wait_time = delay; diff --git a/src/utils/accessor.cc b/src/utils/accessor.cc new file mode 100644 index 0000000..86484c5 --- /dev/null +++ b/src/utils/accessor.cc @@ -0,0 +1,8 @@ +#include "accessor.h" +#include + +std::ostream &operator<<(std::ostream &os, Accessor a) +{ + const std::string nameA[] = {"IDLE", "MEM", "FETCH", "L1CACHE", "SIDE"}; + return os << nameA[a]; +} diff --git a/src/utils/response.cc b/src/utils/response.cc new file mode 100644 index 0000000..def6578 --- /dev/null +++ b/src/utils/response.cc @@ -0,0 +1,8 @@ +#include "response.h" +#include + +std::ostream &operator<<(std::ostream &os, Response r) +{ + const std::string nameR[] = {"OK", "WAIT", "BLOCKED"}; + return os << nameR[r]; +} diff --git a/tests/cache.cc b/tests/cache.cc index fc467c7..e8a257f 100644 --- a/tests/cache.cc +++ b/tests/cache.cc @@ -27,7 +27,6 @@ TEST_CASE("no delay stores instantly", "[cache]") r = c->write(MEM, w, 0b0); CHECK(r == OK); - d->resolve(); c->resolve(); actual = d->view(0, 1)[0]; @@ -61,12 +60,10 @@ TEST_CASE("cache takes \"forever\"", "[cache]") actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } r = c->write(MEM, w, 0b0); CHECK(r == OK); - d->resolve(); actual = d->view(0, 1)[0]; // we do NOT write back now! @@ -99,12 +96,10 @@ TEST_CASE("dram takes \"forever\"", "[cache]") actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } r = c->write(MEM, w, 0b0); CHECK(r == OK); - d->resolve(); actual = d->view(0, 1)[0]; // we do NOT write back now! @@ -137,7 +132,6 @@ TEST_CASE("dram and cache take \"forever\"", "[cache]") actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } for (i = 0; i < delay; ++i) { @@ -147,13 +141,11 @@ TEST_CASE("dram and cache take \"forever\"", "[cache]") actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } r = c->write(MEM, w, 0b0); CHECK(r == OK); c->resolve(); - d->resolve(); actual = d->view(0, 1)[0]; // we do NOT write back now! @@ -190,7 +182,6 @@ TEST_CASE( actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } r = c->write(MEM, w, 0b0); @@ -199,7 +190,6 @@ TEST_CASE( CHECK(r == WAIT); c->resolve(); - d->resolve(); actual = d->view(0, 1)[0]; // we do NOT write back now! @@ -214,7 +204,6 @@ TEST_CASE( CHECK(r == OK); c->resolve(); - d->resolve(); expected.at(1) = w; actual = c->view(0, 1)[0]; @@ -248,7 +237,6 @@ TEST_CASE( actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } r = c->write(MEM, w, 0b0); @@ -257,7 +245,6 @@ TEST_CASE( CHECK(r == WAIT); c->resolve(); - d->resolve(); actual = d->view(0, 1)[0]; // we do NOT write back now! @@ -274,14 +261,12 @@ TEST_CASE( actual = c->view(0, 1)[0]; REQUIRE(expected == actual); c->resolve(); - d->resolve(); } r = c->write(FETCH, w, 0b1); CHECK(r == OK); c->resolve(); - d->resolve(); expected.at(1) = w; actual = c->view(0, 1)[0]; -- cgit v1.2.3 From 11182ea41cc5f453b06d769dea6e157aa9c3cf86 Mon Sep 17 00:00:00 2001 From: bd Date: Tue, 11 Mar 2025 00:09:12 -0400 Subject: cli display clock cycle, parse ';' delimited commands --- CMakeLists.txt | 3 ++ inc/cli.h | 28 ++++++----- inc/definitions.h | 6 +-- inc/utils.h | 9 ++++ src/cli/cli.cc | 139 ++++++++++++++++++++++++++++++--------------------- src/main.cc | 5 +- src/storage/cache.cc | 1 - src/storage/dram.cc | 1 - src/utils/utils.cc | 25 +++++++-- 9 files changed, 139 insertions(+), 78 deletions(-) (limited to 'inc/cli.h') diff --git a/CMakeLists.txt b/CMakeLists.txt index 17ec29f..6a48922 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -3,6 +3,9 @@ project(risc_vector) set(CMAKE_EXPORT_COMPILE_COMMANDS ON) +add_compile_options(-Wall -lstdc++) +add_compile_options(-Wextra -Wpedantic) + # cpp standard set(CMAKE_CXX_STANDARD 17) set(CMAKE_CXX_STANDARD_REQUIRED ON) diff --git a/inc/cli.h b/inc/cli.h index 2194228..a0c698a 100644 --- a/inc/cli.h +++ b/inc/cli.h @@ -25,13 +25,14 @@ class Cli * @param memory_address address of the memory where data needs to be loaded * from */ - void load(int memory_address); + void load(Accessor accessor, int memory_address); /** * Stores data into memory at the specified address. * @param accessor the pipline stage that is making this request * @param data data value to be written to the memory * @param address address of the memory where data needs to be stored + * @return the response from the storage device */ void store(Accessor accessor, int data, int address); @@ -42,15 +43,6 @@ class Cli */ void reset(); - /** - * Updates the memory at the specified address with the given data. - * This function provides a side door modification interface to the memory - * system, allowing the user to modify the memory configuration directly. - * @param memory_address address of the memory to be updated - * @param data data value to be written to the memory - */ - void update(int memory_address, int data); - /** * Advance the clock one cycle, refreshing the storage devices. */ @@ -79,17 +71,27 @@ class Cli * Initializes the cache object. */ void initialize(); + /** + * Attempts to match string to either fetch or mem, or throws + * std::invalid_argument otherwise. + * @param the string to be converted accessor + * @return the corresponding accessor + * @throws invalid_argument if the string is not fetch or mem + */ + Accessor match_accessor_or_die(std::string s); /** Map of commands and their corresponding functions. * This map is used to store the commands and their corresponding functions. */ - std::unordered_map< - std::string, - std::function)>> + std::unordered_map)>> commands; /** * The cache object to interface with. */ Cache *cache; + /** + * The current cycle. + */ + int cycle; }; #endif /* CLI_H_INCLUDED */ diff --git a/inc/definitions.h b/inc/definitions.h index 62ed0f6..f015ce9 100644 --- a/inc/definitions.h +++ b/inc/definitions.h @@ -10,7 +10,7 @@ /** * The total number of words in a line */ -#define LINE_SIZE (int)pow(2, 2) +#define LINE_SIZE static_cast(pow(2, 2)) /** * The number of bits to specify a memory line @@ -20,7 +20,7 @@ /** * The total number of words in memory */ -#define MEM_SIZE (int)pow(2, MEM_SPEC) +#define MEM_SIZE static_cast(pow(2, MEM_SPEC)) /** * The number of bits to specify a l1 cache line @@ -29,7 +29,7 @@ /** * The total number of words in l1 cache */ -#define L1_CACHE_SIZE (int)pow(2, L1_CACHE_SPEC) +#define L1_CACHE_SIZE static_cast(pow(2, L1_CACHE_SPEC)) /** * The total number of cycles a memory access takes. diff --git a/inc/utils.h b/inc/utils.h index e258ed8..71e515b 100644 --- a/inc/utils.h +++ b/inc/utils.h @@ -1,5 +1,6 @@ #ifndef UTILS_H #define UTILS_H +#include /** * Parse an address into a tag, index into the cache table, and a line @@ -11,4 +12,12 @@ */ void get_bit_fields(int address, int *tag, int *index, int *offset); +/** + * Formats a string using snprintf. + * @param an object that represents the format string + * @param arguments to be formatted + * @return a string object holding the formatted result + */ +const std::string string_format(const char *const zcFormat, ...); + #endif /* UTILS_H_INCLUDED */ diff --git a/src/cli/cli.cc b/src/cli/cli.cc index 272c246..033849e 100644 --- a/src/cli/cli.cc +++ b/src/cli/cli.cc @@ -3,17 +3,21 @@ #include "definitions.h" #include "dram.h" #include "response.h" -#include +#include "utils.h" +#include Cli::Cli() { this->cache = nullptr; + this->cycle = 0; this->initialize(); - commands["l"] = [this](std::vector args) { - if (args.size() >= 1) { + commands['l'] = [this](std::vector args) { + Accessor a; + if (args.size() >= 2) { try { - load(std::stoi(args[0])); + a = match_accessor_or_die(args[0]); + load(a, std::stoi(args[1])); } catch (const std::invalid_argument &e) { std::cerr << "Invalid input: " << e.what() << std::endl; } @@ -23,16 +27,11 @@ Cli::Cli() return; }; - commands["s"] = [this](std::vector args) { + commands['s'] = [this](std::vector args) { Accessor a; if (args.size() >= 3) { try { - if (tolower(args[0][0]) == 'f') - a = FETCH; - else if (tolower(args[0][0]) == 'm') - a = MEM; - else - throw std::invalid_argument(args[0]); + a = match_accessor_or_die(args[0]); store(a, std::stoi(args[1]), std::stoi(args[2])); } catch (const std::invalid_argument &e) { std::cerr << "Invalid input: " << e.what() << std::endl; @@ -43,12 +42,12 @@ Cli::Cli() return; }; - commands["r"] = [this](std::vector args) { + commands['r'] = [this](std::vector args) { reset(); return; }; - commands["p"] = [this](std::vector args) { + commands['p'] = [this](std::vector args) { if (args.size() >= 1) { try { peek(std::stoi(args[0])); @@ -61,12 +60,12 @@ Cli::Cli() return; }; - commands["c"] = [this](std::vector args) { + commands['c'] = [this](std::vector args) { clock(); return; }; - commands["h"] = [this](std::vector args) { + commands['h'] = [this](std::vector args) { help(); return; }; @@ -77,23 +76,29 @@ Cli::~Cli() { delete this->cache; } void Cli::help() { std::cout - << "Available commands:\n" - << " l
- Load data from memory at the specified " - "address\n" - << " s
- Stores data into memory at " - "specified address. Accessor must be one of: \"MEM\", \"FETCH\", " - "\"L1CACHE\".\n" - << " c - manually advances the clock\n" - << " f - advances the clock until one operation reports completion\n" - << " r - side door function that resets the memory configuration and " - "cycles\n" - << " p - side door function that peeks " - "the current status of the entire memory subsystem\n" - << " h - Prints this help text\n" - << " q - Quits the program\n"; + << "Available commands:" << std::endl + << " [l]oad
- Load data from memory at the specified " + "address" + << std::endl + << " [s]tore
- Stores data into memory at " + "specified address. Acessor must be one of: [f]etch, [m]em" + << std::endl + << " [c]ycle - manually advances the clock" << std::endl + << " [f]orce - advances the clock until one operation reports " + "completion" + << std::endl + << " [r]eset - side door function that resets the memory " + "configuration and " + "cycles" + << std::endl + << " [p]eek - side door function that " + "peeks the current status of the entire memory subsystem" + << std::endl + << " [h]elp - Prints this help text" << std::endl + << " [q]uit - Quits the program" << std::endl; } -void Cli::load(int memory_address) { ; } +void Cli::load(Accessor accessor, int address) { ; } void Cli::store(Accessor accessor, int data, int address) { @@ -101,12 +106,11 @@ void Cli::store(Accessor accessor, int data, int address) std::cout << r << " to " << accessor << " storing " << data << '\n'; } -void Cli::update(int memory_address, int data) +void Cli::clock() { - std::cout << "Resetting memory configuration and cycles.\n"; -}; - -void Cli::clock() { this->cache->resolve(); } + this->cache->resolve(); + ++this->cycle; +} void Cli::reset() { @@ -140,33 +144,42 @@ void Cli::run() "commands.\n"; std::string input; - while (true) { - std::cout << "> "; + bool run = true; + while (run) { + std::cout << this->cycle << "> "; std::getline(std::cin, input); - std::istringstream iss(input); - std::vector tokens; + std::istringstream iss1(input); + std::vector words; + std::string sentence; std::string word; - while (iss >> word) - tokens.push_back(word); - if (tokens.empty()) - continue; + while (std::getline(iss1, sentence, ';')) { + words.clear(); + std::istringstream iss2(sentence); - std::string command = tokens[0]; - tokens.erase(tokens.begin()); + while (iss2 >> word) { + words.push_back(word); + } + if (words.empty()) + continue; - if (command == "q") { - std::cout << "Exiting...\n"; - break; - } + std::string command = words[0]; + words.erase(words.begin()); - auto it = commands.find(command); - if (it != commands.end()) { - it->second(tokens); - } else { - std::cout << "Unknown command: '" << command - << "'. Type 'help' for available commands.\n"; + if (command == "q") { + run = false; + break; + } + + auto it = commands.find(tolower(command[0])); + if (it != commands.end()) { + it->second(words); + } else { + std::cout << "Unknown command: '" << command + << "'. Type 'help' for available commands." + << std::endl; + } } } } @@ -174,9 +187,23 @@ void Cli::run() void Cli::initialize() { Logger *global_log = Logger::getInstance(); - global_log->log(INFO, "Resetting memory configuration."); + + global_log->log(INFO, "Resetting memory configuration and cycle."); + if (this->cache != nullptr) delete this->cache; + Dram *d = new Dram(MEM_SIZE, MEM_DELAY); this->cache = new Cache(d, L1_CACHE_DELAY); + this->cycle = 1; +} + +Accessor match_accessor_or_die(std::string s) +{ + if (tolower(s[0]) == 'f') + return FETCH; + else if (tolower(s[0]) == 'm') + return MEM; + else + throw std::invalid_argument(s); } diff --git a/src/main.cc b/src/main.cc index f2f416d..1259729 100644 --- a/src/main.cc +++ b/src/main.cc @@ -21,7 +21,7 @@ static std::string banner = "_/ _/ _/ _/_/_/ _/ _/ _/ _/ _/_/_/ _/ \n" " _/ _/ _/ _/ _/ _/ _/ _/ _/ _/ _/ \n" " _/ _/ _/_/_/_/ _/_/_/ _/ _/_/ _/ _/ _/ \n" - " _/_/ _/_/ \n"; + " _/_/ _/_/ "; static void print_version_number() { std::cout << banner << version_number << '\n'; } static void err() @@ -79,5 +79,8 @@ int main(int argc, char **argv) } cli.run(); + + global_log->log(INFO, "Cleaning up..."); + global_log->log(INFO, "Goodbye!"); return EXIT_SUCCESS; } diff --git a/src/storage/cache.cc b/src/storage/cache.cc index 6dd0b8a..d382c3d 100644 --- a/src/storage/cache.cc +++ b/src/storage/cache.cc @@ -100,7 +100,6 @@ std::ostream &operator<<(std::ostream &os, const Cache &c) c.view(0, L1_CACHE_SIZE); std::array, L1_CACHE_SIZE> meta = c.get_meta(); - cout << data.capacity(); os << " " << std::setfill(' ') << std::setw(L1_CACHE_SPEC + 2) << "INDEX" << " | " << std::setfill(' ') << std::setw((8 + 3) * 4 - 1) << "DATA" << " | " << std::setfill(' ') diff --git a/src/storage/dram.cc b/src/storage/dram.cc index 5c6d719..7353bce 100644 --- a/src/storage/dram.cc +++ b/src/storage/dram.cc @@ -83,7 +83,6 @@ std::ostream &operator<<(std::ostream &os, const Dram &d) std::vector> data = d.view(0, MEM_SIZE); - cout << data.capacity(); os << " " << std::setfill(' ') << std::setw(MEM_SPEC + 2) << "INDEX" << " | " << std::setfill(' ') << std::setw((8 + 3) * 4 - 1) << "DATA" << '\n'; for (int i = 0; i < MEM_SIZE; ++i) { diff --git a/src/utils/utils.cc b/src/utils/utils.cc index dfeb2b3..5de8e89 100644 --- a/src/utils/utils.cc +++ b/src/utils/utils.cc @@ -1,11 +1,30 @@ #include "utils.h" #include "definitions.h" +#include +#include +#include void get_bit_fields(int address, int *tag, int *index, int *offset) { - *tag = - GET_MID_BITS(address, LINE_SPEC + L1_CACHE_SPEC, - MEM_SPEC + LINE_SPEC + L1_CACHE_SPEC); + *tag = GET_MID_BITS( + address, LINE_SPEC + L1_CACHE_SPEC, + MEM_SPEC + LINE_SPEC + L1_CACHE_SPEC); *index = GET_MID_BITS(address, LINE_SPEC, L1_CACHE_SPEC + LINE_SPEC); *offset = GET_LS_BITS(address, LINE_SPEC); } + +const std::string string_format(const char *const zcFormat, ...) +{ + va_list vaArgs; + va_start(vaArgs, zcFormat); + + va_list vaArgsCopy; + va_copy(vaArgsCopy, vaArgs); + const int iLen = std::vsnprintf(NULL, 0, zcFormat, vaArgsCopy); + va_end(vaArgsCopy); + + std::vector zc(iLen + 1); + std::vsnprintf(zc.data(), zc.size(), zcFormat, vaArgs); + va_end(vaArgs); + return std::string(zc.data(), iLen); +} -- cgit v1.2.3