[gem5-dev] Change in gem5/gem5[develop]: python,scons: Only generate pybind if using python

2021-06-11 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/46820 ) Change subject: python,scons: Only generate pybind if using python .. python,scons: Only generate pybind

[gem5-dev] Change in gem5/gem5[develop]: sim: Make SEWorkload an abstract SimObject

2021-06-02 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/46379 ) Change subject: sim: Make SEWorkload an abstract SimObject .. sim: Make SEWorkload an abstract SimObject

[gem5-dev] Change in gem5/gem5[develop]: scons: Put all Params::create funcs in one file

2021-06-02 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/46380 ) Change subject: scons: Put all Params::create funcs in one file .. scons: Put all Params::create funcs

[gem5-dev] Change in gem5/gem5[develop]: cpu-o3: Fix parenthesis in condition

2021-05-27 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/46039 ) Change subject: cpu-o3: Fix parenthesis in condition .. cpu-o3: Fix parenthesis in condition The commit which removed

[gem5-dev] Change in gem5/gem5[develop]: cpu-o3: Fix parenthesis in condition

2021-05-26 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/46039 ) Change subject: cpu-o3: Fix parenthesis in condition .. cpu-o3: Fix parenthesis in condition The commit

[gem5-dev] Re: test breakage for GCN3 on ToT?

2021-05-24 Thread Jason Lowe-Power via gem5-dev
We shouldn't remove that test! It's a important feature that people rely on. The solution would be to run the test within a "known" environment. The docker containers we provide should work perfectly for that. Cheers, Jason On Mon, May 24, 2021 at 11:54 AM Bobby Bruce via gem5-dev wrote: >

[gem5-dev] Re: software floating point library(s)

2021-05-14 Thread Jason Lowe-Power via gem5-dev
I think we can use softfloat for everything now. Years ago, softfloat wasn't BSD. So, we somewhat rolled our own to get the Arm instructions working, but it wasn't comprehensive. Since softfloat was added in the past few months, no one has yet migrated the old code. As far as I know, this would be

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

2021-05-13 Thread Jason Lowe-Power via gem5-dev
I think the main question for (not) dropping support is the LTS for Ubuntu and RHEL. It looks like Ubuntu 16.04 just dropped out of standard support, so we can probably drop support for the default there, now. https://wiki.ubuntu.com/Releases TBH, I can't really tell what's currently supported

[gem5-dev] Re: gem5 namespace

2021-05-06 Thread Jason Lowe-Power via gem5-dev
< > gem5-dev@gem5.org> wrote: > > Hi all, > > I agree with Daniel's analysis and solution, as enforcing snake_case for > namespaces would probably make everyone happy. > We could in theory adopt namespace aliases for backward compatibility, to > transition smoothly

[gem5-dev] Re: gem5 namespace

2021-05-03 Thread Jason Lowe-Power via gem5-dev
t;gem5", and Gabe and I like consistency/patterns. > The (huge) downside to this solution is that it would affect users, but 1) > we'd already be affecting users anyway with the new namespace, and 2) > solving the merge conflicts would be straightforward if the patches were > created w

[gem5-dev] Re: gem5 namespace

2021-05-03 Thread Jason Lowe-Power via gem5-dev
Just a few quick replies below. Overall, I think we're being too focused on "standards" and not on enough on ease of use. It's not a problem if there are exceptions to guidance, if there's good reasons for the exceptions. On Mon, May 3, 2021 at 11:36 AM Daniel Carvalho via gem5-dev <

[gem5-dev] Re: gem5 namespace

2021-05-03 Thread Jason Lowe-Power via gem5-dev
ous >> > when reading code what things are and what they're doing. It's a bit >> > hypocritical to say that we should follow style rules and completely >> ignore >> > the aesthetic rule when capitalizing GEM5_, but then say that the >> aesthetic >> > rule

[gem5-dev] Re: ISA object and ISA specific CPU components

2021-04-30 Thread Jason Lowe-Power via gem5-dev
At a high level, I think this is a good direction. Specifically, I like the idea of making the ISA object a singleton for the simulation, or at least a member of the system, instead of one per CPU is great. We should probably make it per-system in case we want to support a simulation with multiple

[gem5-dev] Re: gem5 roadmap!

2021-04-23 Thread Jason Lowe-Power via gem5-dev
Last call! Let me know if there's anything else to add. Otherwise, I'll push this on Monday. Cheers, Jason On Tue, Apr 20, 2021 at 4:00 PM Jason Lowe-Power wrote: > Hi all, > > It's that time again to create a roadmap for the next gem5 release! I've > put together a draft wit

[gem5-dev] Re: Gem5-GCN3 Checkpointing Support

2021-04-22 Thread Jason Lowe-Power via gem5-dev
mething either in Ruby or elsewhere in the simulator? > > > > Thanks again for all of your help. > > > > Paul > > > -- Jason Lowe-Power (he/him/his) Assistant Professor, Computer Science Department University of California, Davis 3049 Kemper Hall https://arch.cs.ucdavis.edu/

[gem5-dev] gem5 roadmap!

2021-04-20 Thread Jason Lowe-Power via gem5-dev
Hi all, It's that time again to create a roadmap for the next gem5 release! I've put together a draft with the things that we are planning to work on here at UCD and a couple of other things that are on Jira. Please let the community know what you're planning on working on over the next couple

[gem5-dev] LTO causing headaches for users

2021-04-20 Thread Jason Lowe-Power via gem5-dev
Hi all, It seems the recent change to enable LTO by default is causing more harm than good. Especially for people debugging, it's a large overhead to wait 2-5 minutes for gem5 to link in opt mode. I even have encountered this with a 6 core 3.5+GHz Ryzen. It's quite annoying. We've gotten at least

[gem5-dev] Re: gem5 namespace

2021-04-19 Thread Jason Lowe-Power via gem5-dev
variable called gem5, a completely legal and reasonable name >> to use, >> > *especially* in code outside of gem5 which might be using it to refer to >> > something related to gem5 which it is interacting with. >> > >> > There are no other instances

[gem5-dev] Re: tutorial docs

2021-04-16 Thread Jason Lowe-Power via gem5-dev
on the current stable master: 21.0.0.0 >> >> >> >> On Thu, Apr 15, 2021 at 8:18 AM Jason Lowe-Power >> wrote: >> >>> Hi Mike, >>> >>> It's a bit embarrassing how out of date our documentation is. >>> Unfortunately, we h

[gem5-dev] Re: tutorial docs

2021-04-15 Thread Jason Lowe-Power via gem5-dev
Hi Mike, It's a bit embarrassing how out of date our documentation is. Unfortunately, we have a really hard time finding the resources or convincing developers to update the documentation as the code changes. The canonical documentation can be found here: http://www.gem5.org/documentation/ and

[gem5-dev] Re: gem5 namespace

2021-04-14 Thread Jason Lowe-Power via gem5-dev
Thanks for putting this all together, Daniel! My opinion is the same as yours: option 2 and macro A. One other thing we need to do is to standardize and document when and where you need to use the gem5 namespace. For instance, do we need to update *all* headers to be in the gem5 namespace? If

[gem5-dev] Change in gem5/gem5[develop]: python: Improve type annotations in pystats

2021-04-02 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42644 ) Change subject: python: Improve type annotations in pystats .. python: Improve type annotations in pystats This fixes

[gem5-dev] Re: Build failed in Jenkins: Weekly #14

2021-03-26 Thread Jason Lowe-Power via gem5-dev
I think https://gem5-review.googlesource.com/c/public/gem5/+/43287 change caused the breakage. Should be a simple fix to the GPU scripts. Cheers, Jason On Fri, Mar 26, 2021 at 3:17 AM jenkins-no-reply--- via gem5-dev < gem5-dev@gem5.org> wrote: > See

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: misc: Add release notes for v21.0.0.0

2021-03-24 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/43309 ) Change subject: misc: Add release notes for v21.0.0.0 .. misc: Add release notes for v21.0.0.0 Change-Id

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: misc: Add util-gem5art maintainer tag

2021-03-24 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/43044 ) Change subject: misc: Add util-gem5art maintainer tag .. misc: Add util-gem5art maintainer tag Change-Id

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: misc: Add release notes for v21.0.0.0

2021-03-19 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/43309 ) Change subject: misc: Add release notes for v21.0.0.0 .. misc: Add release notes for v21.0.0.0 Change-Id

[gem5-dev] Re: Python level port indirection

2021-03-18 Thread Jason Lowe-Power via gem5-dev
Hi Gabe, Why have a new special "port" and not just a function (e.g., connectCPU or connectMemory depending on if you're CPU or memory-side focused)? I would strongly prefer to have descriptive names and not something like "Export" which could mean many things. Jason On Wed, Mar 17, 2021 at

[gem5-dev] Some gem5 infrastructure down

2021-03-17 Thread Jason Lowe-Power via gem5-dev
Hi all, We're having an issue with the billing for our Google cloud infrastructure. There may be intermittent issues with gem5 infrastructure for a little while. We hope to get this resolved within a few hours. Cheers, Jason ___ gem5-dev mailing list

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: misc: Add util-gem5art maintainer tag

2021-03-15 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/43044 ) Change subject: misc: Add util-gem5art maintainer tag .. misc: Add util-gem5art maintainer tag Change-Id

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: python: Use Pattern from typing

2021-03-10 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42643 ) Change subject: python: Use Pattern from typing .. python: Use Pattern from typing Python <3.7 must import Patt

[gem5-dev] Change in gem5/gem5[develop]: python: Improve type annotations in pystats

2021-03-09 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/42644 ) Change subject: python: Improve type annotations in pystats .. python: Improve type annotations in pystats

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: python: Use Pattern from typing

2021-03-09 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/42643 ) Change subject: python: Use Pattern from typing .. python: Use Pattern from typing Python <3.7 must imp

[gem5-dev] Re: RFC: run python Black on gem5 python code

2021-03-04 Thread Jason Lowe-Power via gem5-dev
by R. Bruce > > Room 2235, > > Kemper Hall, UC Davis > > Davis, > > CA, 95616 > > > > > > web: https://www.bobbybruce.net > > > > > > > > On Wed, Mar 3, 2021 at 10:11 AM Daniel Carvalho via gem5-dev > > mailto:gem5-dev@gem5.

[gem5-dev] RFC: run python Black on gem5 python code

2021-03-03 Thread Jason Lowe-Power via gem5-dev
Hi all, Right now, we don't have an official style guide for python. Our style guide ( http://www.gem5.org/documentation/general_docs/development/coding_style/) is very C++ focused. I would like to propose adopting a relatively strict PEP 8 style guide: https://www.python.org/dev/peps/pep-0008.

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: python: Add search functions to pystats groups

2021-03-03 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42014 ) Change subject: python: Add search functions to pystats groups .. python: Add search functions to pystats groups

[gem5-dev] Re: review backlog

2021-03-01 Thread Jason Lowe-Power via gem5-dev
Hey Gabe, Just FYI, those of use working on the release still have to test and fix bugs that we find on the staging branch. In releases past, this has taken 2-4 weeks of 100% effort. We'll get to these new changes when we can! Cheers, Jason On Mon, Mar 1, 2021 at 3:56 PM Gabe Black via gem5-dev

[gem5-dev] Change in gem5/gem5[release-staging-v21-0]: python: Add search functions to pystats groups

2021-03-01 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/42014 ) Change subject: python: Add search functions to pystats groups .. python: Add search functions to pystats

[gem5-dev] Change in gem5/gem5[develop]: Revert "python: Add search functions to pystats groups"

2021-03-01 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Attention is currently required from: Andreas Sandberg, Bobby R. Bruce, Jason Lowe-Power. Hello Andreas Sandberg, kokoro, Bobby R. Bruce, Jason Lowe-Power, I'd like you to do a code review. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/42013 to review the following

[gem5-dev] Change in gem5/gem5[develop]: python: Add search functions to pystats groups

2021-03-01 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/41603 ) Change subject: python: Add search functions to pystats groups .. python: Add search functions to pystats groups

[gem5-dev] Re: gem5 v21.0.0.0 Staging branch created

2021-03-01 Thread Jason Lowe-Power via gem5-dev
I assume you mean "**NOW** begin running rigorous tests" ;) Cheers, Jason On Mon, Mar 1, 2021 at 10:27 AM Bobby Bruce via gem5-dev wrote: > Dear all, > > As of 10:30AM PST the gem5 Staging Branch for gem5 v21.0.0.0 has been > created from the gem5 develop branch. We will not begin running

[gem5-dev] Re: de-templating the O3 CPU

2021-03-01 Thread Jason Lowe-Power via gem5-dev
Hey Gabe, I love this idea! It would be nice if you could document the code as you go, too. It could serve as a good learning tool in the future. Cheers, Jason On Mon, Mar 1, 2021 at 7:56 AM Giacomo Travaglini via gem5-dev < gem5-dev@gem5.org> wrote: > +2, +1, Merged > >  > > Giacomo > > >

[gem5-dev] Re: Design doc for partial rework of instruction execution mechanism

2021-02-23 Thread Jason Lowe-Power via gem5-dev
Thanks for the effort here, Gabe! I'm excited to dive in. Just FYI, I will probably not have the bandwidth to deeply look at this for at least a few weeks. Between remote teaching and trying to get the release out of the door, I'm completely swamped for the near term. Cheers, Jason On Tue, Feb

[gem5-dev] Change in gem5/gem5[develop]: python: Add find function to pystats groups

2021-02-18 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41603 ) Change subject: python: Add find function to pystats groups .. python: Add find function to pystats groups

[gem5-dev] Re: removing partial linking hurts simulator performance, locality suspected

2021-02-10 Thread Jason Lowe-Power via gem5-dev
This is interesting! My takeaway: we need bigger i-caches to run gem5 ;). Is the binary about the same size for the two conditions? Could it be simply that the instruction working set is bigger when not using partial linking? That said, a 3% performance difference isn't a big problem, in my

[gem5-dev] Re: Upstreaming power-gem5

2021-02-08 Thread Jason Lowe-Power via gem5-dev
Wow! Thanks for all of the work here! It's cool to see all of the excitement around POWER support! Just to be clear, because I think it was a little ambiguous in other conversations, I'm very supportive of improving the POWER support in gem5 given that there are a significant number of people who

[gem5-dev] Re: Testlib six dependency

2021-01-26 Thread Jason Lowe-Power via gem5-dev
tests/ > somewhere instead of keeping it in ext/? > > Cheers, > Andreas > On 26/01/2021 16:21, Jason Lowe-Power wrote: > > Hi Andreas, > > There is no upstream for testlib. It's a purely gem5 project. We should > fix it in tree. > > Jason > > On Tue, Jan 26, 202

[gem5-dev] Re: Testlib six dependency

2021-01-26 Thread Jason Lowe-Power via gem5-dev
Hi Andreas, There is no upstream for testlib. It's a purely gem5 project. We should fix it in tree. Jason On Tue, Jan 26, 2021 at 4:56 AM Andreas Sandberg via gem5-dev < gem5-dev@gem5.org> wrote: > Hi Everyone, > > I have just posted a series of patches [1] that get rid of 'six' as a >

[gem5-dev] Re: Nightly stuck

2021-01-18 Thread Jason Lowe-Power via gem5-dev
Thanks, Giacomo! We may be a bit slow to respond (it's a holiday here in the states today and Bobby is off the next couple of days). I don't think we want to do a global timeout since we have some tests that take days. The compiler test takes about 3 days, IIRC. I think per-test timeout is going

[gem5-dev] Re: Re-join debug flags headers

2021-01-18 Thread Jason Lowe-Power via gem5-dev
Thanks for bringing this up, Daniel! I don't have a strong opinion, personally. I see how this makes things a little easier for developers. No longer would we have to teach people that to add a new debug flag you both declare it in the SConscript file and include the named header file. Given that

[gem5-dev] Re: Testing fail on GCN3_X86 on Arm machine

2021-01-11 Thread Jason Lowe-Power via gem5-dev
Hi all, It doesn't surprise me at all that GCN3 doesn't build on Arm. Nathanael, you should be able to ignore those errors. We don't test that combination of host-target and I have not heard of anyone trying to use that combination of host-target. If you're trying to use the GPU model, I'd

[gem5-dev] Change in gem5/gem5[develop]: cpu,stats: Update stats for tage_sc_l to new style stats

2020-12-10 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/36336 ) Change subject: cpu,stats: Update stats for tage_sc_l to new style stats .. cpu,stats: Update stats for tage_sc_l

[gem5-dev] Change in gem5/gem5[develop]: util: Use MAINTAINERS.yaml for valid tags in git hook

2020-11-20 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/37220 ) Change subject: util: Use MAINTAINERS.yaml for valid tags in git hook .. util: Use MAINTAINERS.yaml for valid tags

[gem5-dev] Change in gem5/gem5[develop]: util: Relax commit message checker to allow fixups

2020-11-20 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/37598 ) Change subject: util: Relax commit message checker to allow fixups .. util: Relax commit message checker to allow

[gem5-dev] Change in gem5/gem5[develop]: misc: Update maintainers file

2020-11-05 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/36886 ) Change subject: misc: Update maintainers file .. misc: Update maintainers file Change-Id

[gem5-dev] Re: Build failed in Jenkins: Nightly #118

2020-11-04 Thread Jason Lowe-Power via gem5-dev
Hi all, I'm pretty sure https://gem5-review.googlesource.com/c/public/gem5/+/34984 is the breaking change on last night's build. It's unfortunate how much time/effort supporting MIPS and other ISAs takes... Cheers, Jason On Tue, Nov 3, 2020 at 11:03 PM jenkins-no-reply--- via gem5-dev <

[gem5-dev] Change in gem5/gem5[develop]: misc: Update maintainers file

2020-11-03 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/36886 ) Change subject: misc: Update maintainers file .. misc: Update maintainers file Change-Id

[gem5-dev] Re: SimObject params() method

2020-10-22 Thread Jason Lowe-Power via gem5-dev
y new > > > > Class MyObject : public SimObject {} > > > > I would use > > > > auto p = params(); > > > > You still have to type your param type so that doesn’t make it the best > but I believe it’s the simplest one > > > > Giacomo >

[gem5-dev] Re: SimObject params() method

2020-10-22 Thread Jason Lowe-Power via gem5-dev
Hey Gabe, Thanks for bringing this up. I have also been bothered by the lack of consistency with how params are used. I can't think of an example of when you need to store the params object. I would be all for getting rid of the params() function and updating the documentation to say that it's

[gem5-dev] Change in gem5/gem5[develop]: cpu-kvm, arch-x86: Fix KVM on Intel platforms

2020-10-20 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/12278 ) Change subject: cpu-kvm, arch-x86: Fix KVM on Intel platforms .. cpu-kvm, arch-x86: Fix KVM on Intel platforms

[gem5-dev] Re: Proposal for new committing policies

2020-10-19 Thread Jason Lowe-Power via gem5-dev
ere: https://gem5-review.googlesource.com/c/public/gem5/+/34737 Cheers, Jason On Fri, Oct 16, 2020 at 3:50 PM Jason Lowe-Power wrote: > Hi all, > > I have just submitted a changeset to gerrit which updates the CONTRIBUTING > documentation with a proposal for updating the committi

[gem5-dev] Change in gem5/gem5[develop]: misc: Minor updates to CONTRIBUTING.md

2020-10-19 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/36255 ) Change subject: misc: Minor updates to CONTRIBUTING.md .. misc: Minor updates to CONTRIBUTING.md This brings the file

[gem5-dev] Proposal for new committing policies

2020-10-16 Thread Jason Lowe-Power via gem5-dev
Hi all, I have just submitted a changeset to gerrit which updates the CONTRIBUTING documentation with a proposal for updating the committing policies. Issue: https://gem5.atlassian.net/browse/GEM5-799 Changeset: https://gem5-review.googlesource.com/c/public/gem5/+/36256 The gem5 project is

[gem5-dev] Change in gem5/gem5[develop]: misc: Add new policies to CONTRIBUTING.md

2020-10-16 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/36256 ) Change subject: misc: Add new policies to CONTRIBUTING.md .. misc: Add new policies to CONTRIBUTING.md

[gem5-dev] Change in gem5/gem5[develop]: misc: Minor updates to CONTRIBUTING.md

2020-10-16 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/36255 ) Change subject: misc: Minor updates to CONTRIBUTING.md .. misc: Minor updates to CONTRIBUTING.md

[gem5-dev] A call for maintainers

2020-10-16 Thread Jason Lowe-Power via gem5-dev
Hi everyone! As you all are aware, we're pushing to increase the stability of gem5. However, at the same time, we're seeing a significant increase in the rate of gem5 changes. Between gem5-20.0 and gem5-20.1 we had over 650 commits! And, that doesn't count the many changes on gerrit that never

[gem5-dev] Change in gem5/gem5[develop]: sim,python: Flip logic on loopback listeners

2020-10-16 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/36175 ) Change subject: sim,python: Flip logic on loopback listeners .. sim,python: Flip logic on loopback listeners People

[gem5-dev] Change in gem5/gem5[develop]: sim,python: Flip logic on loopback listeners

2020-10-15 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/36175 ) Change subject: sim,python: Flip logic on loopback listeners .. sim,python: Flip logic on loopback

[gem5-dev] Determining the APIC id for a CPU

2020-10-15 Thread Jason Lowe-Power via gem5-dev
Hi all (and specifically our AMD colleagues), Does anyone know how real hardware assigns APIC IDs in x86? We need to do something more than just use the CPU number if we want to support multiple hardware threads. We have a proposal here:

[gem5-dev] Re: Implementing PSHUFB instruction

2020-10-13 Thread Jason Lowe-Power via gem5-dev
> > On Tue, Oct 13, 2020 at 8:27 AM Jason Lowe-Power > wrote: > >> Hi Gabe, >> >> Thanks for the info! This is a bit helpful. Although, I'm still not sure >> what the next steps would be or how to even start on (1) or (2) that you >> listed. >>

[gem5-dev] Re: Build failed in Jenkins: Nightly #91

2020-10-13 Thread Jason Lowe-Power via gem5-dev
Hi all, Yeah, this is 100% a scons problem. I'm not sure what we can do about it. To be more straightforward here are the possible solutions: 1. install python-is-python3 2. pip install scons 3. Edit /usr/bin/scons to use python3 4. Add `alias scons3="/usr/bin/env python3 $(which scons)"` to

[gem5-dev] Re: Implementing PSHUFB instruction

2020-10-13 Thread Jason Lowe-Power via gem5-dev
Hi Gabe, Thanks for the info! This is a bit helpful. Although, I'm still not sure what the next steps would be or how to even start on (1) or (2) that you listed. Is it possible to focus on functional correctness first and then work on the timing correctness? The problem is that modern

[gem5-dev] Re: simple simulator performance check?

2020-10-07 Thread Jason Lowe-Power via gem5-dev
I think Linux boot is pretty reasonable. Or, Linux boot + some multithreaded tests (parsec is available for x86 in gem5-resources). If there isn't much performance impact there, I think that would be strong evidence for little performance impact, generally. Cheers, Jason On Mon, Oct 5, 2020 at

[gem5-dev] Re: MOESI_CMP_directory coherence protocol

2020-10-05 Thread Jason Lowe-Power via gem5-dev
Hello, Without digging deeper into things, I *believe* `Writeback_Ack_Data` has the data with the acknowledgement, but `Writeback_Ack` just has the ack. That's why you have to send the data from the TBE if you receive a Writeback_Ack_Data. Cheers, Jason On Fri, Sep 25, 2020 at 6:26 AM

[gem5-dev] Status of Ruby tester and DMA

2020-10-05 Thread Jason Lowe-Power via gem5-dev
Hi all (specifically Matt P. and other AMDers and Tuan), What's the status of adding DMA to the ruby tester? I see a couple of issues on Jira (e.g., https://gem5.atlassian.net/browse/GEM5-740) and there are some patches on gerrit for the GPU tester (e.g.,

[gem5-dev] Change in gem5/gem5[develop]: base: Notify stats output of first/last dump

2020-10-03 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/35576 ) Change subject: base: Notify stats output of first/last dump .. base: Notify stats output of first/last

[gem5-dev] Change in gem5/gem5[develop]: base: Add JSON output for stats

2020-10-03 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/35575 ) Change subject: base: Add JSON output for stats .. base: Add JSON output for stats This patch adds

[gem5-dev] Re: PCI memory BARs broken on ARM

2020-10-02 Thread Jason Lowe-Power via gem5-dev
Hey Gabe, I believe the developers at AMD have already implemented a lot of similar things to get their GPU model working. The relevant Jira issue is https://gem5.atlassian.net/browse/GEM5-470 (I think). There may be other relevant issues on this Epic: https://gem5.atlassian.net/browse/GEM5-195.

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: misc: Add release notes for 20.1

2020-09-30 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/35375 ) Change subject: misc: Add release notes for 20.1 .. misc: Add release notes for 20.1 Change-Id

[gem5-dev] gem5-20.1 release imminent

2020-09-29 Thread Jason Lowe-Power via gem5-dev
Hi all, We're planning on making the gem5-20.1 release official tomorrow. Please let us know if there's any last minute blocking issues. There are a couple of small things we're still waiting on, but we're confident they'll be resolved by tomorrow morning (US west coast time). In the meantime,

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: misc: Add release notes for 20.1

2020-09-29 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/35375 ) Change subject: misc: Add release notes for 20.1 .. misc: Add release notes for 20.1 Change-Id

[gem5-dev] Re: MMU object vs. DTB and ITB

2020-09-29 Thread Jason Lowe-Power via gem5-dev
g the TLB hierarchy to the rest of the cpu code. > > The remaining thing after this patches would be to move most TLB methods > to the MMU class and to make the TLB a simple passive translation cache. > That will allow us to stack them in any way we want > > > > Giacomo >

[gem5-dev] Re: Build failed in Jenkins: Nightly #75

2020-09-22 Thread Jason Lowe-Power via gem5-dev
I think the problem is that google perftools isn't installed in this image. I guess we could skip building perf for the unittests? On Mon, Sep 21, 2020 at 11:26 PM jenkins-no-reply--- via gem5-dev < gem5-dev@gem5.org> wrote: > See

[gem5-dev] Re: MMU object vs. DTB and ITB

2020-09-21 Thread Jason Lowe-Power via gem5-dev
We (well, mostly Ayaz) have also been looking at this interface. We've been thinking more about x86 and RISC-V, but would also like to be kept up to date! We were also thinking that many of the TLB/MMU concepts are shared between ISAs (or are microarchitecture details). So, it would be nice to be

[gem5-dev] Re: gem5 python version?

2020-09-20 Thread Jason Lowe-Power via gem5-dev
Hi Mike, I believe we've decided to drop 2.7 support after the 20.1 release. This hasn't been checked into develop, yet, but I think it's a safe assumption that develop now only supports python3. Cheers, Jason On Sun, Sep 20, 2020 at 10:52 AM mike upton via gem5-dev wrote: > > Do we still

[gem5-dev] Re: A few quick thoughts

2020-09-19 Thread Jason Lowe-Power via gem5-dev
e merits and is worth considering. If in the future somebody does > either figure out some crazy clever way to do this without the templates, > or the standard evolves to the point where we can use something more > elegant, then it should be relatively easy to move to that instead. > > G

[gem5-dev] Re: A few quick thoughts

2020-09-18 Thread Jason Lowe-Power via gem5-dev
Sorry for the spam... One last thing: We have to keep both bits(val, first, last) *and* bits(val) because sometimes first and last are *not* constexpr. If they were *always* constexpr, this would be much simpler (I think). Cheers, Jason On Fri, Sep 18, 2020 at 8:33 AM Jason Lowe-Power wrote

[gem5-dev] Re: A few quick thoughts

2020-09-18 Thread Jason Lowe-Power via gem5-dev
this somewhere but what is the reason for > such a strong aversion to the template approach? It seems to solve the > issue nicely with what seems to be a minor change in syntax. gem5 is C++, > so we should allow users to write C++. > > > > Tony > > > > *From:* Jason

[gem5-dev] Re: A few quick thoughts

2020-09-18 Thread Jason Lowe-Power via gem5-dev
k. > > Gabe > > On Thu, Sep 17, 2020 at 2:53 PM Jason Lowe-Power > wrote: > >> >> >> On Thu, Sep 17, 2020 at 2:48 PM Gabe Black via gem5-dev < >> gem5-dev@gem5.org> wrote: >> >>> 1. Sounds good, I'll hopefully have some time to put

[gem5-dev] Re: A few quick thoughts

2020-09-17 Thread Jason Lowe-Power via gem5-dev
https://gem5.googlesource.com/public/gem5/+/refs/heads/develop/src/base/compiler.hh#69 > >> -- >> Dr. Bobby R. Bruce >> Room 2235, >> Kemper Hall, UC Davis >> Davis, >> CA, 95616 >> >> web: https://www.bobbybruce.net >> >> >&

[gem5-dev] Re: A few quick thoughts

2020-09-17 Thread Jason Lowe-Power via gem5-dev
Hey Gabe, On Thu, Sep 17, 2020 at 4:46 AM Gabe Black via gem5-dev wrote: > 1. Use __builtin_expect() for panic, fatal, etc. Preexisting library > functions like assert probably already have this, but our versions don't > and have similar behavior patterns. This should improve performance. >

[gem5-dev] Change in gem5/gem5[develop]: misc: Add Matt Poremba as GPU maintainer

2020-09-16 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/34655 ) Change subject: misc: Add Matt Poremba as GPU maintainer .. misc: Add Matt Poremba as GPU maintainer Change-Id

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: configs: Add special case in MemConfig

2020-09-16 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/34595 ) Change subject: configs: Add special case in MemConfig .. configs: Add special case in MemConfig SimpleMemory doesn't

[gem5-dev] Change in gem5/gem5[develop]: misc: Add Matt Poremba as GPU maintainer

2020-09-16 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/34655 ) Change subject: misc: Add Matt Poremba as GPU maintainer .. misc: Add Matt Poremba as GPU maintainer

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: configs: Add special case in MemConfig

2020-09-15 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/34595 ) Change subject: configs: Add special case in MemConfig .. configs: Add special case in MemConfig

[gem5-dev] Re: MessageBuffer double counting delay between arrival and dequeue ticks?

2020-09-15 Thread Jason Lowe-Power via gem5-dev
;> >> In any case, it would be great if you can post a fix. >> >> Thanks, >> Srikant >> >> On Mon, Sep 14, 2020 at 10:48 AM Jason Lowe-Power via gem5-dev < >> gem5-dev@gem5.org> wrote: >> >>> Hey Ryan, >>> >>> Sorry

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: mem-ruby: Update port names in Ruby

2020-09-14 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/34417 ) Change subject: mem-ruby: Update port names in Ruby .. mem-ruby: Update port names in Ruby After the terminology

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: tests: Remove MIPS from Learning gem5 tests

2020-09-14 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/34415 ) Change subject: tests: Remove MIPS from Learning gem5 tests .. tests: Remove MIPS from Learning gem5 tests Change-Id

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: dev: Fix port name in x86 device

2020-09-14 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/34435 ) Change subject: dev: Fix port name in x86 device .. dev: Fix port name in x86 device Change-Id

[gem5-dev] Re: MessageBuffer double counting delay between arrival and dequeue ticks?

2020-09-14 Thread Jason Lowe-Power via gem5-dev
Hey Ryan, Sorry for the slow reply. Yes, it looks like delayedTicks may be double counting in some cases. I wonder if a little microbenchmark might be able to confirm more clearly? Assuming it is being double counted, we'd welcome a fix! @Bharadwaj, Srikant might be able to help as well.

[gem5-dev] Change in gem5/gem5[release-staging-v20.1.0.0]: dev: Fix port name in x86 device

2020-09-11 Thread Jason Lowe-Power (Gerrit) via gem5-dev
Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/34435 ) Change subject: dev: Fix port name in x86 device .. dev: Fix port name in x86 device Change-Id

<    1   2   3   4   5   6   7   8   9   10   >