Re: [gem5-dev] Review Request 3800: x86: fix Mul1u instruction

2017-02-03 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3800/ --- (Updated Feb. 3, 2017, 4:13 p.m.) Review request for Default. Summary (updated)

Re: [gem5-dev] Review Request 2557: x86: kvm: Fix the KVM CPU in SE and FS on Intel CPUs.

2017-02-03 Thread Jason Lowe-Power
Oops, those patches don't apply cleanly to the mainline. Here's the updated patches. I tested these three patches on Intel (3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux) and AMD (4.0.0-040050-generic #201505271752 SMP Wed May 27 17:53:58 IDT 2015

Re: [gem5-dev] Review Request 3803: sim: Patch to fix the statfs build

2017-02-03 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3803/#review9394 --- Ship it! Ship It! - Tony Gutierrez On Feb. 3, 2017, 8:25 a.m.,

Re: [gem5-dev] Review Request 3800: x86: fix Mul1u and Mul1s instructions

2017-02-03 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3800/ --- (Updated Feb. 3, 2017, 11:29 a.m.) Review request for Default. Summary (updated)

Re: [gem5-dev] Review Request 2557: x86: kvm: Fix the KVM CPU in SE and FS on Intel CPUs.

2017-02-03 Thread Jason Lowe-Power
Here's the three patches I've been using. I've done most of my testing on an Intel platform, BTW. I have an AMD platform, though it's been a long time since I've tested gem5-KVM on it. If I get some extra time today, I'll run a quick test. Cheers, Jason On Fri, Feb 3, 2017 at 12:10 PM Gabe Black

Re: [gem5-dev] Review Request 3806: sparc: fix bugs caused by cd7f3a1dbf55

2017-02-03 Thread Brandon Potter
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3806/ --- (Updated Feb. 3, 2017, 6:48 p.m.) Review request for Default, Ali Saidi, Gabe

Re: [gem5-dev] Review Request 3804: sim: fix build breakage in process.cc after brandon@11801

2017-02-03 Thread Brandon Potter
> On Feb. 3, 2017, 5:31 p.m., Brandon Potter wrote: > > Ship It! I just posted another patch related to the cd7f3a1dbf55 (brandon@11801) changeset that is needed to get the regression tester working (http://reviews.gem5.org/r/3806/). This patch, 3804, is a preliminary patch that is needed as

[gem5-dev] Review Request 3806: sparc: fix bugs caused by cd7f3a1dbf55

2017-02-03 Thread Brandon Potter
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3806/ --- Review request for Default. Repository: gem5 Description --- Changeset

Re: [gem5-dev] Review Request 2557: x86: kvm: Fix the KVM CPU in SE and FS on Intel CPUs.

2017-02-03 Thread Gabe Black
ping... On Thu, Feb 2, 2017 at 12:11 PM, Gabe Black wrote: > Any chance somebody can send me a patch? > > Gabe > > > On Tue, Jan 31, 2017 at 2:34 PM, Gabe Black wrote: > >> I have no plans to attend HPCA at the moment. >> >> There are folks here who

Re: [gem5-dev] Review Request 3804: sim: fix build breakage in process.cc after brandon@11801

2017-02-03 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3804/#review9392 --- Ship it! Ship It! - Tony Gutierrez On Feb. 3, 2017, 8:29 a.m.,

Re: [gem5-dev] Review Request 3804: sim: fix build breakage in process.cc after brandon@11801

2017-02-03 Thread Brandon Potter
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3804/#review9391 --- Ship it! Ship It! - Brandon Potter On Feb. 3, 2017, 4:29 p.m.,

Re: [gem5-dev] Review Request 3803: sim: Patch to fix the statfs build

2017-02-03 Thread Brandon Potter
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3803/#review9390 --- Ship it! Ship It! - Brandon Potter On Feb. 3, 2017, 4:25 p.m.,

[gem5-dev] Review Request 3805: mem: fix fast build breakage from CommMonitor refactoring

2017-02-03 Thread Curtis Dunham
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3805/ --- Review request for Default. Repository: gem5 Description --- mem: fix fast

[gem5-dev] Can someone please commit these three?

2017-02-03 Thread Bjoern A. Zeeb
Hi, getting these tiny things in and out of a tree is really helpful; could someone please commit them? http://reviews.gem5.org/r/3667/ (ARM, 3 months Ship it!) http://reviews.gem5.org/r/3791/ (e1000) http://reviews.gem5.org/r/3378/ (seems everyone is fine with it now) Thanks, Bjoern

[gem5-dev] Review Request 3804: sim: fix build breakage in process.cc after brandon@11801

2017-02-03 Thread Bjoern A. Zeeb
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3804/ --- Review request for Default. Repository: gem5 Description --- Seeing build

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-02-03 Thread Bjoern A. Zeeb
On 2 Feb 2017, at 16:31, Bjoern A. Zeeb wrote: Hi, OK, I updated the diff. Can everyone please check if the diff from http://reviews.gem5.org/r/3803/ works for you now? /bz ___ gem5-dev mailing list gem5-dev@gem5.org

[gem5-dev] Review Request 3803: sim: Patch to fix the statfs build

2017-02-03 Thread Bjoern A. Zeeb
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3803/ --- Review request for Default. Repository: gem5 Description --- See developers

Re: [gem5-dev] Review Request 3378: scons: make build better on FreeBSD

2017-02-03 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3378/#review9389 --- Ship it! Ship It! - Andreas Hansson On Feb. 3, 2017, 4:20 p.m.,

Re: [gem5-dev] Review Request 3378: scons: make build better on FreeBSD

2017-02-03 Thread Bjoern A. Zeeb
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3378/ --- (Updated Feb. 3, 2017, 4:20 p.m.) Review request for Default. Changes ---

[gem5-dev] Review Request 3802: mem, stats: fix typos in CommMonitor and Stats

2017-02-03 Thread Pierre-Yves Péneau
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3802/ --- Review request for Default. Repository: gem5 Description --- Changeset

Re: [gem5-dev] Review Request 3801: mem, misc: fix building issue with CommMonitor (unused variables)

2017-02-03 Thread Pierre-Yves Péneau
> On Feb. 3, 2017, 9:58 a.m., Andreas Hansson wrote: > > Thanks for fixing. Could you make it "mem:" and also include CommMonitor in > > the description. I also expect this is due to unused variables in fast > > builds? Not undefined variables. Yes, this is unused variables, thank you for

Re: [gem5-dev] Review Request 3801: mem, misc: fix building issue with CommMonitor (unused variables)

2017-02-03 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/#review9387 --- Ship it! Ship It! - Andreas Hansson On Feb. 3, 2017, 9:04 a.m.,

Re: [gem5-dev] Review Request 3801: mem, misc: fix building issue with CommMonitor (unused variables)

2017-02-03 Thread Pierre-Yves Péneau
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/ --- (Updated Feb. 3, 2017, 10:04 a.m.) Review request for Default. Summary (updated)

Re: [gem5-dev] Review Request 3801: misc: fix building issue (undefined variables)

2017-02-03 Thread Pierre-Yves Péneau
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/ --- (Updated Feb. 3, 2017, 10:02 a.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 3801: misc: fix building issue (undefined variables)

2017-02-03 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/#review9386 --- Thanks for fixing. Could you make it "mem:" and also include CommMonitor

Re: [gem5-dev] Review Request 3801: misc: fix building issue (undefined variables)

2017-02-03 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/#review9385 --- src/mem/comm_monitor.cc (line 276)

[gem5-dev] Review Request 3801: misc: fix building issue (undefined variables)

2017-02-03 Thread Pierre-Yves Péneau
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3801/ --- Review request for Default. Repository: gem5 Description --- Changeset