[gem5-dev] RubyPort::MemSlavePort::recvTimingReq(PacketPtr): Assertion `getOffset(pkt->getAddr()) + pkt->getSize() <= RubySystem::getBlockSizeBytes()' failed on X86_GCN3.

2018-08-13 Thread Tsungtai Yeh
The following debug trace tells us the "getOffset(pkt->getAddr()) + pkt->getSize() <= RubySystem::getBlockSizeBytes()". Why does this exception happen? Do you give me some hints I can continue to dig in this error? Thank you. 1428070516000: system.ruby.tcp_cntrl2.coalescer.slave0:

Re: [gem5-dev] RubyPort::MemSlavePort::recvTimingReq(PacketPtr): Assertion `getOffset(pkt->getAddr()) + pkt->getSize() <= RubySystem::getBlockSizeBytes()' failed on X86_GCN3.

2018-08-13 Thread Tsungtai Yeh
Matt, 1. This error occurs in DeepBench RNN LSTM app. 2. The (hidden size, batch size, time step) is (256, 1, 1), (512, 1, 1), and (1024, 1, 1) 3. This error happens in MIOpenGEMM "miog_alphaab" kernel 4. I don't think this error caused by atomic operation, but I can check. Tsung

Re: [gem5-dev] RubyPort::MemSlavePort::recvTimingReq(PacketPtr): Assertion `getOffset(pkt->getAddr()) + pkt->getSize() <= RubySystem::getBlockSizeBytes()' failed on X86_GCN3.

2018-08-13 Thread Matt Sinclair
Hi Tsung, In general this error happens when a memory request is trying to issue a request that spans multiple cache lines. From the line you included, it appears that this is the case. I thought I'd added the appropriate support for uncoalesced accesses that should resolve this problem, but

[gem5-dev] Gem5 Compile issue : Commit : e02ec0c24d56bce4a0d8636a340e15cd223d1930

2018-08-13 Thread Sampad Mohapatra
Hi All, I am facing compile issues while trying to compile commit e02ec0c24d56bce4a0d8636a340e15cd223d1930 . There are two types of errors in the Gem5 source: (1) *no matching function for call to : find(uint64_t&)* build/HSAIL_X86/gpu-compute/hsail_code.cc: In member function 'StorageElement*

Re: [gem5-dev] RubyPort::MemSlavePort::recvTimingReq(PacketPtr): Assertion `getOffset(pkt->getAddr()) + pkt->getSize() <= RubySystem::getBlockSizeBytes()' failed on X86_GCN3.

2018-08-13 Thread Tsungtai Yeh
Tony, This SDWA instruction causes this Ruby error. Please check if there is a fix for this error and please check it in publicly. Thank you. Tsung Tai arch/gcn3/insts/inst_util.hh:599: T Gcn3ISA::sdwaInstSrcImpl_helper(T, T, SDWASelVals, bool) [with T = unsigned int]: Assertion

[gem5-dev] Change in gem5/gem5[master]: tests: Add test for the m5-exit instruction.

2018-08-13 Thread Sean Wilson (Gerrit)
Hello Anthony Gutierrez, Jason Lowe-Power, Brandon Potter, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/4423 to look at the new patch set (#17). Change subject: tests: Add test for the m5-exit instruction.

[gem5-dev] Change in gem5/gem5[master]: tests: Add explicit build test

2018-08-13 Thread Sean Wilson (Gerrit)
Sean Wilson has uploaded a new patch set (#8) to the change originally created by Jason Lowe-Power. ( https://gem5-review.googlesource.com/c/public/gem5/+/10121 ) Change subject: tests: Add explicit build test .. tests: Add

[gem5-dev] Change in gem5/gem5[master]: tests: Add a simple example test

2018-08-13 Thread Sean Wilson (Gerrit)
Hello Anthony Gutierrez, Jason Lowe-Power, Brandon Potter, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/4422 to look at the new patch set (#17). Change subject: tests: Add a simple example test

[gem5-dev] Change in gem5/gem5[master]: tests: Add documentation for new testing code

2018-08-13 Thread Sean Wilson (Gerrit)
Sean Wilson has uploaded a new patch set (#17) to the change originally created by Jason Lowe-Power. ( https://gem5-review.googlesource.com/c/public/gem5/+/4882 ) Change subject: tests: Add documentation for new testing code

[gem5-dev] Change in gem5/gem5[master]: tests, ext: Add a new testing library proposal

2018-08-13 Thread Sean Wilson (Gerrit)
Hello Gabe Black, Anthony Gutierrez, Jason Lowe-Power, Giacomo Travaglini, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/4421 to look at the new patch set (#16). Change subject: tests,ext: Add a new testing

Re: [gem5-dev] RubyPort::MemSlavePort::recvTimingReq(PacketPtr): Assertion `getOffset(pkt->getAddr()) + pkt->getSize() <= RubySystem::getBlockSizeBytes()' failed on X86_GCN3.

2018-08-13 Thread Matt Sinclair
Hi Tsung, That helper function is actually used by all vector instructions that have SDWA support, so posting the error function does not tell us which specific instruction is causing the failure. We would need a gdb backtrace for that. However, I don't think getting a gdb backtrace is the

[gem5-dev] Cron /z/m5/regression/do-regression quick

2018-08-13 Thread Cron Daemon
* build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/minor-timing: FAILED! * build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/o3-timing: FAILED! * build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/simple-atomic: FAILED! *

[gem5-dev] Change in gem5/gem5[master]: mem: Add a QoS-aware Memory Controller type

2018-08-13 Thread Giacomo Travaglini (Gerrit)
Hello Gabe Black, Daniel Carvalho, Jason Lowe-Power, Nikos Nikoleris, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/11970 to look at the new patch set (#3). Change subject: mem: Add a QoS-aware Memory Controller

[gem5-dev] Change in gem5/gem5[master]: mem: Add a QoS-aware Memory Controller type

2018-08-13 Thread Giacomo Travaglini (Gerrit)
Hello Gabe Black, Daniel Carvalho, Jason Lowe-Power, Nikos Nikoleris, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/11970 to look at the new patch set (#4). Change subject: mem: Add a QoS-aware Memory Controller