---
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)
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
---
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.,
---
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)
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
---
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
> 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
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3806/
---
Review request for Default.
Repository: gem5
Description
---
Changeset
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
---
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.,
---
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.,
---
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.,
---
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
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
---
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
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
---
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
---
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.,
---
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
---
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3802/
---
Review request for Default.
Repository: gem5
Description
---
Changeset
> 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
---
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.,
---
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)
---
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
---
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
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3801/#review9385
---
src/mem/comm_monitor.cc (line 276)
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3801/
---
Review request for Default.
Repository: gem5
Description
---
Changeset
27 matches
Mail list logo