Gabe Black has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/21460 )
Change subject: base: ELF segment types are not bitfields.
..
base: ELF segment types are not bitfields.
The ELF segment
Gabe Black has uploaded a new patch set (#2). (
https://gem5-review.googlesource.com/c/public/gem5/+/21505 )
Change subject: fastmodel: Add a CortexA76x2 model.
..
fastmodel: Add a CortexA76x2 model.
This is a WIP and
Hello Andreas Sandberg, Giacomo Travaglini, Jason Lowe-Power, Chun-Chen TK
Hsu,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/21049
to look at the new patch set (#6).
Change subject: fastmodel: Make CortexA76x1's interrupts use
Gabe Black has uploaded a new patch set (#2). (
https://gem5-review.googlesource.com/c/public/gem5/+/21500 )
Change subject: fastmodel: Expose all CPU communication ports from the GIC.
..
fastmodel: Expose all CPU
Hello Andreas Sandberg, Giacomo Travaglini, Jason Lowe-Power, Chun-Chen TK
Hsu,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/21050
to look at the new patch set (#6).
Change subject: fastmodel: Export GICV3Comms directly.
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21503 )
Change subject: fastmodel: Templatize the xn versions of the CortexA76.
..
fastmodel: Templatize the xn versions
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21505 )
Change subject: fastmodel: Add a CortexA76x2 model.
..
fastmodel: Add a CortexA76x2 model.
This is a WIP and
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21504 )
Change subject: fastmodel: Enable auto bridging and use it to simplify
CortexA76x1.
..
fastmodel: Enable auto
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21502 )
Change subject: fastmodel: Add string constructors which delegate to const
char * ones.
..
fastmodel: Add
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21500 )
Change subject: fastmodel: Expose all CPU communication ports from the GIC.
..
fastmodel: Expose all CPU
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21501 )
Change subject: fastmodel: Refactor the CortexA76x1 model for MP support.
..
fastmodel: Refactor the CortexA76x1
Thanks for the feedback Brad.
I think you are correct. We should probably give 2 weeks. I'll update the
guidelines accordingly.
Bobby
--
Dr. Bobby R. Bruce
Room 2236,
Kemper Hall, UC Davis
Davis,
CA, 95616
From: gem5-dev on behalf of Beckmann, Brad
Sent:
Thanks Bobby.
Most of this looks good to me, except can we extend the "no response period" in
1) from 48 hours to 2 weeks? Abandoning a patchset is a big step, so we should
make sure the contributor is unreachable beforehand.
Brad
-Original Message-
From: gem5-dev On Behalf Of
Dear all,
I'm currently in the process of updating CONTRIBUTING.md with some Gerrit Code
Review guidelines (please see my WIP change here:
https://gem5-review.googlesource.com/c/public/gem5/+/21419 ). My reason for
doing this is finding out no one really knows the procedure for addressing
* build/NULL/tests/opt/quick/se/80.dram-openpage/null/none/dram-lowp:
FAILED!
* build/NULL/tests/opt/quick/se/80.dram-closepage/null/none/dram-lowp:
FAILED!
* build/NULL/tests/opt/quick/se/60.rubytest/null/none/rubytest-ruby: FAILED!
*
Hello Andreas Sandberg, kokoro, Giacomo Travaglini, Jason Lowe-Power,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/21299
to look at the new patch set (#3).
Change subject: config: skip access to branchPred in ARM KVM
Hello Andreas Sandberg, kokoro, Giacomo Travaglini, Jason Lowe-Power,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/21299
to look at the new patch set (#2).
Change subject: config: skip access to branchPred in ARM KVM
Yeah, that's not much. I think you could hammer out some of that
incompatibility between versions by using data structure layout information
in the debug info in the kernel binary. I don't know how to read or
interpret that information, but gdb does it somehow so it must be in there.
Gabe
On
Hi folks. For a long time I've had my eye on factoring process and kernel
loading out of the system object so that it can be consistent between SE
and FS modes, and while I had a little spare time I decided to try to make
a little progress on that. I ended up going down a bit of a rabbit hole
with
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21466 )
Change subject: arch,base: Separate the idea of a memory image and object
file.
..
arch,base: Separate the
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21460 )
Change subject: base: ELF segment types are not bitfields.
..
base: ELF segment types are not bitfields.
The ELF
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21465 )
Change subject: arch,base: Stop loading the interpreter in ElfObject.
..
arch,base: Stop loading the interpreter
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21467 )
Change subject: arch,base: Restructure the object file loaders.
..
arch,base: Restructure the object file
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21463 )
Change subject: arch, base: Stop assuming object files have three segments.
..
arch, base: Stop assuming object
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21464 )
Change subject: arch-mips,arch-riscv,base: Get rid of the unused HexFile
class.
..
arch-mips,arch-riscv,base:
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21459 )
Change subject: base: Ensure %p format arguments are printed as pointers.
..
base: Ensure %p format arguments are
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21461 )
Change subject: base: Get rid of the unused global pointer in object files.
..
base: Get rid of the unused global
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21468 )
Change subject: arch,base,sim: Move Process loader hooks into the Process
class.
..
arch,base,sim: Move
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21462 )
Change subject: base: Rename Section to Segment, and some of its members.
..
base: Rename Section to Segment, and
Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/21469 )
Change subject: sparc: Create a helper functions to install firmware images.
..
sparc: Create a helper functions
I haven't found a working solution yet, the current state of my
research in that area is summarized at:
https://stackoverflow.com/questions/9561546/thread-aware-gdb-for-the-linux-kernel/51515470#51515470
On Fri, Oct 4, 2019 at 12:19 AM Gabe Black wrote:
>
> Hi folks. Recently I was trying to
Hello Andreas Sandberg, Giacomo Travaglini, Jason Lowe-Power,
I'd like you to reexamine a change. Please visit
https://gem5-review.googlesource.com/c/public/gem5/+/21439
to look at the new patch set (#2).
Change subject: configs: Isolate ISA related object lists
Tommaso Marinelli has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/21379 )
Change subject: kvm: Rename gettid() to build with glibc 2.30+
..
kvm: Rename gettid() to build with glibc 2.30+
33 matches
Mail list logo