[gem5-dev] Change in public/gem5[master]: mem-cache: Create FIFO replacement policy

[Daniel Carvalho (Gerrit)]

Daniel Carvalho has submitted this change and it was merged. (  
https://gem5-review.googlesource.com/ )


Change subject: mem-cache: Create FIFO replacement policy
..

mem-cache: Create FIFO replacement policy

Implementation of a First-In, First-Out replacement policy.

Change-Id: Id234ec9d29c092dd4516e609da14b8a75a96b5e4
Reviewed-on: https://gem5-review.googlesource.com/
Reviewed-by: Nikos Nikoleris 
Maintainer: Nikos Nikoleris 
---
M src/mem/cache/replacement_policies/ReplacementPolicies.py
M src/mem/cache/replacement_policies/SConscript
A src/mem/cache/replacement_policies/fifo_rp.cc
A src/mem/cache/replacement_policies/fifo_rp.hh
4 files changed, 144 insertions(+), 0 deletions(-)

Approvals:
  Nikos Nikoleris: Looks good to me, approved; Looks good to me, approved



diff --git a/src/mem/cache/replacement_policies/ReplacementPolicies.py  
b/src/mem/cache/replacement_policies/ReplacementPolicies.py

index 1907c36..be9f2de 100644
--- a/src/mem/cache/replacement_policies/ReplacementPolicies.py
+++ b/src/mem/cache/replacement_policies/ReplacementPolicies.py
@@ -35,6 +35,11 @@
 abstract = True
 cxx_header = "mem/cache/replacement_policies/base.hh"

+class FIFORP(BaseReplacementPolicy):
+type = 'FIFORP'
+cxx_class = 'FIFORP'
+cxx_header = "mem/cache/replacement_policies/fifo_rp.hh"
+
 class LRURP(BaseReplacementPolicy):
 type = 'LRURP'
 cxx_class = 'LRURP'
diff --git a/src/mem/cache/replacement_policies/SConscript  
b/src/mem/cache/replacement_policies/SConscript

index 6f99110..45a7b4f 100644
--- a/src/mem/cache/replacement_policies/SConscript
+++ b/src/mem/cache/replacement_policies/SConscript
@@ -33,6 +33,7 @@
 SimObject('ReplacementPolicies.py')

 Source('base.cc')
+Source('fifo_rp.cc')
 Source('lru_rp.cc')
 Source('mru_rp.cc')
 Source('random_rp.cc')
diff --git a/src/mem/cache/replacement_policies/fifo_rp.cc  
b/src/mem/cache/replacement_policies/fifo_rp.cc

new file mode 100644
index 000..fd320be
--- /dev/null
+++ b/src/mem/cache/replacement_policies/fifo_rp.cc
@@ -0,0 +1,69 @@
+/**
+ * Copyright (c) 2018 Inria
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: Daniel Carvalho
+ */
+
+#include "mem/cache/replacement_policies/fifo_rp.hh"
+
+#include "debug/CacheRepl.hh"
+
+FIFORP::FIFORP(const Params *p)
+: BaseReplacementPolicy(p)
+{
+}
+
+CacheBlk*
+FIFORP::getVictim(const ReplacementCandidates& candidates)
+{
+// There must be at least one replacement candidate
+assert(candidates.size() > 0);
+
+// Visit all candidates to find victim
+CacheBlk* blk = candidates[0];
+for (const auto& candidate : candidates) {
+// Stop iteration if found an invalid block
+if (!candidate->isValid()) {
+blk = candidate;
+break;
+// Update victim block if necessary
+} else if (candidate->tickInserted < blk->tickInserted) {
+blk = candidate;
+}
+}
+
+DPRINTF(CacheRepl, "set %x, way %x: selecting blk for replacement\n",
+blk->set, blk->way);
+
+return blk;
+}
+
+FIFORP*
+FIFORPParams::create()
+{
+return new FIFORP(this);
+}
diff --git a/src/mem/cache/replacement_policies/fifo_rp.hh  
b/src/mem/cache/replacement_policies/fifo_rp.hh

new file mode 100644
index 000..1649371
--- /dev/null
+++ b/src/mem/cache/replacement_policies/fifo_rp.hh
@@ -0,0 +1,69 @@
+/**
+ * 

[gem5-dev] Change in public/gem5[master]: mem-cache: Create FIFO replacement policy

[Daniel Carvalho (Gerrit)]

Hello Jason Lowe-Power, Nikos Nikoleris,

I'd like you to reexamine a change. Please visit

https://gem5-review.googlesource.com/

to look at the new patch set (#5).

Change subject: mem-cache: Create FIFO replacement policy
..

mem-cache: Create FIFO replacement policy

Implementation of a First-In, First-Out replacement policy.

Change-Id: Id234ec9d29c092dd4516e609da14b8a75a96b5e4
---
M src/mem/cache/replacement_policies/ReplacementPolicies.py
M src/mem/cache/replacement_policies/SConscript
A src/mem/cache/replacement_policies/fifo_rp.cc
A src/mem/cache/replacement_policies/fifo_rp.hh
4 files changed, 144 insertions(+), 0 deletions(-)


--
To view, visit https://gem5-review.googlesource.com/
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Id234ec9d29c092dd4516e609da14b8a75a96b5e4
Gerrit-Change-Number: 
Gerrit-PatchSet: 5
Gerrit-Owner: Daniel Carvalho 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Nikos Nikoleris 
Gerrit-MessageType: newpatchset
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in public/gem5[master]: mem-cache: Create FIFO replacement policy

[Daniel Carvalho (Gerrit)]

Hello Jason Lowe-Power, Nikos Nikoleris,

I'd like you to reexamine a change. Please visit

https://gem5-review.googlesource.com/

to look at the new patch set (#4).

Change subject: mem-cache: Create FIFO replacement policy
..

mem-cache: Create FIFO replacement policy

Implementation of a First-In, First-Out replacement policy.

Change-Id: Id234ec9d29c092dd4516e609da14b8a75a96b5e4
---
M src/mem/cache/replacement_policies/ReplacementPolicies.py
M src/mem/cache/replacement_policies/SConscript
A src/mem/cache/replacement_policies/fifo_rp.cc
A src/mem/cache/replacement_policies/fifo_rp.hh
4 files changed, 144 insertions(+), 0 deletions(-)


--
To view, visit https://gem5-review.googlesource.com/
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Id234ec9d29c092dd4516e609da14b8a75a96b5e4
Gerrit-Change-Number: 
Gerrit-PatchSet: 4
Gerrit-Owner: Daniel Carvalho 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Nikos Nikoleris 
Gerrit-MessageType: newpatchset
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in public/gem5[master]: mem-cache: Create FIFO replacement policy

[Daniel Carvalho (Gerrit)]

Daniel Carvalho has uploaded this change for review. (  
https://gem5-review.googlesource.com/



Change subject: mem-cache: Create FIFO replacement policy
..

mem-cache: Create FIFO replacement policy

Implementation of a First-In, First-Out replacement policy.

Change-Id: Id234ec9d29c092dd4516e609da14b8a75a96b5e4
---
M src/mem/cache/replacement_policies/ReplacementPolicies.py
M src/mem/cache/replacement_policies/SConscript
A src/mem/cache/replacement_policies/fifo_rp.cc
A src/mem/cache/replacement_policies/fifo_rp.hh
4 files changed, 145 insertions(+), 0 deletions(-)



diff --git a/src/mem/cache/replacement_policies/ReplacementPolicies.py  
b/src/mem/cache/replacement_policies/ReplacementPolicies.py

index 0faa524..51e2ef7 100644
--- a/src/mem/cache/replacement_policies/ReplacementPolicies.py
+++ b/src/mem/cache/replacement_policies/ReplacementPolicies.py
@@ -35,6 +35,11 @@
 abstract = True
 cxx_header = "mem/cache/replacement_policies/base.hh"

+class FIFORP(BaseReplacementPolicy):
+type = 'FIFORP'
+cxx_class = 'FIFORP'
+cxx_header = "mem/cache/replacement_policies/fifo_rp.hh"
+
 class LRURP(BaseReplacementPolicy):
 type = 'LRURP'
 cxx_class = 'LRURP'
diff --git a/src/mem/cache/replacement_policies/SConscript  
b/src/mem/cache/replacement_policies/SConscript

index c3c8992..31b1d7b 100644
--- a/src/mem/cache/replacement_policies/SConscript
+++ b/src/mem/cache/replacement_policies/SConscript
@@ -33,5 +33,6 @@
 SimObject('ReplacementPolicies.py')

 Source('base.cc')
+Source('fifo_rp.cc')
 Source('lru_rp.cc')
 Source('random_rp.cc')
diff --git a/src/mem/cache/replacement_policies/fifo_rp.cc  
b/src/mem/cache/replacement_policies/fifo_rp.cc

new file mode 100644
index 000..96ce636
--- /dev/null
+++ b/src/mem/cache/replacement_policies/fifo_rp.cc
@@ -0,0 +1,70 @@
+/**
+ * Copyright (c) 2018 Inria
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: Daniel Carvalho
+ */
+
+#include "mem/cache/replacement_policies/fifo_rp.hh"
+
+#include "debug/CacheRepl.hh"
+
+FIFORP::FIFORP(const Params *p)
+: BaseReplacementPolicy(p)
+{
+}
+
+CacheBlk*
+FIFORP::getVictim(ReplacementCandidates cands)
+{
+// There must be at least one replacement candidate
+assert(cands.size() > 0);
+
+// Visit all candidates to find victim
+CacheBlk* blk = cands[0];
+for (auto it = cands.begin(); it != cands.end(); ++it) {
+// Stop iteration if found an invalid block
+if (!(*it)->isValid()) {
+blk = *it;
+break;
+// Update victim block if necessary
+} else if (!(*it)->isValid() or
+   ((*it)->tickInserted < blk->tickInserted)) {
+blk = *it;
+}
+}
+
+DPRINTF(CacheRepl, "set %x, way %x: selecting blk for replacement\n",
+blk->set, blk->way);
+
+return blk;
+}
+
+FIFORP*
+FIFORPParams::create()
+{
+return new FIFORP(this);
+}
diff --git a/src/mem/cache/replacement_policies/fifo_rp.hh  
b/src/mem/cache/replacement_policies/fifo_rp.hh

new file mode 100644
index 000..7c94491
--- /dev/null
+++ b/src/mem/cache/replacement_policies/fifo_rp.hh
@@ -0,0 +1,69 @@
+/**
+ * Copyright (c) 2018 Inria
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above