Re: [gem5-dev] Does anyone care about X86 ISA simulations?

2016-04-12 Thread Steve Reinhardt
Hi Bjoern, As Jason said, we do use the x86 ISA regularly inside AMD, though we primarily use SE mode. The FS mode support was developed with the goal of getting Linux to boot, so I'm not surprised it doesn't work for other OSes. I don't know how widely used x86 FS is. The out-of-sequence Exec

Re: [gem5-dev] Review Request 3354: mem: Add unified queue to DRAMCtrl

2016-04-12 Thread Joel Hestness
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3354/#review8195 --- If I'm reading this patch correctly, I'm afraid I feel similarly about

Re: [gem5-dev] Memory consistency verification (and some bugs)

2016-04-12 Thread Beckmann, Brad
Hi Marco, Thank you for initiating the discussion. You mentioned concerns with checking in your patch (http://homepages.inf.ed.ac.uk/s0787712/res/mcversi_gem5.patch). Why do you think it needs more work? You already have two ship its for your Ruby changes (review 3398). I believe most of

Re: [gem5-dev] Memory consistency verification (and some bugs)

2016-04-12 Thread Andreas Hansson
Hi Marco, This sounds good. Regarding the external component, I certainly do not mind. I would suggest we prune the old and crufty McPat. That should free up some 45k LoC, and I there is not really any integration in place in any case. Do you also have any updates to the MemChecker? It is

[gem5-dev] Memory consistency verification (and some bugs)

2016-04-12 Thread Marco Elver
Hi all, I've been working on integrating McVerSi [1] (a framework for simulation-based memory consistency verification) into the latest Gem5 version; in doing so I have also discovered new bugs (see end of email). At [1] I provide a patch and instructions on how to use McVerSi with the current

Re: [gem5-dev] Review Request 3356: mem: Implement alternative flow control between MemObject ports

2016-04-12 Thread Joel Hestness
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3356/#review8192 --- Hi Matt. Finally getting back to this. Thanks for posting it, so we can

Re: [gem5-dev] Review Request 3446: hsail, gpu-compute: fixes to appease clang++

2016-04-12 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3446/#review8191 --- Ship it! Thanks! - Andreas Hansson On April 12, 2016, 5:20 p.m.,

Re: [gem5-dev] Review Request 3446: hsail, gpu-compute: fixes to appease clang++

2016-04-12 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3446/ --- (Updated April 12, 2016, 10:20 a.m.) Review request for Default. Repository:

[gem5-dev] Review Request 3446: hsail, gpu-compute: fixes to appease clang++

2016-04-12 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3446/ --- Review request for Default. Repository: gem5 Description --- Changeset

Re: [gem5-dev] Review Request 3231: dist: config file for distributed switch

2016-04-12 Thread Mohammad Alian
> On Feb. 6, 2016, 6:23 p.m., Tony Gutierrez wrote: > > Andreas, are you ok with this being shipped now? I am planning on shipping > > the switch, and it'd be nice if this can go in as well. > > Andreas Hansson wrote: > No objections. That said, I do think that a separate directory is more

[gem5-dev] changeset in gem5: misc: Appease clang...again

2016-04-12 Thread Andreas Hansson
changeset b31738224fb0 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=b31738224fb0 description: misc: Appease clang...again Once again, clang is having issues with recently committed code. Unfortunately HSAIL_X86 is still broken. diffstat:

[gem5-dev] HSAIL_X86 not compiling with clang

2016-04-12 Thread Andreas Hansson
Hi all, Is there someone at AMD that could have a look? It’s been broken for quite some time now. Thanks, Andreas IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender