Re: [gem5-dev] gem5 wiki can't accept new files?

2017-11-05 Thread Ali Saidi
Yep.. the permissions were wrong. Give it a try now. Ali > On Nov 2, 2017, at 5:54 PM, Gabe Black wrote: > > I think I saw an email from Ali recently that he updated the wiki? Maybe it > has something to do with that? Maybe some file or directory permissions > were clobbered in the process? >

[gem5-dev] Wiki updated

2017-10-21 Thread Ali Saidi
The wiki has been updated to the latest version of mediawiki. Everything appears to be working, but if you run into issues please let me know. Ali ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

Re: [gem5-dev] Issue with uploaded images on the wiki

2017-04-27 Thread Ali Saidi
It’s been fixed, sorry about that. Ali > On Apr 27, 2017, at 8:21 AM, Ali Saidi wrote: > > I upgraded the wiki about a week ago to address some security issues and > something much have gone wrong, I’ll take a look at what is happening. > > Ali > > > >

Re: [gem5-dev] Issue with uploaded images on the wiki

2017-04-27 Thread Ali Saidi
I upgraded the wiki about a week ago to address some security issues and something much have gone wrong, I’ll take a look at what is happening. Ali > On Apr 27, 2017, at 5:53 AM, Andreas Sandberg > wrote: > > I had a quick look at the MediaWiki config and it seems like there is > something

Re: [gem5-dev] Review Request 3814: misc: Add a CONTRIBUTING document

2017-02-28 Thread Ali Saidi
988> I like Andreas' comments, but actually I don't think the original is negative either. Andreas 1,2 is probably a good reason to do it and the current 1,2 are the carrots of why you should. - Ali Saidi On Feb. 21, 2017, 5:34 p.m., Jason

Re: [gem5-dev] Review Request 3688: alpha: Remove ALPHA tru64 support and associated tests

2016-11-14 Thread Ali Saidi
688/#comment7787> Both Linux and Tru64 use the same callpal to trigger a system call. - Ali Saidi On Oct. 27, 2016, 7:45 a.m., Andreas Hansson wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http:

Re: [gem5-dev] Review Request 3693: riscv: [Patch 7/5] Corrected LRSC semantics

2016-11-02 Thread Ali Saidi
> On Nov. 2, 2016, 7:38 p.m., Alec Roelke wrote: > > This doesn't work with the O3 CPU model; I wrote a simple program that > > performs a lr.w followed by sc.w that works with the atomic-simple, > > timing-simple, and minor CPU models, but with the O3 model I get this error: > > > > gem5.debu

[gem5-dev] Wiki Upgrade

2016-06-24 Thread Ali Saidi
Hi Everyone, The wiki has just been upgraded. Please let us know if there are any problems. Thanks, Ali ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

Re: [gem5-dev] Review Request 3228: dist: Distributed Ethernet link support for distributed gem5 simulations

2015-11-28 Thread Ali Saidi
ng, we probably shouldn't repeat it here. - Ali Saidi On Nov. 19, 2015, 5:44 p.m., Curtis Dunham wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://

Re: [gem5-dev] Review Request 3235: stats: make sim_insts and sim_ops respect stats reset

2015-11-28 Thread Ali Saidi
> On Nov. 25, 2015, 6:42 p.m., Andreas Hansson wrote: > > Traditionally, it's not reset, as it accounting for all instructions > > executed. Perhaps hostSeconds shouldn't be reset as well and then the > > calculation would be correct? sim_insts isn't supposed to be used for > > anything other

Re: [gem5-dev] Review Request 3207: arm: remote GDB: rationalize structure of register offsets

2015-11-14 Thread Ali Saidi
207/#comment6516> would like to see doxygen documentation for these methods Overall this seems like a sensible approach to me - Ali Saidi On Nov. 13, 2015, 5:21 p.m., Boris Shingarov wrote: > > --- > This is an automatically g

Re: [gem5-dev] Review Request 3200: sparc: Make remote debugging with gdb work

2015-11-07 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3200/#review7531 --- Ship it! Ship It! - Ali Saidi On Nov. 7, 2015, 9:22 p.m., Palle

Re: [gem5-dev] Review Request 3150: cpu: remove O3's fetch address validity check

2015-10-12 Thread Ali Saidi
> On Oct. 12, 2015, 4:58 p.m., Nilay Vaish wrote: > > Can you explain why it is no longer necessary? > > Curtis Dunham wrote: > Empirically, all regressions still pass. :-) > > It might be easier to explain why it was ever necessary, which is > definitely before my time with gem5.

Re: [gem5-dev] including reviewboard URL in commit message?

2015-09-12 Thread Ali Saidi
> On Sep 4, 2015, at 12:54 PM, Steve Reinhardt wrote: > > To answer Jason's question: it would be nice to have an automation system > where someone with sufficient permissions could just click on a patch on > reviewboard and the patch would automatically be committed; with that > process in pla

Re: [gem5-dev] Review Request 3034: base: fix small memory leak in the ELF loader

2015-08-13 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3034/#review6936 --- Ship it! Ship It! - Ali Saidi On Aug. 12, 2015, 9:19 p.m., Brandon

Re: [gem5-dev] UnserliazeMemory

2015-08-12 Thread Ali Saidi
Much like the comment says, the virtual memory layer in the operating system lazily allocates pages. If they're zero they'll get lazily allocated so there isn't any reason for us to use up a lot of extra system memory storing 0s. Ali Sent from my ARM powered mobile device > On Aug 12, 2015, at

Re: [gem5-dev] Review Request 2741: arch, base, dev, kern, sym: FreeBSD support

2015-04-26 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2741/#review6083 --- Ship it! Ship It! - Ali Saidi On April 24, 2015, 5:45 p.m., Ruslan

Re: [gem5-dev] Review Request 2741: arch, base, dev, kern, sym: FreeBSD support

2015-04-23 Thread Ali Saidi
can all just panic() with unimplmented if there isn't a good corallary in FreeBSD - Ali Saidi On April 23, 2015, 3:26 p.m., Ruslan Bukin wrote: > > --- > This is an automatically gener

Re: [gem5-dev] Review Request 2741: arch, base, dev, kern, sym: FreeBSD support

2015-04-19 Thread Ali Saidi
Linux (in which case nothing is done) or FreeBSD which if(!FreeBSD) is added to a few of the lines? Thanks Ruslan. This is getting a lot closer, and it will be great to have FreeBSD support in gem5. I have a couple questions above about code duplication between FreeBSD/Linux. Thanks again, Ali -

Re: [gem5-dev] Review Request 2732: arch: do not set Linux-specific AT elf relocation types on FreeBSD

2015-04-09 Thread Ali Saidi
> On April 9, 2015, 6:06 p.m., Ali Saidi wrote: > > Sorry, my comments disappeared somewhere. Does this change allow for gem5 to compile on FreeBSD? gem5 to run a FreeBSD binary? Both? If it's the prior, it would be good to allow any type of binary to run on any host OS. If i

Re: [gem5-dev] Review Request 2732: arch: do not set Linux-specific AT elf relocation types on FreeBSD

2015-04-09 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2732/#review6033 --- - Ali Saidi On April 9, 2015, 4:42 p.m., Ruslan Bukin wrote

Re: [gem5-dev] Review Request 2728: base: fix includes for FreeBSD

2015-04-09 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2728/#review6032 --- Ship it! Ship It! - Ali Saidi On April 9, 2015, 9:07 a.m., Ruslan

Re: [gem5-dev] Review Request 2729: base: include header file for FreeBSD

2015-04-09 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2729/#review6031 --- Ship it! Ship It! - Ali Saidi On April 9, 2015, 9:29 a.m., Ruslan

Re: [gem5-dev] Review Request 2730: base: fix FreeBSD OS detection

2015-04-09 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2730/#review6030 --- Ship it! Ship It! - Ali Saidi On April 9, 2015, 9:37 a.m., Ruslan

Re: [gem5-dev] Review Request 2731: dev: dont panic on GIC_EOIR

2015-04-09 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2731/#review6029 --- Ship it! Ship It! - Ali Saidi On April 9, 2015, 9:51 a.m., Ruslan

Re: [gem5-dev] Review Request 2710: Correct the endianess detection

2015-03-27 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2710/#review5978 --- Ship it! Ship It! - Ali Saidi On March 27, 2015, 2 p.m., Ruslan

Re: [gem5-dev] Review Request 2705: Extend access width for IDE control registers

2015-03-27 Thread Ali Saidi
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2705/#review5977 --- Ship it! Thanks for the change! - Ali Saidi On March 25, 2015, 11:10

[gem5-dev] gem5.org Maintenance

2015-03-08 Thread Ali Saidi
gem5.org is undergoing some maintenance. Please let us know if you have any trouble accessing services on gem5.org Monday morning. Thanks, Ali ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

Re: [gem5-dev] Configuration GUI for gem5

2015-03-03 Thread Ali Saidi via gem5-dev
Hi Marcus, Option 1 is probably the most preferred route as the code is much more likely to say in-sync and compatible if it¹s in the same repository and tested along side gem5. The best way to proceed with that option is to post your patches to the gem5 review board http://reviews.gem5.org. If yo

[gem5-dev] changeset in gem5: cpu: add support for outputing a protobuf for...

2015-02-16 Thread Ali Saidi via gem5-dev
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + * + * Authors: Ali Saidi + */ + +#include "cpu/inst_pb_trace.hh" + +#include "base/callback.hh" +#include "base/output.hh&q

[gem5-dev] changeset in gem5: sim: Clean up InstRecord

2015-01-25 Thread Ali Saidi via gem5-dev
changeset aef704eaedd2 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=aef704eaedd2 description: sim: Clean up InstRecord Track memory size and flags as well as add some comments and consts. diffstat: src/cpu/base_dyn_inst.hh| 10 +- src/c

[gem5-dev] changeset in gem5: cpu: Remove all notion that we know when the ...

2015-01-25 Thread Ali Saidi via gem5-dev
changeset 61a0b02aa800 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=61a0b02aa800 description: cpu: Remove all notion that we know when the cpu is misspeculating. We have no way of knowing if a CPU model is on the wrong path with our execute-in-execu

[gem5-dev] changeset in gem5: arm: always set the IsFirstMicroop flag

2015-01-25 Thread Ali Saidi via gem5-dev
changeset 3c42be107634 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=3c42be107634 description: arm: always set the IsFirstMicroop flag While the IsFirstMicroop flag exists it was only occasionally used in the ARM instructions that gem5 microOps and

[gem5-dev] changeset in gem5: cpu: remove legion tracer

2015-01-25 Thread Ali Saidi via gem5-dev
changeset c3fd4c020e49 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=c3fd4c020e49 description: cpu: remove legion tracer If someone wants to debug with legion again they can restore the code from the repository, but no need to have it hang around in

[gem5-dev] changeset in gem5: cpu: Put all CPU instruction tracers in a sin...

2015-01-25 Thread Ali Saidi via gem5-dev
changeset fae54a666162 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=fae54a666162 description: cpu: Put all CPU instruction tracers in a single file diffstat: src/arch/arm/ArmNativeTrace.py | 2 +- src/arch/sparc/SparcNativeTrace.py | 2 +- src/arch/x86/X8

Re: [gem5-dev] Review Request 2604: cpu: commit probe notification on every microop or macroop

2015-01-14 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2604/#review5754 --- Ship it! Ship It! - Ali Saidi On Jan. 14, 2015, 6:13 p.m., Nikos

Re: [gem5-dev] simpoints and KVM

2015-01-13 Thread Ali Saidi via gem5-dev
That is doable. You can created checkpoints, there is just no way to instrument every basic-block. Ali On 1/13/15, 3:13 PM, "mike upton via gem5-dev" wrote: >Oh, Bummer. > >I was hoping to be able to get simpoints out of a KvmCPU run. >It seems like that is not possible. > >Is it possible to

Re: [gem5-dev] gem5 on redhat6

2015-01-09 Thread Ali Saidi via gem5-dev
It should probably be /usr/bin/env python Ali On 1/9/15, 1:12 PM, "mike upton via gem5-dev" wrote: >OK, thanks. > >I don't understand what is going on, but blindly following the suggestion >made it work. > >I needed to do: > >python /usr/bin/scons > > >Oh, I see what is going on. >/usr/bin/sco

Re: [gem5-dev] Review Request 2305: dev: add an ethernet switch model

2015-01-05 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2305/#review5734 --- Ship it! Ship It! - Ali Saidi On July 24, 2014, 7:44 p.m., Anthony

Re: [gem5-dev] Review Request 2585: test: Add a unittest for the BitUnion types.

2014-12-26 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2585/#review5715 --- Ship it! Ship It! - Ali Saidi On Dec. 26, 2014, 9:32 p.m., Gabe

Re: [gem5-dev] Review Request 2585: test: Add a unittest for the BitUnion types.

2014-12-26 Thread Ali Saidi via gem5-dev
the other files? - Ali Saidi On Dec. 17, 2014, 7:08 a.m., Gabe Black wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem

Re: [gem5-dev] Review Request 2525: x86: Add a cirrus graphics adapter to x86 systems.

2014-12-21 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2525/#review5701 --- Ship it! Seems fine... Anyone who deals with Ruby OK? - Ali Saidi On

Re: [gem5-dev] Review Request 2519: x86: i8042: Give the keyboard controller a little TLC.

2014-12-21 Thread Ali Saidi via gem5-dev
well - Ali Saidi On Dec. 17, 2014, 7:35 a.m., Gabe Black wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2519/ > -

Re: [gem5-dev] InOrder CPU with ARM

2014-12-17 Thread Ali Saidi via gem5-dev
The in-order cpu model is deprecated and will soon be removed from the tree. Id you'd like an in order cpu model use the minor cpu model which supports ARM. Ali Sent from my ARM powered mobile device > On Dec 17, 2014, at 2:18 PM, Anastasiia via gem5-dev > wrote: > > Hi, > > I try to run

[gem5-dev] Review Request 2563: cpu: add support for outputing a protobuf formatted CPU trace

2014-12-10 Thread Ali Saidi via gem5-dev
4e09ae443c96 src/proto/inst.proto PRE-CREATION util/decode_inst_trace.py PRE-CREATION Diff: http://reviews.gem5.org/r/2563/diff/ Testing --- Thanks, Ali Saidi ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo

[gem5-dev] Review Request 2562: arm: always set the IsFirstMicroop flag

2014-12-10 Thread Ali Saidi via gem5-dev
em.cc 4e09ae443c96 src/arch/arm/isa/templates/mem.isa 4e09ae443c96 src/arch/arm/isa/templates/mem64.isa 4e09ae443c96 src/cpu/static_inst.hh 4e09ae443c96 Diff: http://reviews.gem5.org/r/2562/diff/ Testing --- Thanks, Ali Saidi ___ gem5-dev ma

[gem5-dev] Review Request 2561: sim: Clean up InstRecord

2014-12-10 Thread Ali Saidi via gem5-dev
/minor/lsq.cc 4e09ae443c96 src/cpu/simple/atomic.cc 4e09ae443c96 src/cpu/simple/timing.cc 4e09ae443c96 src/sim/insttracer.hh 4e09ae443c96 Diff: http://reviews.gem5.org/r/2561/diff/ Testing --- Thanks, Ali Saidi ___ gem5-dev mailing list

[gem5-dev] Review Request 2560: cpu: Remove all notion that we know when the cpu is misspeculating.

2014-12-10 Thread Ali Saidi via gem5-dev
/diff/ Testing --- Thanks, Ali Saidi ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Review Request 2559: cpu: Put all CPU instruction tracers in a single file

2014-12-10 Thread Ali Saidi via gem5-dev
/cpu/CPUTracers.py PRE-CREATION src/cpu/ExeTracer.py 4e09ae443c96 src/cpu/IntelTrace.py 4e09ae443c96 src/cpu/NativeTrace.py 4e09ae443c96 src/cpu/SConscript 4e09ae443c96 Diff: http://reviews.gem5.org/r/2559/diff/ Testing --- Thanks, Ali Saidi

[gem5-dev] Review Request 2558: cpu: remove legion tracer

2014-12-10 Thread Ali Saidi via gem5-dev
/2558/diff/ Testing --- Thanks, Ali Saidi ___ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev

Re: [gem5-dev] Review Request 2510: Let other objects set up memory like regions in a KVM VM.

2014-12-06 Thread Ali Saidi via gem5-dev
I think it’s fine that a device wants to do this, I’d just like it to use an thin interface on the System object as a matter of clean interfaces in the object hierarchy, so unrelated objects don’t have to know about each other. Ali On Dec 3, 2014, at 11:54 AM, Gabe Black via gem5-dev wrote:

Re: [gem5-dev] Review Request 2556: misc: Generalize GDB single stepping.

2014-12-05 Thread Ali Saidi via gem5-dev
but I haven't had a chance to actually test it yet. - Ali Saidi On Dec. 5, 2014, 11:15 a.m., Gabe Black wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://review

Re: [gem5-dev] Review Request 2555: misc: Add some utility functions for schedule inst commit events.

2014-12-05 Thread Ali Saidi via gem5-dev
555/#comment5044> can these be doxygen comment? (e.g. /// or /*** one minor request, no need to repost - Ali Saidi On Dec. 5, 2014, 10:36 a.m., Gabe Black wrote: > > --- > This is an automatically generated e-mail.

Re: [gem5-dev] Review Request 2554: misc: Rename the GDB "Event" event class to InputEvent.

2014-12-05 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2554/#review5639 --- Ship it! Ship It! - Ali Saidi On Dec. 5, 2014, 10 a.m., Gabe Black

Re: [gem5-dev] Review Request 2548: arm: Add unlinkat syscall implementation

2014-12-04 Thread Ali Saidi via gem5-dev
> On Dec. 4, 2014, 3:03 p.m., Nilay Vaish wrote: > > src/sim/syscall_emul.hh, line 201 > > > > > > How will the compiler choose between the two versions of unlinkFunc? I > > think we should either drop the default argument

Re: [gem5-dev] testing

2014-12-04 Thread Ali Saidi via gem5-dev
Hi Gabe/Steve, As Steve mentioned I¹ve been working on a new take on the regression system in my spare-time. I want to get it a bit more complete before I show it to the world, but some of the goals I set out with align with yours. In particular I¹m targeting the following: (1) Success/Failure b

Re: [gem5-dev] Review Request 2548: arm: Add unlinkat syscall implementation

2014-12-04 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2548/#review5616 --- Ship it! Ship It! - Ali Saidi On Dec. 3, 2014, 7:10 p.m., mike upton

Re: [gem5-dev] Review Request 2516: config: Add options to take/resume from SimPoint checkpoints

2014-11-28 Thread Ali Saidi via gem5-dev
> On Nov. 25, 2014, 10:02 p.m., Nilay Vaish wrote: > > configs/common/Options.py, line 155 > > > > > > Can you explain why we need this separate option for restoring from a > > checkpoint taken using the take-simpoint-check

Re: [gem5-dev] Review Request 2517: x86: vnc: Add a VNC server to x86 systems.

2014-11-28 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2517/#review5562 --- Ship it! Ship It! - Ali Saidi On Nov. 22, 2014, 11:39 a.m., Gabe

Re: [gem5-dev] Review Request 2521: ps2: Support translating left and right ALT keys.

2014-11-28 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2521/#review5561 --- Ship it! Ship It! - Ali Saidi On Nov. 22, 2014, 1:36 p.m., Gabe

Re: [gem5-dev] Review Request 2523: config: Get rid of some extra spaces around default arguments.

2014-11-28 Thread Ali Saidi via gem5-dev
On Nov 26, 2014, at 12:11 PM, Steve Reinhardt via gem5-dev wrote: > On Wed, Nov 26, 2014 at 4:30 AM, Gabe Black via gem5-dev > wrote: > >> >> >> Just to make sure we're all on the same page, I removed the spaces here >> because these are default values for arguments. My brain might have jus

Re: [gem5-dev] KvmCPU Behaviour

2014-11-28 Thread Ali Saidi via gem5-dev
There are functions that correspond to this functionality although they’re poorly named for the purpose, but memWriteback() -> beginning of KVM simulation; memInvalidate -> end of KVM simulation. Another option would be to check for the memory time atomic_noncaching which I believe only KVM uses

Re: [gem5-dev] "Search source" link broken

2014-11-28 Thread Ali Saidi via gem5-dev
daystrom is very low on ram at the moment with the various services we’re running. We have a new machine that has substantially more ram that we’ll be switching to in the near term which should address the issue. For the moment I’ve restarted tomcat and it seems to be working. Ali On Nov 28, 2

Re: [gem5-dev] VNC errors

2014-11-25 Thread Ali Saidi via gem5-dev
Looking at the code it looks like it would be possible for there to be <12 bytes in the socket, so the read will return less that 12 bytes (because you got very unlucky and the bytes were split across two packets). Looking at the code fixing that assumption will be a bit of a pain. Ali On 11/

Re: [gem5-dev] some rough edges on remote GDB support

2014-11-24 Thread Ali Saidi via gem5-dev
On Nov 24, 2014, at 11:51 PM, Gabe Black via gem5-dev wrote: > I'm working on adding remote GDB support for x86, and I've noticed a couple > of rough edges I'd like to smooth out somehow. > > First, when GDB attaches, a pollEvent will be added which will give the GDB > stub control immediately.

Re: [gem5-dev] Review Request 2528: config: make M5_PATH a real search path

2014-11-23 Thread Ali Saidi via gem5-dev
iltin function file() as an identifier/parameter in this code - Ali Saidi On Nov. 24, 2014, 2:48 a.m., Steve Reinhardt wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://review

Re: [gem5-dev] Review Request 2513: KVM: Build in most of the KVM stuff even if we're not going to use it.

2014-11-18 Thread Ali Saidi via gem5-dev
On 11/18/14, 5:02 PM, "Steve Reinhardt via gem5-dev" wrote: > >I don't follow your question, Nilay. The former suggestion is really a >generalization of Andreas's idea, so the same answer holds. In the latter >case, you'd still need to ifdef some C++ code with USE_KVM (as Ali >originally sugge

Re: [gem5-dev] Review Request 2462: mem: Page Table map api modification

2014-11-18 Thread Ali Saidi via gem5-dev
926> setting AP=3 makes the page table user-read-only. one minor comment - Ali Saidi On Oct. 6, 2014, 6:58 p.m., Alexandru Dutu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http:

Re: [gem5-dev] Review Request 2513: KVM: Build in most of the KVM stuff even if we're not going to use it.

2014-11-17 Thread Ali Saidi via gem5-dev
der that can be used in case !USE_KVM, but as you allude to this is very ugly. Off the top of my head the way I'd prefer to see it done is let objects register ranges with the System object and then the KVM code could iterate that list if it was compiled in. Sound reasonable? - Ali Saidi On

Re: [gem5-dev] Error while building gem5.fast on latest on copy of gem5

2014-11-17 Thread Ali Saidi via gem5-dev
What compiler and version are you using? Ali On 11/17/14, 2:37 PM, "Urmish Ajit Thakker via gem5-dev" wrote: >Hi, > >I pulled up the latest copy of gem5 and got the following error while >building gem5.fast. > >build/ARM/proto/protoio.fo (symbol from plugin): warning: memset used >with constan

Re: [gem5-dev] Review Request 2508: vnc: Add a conversion function for bgr888.

2014-11-17 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2508/#review5459 --- Ship it! Ship It! - Ali Saidi On Nov. 17, 2014, 9:45 a.m., Gabe

[gem5-dev] changeset in gem5: arm: Fix timing wakeup with LLSC

2014-11-12 Thread Ali Saidi via gem5-dev
changeset d1dce0b728b6 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=d1dce0b728b6 description: arm: Fix timing wakeup with LLSC diffstat: src/cpu/simple/timing.cc | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diffs (29 lines): diff -r 6645

Re: [gem5-dev] Review Request 2474: minor: fixed LSQ MasterPortID

2014-11-10 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2474/#review5450 --- Ship it! Ship It! - Ali Saidi On Nov. 10, 2014, 5:20 p.m., Andrew

Re: [gem5-dev] Review Request 2468: Solved bugs while switching cpu models with the --repeat-switch command

2014-11-08 Thread Ali Saidi via gem5-dev
would need to comment on the other one. - Ali Saidi On Nov. 7, 2014, 9:35 p.m., Alberto Javier Naranjo Carmona wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem

Re: [gem5-dev] Review Request 2430: config: Add the ability to read a config file using C++ and Python

2014-11-07 Thread Ali Saidi via gem5-dev
d()) { > > (gdb) > > > > Trying it with other cpu-type and other mem-type results in similar seg > > faults. It looks like this patch is missing some state initialization or > > some other checkpoint restore functionality for X86. > > Ali Saidi wrote: >

Re: [gem5-dev] Review Request 2430: config: Add the ability to read a config file using C++ and Python

2014-11-06 Thread Ali Saidi via gem5-dev
> On Nov. 6, 2014, 7:48 p.m., Cagdas Dirik wrote: > > This patch seems to be broken for X86 when restoring from checkpoints. A > > sample test crashes with segmentation fault. Here are the steps: > > 0. Sample test program does int array manipulation and creates a checkpoint > > before computat

Re: [gem5-dev] Cron /z/m5/regression/do-regression --scratch all

2014-11-03 Thread Ali Saidi via gem5-dev
These should now be fixed. Thanks, Ali On 11/2/14, 3:20 PM, "Cron Daemon via gem5-dev" wrote: >* >build/SPARC/tests/opt/long/fs/80.solaris-boot/sparc/solaris/t1000-simple-a >tomic CHANGED! >* >build/ARM/tests/opt/long/fs/10.linux-boot/arm/linux/realview-minor-dual >CHANGED! >* buil

[gem5-dev] changeset in gem5: tests: Update stats no match.

2014-11-03 Thread Ali Saidi via gem5-dev
changeset ba51f8572571 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=ba51f8572571 description: tests: Update stats no match. Bootloader I had on my sytem was an older version with a couple of instruction differences. diffstat: tests/long/fs/10.lin

Re: [gem5-dev] Cron /z/m5/regression/do-regression quick

2014-10-30 Thread Ali Saidi via gem5-dev
I'll see if it clears up today, and if not I'll investigate. Ali On 30.10.2014 03:19, Cron Daemon via gem5-dev wrote: > * > build/ARM/tests/opt/quick/fs/10.linux-boot/arm/linux/realview-simple-atomic > CHANGED! > * > build/ARM/tests/opt/quick/fs/10.linux-boot/arm/linux/realview-

[gem5-dev] changeset in gem5: arm, tests: Forgot the system.terminal files ...

2014-10-29 Thread Ali Saidi via gem5-dev
changeset cae494887847 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=cae494887847 description: arm, tests: Forgot the system.terminal files for the new regressions. diffstat: tests/long/fs/10.linux-boot/ref/arm/linux/realview64-minor-dual/system.terminal

[gem5-dev] changeset in gem5: arm, tests: Add 64-bit ARM regression tests

2014-10-29 Thread Ali Saidi via gem5-dev
changeset bd7c2aa12122 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=bd7c2aa12122 description: arm, tests: Add 64-bit ARM regression tests diffstat: tests/long/fs/10.linux-boot/ref/arm/linux/realview64-minor-dual/config.ini | 2431 +++ tests/long/fs

[gem5-dev] changeset in gem5: tests: Update regressions for the new kernels...

2014-10-29 Thread Ali Saidi via gem5-dev
changeset ca4438b6e39a in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=ca4438b6e39a description: tests: Update regressions for the new kernels and various preceeding fixes. diffstat: tests/long/fs/10.linux-boot/ref/alpha/linux/tsunami-minor/config.ini

[gem5-dev] changeset in gem5: arm, tests: Update config files to more recen...

2014-10-29 Thread Ali Saidi via gem5-dev
changeset b423e1d0735e in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=b423e1d0735e description: arm, tests: Update config files to more recent kernels and create 64-bit regressions. This changes the default ARM system to a Versatile Express-like system that

[gem5-dev] changeset in gem5: automated merge

2014-10-29 Thread Ali Saidi via gem5-dev
changeset 2b416ef3b400 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=2b416ef3b400 description: automated merge diffstat: src/arch/alpha/linux/process.cc |4 +- src/arch/arm/linux/process.cc|4 +- src/arch/mips/linux/process.cc |4 +- src/arch/po

[gem5-dev] changeset in gem5: arm: fix bare-metal memory setup.

2014-10-29 Thread Ali Saidi via gem5-dev
changeset f33fab6214c4 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=f33fab6214c4 description: arm: fix bare-metal memory setup. The bare-metal configuration option still configured memory with the old scheme that no-longer works. This change unifie

[gem5-dev] changeset in gem5: cpu: Add support to checker for CACHE_BLOCK_Z...

2014-10-29 Thread Ali Saidi via gem5-dev
changeset 38c7a9ea7729 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=38c7a9ea7729 description: cpu: Add support to checker for CACHE_BLOCK_ZERO commands. The checker didn't know how to properly validate these new commands. diffstat: src/cpu/checker/cpu.cc

[gem5-dev] changeset in gem5: arm, mem: Fix drain bug and provide drain pri...

2014-10-29 Thread Ali Saidi via gem5-dev
changeset d5554f97c451 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=d5554f97c451 description: arm, mem: Fix drain bug and provide drain prints for more components. diffstat: src/arch/arm/table_walker.cc | 5 ++--- src/mem/cache/mshr_queue.cc | 3 +++ src/mem/d

[gem5-dev] changeset in gem5: arm: Mark some miscregs (timer counter) regis...

2014-10-29 Thread Ali Saidi via gem5-dev
changeset aa23216161fa in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=aa23216161fa description: arm: Mark some miscregs (timer counter) registers at unverifiable. The checker can't verify timer registers, so it should just grab the version from the e

[gem5-dev] changeset in gem5: arm: Fix multi-system AArch64 boot w/caches.

2014-10-29 Thread Ali Saidi via gem5-dev
changeset aa46a8ae3487 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=aa46a8ae3487 description: arm: Fix multi-system AArch64 boot w/caches. Automatically extract cpu release address from DTB file. Check SCTLR_EL1 to verify all caches are enabled. di

Re: [gem5-dev] Review Request 2470: cpu: Minor Draining Bug

2014-10-29 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2470/#review5436 --- Ship it! Ship It! - Ali Saidi On Oct. 29, 2014, 3:01 p.m., Andrew

Re: [gem5-dev] Review Request 2468: Solved bugs while switching cpu models with the --repeat-switch command

2014-10-27 Thread Ali Saidi via gem5-dev
915> If you have a chance to test the dev repository that would be great. - Ali Saidi On Oct. 26, 2014, 5:16 p.m., Alberto Javier Naranjo Carmona wrote: > > --- > This is an automatically generated e-mail. To reply

Re: [gem5-dev] Review Request 2468: Solved bugs while switching cpu models with the --repeat-switch command

2014-10-27 Thread Ali Saidi via gem5-dev
912> I'm not sure which version of gem5 you're using, but in the latest dev repository it looks like this if block is completely gone. I'm not sure if that is because the check has been moved elsewhere or otherwise. - Ali Saidi On Oct. 26, 2014, 5:16 p.m., Alberto J

Re: [gem5-dev] Review Request 2468: Solved bugs while switching cpu models with the --repeat-switch command

2014-10-27 Thread Ali Saidi via gem5-dev
hange it supposed to be in there. otherwise I think it's good to go. src/mem/ruby/system/RubyMemoryControl.cc <http://reviews.gem5.org/r/2468/#comment4911> is this change supposed to be in here? - Ali Saidi On Oct. 26, 2014, 5:16 p.m., Alberto Javier Naran

Re: [gem5-dev] HTML output received on pushing patches to repo

2014-10-20 Thread Ali Saidi via gem5-dev
Hi Nilay, The idea is to automatically update a dockerized container that includes gem5. I¹m not quite sure why it¹s not working but I²ll look into it. Thanks, Ali On 10/20/14, 5:00 PM, "Nilay Vaish via gem5-dev" wrote: >Hi > >The last two times I have pushed patches to gem5's repo, I receiv

[gem5-dev] WriteInvalidate

2014-10-17 Thread Ali Saidi via gem5-dev
Hi Everyone, We've noticed an issue with the WriteInvalidate patch where the cpu reads or writes the same cache line that is being write invalidated. We're working on a fix and hope to have it out next week. If not we'll disable the code for the time being. Thanks, Ali

[gem5-dev] Review Request 2463: dev: refactor pci config space for sysfs scanning

2014-10-10 Thread Ali Saidi via gem5-dev
space. This change allows the extended PCI config space to be accessed by sysfs properly. Diffs - src/dev/pcidev.hh d96740732a61 src/dev/pcidev.cc d96740732a61 src/dev/pcireg.h d96740732a61 Diff: http://reviews.gem5.org/r/2463/diff/ Testing --- Thanks, Ali Saidi

Re: [gem5-dev] Review Request 2412: syscall emulation: minor style fix to LiveProcess constructor

2014-09-24 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2412/#review5352 --- Ship it! Ship It! - Ali Saidi On Sept. 23, 2014, 7:47 p.m., Steve

Re: [gem5-dev] Review Request 2410: syscall emulation: Put BufferArg classes in a separate header.

2014-09-24 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2410/#review5350 --- Ship it! Ship It! - Ali Saidi On Sept. 23, 2014, 7:47 p.m., Steve

Re: [gem5-dev] Review Request 2411: syscall emulation: devirtualize BaseBufferArg methods

2014-09-24 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2411/#review5351 --- Ship it! Ship It! - Ali Saidi On Sept. 23, 2014, 7:47 p.m., Steve

Re: [gem5-dev] Review Request 2409: syscall emulation: add EmulatedDriver object

2014-09-24 Thread Ali Saidi via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2409/#review5349 --- Ship it! Ship It! - Ali Saidi On Sept. 23, 2014, 7:46 p.m., Steve

  1   2   3   4   5   6   7   8   9   10   >