From 6eabbd49ebe47f4ba77b75d5b3913cdf85401430 Mon Sep 17 00:00:00 2001 From: "Daniel R. Carvalho" Date: Thu, 22 Aug 2019 14:15:46 +0200 Subject: [PATCH] mem-cache: Add getter for the number of valid sub-blks Add a getter function so that the number of valid sub-blocks can be retrieved. As a side effect, make the respective counter private. Change-Id: Icef8b51164c8e165872dcaebc65f5c330f16cb29 Signed-off-by: Daniel R. Carvalho Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/22605 Tested-by: kokoro Reviewed-by: Nikos Nikoleris Maintainer: Nikos Nikoleris --- src/mem/cache/tags/sector_blk.cc | 8 +++++++- src/mem/cache/tags/sector_blk.hh | 20 ++++++++++++++------ 2 files changed, 21 insertions(+), 7 deletions(-) diff --git a/src/mem/cache/tags/sector_blk.cc b/src/mem/cache/tags/sector_blk.cc index 7e4468dc3..13c90b68c 100644 --- a/src/mem/cache/tags/sector_blk.cc +++ b/src/mem/cache/tags/sector_blk.cc @@ -116,7 +116,7 @@ SectorSubBlk::print() const } SectorBlk::SectorBlk() - : ReplaceableEntry(), _tag(MaxAddr), _validCounter(0), _secureBit(false) + : ReplaceableEntry(), _validCounter(0), _tag(MaxAddr), _secureBit(false) { } @@ -127,6 +127,12 @@ SectorBlk::isValid() const return _validCounter > 0; } +uint8_t +SectorBlk::getNumValid() const +{ + return _validCounter; +} + bool SectorBlk::isSecure() const { diff --git a/src/mem/cache/tags/sector_blk.hh b/src/mem/cache/tags/sector_blk.hh index a30fb817c..cfe67729c 100644 --- a/src/mem/cache/tags/sector_blk.hh +++ b/src/mem/cache/tags/sector_blk.hh @@ -144,18 +144,19 @@ class SectorSubBlk : public CacheBlk */ class SectorBlk : public ReplaceableEntry { - protected: - /** - * Sector tag value. A sector's tag is the tag of all its sub-blocks. - */ - Addr _tag; - + private: /** * Counter of the number of valid sub-blocks. The sector is valid if any * of its sub-blocks is valid. */ uint8_t _validCounter; + protected: + /** + * Sector tag value. A sector's tag is the tag of all its sub-blocks. + */ + Addr _tag; + /** * Whether sector blk is in secure-space or not. */ @@ -177,6 +178,13 @@ class SectorBlk : public ReplaceableEntry */ bool isValid() const; + /** + * Get the number of sub-blocks that have been validated. + * + * @return The number of valid sub-blocks. + */ + uint8_t getNumValid() const; + /** * Checks that a sector block is secure. A single secure block suffices * to imply that the whole sector is secure, as the insertion proccess -- 2.30.2