Re: [m5-dev] 20 hour long regression test

2011-02-05 Thread Gabe Black
That's actually what we're talking about. All the script bits are already there, I'd just be adding reference outputs. Gabe On 02/04/11 23:49, nathan binkert wrote: Can't he commit the regression without any reference stats? I thought it wouldn't run if there were no refs. Perhaps the script

[m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-02-05 Thread Cron Daemon
* build/X86_SE/tests/fast/quick/00.hello/x86/linux/o3-timing FAILED! * build/ALPHA_SE_MOESI_hammer/tests/fast/quick/60.rubytest/alpha/linux/rubytest-ruby-MOESI_hammer passed. * build/ALPHA_SE/tests/fast/quick/00.hello/alpha/linux/simple-atomic passed. *

Re: [m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-02-05 Thread Gabe Black
The X86 o3 regressions will fail until somebody blasts the build directory. Building O3 is now default, but it won't be picked up into existing build directories. Gabe On 02/05/11 00:10, Cron Daemon wrote: * build/X86_SE/tests/fast/quick/00.hello/x86/linux/o3-timing FAILED! *

Re: [m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-02-05 Thread nathan binkert
The X86 o3 regressions will fail until somebody blasts the build directory. Building O3 is now default, but it won't be picked up into existing build directories. Couldn't that somebody be you? :) Nate ___ m5-dev mailing list m5-dev@m5sim.org

Re: [m5-dev] Review Request: X86: ISA bug fixes identified when bringing up Barrelfish OS

2011-02-05 Thread nathan binkert
!entry-writable (inUser || cr0.wp) is used here and in the if below. This would be easier to read if that was in a temporary variable, maybe called badWrite. Nate would probably yell at me that it should be bad_write because it's local. He'd be right, but that would be inconsistent with

Re: [m5-dev] 20 hour long regression test

2011-02-05 Thread Steve Reinhardt
So what's the point in having it without reference outputs? Sure you could run it, but as a regression it wouldn't be very valuable since you wouldn't know if anything changed or not. I guess you could run it to see if it completed, but I'm not sure how much value is in that (or whether anyone

Re: [m5-dev] Review Request: X86: ISA bug fixes identified when bringing up Barrelfish OS

2011-02-05 Thread Steve Reinhardt
!entry-writable (inUser || cr0.wp) is used here and in the if below. This would be easier to read if that was in a temporary variable, maybe called badWrite. Nate would probably yell at me that it should be bad_write because it's local. He'd be right, but that would be inconsistent with

Re: [m5-dev] 20 hour long regression test

2011-02-05 Thread nathan binkert
All true. I guess I'm thinking of the regressions as an example of how to run a job, but I really shouldn't since that's not what they're for. Nate On Sat, Feb 5, 2011 at 10:00 AM, Steve Reinhardt ste...@gmail.com wrote: So what's the point in having it without reference outputs? Sure you

Re: [m5-dev] Review Request: X86: Don't read in dest regs if all bits are replaced.

2011-02-05 Thread Nathan Binkert
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/464/#review838 --- The summary line caught my eye and I had no idea what it really meant

Re: [m5-dev] 20 hour long regression test

2011-02-05 Thread Gabe Black
My point is that you -don't- have it without the reference outputs. The reference outputs are how you make it exist in this case. Gabe On 02/05/11 10:00, Steve Reinhardt wrote: So what's the point in having it without reference outputs? Sure you could run it, but as a regression it wouldn't

Re: [m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-02-05 Thread Gabe Black
On 02/05/11 07:23, nathan binkert wrote: The X86 o3 regressions will fail until somebody blasts the build directory. Building O3 is now default, but it won't be picked up into existing build directories. Couldn't that somebody be you? :) Nate

Re: [m5-dev] Review Request: X86: ISA bug fixes identified when bringing up Barrelfish OS

2011-02-05 Thread Gabe Black
On 02/05/11 10:05, Steve Reinhardt wrote: !entry-writable (inUser || cr0.wp) is used here and in the if below. This would be easier to read if that was in a temporary variable, maybe called badWrite. Nate would probably yell at me that it should be bad_write because it's local. He'd

[m5-dev] Review Request: Ruby: Change PerfectSwitch's wakeup function

2011-02-05 Thread Nilay Vaish
--- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/328/ --- Review request for Default. Summary --- Currently the wakeup function for the

Re: [m5-dev] Review Request: X86: ISA bug fixes identified when bringing up Barrelfish OS

2011-02-05 Thread Steve Reinhardt
On Sat, Feb 5, 2011 at 12:35 PM, Gabe Black gbl...@eecs.umich.edu wrote: On 02/05/11 10:05, Steve Reinhardt wrote: !entry-writable (inUser || cr0.wp) is used here and in the if below. This would be easier to read if that was in a temporary variable, maybe called badWrite. Nate would

Re: [m5-dev] 20 hour long regression test

2011-02-05 Thread Steve Reinhardt
I see... so all the bzip2 infrastructure is there as a combination of other x86 o3 regressions and other non-x86 bzip2 regressions, it's just populating that intersection with a reference output that enables it. OK, so I guess we're in a agreement that we shouldn't do that. Steve On Sat, Feb 5,

Re: [m5-dev] Cron m5test@zizzer /z/m5/regression/do-regression quick

2011-02-05 Thread nathan binkert
I don't know for sure where the right directory is, and I don't want to go deleting things willy nilly. zizzer:/z/m5/regression/zizzer/m5/build Tomorrow night (or maybe tonight?) it will run from scratch anyway. Nate ___ m5-dev mailing list