[gem5-dev] Re: deprecating warn_once

2021-07-27 Thread Gabe Black via gem5-dev
No, not that I'm aware of. It would just be to make it feasible to implement the warn_once functionality without using macros. With c++20, I can more or less get it to work with some minor template syntax, warn("xyz"), but that relies on the source location (file, line, column which may be iffy)

[gem5-dev] Re: deprecating warn_once

2021-07-27 Thread Gabe Black via gem5-dev
Or I should say without un-namespaced macros (GEM5_ prefixed), since GEM5_ONCE itself would be a macro. Gabe On Mon, Jul 26, 2021 at 11:01 PM Gabe Black wrote: > No, not that I'm aware of. It would just be to make it feasible to > implement the warn_once functionality without using macros.

[gem5-dev] Change in gem5/gem5[develop]: fastmodel: Minimally implement reading MiscRegs for the CortexR52.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/46139 ) Change subject: fastmodel: Minimally implement reading MiscRegs for the CortexR52. .. fastmodel: Minimally implement

[gem5-dev] Change in gem5/gem5[develop]: dev: Drop include of drm/drm.h in kfd_ioctl.h.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48603 ) Change subject: dev: Drop include of drm/drm.h in kfd_ioctl.h. .. dev: Drop include of drm/drm.h in kfd_ioctl.h. I don't

[gem5-dev] Change in gem5/gem5[develop]: mem: Use the new "debug" namespace, and not "Debug".

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48647 ) Change subject: mem: Use the new "debug" namespace, and not "Debug". .. mem: Use the new "debug" namespace, and not "Debug".

[gem5-dev] Change in gem5/gem5[develop]: scons: Increase the minimum version of gcc to 7.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48512 ) Change subject: scons: Increase the minimum version of gcc to 7. .. scons: Increase the minimum version of gcc to 7. This

[gem5-dev] Change in gem5/gem5[develop]: fastmodel: Use Iris API to access memory

2021-07-27 Thread Yu-hsin Wang (Gerrit) via gem5-dev
Yu-hsin Wang has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45581 ) Change subject: fastmodel: Use Iris API to access memory .. fastmodel: Use Iris API to access memory Memory space is not

[gem5-dev] Re: deprecating warn_once

2021-07-27 Thread Gabe Black via gem5-dev
Yeah, that's not a bad option either. Between the two, I suggested GEM5_ONCE so that the _once version wouldn't be as inconsistent with the normal version of warn, and even though the GEM5_ONCE macro probably wouldn't be useful outside of warn, it's a pretty generic mechanism and could be. For

[gem5-dev] Change in gem5/gem5[develop]: scons,test: Don't -Wall or -Werror for googletest libraries.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48368 ) Change subject: scons,test: Don't -Wall or -Werror for googletest libraries. .. scons,test: Don't -Wall or -Werror for

[gem5-dev] Change in gem5/gem5[develop]: scons: Get rid of some redundant text in a print out.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48644 ) Change subject: scons: Get rid of some redundant text in a print out. .. scons: Get rid of some redundant text in a print

[gem5-dev] Change in gem5/gem5[develop]: scons: Add the "print_suppressions=0" option to the LSAN warning.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48645 ) Change subject: scons: Add the "print_suppressions=0" option to the LSAN warning. .. scons: Add the

[gem5-dev] Change in gem5/gem5[develop]: util: Make --with-asan work with more versions of the python lib.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/48643 ) Change subject: util: Make --with-asan work with more versions of the python lib. .. util: Make --with-asan

[gem5-dev] Change in gem5/gem5[develop]: mem: Use the new "debug" namespace, and not "Debug".

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/48647 ) Change subject: mem: Use the new "debug" namespace, and not "Debug". .. mem: Use the new "debug" namespace, and

[gem5-dev] Change in gem5/gem5[develop]: scons: Add the "print_suppressions=0" option to the LSAN warning.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/48645 ) Change subject: scons: Add the "print_suppressions=0" option to the LSAN warning. .. scons: Add the

[gem5-dev] Re: "epic" Jira issue to track multi-ISA gem5

2021-07-27 Thread Giacomo Travaglini via gem5-dev
This is great, thanks Gabe! > -Original Message- > From: Gabe Black via gem5-dev > Sent: 25 July 2021 12:45 > To: gem5 Developer List ; Earl Ou > > Cc: Gabe Black > Subject: [gem5-dev] "epic" Jira issue to track multi-ISA gem5 > > To help lay out a roadmap and scope for my ongoing

[gem5-dev] Change in gem5/gem5[develop]: scons: Disable the free-nonheap-object warning for gcc.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/48646 ) Change subject: scons: Disable the free-nonheap-object warning for gcc. .. scons: Disable the free-nonheap-object

[gem5-dev] Change in gem5/gem5[develop]: scons: Get rid of some redundant text in a print out.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/48644 ) Change subject: scons: Get rid of some redundant text in a print out. .. scons: Get rid of some redundant text in

[gem5-dev] Minor CPU decoding instructions in fetch?

2021-07-27 Thread Gabe Black via gem5-dev
Hi folks, specifically ARM folks. I'm trying to figure out what's going on with this in the minor CPU: #if THE_ISA == X86_ISA || THE_ISA == ARM_ISA /* In SE mode, it's possible to branch to a microop when * replaying faults such as page faults (or simply

[gem5-dev] Change in gem5/gem5[develop]: util: Fix gerrit bot commit subject parser

2021-07-27 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48586 ) Change subject: util: Fix gerrit bot commit subject parser .. util: Fix gerrit bot commit subject parser Currently, if

[gem5-dev] Change in gem5/gem5[develop]: scons: Update the minimum version of python to 3.6.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48513 ) Change subject: scons: Update the minimum version of python to 3.6. .. scons: Update the minimum version of python to 3.6.

[gem5-dev] Change in gem5/gem5[develop]: util: Make --with-asan work with more versions of the python lib.

2021-07-27 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48643 ) Change subject: util: Make --with-asan work with more versions of the python lib. .. util: Make --with-asan work with

[gem5-dev] Re: deprecating warn_once

2021-07-27 Thread Steve Reinhardt via gem5-dev
Wouldn't we get the same net result by just renaming warn_once() to GEM5_WARN_ONCE()? Seems simpler . Steve On Mon, Jul 26, 2021 at 11:03 PM Gabe Black wrote: > Or I should say without un-namespaced macros (GEM5_ prefixed), since > GEM5_ONCE itself would be a macro. > > Gabe > > On Mon, Jul

[gem5-dev] Re: "epic" Jira issue to track multi-ISA gem5

2021-07-27 Thread Bobby Bruce via gem5-dev
This will make things so much better. I very much support it. Thanks for creating the Jira epic for tracking this. -- Dr. Bobby R. Bruce Room 3050, Kemper Hall, UC Davis Davis, CA, 95616 web: https://www.bobbybruce.net On Tue, Jul 27, 2021 at 3:42 AM Giacomo Travaglini via gem5-dev <

[gem5-dev] Change in gem5/gem5[release-staging-v21-1]: misc: Update the version to v21.1.0.0

2021-07-27 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48583 ) Change subject: misc: Update the version to v21.1.0.0 .. misc: Update the version to v21.1.0.0 Change-Id:

[gem5-dev] Change in gem5/gem5[release-staging-v21-1]: scons: Remove -Werror for the gem5 21.1 release

2021-07-27 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48584 ) Change subject: scons: Remove -Werror for the gem5 21.1 release .. scons: Remove -Werror for the gem5 21.1 release

[gem5-dev] Change in gem5/gem5[release-staging-v21-1]: docker-util: Update the gcn-gpu docker image Cloud Bucket

2021-07-27 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/48663 ) Change subject: docker-util: Update the gcn-gpu docker image Cloud Bucket .. docker-util: Update the gcn-gpu

[gem5-dev] Re: deprecating warn_once

2021-07-27 Thread Daniel Carvalho via gem5-dev
Hello, I like the idea of GEM5_ONCE, but as it would become part of the API users could definitely use in unforeseen ways (e.g., debugging). I am not saying this is a bad thing, but it is something to consider. Regards,Daniel ___ gem5-dev mailing list