Re: [gem5-dev] Namespace Inconvenience

2020-04-17 Thread Gabe Black
Hi Daneil, thanks for the heads up. I've seen similar things like that every once in a while in the past and had hoped to some day nail down where the stale files were. It seems to be the param structure definitions for you? I can see how that would happen since those are generated from python,

[gem5-dev] Design doc - Merge gem5’s VPtr and BufferArg classes

2020-04-17 Thread Gabe Black
One of these types is no longer used since Alpha was deleted, and they're pretty similar to one another but with some complementary capabilities. I put together a design doc which lays out how to merge them, and add some minor new features like teaching the GuestABI to construct them automatically

Re: [gem5-dev] How to get the size in bytes of a guest int or unsigned value to implement a system call?

2020-04-17 Thread Gabe Black
Hi Ciro. There are some folks from AMD I think who have been doing quite a lot with syscall emulation and should probably chime in too, but here is my understanding. If you're accepting an address, you should use the Addr type. This is sometimes treated specially, and even though it's a 64 bit

[gem5-dev] Change in gem5/gem5[develop]: sim: Use off_t for mmap offset arguments

2020-04-17 Thread Matthew Poremba (Gerrit)
Matthew Poremba has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27367 ) Change subject: sim: Use off_t for mmap offset arguments .. sim: Use off_t for mmap offset arguments The GuestABI used

[gem5-dev] How to get the size in bytes of a guest int or unsigned value to implement a system call?

2020-04-17 Thread Ciro Santilli
I want to implement the simple getcpu syscall. According to man getcpu and kernel source in kernel/sys.c, that function as an argument an unsigned *cpu to which it writes the value. I was looking through the other syscall implementations in syscall_emul.hh but all I've seen so far appear to

[gem5-dev] Change in gem5/gem5[develop]: mem-cache: Create Prefetcher namespace

2020-04-17 Thread Daniel Carvalho (Gerrit)
Daniel Carvalho has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/24537 ) Change subject: mem-cache: Create Prefetcher namespace .. mem-cache: Create Prefetcher namespace Create a namespace

[gem5-dev] Namespace Inconvenience

2020-04-17 Thread Daniel Carvalho
Hello all, I have just merged https://gem5-review.googlesource.com/c/public/gem5/+/24537, which may cause the following inconvenience: In file included from build/ISA/python/_m5/param_BaseCache.cc:4:0:build/ISA/params/BaseCache.hh:66:5: error: 'BasePrefetcher' does not name a type

Re: [gem5-dev] gem5-resources: Feedback appreciated!

2020-04-17 Thread Ciro Santilli
Yes, my main point is that hopefully when you run "make ISA=arm64" / or scons it will build at least all single input source files that we have in one go e.g.: main.c -> out/arm64/main.out test2.c -> out/arm64/test2.out for the given ISA, as I did in the mentioned patchset, without me having

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby, mem-garnet: Multiple networks per RubySystem

2020-04-17 Thread Matthew Poremba (Gerrit)
Matthew Poremba has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27927 ) Change subject: mem-ruby,mem-garnet: Multiple networks per RubySystem .. mem-ruby,mem-garnet: Multiple

[gem5-dev] Change in gem5/gem5[develop]: sim-se: add missing path redirection to mmap createObjectFile

2020-04-17 Thread Ciro Santilli (Gerrit)
Ciro Santilli has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27887 ) Change subject: sim-se: add missing path redirection to mmap createObjectFile .. sim-se: add missing path redirection

[gem5-dev] Change in gem5/gem5[develop]: configs: make --disk-images optional in fs.py

2020-04-17 Thread Ciro Santilli (Gerrit)
Ciro Santilli has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27867 ) Change subject: configs: make --disk-images optional in fs.py .. configs: make --disk-images optional in fs.py The main