[gem5-dev] Memory fence neglected/obviated after squashing a younger/later fence

2020-01-15 Thread Eduardo Jose Gomez Hernandez
Hi everybody, During  running  some  benchmarks  (actually, a  modified version  of ocean-contiguous-partitions    from Splash-3 "https://github.com/SakalisC/Splash-3";), we encountered a deadlock. After diving into the trace files, we found that an atomic instruction was locking the

[gem5-dev] Change in gem5/gem5[master]: arch-arm: ELIsInHost, check VHE and SecEL2

2020-01-15 Thread Giacomo Travaglini (Gerrit)
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/24046 ) Change subject: arch-arm: ELIsInHost, check VHE and SecEL2 .. arch-arm: ELIsInHost, check VHE and SecEL2 This patch

[gem5-dev] Change in gem5/gem5[master]: arch-arm: Virtualization Host Extensions checking

2020-01-15 Thread Giacomo Travaglini (Gerrit)
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/24045 ) Change subject: arch-arm: Virtualization Host Extensions checking .. arch-arm: Virtualization Host Extensions checkin

[gem5-dev] Change in gem5/gem5[master]: tests: Migrated old quick/se/00.hello tests

2020-01-15 Thread Bobby R. Bruce (Gerrit)
Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/23024 ) Change subject: tests: Migrated old quick/se/00.hello tests .. tests: Migrated old quick/se/00.hello tests Migrated old

[gem5-dev] Change in gem5/gem5[master]: tests: Renaming tests to include dash between words

2020-01-15 Thread Bobby R. Bruce (Gerrit)
Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/23025 ) Change subject: tests: Renaming tests to include dash between words .. tests: Renaming tests to include dash between word

[gem5-dev] Change in gem5/gem5[refs/meta/config]: Review access change

2020-01-15 Thread Bobby R. Bruce (Gerrit)
Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/2 ) Change subject: Review access change .. Review access change Change-Id: I4569266fb64b3b3aea156091b7f82fff7d14

[gem5-dev] Change in gem5/gem5[refs/meta/config]: Review access change

2020-01-15 Thread Bobby R. Bruce (Gerrit)
Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/24443 ) Change subject: Review access change .. Review access change Change-Id: I4c9898d05fd2fe8f78e3e8a1abd80d771953

[gem5-dev] Change in gem5/gem5[refs/meta/config]: misc: Updated Gerrit permissions to restrict editing of master

2020-01-15 Thread Bobby R. Bruce (Gerrit)
Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/2 ) Change subject: misc: Updated Gerrit permissions to restrict editing of master .. misc: Updated Gerrit permissions to

[gem5-dev] Change in gem5/gem5[refs/meta/config]: Revert "misc: Updated Gerrit permissions to restrict editing of master"

2020-01-15 Thread Bobby R. Bruce (Gerrit)
Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/24445 ) Change subject: Revert "misc: Updated Gerrit permissions to restrict editing of master" .. Revert "misc: Up

Re: [gem5-dev] Change in gem5/gem5[refs/meta/config]: misc: Updated Gerrit permissions to restrict editing of master

2020-01-15 Thread Bobby Bruce
Apologies to all for this force push on Gerrit. Complete accident! This commit was supposed to stop non-maintainers from modifying the master branch as part of our "master-as-stable/develop-branch" transition. Though from playing around it doesn't appear to have had this effect (a non-maintainer i

[gem5-dev] Change in gem5/gem5[refs/meta/config]: Revert "misc: Updated Gerrit permissions to restrict editing of master"

2020-01-15 Thread Gabe Black (Gerrit)
Hello Bobby R. Bruce, I'd like you to do a code review. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/24463 to review the following change. Change subject: Revert "misc: Updated Gerrit permissions to restrict editing of master" ..

Re: [gem5-dev] Change in gem5/gem5[refs/meta/config]: misc: Updated Gerrit permissions to restrict editing of master

2020-01-15 Thread Gabe Black
I was able to revert it. Gabe On Wed, Jan 15, 2020 at 2:54 PM Bobby Bruce wrote: > Apologies to all for this force push on Gerrit. Complete accident! > > This commit was supposed to stop non-maintainers from modifying the master > branch as part of our "master-as-stable/develop-branch" transiti

Re: [gem5-dev] Change in gem5/gem5[refs/meta/config]: misc: Updated Gerrit permissions to restrict editing of master

2020-01-15 Thread Gabe Black
Hopefully I didn't just revert a new fixed version? I didn't see the second email (it was in my spam) until after I did the revert. Gabe On Wed, Jan 15, 2020 at 4:05 PM Gabe Black wrote: > I was able to revert it. > > Gabe > > On Wed, Jan 15, 2020 at 2:54 PM Bobby Bruce wrote: > >> Apologies t

[gem5-dev] Change in gem5/gem5[master]: arm: A couple small fixes for the arm64 bootloader makefile.

2020-01-15 Thread Gabe Black (Gerrit)
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/24329 ) Change subject: arm: A couple small fixes for the arm64 bootloader makefile. .. arm: A couple small fixes for the arm64 bootl

Re: [gem5-dev] Change in gem5/gem5[refs/meta/config]: misc: Updated Gerrit permissions to restrict editing of master

2020-01-15 Thread Bobby Bruce
Thanks Gabe! No, it's fine. I'm glad it's been reverted. -- Dr. Bobby R. Bruce Room 2235, Kemper Hall, UC Davis Davis, CA, 95616 web: https://www.bobbybruce.net On Wed, Jan 15, 2020 at 4:07 PM Gabe Black wrote: > Hopefully I didn't just revert a new fixed version? I didn't see the second > e

[gem5-dev] Design document for fast model (and ARM in general) pseudo instruction/checkpointing support

2020-01-15 Thread Gabe Black
Hi folks. The existing mechanisms for supporting pseudo instructions in the fast model based CPUs won't work, and so a new mechanism is needed. Also, because there are already a few different ways to call pseudo instructions, it would be nice to consolidate or at least coordinate them, and to avoid