[gem5-dev] Change in gem5/gem5[develop]: mem,configs,ext,util: Replace ExternalMaster by ExternalRequestor

2021-06-07 Thread Hoa Nguyen (Gerrit) via gem5-dev 
Hoa Nguyen has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/46581 )



Change subject: mem,configs,ext,util: Replace ExternalMaster by  
ExternalRequestor

..

mem,configs,ext,util: Replace ExternalMaster by ExternalRequestor

Signed-off-by: Hoa Nguyen 
Change-Id: Icc404e23df1026ec84250520007ce6f4b8f1edd4
---
M configs/common/FSConfig.py
M ext/sst/ExtMaster.cc
M ext/sst/ExtMaster.hh
M ext/sst/README
M ext/sst/gem5.cc
M ext/sst/gem5.hh
R src/mem/ExternalRequestor.py
M src/mem/SConscript
R src/mem/external_requestor.cc
R src/mem/external_requestor.hh
M util/tlm/README
M util/tlm/conf/tlm_master.py
M util/tlm/src/sc_master_port.cc
M util/tlm/src/sc_master_port.hh
M util/tlm/src/sim_control.cc
15 files changed, 62 insertions(+), 60 deletions(-)



diff --git a/configs/common/FSConfig.py b/configs/common/FSConfig.py
index 4c1afce..cc93322 100644
--- a/configs/common/FSConfig.py
+++ b/configs/common/FSConfig.py
@@ -299,8 +299,8 @@

 if external_memory:
 # I/O traffic enters iobus
-self.external_io = ExternalMaster(port_data="external_io",
-  port_type=external_memory)
+self.external_io = ExternalRequestor(port_data="external_io",
+ port_type=external_memory)
 self.external_io.port = self.iobus.slave

 # Ensure iocache only receives traffic destined for (actual)  
memory.

diff --git a/ext/sst/ExtMaster.cc b/ext/sst/ExtMaster.cc
index 3afd6b4..4d6b54f 100644
--- a/ext/sst/ExtMaster.cc
+++ b/ext/sst/ExtMaster.cc
@@ -56,7 +56,7 @@
 using namespace SST::gem5;
 using namespace SST::MemHierarchy;

-ExtMaster::ExtMaster(gem5Component *g, Output , ::ExternalMaster& p,
+ExtMaster::ExtMaster(gem5Component *g, Output , ::ExternalRequestor& p,
 std::string ) :
 Port(n, p), out(o), port(p), simPhase(CONSTRUCTION),
 gem5(g), name(n)
diff --git a/ext/sst/ExtMaster.hh b/ext/sst/ExtMaster.hh
index 04e98e5..ec73993 100644
--- a/ext/sst/ExtMaster.hh
+++ b/ext/sst/ExtMaster.hh
@@ -70,12 +70,12 @@

 class gem5Component;

-class ExtMaster : public ExternalMaster::Port {
+class ExtMaster : public ExternalRequestor::Port {

 enum Phase { CONSTRUCTION, INIT, RUN };

 Output& out;
-const ExternalMaster& port;
+const ExternalRequestor& port;
 Phase simPhase;

 gem5Component *const gem5;
@@ -97,7 +97,7 @@
 bool recvTimingResp(PacketPtr);
 void recvReqRetry();

-ExtMaster(gem5Component*, Output&, ExternalMaster&, std::string&);
+ExtMaster(gem5Component*, Output&, ExternalRequestor&, std::string&);
 void init(unsigned phase);
 void setup();
 void finish();
@@ -105,7 +105,8 @@
 void clock();

 // receive Requests from SST bound for a gem5 slave;
-// this module is "external" from gem5's perspective, thus  
ExternalMaster.

+// this module is "external" from gem5's perspective, thus
+// ExternalRequestor.
 void handleEvent(SST::Event*);

 protected:
diff --git a/ext/sst/README b/ext/sst/README
index 0dcc45e..834f46b 100644
--- a/ext/sst/README
+++ b/ext/sst/README
@@ -6,7 +6,7 @@
 a notion of master and slave. This distinction is important to gem5, so
 when connecting a gem5 CPU to an SST cache, an ExternalResponder must be  
used,

 and similarly when connecting the memory side of SST cache to a gem5 port
-(for memory <-> I/O), an ExternalMaster must be used.
+(for memory <-> I/O), an ExternalRequestor must be used.

 The connector handles the administrative aspects of gem5
 (initialization, simulation, shutdown) as well as translating
@@ -45,7 +45,7 @@

 This directory provides:
 1. an SST "Component" for gem5;
-2. a class that implements gem5's "ExternalMaster" interface to connect  
with
+2. a class that implements gem5's "ExternalRequestor" interface to connect  
with

SST "Link"s exchanging "memEvents"
(sst/elements/memHierarchy stuff - caches, memories, etc.)
This lets gem5 receive packets from SST, as in
@@ -60,6 +60,6 @@
v
SST cache hierarchy <-> SST memory
^
-   | [ExternalMaster]
+   | [ExternalRequestor]
v
gem5 I/O devices (terminal, disk, etc.)
diff --git a/ext/sst/gem5.cc b/ext/sst/gem5.cc
index b18eadc..e5c2a9f 100644
--- a/ext/sst/gem5.cc
+++ b/ext/sst/gem5.cc
@@ -111,7 +111,7 @@
 setDebugFlag(flag);
 }

-ExternalMaster::registerHandler("sst", this); // these are idempotent
+ExternalRequestor::registerHandler("sst", this); // these are  
idempotent

 ExternalResponder ::registerHandler("sst", this);

 // Initialize m5 special signal handling.
@@ -248,9 +248,9 @@
 }
 }

-ExternalMaster::Port*
+ExternalRequestor::Port*
 gem5Component::getExternalPort(const std::string ,
-ExternalMaster , const std::string _data)
+ExternalRequestor , const std::string _data)
 {
 std::string s(name); // bridges non-& result

[gem5-dev] Change in gem5/gem5[develop]: util: Make sorted includes verifier less confusing

2021-06-07 Thread Hoa Nguyen (Gerrit) via gem5-dev 
Hoa Nguyen has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/46580 )



Change subject: util: Make sorted includes verifier less confusing
..

util: Make sorted includes verifier less confusing

verifiers.py complains about unordered includes where there is
more than one empty line under the #include region, even if
the includes are sorted.

This change adds a note about the fact.

Change-Id: I7a8dbc12fd82db0f0cadcfec270e42f6e0de4aea
Signed-off-by: Hoa Nguyen 
---
M util/style/verifiers.py
1 file changed, 4 insertions(+), 2 deletions(-)



diff --git a/util/style/verifiers.py b/util/style/verifiers.py
index efc347e..7ab7344 100644
--- a/util/style/verifiers.py
+++ b/util/style/verifiers.py
@@ -361,8 +361,10 @@

 if modified:
 if not silent:
-self.ui.write("invalid sorting of includes in %s\n"
-% (filename))
+self.ui.write("invalid sorting of includes in %s. Note:  
If "
+  "there is more than one empty line under  
the "

+  "#include region, please reduce it to one.\n"
+  % (filename))
 if self.ui.verbose:
 for start, end in modified.regions:
 self.ui.write("bad region [%d, %d)\n" % (start,  
end))


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


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I7a8dbc12fd82db0f0cadcfec270e42f6e0de4aea
Gerrit-Change-Number: 46580
Gerrit-PatchSet: 1
Gerrit-Owner: Hoa Nguyen 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Re: Build failed in Jenkins: compiler-checks #86

2021-06-07 Thread Bobby Bruce via gem5-dev 
Just an FYI: The compiler tests are occasionally failing due to this bug:
https://gem5.atlassian.net/browse/GEM5-1006 (in short, gem5 targeting X86
can't be compiled in GCC 5).

So we don't compile every variation of gem5 against every version of GCC,
for GCC 5 we randomly select a single gem5 target to check each time the
compiler checks are run. This is why these tests are acting in a flakey
manner --- they're only failing when that randomly selected gem5 target is
X86/GCN3_X86.

We hope we can find a solution to this soon, but these tests might continue
for a little while yet.

--
Dr. Bobby R. Bruce
Room 3050,
Kemper Hall, UC Davis
Davis,
CA, 95616

web: https://www.bobbybruce.net


On Mon, Jun 7, 2021 at 2:30 AM jenkins-no-reply--- via gem5-dev <
gem5-dev@gem5.org> wrote:

> See 
>
> Changes:
>
>
> --
> Started by timer
> Running as SYSTEM
> Building in workspace 
> Selected Git installation does not exist. Using Default
> The recommended git tool is: NONE
> No credentials specified
>  > git rev-parse --resolve-git-dir <
> https://jenkins.gem5.org/job/compiler-checks/ws/.git> # timeout=10
> Fetching changes from the remote Git repository
>  > git config remote.origin.url https://gem5.googlesource.com/public/gem5
> # timeout=10
> Fetching upstream changes from https://gem5.googlesource.com/public/gem5
>  > git --version # timeout=10
>  > git --version # 'git version 2.25.1'
>  > git fetch --tags --force --progress --
> https://gem5.googlesource.com/public/gem5
> +refs/heads/*:refs/remotes/origin/* # timeout=10
>  > git rev-parse refs/remotes/origin/develop^{commit} # timeout=10
> Checking out Revision c01b9cd62b981584612b44cefa6d0c94493ddf57
> (refs/remotes/origin/develop)
>  > git config core.sparsecheckout # timeout=10
>  > git checkout -f c01b9cd62b981584612b44cefa6d0c94493ddf57 # timeout=10
> Commit message: "mem: Fix bandwidth-delay calculation in AMPM prefetcher"
>  > git rev-list --no-walk c01b9cd62b981584612b44cefa6d0c94493ddf57 #
> timeout=10
> [compiler-checks] $ /bin/sh -xe /tmp/jenkins13168994104860813295.sh
> + ./util/compiler-tests.sh -j 12
> Starting build tests with 'gcc-version-10'...
> 'gcc-version-10' was found in the comprehensive tests. All ISAs will be
> built.
>   * Building target 'GCN3_X86.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'GCN3_X86.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'ARM_MESI_Three_Level.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'ARM_MESI_Three_Level.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'SPARC.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'SPARC.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MOESI_hammer.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MOESI_hammer.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'X86.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'X86.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MESI_Two_Level.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MESI_Two_Level.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'RISCV.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'RISCV.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'Garnet_standalone.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'Garnet_standalone.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MOESI_CMP_token.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MOESI_CMP_token.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'MIPS.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'MIPS.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'ARM.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'ARM.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'X86_MOESI_AMD_Base.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'X86_MOESI_AMD_Base.fast' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MOESI_CMP_directory.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'NULL_MOESI_CMP_directory.fast' with
> 'gcc-version-10'...
> Done.
>   * Building target 'POWER.opt' with 'gcc-version-10'...
> Done.
>   * Building target 'POWER.fast' with 'gcc-version-10'...
> Done.
> Starting build tests with 'gcc-version-9'...
>   * Building target 'GCN3_X86.opt' with 'gcc-version-9'...
> Done.
>   * Building target 'GCN3_X86.fast' with 'gcc-version-9'...
> Done.
> Starting build tests with 'gcc-version-8'...
>   * Building target 'NULL_MOESI_CMP_directory.opt' with 'gcc-version-8'...
> Done.
>   * Building target 'NULL_MOESI_CMP_directory.fast' with 'gcc-version-8'...
>

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: Fix RubySystem::functionalRead with partial data

2021-06-07 Thread Gabriel Busnot (Gerrit) via gem5-dev 
Gabriel Busnot has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/46561 )



Change subject: mem-ruby: Fix RubySystem::functionalRead with partial data
..

mem-ruby: Fix RubySystem::functionalRead with partial data

Some protocol other than CHI require the read-write and/or read-only and/or
backing-store controller's buffers to be checked if the system is busy.

More details in issue GEM5-1000

Change-Id: I0ad6385ad5a88fc158e68e4c63c540504b817ccb
---
M src/mem/ruby/system/RubySystem.cc
1 file changed, 6 insertions(+), 0 deletions(-)



diff --git a/src/mem/ruby/system/RubySystem.cc  
b/src/mem/ruby/system/RubySystem.cc

index 239b652..2129e53 100644
--- a/src/mem/ruby/system/RubySystem.cc
+++ b/src/mem/ruby/system/RubySystem.cc
@@ -657,6 +657,12 @@
 // and/or dirty copy of the line might be in a message buffer or the
 // network
 if (!ctrl_busy.empty() || !bytes.isFull()) {
+if (ctrl_rw != nullptr)
+ctrl_rw->functionalReadBuffers(pkt, bytes);
+for (auto ctrl : ctrl_ro)
+ctrl->functionalReadBuffers(pkt, bytes);
+if (ctrl_bs != nullptr)
+ctrl_bs->functionalReadBuffers(pkt, bytes);
 DPRINTF(RubySystem, "Reading from busy controllers and network\n");
 for (auto ctrl : ctrl_busy) {
 ctrl->functionalRead(line_address, pkt, bytes);

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


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I0ad6385ad5a88fc158e68e4c63c540504b817ccb
Gerrit-Change-Number: 46561
Gerrit-PatchSet: 1
Gerrit-Owner: Gabriel Busnot 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: Rename WriteMask::cmpMask to containsMask

2021-06-07 Thread Gabriel Busnot (Gerrit) via gem5-dev 
Gabriel Busnot has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/46560 )



Change subject: mem-ruby: Rename WriteMask::cmpMask to containsMask
..

mem-ruby: Rename WriteMask::cmpMask to containsMask

Avoids confusion as the function tests for inclusions and not for equality.

Change-Id: I4cd10e08af46f69feed26afc2d6c7f809bc5192b
---
M src/mem/ruby/common/WriteMask.hh
M src/mem/ruby/protocol/GPU_VIPER-TCP.sm
M src/mem/ruby/protocol/RubySlicc_Exports.sm
M src/mem/ruby/protocol/chi/CHI-cache-funcs.sm
M src/mem/ruby/protocol/chi/CHI-msg.sm
5 files changed, 5 insertions(+), 5 deletions(-)



diff --git a/src/mem/ruby/common/WriteMask.hh  
b/src/mem/ruby/common/WriteMask.hh

index 895584a..1cb3f46 100644
--- a/src/mem/ruby/common/WriteMask.hh
+++ b/src/mem/ruby/common/WriteMask.hh
@@ -126,7 +126,7 @@
 }

 bool
-cmpMask(const WriteMask ) const
+containsMask(const WriteMask ) const
 {
 bool tmp = true;
 assert(mSize == readMask.mSize);
diff --git a/src/mem/ruby/protocol/GPU_VIPER-TCP.sm  
b/src/mem/ruby/protocol/GPU_VIPER-TCP.sm

index 5e987c8..a231da4 100644
--- a/src/mem/ruby/protocol/GPU_VIPER-TCP.sm
+++ b/src/mem/ruby/protocol/GPU_VIPER-TCP.sm
@@ -362,7 +362,7 @@

   action(norl_issueRdBlkOrloadDone, "norl", desc="local load done") {
 peek(mandatoryQueue_in, RubyRequest){
-  if (cache_entry.writeMask.cmpMask(in_msg.writeMask)) {
+  if (cache_entry.writeMask.containsMask(in_msg.writeMask)) {
   if (use_seq_not_coal) {
 sequencer.readCallback(address, cache_entry.DataBlk, false,  
MachineType:L1Cache);

   } else {
diff --git a/src/mem/ruby/protocol/RubySlicc_Exports.sm  
b/src/mem/ruby/protocol/RubySlicc_Exports.sm

index 7706f57..cea6c04 100644
--- a/src/mem/ruby/protocol/RubySlicc_Exports.sm
+++ b/src/mem/ruby/protocol/RubySlicc_Exports.sm
@@ -54,7 +54,7 @@

 structure(WriteMask, external="yes", desc="...") {
   void clear();
-  bool cmpMask(WriteMask);
+  bool containsMask(WriteMask);
   bool isEmpty();
   bool isFull();
   bool isOverlap(WriteMask);
diff --git a/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm  
b/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm

index 40f33ce..adf4e1c 100644
--- a/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm
+++ b/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm
@@ -215,7 +215,7 @@

   WriteMask test_mask := mask;
   test_mask.orMask(read_mask);
-  if ((mask.cmpMask(test_mask) == false) || dirty) {
+  if ((mask.containsMask(test_mask) == false) || dirty) {
 if (from_tbe) {
   if(testAndReadMask(addr, tbe.dataBlk, read_mask, pkt)) {
 DPRINTF(RubySlicc, "functionalRead tbe %x %s dirty=%d %s %s\n",  
addr, tbe.dataBlk, tbe.dataDirty, read_mask, mask);
diff --git a/src/mem/ruby/protocol/chi/CHI-msg.sm  
b/src/mem/ruby/protocol/chi/CHI-msg.sm

index 22fc508..19cf343 100644
--- a/src/mem/ruby/protocol/chi/CHI-msg.sm
+++ b/src/mem/ruby/protocol/chi/CHI-msg.sm
@@ -217,7 +217,7 @@
 assert(bitMask.isEmpty() == false);
 WriteMask test_mask := mask;
 test_mask.orMask(bitMask);
-if ((mask.cmpMask(test_mask) == false) || is_dirty) {
+if ((mask.containsMask(test_mask) == false) || is_dirty) {
   if (testAndReadMask(addr, dataBlk, bitMask, pkt)) {
 mask.orMask(bitMask);
 return true;

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


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I4cd10e08af46f69feed26afc2d6c7f809bc5192b
Gerrit-Change-Number: 46560
Gerrit-PatchSet: 1
Gerrit-Owner: Gabriel Busnot 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: Fix wrong test in CHI functional reads

2021-06-07 Thread Gabriel Busnot (Gerrit) via gem5-dev 
Gabriel Busnot has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/46559 )



Change subject: mem-ruby: Fix wrong test in CHI functional reads
..

mem-ruby: Fix wrong test in CHI functional reads

A bad write mask inclusion test in CHI cache functionalRead and CHI data  
message
functionalRead was causing clean data not to be read in some cases. The  
issue is

detailed in issue GEM5-1002.

Change-Id: I91254fa87636e8d22a8b2f27ad375f68f997932d
---
M src/mem/ruby/protocol/chi/CHI-cache-funcs.sm
M src/mem/ruby/protocol/chi/CHI-msg.sm
2 files changed, 2 insertions(+), 2 deletions(-)



diff --git a/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm  
b/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm

index db008b0..40f33ce 100644
--- a/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm
+++ b/src/mem/ruby/protocol/chi/CHI-cache-funcs.sm
@@ -215,7 +215,7 @@

   WriteMask test_mask := mask;
   test_mask.orMask(read_mask);
-  if ((test_mask.cmpMask(mask) == false) || dirty) {
+  if ((mask.cmpMask(test_mask) == false) || dirty) {
 if (from_tbe) {
   if(testAndReadMask(addr, tbe.dataBlk, read_mask, pkt)) {
 DPRINTF(RubySlicc, "functionalRead tbe %x %s dirty=%d %s %s\n",  
addr, tbe.dataBlk, tbe.dataDirty, read_mask, mask);
diff --git a/src/mem/ruby/protocol/chi/CHI-msg.sm  
b/src/mem/ruby/protocol/chi/CHI-msg.sm

index d51fb76..22fc508 100644
--- a/src/mem/ruby/protocol/chi/CHI-msg.sm
+++ b/src/mem/ruby/protocol/chi/CHI-msg.sm
@@ -217,7 +217,7 @@
 assert(bitMask.isEmpty() == false);
 WriteMask test_mask := mask;
 test_mask.orMask(bitMask);
-if ((test_mask.cmpMask(mask) == false) || is_dirty) {
+if ((mask.cmpMask(test_mask) == false) || is_dirty) {
   if (testAndReadMask(addr, dataBlk, bitMask, pkt)) {
 mask.orMask(bitMask);
 return true;

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


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I91254fa87636e8d22a8b2f27ad375f68f997932d
Gerrit-Change-Number: 46559
Gerrit-PatchSet: 1
Gerrit-Owner: Gabriel Busnot 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Build failed in Jenkins: compiler-checks #86

2021-06-07 Thread jenkins-no-reply--- via gem5-dev 
See 

Changes:


--
Started by timer
Running as SYSTEM
Building in workspace 
Selected Git installation does not exist. Using Default
The recommended git tool is: NONE
No credentials specified
 > git rev-parse --resolve-git-dir 
 >  # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url https://gem5.googlesource.com/public/gem5 # 
 > timeout=10
Fetching upstream changes from https://gem5.googlesource.com/public/gem5
 > git --version # timeout=10
 > git --version # 'git version 2.25.1'
 > git fetch --tags --force --progress -- 
 > https://gem5.googlesource.com/public/gem5 
 > +refs/heads/*:refs/remotes/origin/* # timeout=10
 > git rev-parse refs/remotes/origin/develop^{commit} # timeout=10
Checking out Revision c01b9cd62b981584612b44cefa6d0c94493ddf57 
(refs/remotes/origin/develop)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f c01b9cd62b981584612b44cefa6d0c94493ddf57 # timeout=10
Commit message: "mem: Fix bandwidth-delay calculation in AMPM prefetcher"
 > git rev-list --no-walk c01b9cd62b981584612b44cefa6d0c94493ddf57 # timeout=10
[compiler-checks] $ /bin/sh -xe /tmp/jenkins13168994104860813295.sh
+ ./util/compiler-tests.sh -j 12
Starting build tests with 'gcc-version-10'...
'gcc-version-10' was found in the comprehensive tests. All ISAs will be built.
  * Building target 'GCN3_X86.opt' with 'gcc-version-10'...
Done.
  * Building target 'GCN3_X86.fast' with 'gcc-version-10'...
Done.
  * Building target 'ARM_MESI_Three_Level.opt' with 'gcc-version-10'...
Done.
  * Building target 'ARM_MESI_Three_Level.fast' with 'gcc-version-10'...
Done.
  * Building target 'SPARC.opt' with 'gcc-version-10'...
Done.
  * Building target 'SPARC.fast' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MOESI_hammer.opt' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MOESI_hammer.fast' with 'gcc-version-10'...
Done.
  * Building target 'X86.opt' with 'gcc-version-10'...
Done.
  * Building target 'X86.fast' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MESI_Two_Level.opt' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MESI_Two_Level.fast' with 'gcc-version-10'...
Done.
  * Building target 'RISCV.opt' with 'gcc-version-10'...
Done.
  * Building target 'RISCV.fast' with 'gcc-version-10'...
Done.
  * Building target 'Garnet_standalone.opt' with 'gcc-version-10'...
Done.
  * Building target 'Garnet_standalone.fast' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MOESI_CMP_token.opt' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MOESI_CMP_token.fast' with 'gcc-version-10'...
Done.
  * Building target 'MIPS.opt' with 'gcc-version-10'...
Done.
  * Building target 'MIPS.fast' with 'gcc-version-10'...
Done.
  * Building target 'ARM.opt' with 'gcc-version-10'...
Done.
  * Building target 'ARM.fast' with 'gcc-version-10'...
Done.
  * Building target 'X86_MOESI_AMD_Base.opt' with 'gcc-version-10'...
Done.
  * Building target 'X86_MOESI_AMD_Base.fast' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MOESI_CMP_directory.opt' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MOESI_CMP_directory.fast' with 'gcc-version-10'...
Done.
  * Building target 'POWER.opt' with 'gcc-version-10'...
Done.
  * Building target 'POWER.fast' with 'gcc-version-10'...
Done.
Starting build tests with 'gcc-version-9'...
  * Building target 'GCN3_X86.opt' with 'gcc-version-9'...
Done.
  * Building target 'GCN3_X86.fast' with 'gcc-version-9'...
Done.
Starting build tests with 'gcc-version-8'...
  * Building target 'NULL_MOESI_CMP_directory.opt' with 'gcc-version-8'...
Done.
  * Building target 'NULL_MOESI_CMP_directory.fast' with 'gcc-version-8'...
Done.
Starting build tests with 'gcc-version-7'...
  * Building target 'SPARC.opt' with 'gcc-version-7'...
Done.
  * Building target 'SPARC.fast' with 'gcc-version-7'...
Done.
Starting build tests with 'gcc-version-6'...
  * Building target 'ARM_MESI_Three_Level.opt' with 'gcc-version-6'...
Done.
  * Building target 'ARM_MESI_Three_Level.fast' with 'gcc-version-6'...
Done.
Starting build tests with 'gcc-version-5'...
  * Building target 'X86.opt' with 'gcc-version-5'...
  ! Failed with exit code 2.
  * Building target 'X86.fast' with 'gcc-version-5'...
  ! Failed with exit code 2.
Starting build tests with 'clang-version-11'...
'clang-version-11' was found in the comprehensive tests. All ISAs will be built.
  * Building target 'POWER.opt' with 'clang-version-11'...
Done.
  * Building target 'POWER.fast' with 'clang-version-11'...
Done.
  * Building target 'X86_MOESI_AMD_Base.opt' with 'clang-version-11'...
Done.
  * Building target