[gem5-dev] Change in gem5/gem5[master]: base, mem: Disambiguate if an addr range is contained or overlaps

Hello Gabe Black, Jason Lowe-Power, Daniel Carvalho, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/5 to look at the new patch set (#3). Change subject: base, mem: Disambiguate if an addr range is contained or overlaps

[gem5-dev] Change in gem5/gem5[master]: scons: Generalize building binaries.

Hello Jason Lowe-Power, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10915 to look at the new patch set (#2). Change subject: scons: Generalize building binaries.

Re: [gem5-dev] systemc reviews

Ok, I'll add you to future reviews. All the existing ones (or at least most of them) are part of the systemc topic branch on gerrit, so hopefully they should be pretty easy to find. There are about 50 of them right now I think, so to avoid lots of tedious clicking around I'll leave the existing

[gem5-dev] Change in gem5/gem5[master]: mem: Use address range to find the destination port in the xbar

Hello Gabe Black, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/7 to look at the new patch set (#2). Change subject: mem: Use address range to find the destination port in the xbar

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Promote targets that don't require writable

Hello Jason Lowe-Power, Daniel Carvalho, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11017 to look at the new patch set (#3). Change subject: mem-cache: Promote targets that don't require writable

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

* build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/simple-timing: FAILED! * build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64f/o3-timing: FAILED! * build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/o3-timing: FAILED! *

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Forward declare ReplaceableEntry

Daniel Carvalho has submitted this change and it was merged. ( https://gem5-review.googlesource.com/11096 ) Change subject: mem-cache: Forward declare ReplaceableEntry .. mem-cache: Forward declare ReplaceableEntry Forward

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Initialize CacheBlk data pointer

Daniel Carvalho has submitted this change and it was merged. ( https://gem5-review.googlesource.com/11097 ) Change subject: mem-cache: Initialize CacheBlk data pointer .. mem-cache: Initialize CacheBlk data pointer

[gem5-dev] Change in gem5/gem5[master]: arch-arm: BadMode checking if corresponding EL is implemented

Hello Andreas Sandberg, I'd like you to do a code review. Please visit https://gem5-review.googlesource.com/11196 to review the following change. Change subject: arch-arm: BadMode checking if corresponding EL is implemented

[gem5-dev] Change in gem5/gem5[master]: dev-arm: Fix the address range for some I/O devices

Nikos Nikoleris has submitted this change and it was merged. ( https://gem5-review.googlesource.com/6 ) Change subject: dev-arm: Fix the address range for some I/O devices .. dev-arm: Fix the address range for some I/O

[gem5-dev] Change in gem5/gem5[master]: arch-arm: AArch32 execution triggering AArch64 SW Break

Hello Andreas Sandberg, I'd like you to do a code review. Please visit https://gem5-review.googlesource.com/11197 to review the following change. Change subject: arch-arm: AArch32 execution triggering AArch64 SW Break ..

Re: [gem5-dev] systemc reviews

> If you also want to be on all the reviews, > please let me know. I don't want to unilaterally > bomb people's inboxes if they're not interested. Wait. I am not sure what you are saying here. Are you proposing to *add* so Andreas, Jason and Matthias will receive more emails over those review

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Create cache compressor

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11100 to look at the new patch set (#3). Change subject: mem-cache: Create cache compressor ..

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Stub compression framework

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10763 to look at the new patch set (#4). Change subject: mem-cache: Stub compression framework

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Add compression stats

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11103 to look at the new patch set (#3). Change subject: mem-cache: Add compression stats ..

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Data pointer in findVictim and insertBlock

Daniel Carvalho has uploaded this change for review. ( https://gem5-review.googlesource.com/11198 Change subject: mem-cache: Data pointer in findVictim and insertBlock .. mem-cache: Data pointer in findVictim and insertBlock

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Create CacheComp debug flag

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11104 to look at the new patch set (#3). Change subject: mem-cache: Create CacheComp debug flag

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Return cycles in findVictim

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10762 to look at the new patch set (#3). Change subject: mem-cache: Return cycles in findVictim

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Remove Packet dependency in Tags

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11098 to look at the new patch set (#2). Change subject: mem-cache: Remove Packet dependency in Tags

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Create CPack compressor

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11105 to look at the new patch set (#3). Change subject: mem-cache: Create CPack compressor ..

[gem5-dev] Change in gem5/gem5[master]: mem-cache: Add compression bit to CacheBlk

Hello Jason Lowe-Power, Nikos Nikoleris, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/11102 to look at the new patch set (#3). Change subject: mem-cache: Add compression bit to CacheBlk

Re: [gem5-dev] mount host FS in simulation?

Hi Gabe, You can use 9p and VirtiIO for this. There is a 9p implementation in gem5 that uses diod (you'll have to install the diod package). There is an old, and probably broken, description on the Wiki: http://gem5.org/WA-gem5 The modern Arm platforms come with two unused VirtIO MMIO devices

Re: [gem5-dev] mount host FS in simulation?

Ok, *that's* what I was trying to remember. Thanks. Gabe On Fri, Jun 15, 2018 at 10:39 AM Andreas Sandberg wrote: > Hi Gabe, > > You can use 9p and VirtiIO for this. There is a 9p implementation in > gem5 that uses diod (you'll have to install the diod package). > > There is an old, and

Re: [gem5-dev] systemc reviews

Hey Boris, I think that gerrit always sends the first message when a patch is uploaded to the whole list. Then, when adding comments or updating patches gerrit only sends emails to people that are cc'ed in gerrit. I'm not sure what the behavior is on merges... I'm not sure what an easy way to be

Re: [gem5-dev] systemc reviews

Hey Jason, thanks for taking a look. As far as comments go, there aren't really that many comments in the Accellera implementation either, at least not doxygen style, per function args, ret style comments. I think for things which are defined in the spec and/or in systemc books, tutorials, etc.,

[gem5-dev] Change in gem5/gem5[master]: systemc: Add stubbed out versions of sc_port and sc_export.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10829 to look at the new patch set (#3). Change subject: systemc: Add stubbed out versions of sc_port and sc_export.

[gem5-dev] Change in gem5/gem5[master]: systemc: Make a test stop including an internal path for sc_vector.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11176 Change subject: systemc: Make a test stop including an internal path for sc_vector. .. systemc: Make a test stop including an

[gem5-dev] Change in gem5/gem5[master]: systemc: Add a stubbed out sc_object class.

Hello Jason Lowe-Power, Matthias Jung, Giacomo Travaglini, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10821 to look at the new patch set (#3). Change subject: systemc: Add a stubbed out sc_object class.

[gem5-dev] Change in gem5/gem5[master]: systemc: Hook up sc_main.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10824 to look at the new patch set (#3). Change subject: systemc: Hook up sc_main.

[gem5-dev] Change in gem5/gem5[master]: systemc: Seperate the "external" header interface.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10835 to look at the new patch set (#3). Change subject: systemc: Seperate the "external" header interface.

[gem5-dev] Change in gem5/gem5[master]: systemc: Add a stub implementation for sc_attr related classes.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10832 to look at the new patch set (#3). Change subject: systemc: Add a stub implementation for sc_attr related classes.

[gem5-dev] Change in gem5/gem5[master]: systemc: Add some missing enums to the utils _using.hh

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11175 Change subject: systemc: Add some missing enums to the utils _using.hh .. systemc: Add some missing enums to the utils _using.hh

[gem5-dev] Change in gem5/gem5[master]: systemc: Stub out all the standard utilility classes and functions.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10841 to look at the new patch set (#3). Change subject: systemc: Stub out all the standard utilility classes and functions.

[gem5-dev] Change in gem5/gem5[master]: systemc: Remove more Accellera source specific include paths in tests.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11179 Change subject: systemc: Remove more Accellera source specific include paths in tests. .. systemc: Remove more Accellera source

[gem5-dev] Change in gem5/gem5[master]: systemc: Stub out the sc_module class and related functions.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10834 to look at the new patch set (#3). Change subject: systemc: Stub out the sc_module class and related functions.

[gem5-dev] Change in gem5/gem5[master]: systemc: Add a stub kernel SimObject.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10822 to look at the new patch set (#3). Change subject: systemc: Add a stub kernel SimObject.

[gem5-dev] Change in gem5/gem5[master]: systemc: Add systemc and systemc.h header files.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10838 to look at the new patch set (#3). Change subject: systemc: Add systemc and systemc.h header files.

[gem5-dev] Change in gem5/gem5[master]: systemc: Stub out the predefined channels.

Hello Jason Lowe-Power, Matthias Jung, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/10839 to look at the new patch set (#3). Change subject: systemc: Stub out the predefined channels.

[gem5-dev] Change in gem5/gem5[master]: systemc: Partially implement SC_FORK and SC_JOIN.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11184 Change subject: systemc: Partially implement SC_FORK and SC_JOIN. .. systemc: Partially implement SC_FORK and SC_JOIN. These

[gem5-dev] Change in gem5/gem5[master]: systemc: Add some deprecated features to sc_clock.*.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11183 Change subject: systemc: Add some deprecated features to sc_clock.*. .. systemc: Add some deprecated features to sc_clock.*. Add

[gem5-dev] Change in gem5/gem5[master]: systemc: Add the nonstandard variable sc_allow_process_control_corners.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11186 Change subject: systemc: Add the nonstandard variable sc_allow_process_control_corners. .. systemc: Add the nonstandard variable

[gem5-dev] Change in gem5/gem5[master]: systemc: Fill out some of the sc_vector.hh classes.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11177 Change subject: systemc: Fill out some of the sc_vector.hh classes. .. systemc: Fill out some of the sc_vector.hh classes. The

[gem5-dev] Change in gem5/gem5[master]: systemc: Add some "implementation defined" stuff to sc_attr_cltn.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11185 Change subject: systemc: Add some "implementation defined" stuff to sc_attr_cltn. .. systemc: Add some "implementation defined"

[gem5-dev] Change in gem5/gem5[master]: systemc: Turn some macros into enums to fix regression test building.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11180 Change subject: systemc: Turn some macros into enums to fix regression test building. .. systemc: Turn some macros into enums to

[gem5-dev] Change in gem5/gem5[master]: systemc: Include boost/bind.hpp in the systemc header.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11178 Change subject: systemc: Include boost/bind.hpp in the systemc header. .. systemc: Include boost/bind.hpp in the systemc header.

[gem5-dev] Change in gem5/gem5[master]: systemc: Add some non-standard sc_trace variants.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11182 Change subject: systemc: Add some non-standard sc_trace variants. .. systemc: Add some non-standard sc_trace variants. These

[gem5-dev] Change in gem5/gem5[master]: systemc: Remove references to internal constants from the tests.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11181 Change subject: systemc: Remove references to internal constants from the tests. .. systemc: Remove references to internal

[gem5-dev] Change in gem5/gem5[master]: systemc: Add the nonstandard triggered function sc_event.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/11187 Change subject: systemc: Add the nonstandard triggered function sc_event. .. systemc: Add the nonstandard triggered function

Re: [gem5-dev] systemc reviews

Oh, by the way, I've been adding Andreas, Jason and Matthias Jung to all the systemc reviews I've been posting. I think those folks should definitely be in the loop, but I don't want to exclude anybody else. If you also want to be on all the reviews, please let me know. I don't want to

[gem5-dev] Change in gem5/gem5[master]: base: Add an asymmetrical Coroutine class

Giacomo Travaglini has uploaded this change for review. ( https://gem5-review.googlesource.com/11195 Change subject: base: Add an asymmetrical Coroutine class .. base: Add an asymmetrical Coroutine class This patch is

[gem5-dev] Change in gem5/gem5[master]: base: Add a class which encapsulates Fibers.

Giacomo Travaglini has uploaded a new patch set (#4) to the change originally created by Gabe Black. ( https://gem5-review.googlesource.com/10935 ) Change subject: base: Add a class which encapsulates Fibers. .. base: Add a

Re: [gem5-dev] systemc reviews

Hi Gabe, please consider myself interested on reviewing SystemC patches ?? From: gem5-dev on behalf of Gabe Black Sent: 15 June 2018 09:55:30 To: gem5 Developer List Subject: Re: [gem5-dev] systemc reviews Oh, by the way, I've been adding Andreas, Jason and