See /z/m5/regression/regress-2008-09-25-03:00:01 for details.
___
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev
Hi all,
So I am in process of getting the in-order model from mips to compile so
we can put it into the tree. I am having the following error:
Reading /Users/rickstrong/work/m5-dev/src/cpu/simple/SConsopts
scons: warning: The env.Copy() method is deprecated; use the env.Clone()
method
I'm not sure how it's supposed to work (perhaps only Korey can answer that),
but I think the basic problem here is that memAccFlags is only defined in
the Memory subclass of MipsStaticInst while what you are trying to
dereference is a pointer to the base StaticInst class.
Steve
On Fri, Sep 26,
changeset c9f52fae6b37 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=c9f52fae6b37
description:
O3CPU: Add a hack to ensure that nextPC is set correctly after syscalls.
Just check CPU's nextPC before and after syscall and if it changes,
update this
Yea, I think there might be a fix for thatin the patch that I sent
did it have any of the arch/mips/isa files?
I did change some stuff in there to get the memory system to function
correctly with M5, because in the Mixie model you need to be able to
have a memory operation happen in a
changeset cb98f0fcc6c6 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=cb98f0fcc6c6
description:
style: These files didn't even come close to following the M5 style
guide.
diffstat:
2 files changed, 27 insertions(+), 145 deletions(-)
src/arch/mips/dsp.cc | 32
changeset 660fa7b652f5 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=660fa7b652f5
description:
scons: disable several gcc warnings for swig autogenerated wrapper code.
diffstat:
1 file changed, 1 insertion(+), 1 deletion(-)
src/SConscript |2 +-
diffs (15 lines):
changeset c2db27fc4f27 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=c2db27fc4f27
description:
style: bring this file into M5 style, use the new pte translate
function.
diffstat:
1 file changed, 54 insertions(+), 10 deletions(-)
src/arch/sparc/vtophys.cc | 64
changeset 875cb7d09831 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=875cb7d09831
description:
When nesting if statements, use braces to avoid ambiguous else clauses.
diffstat:
0 files changed
diffs (40 lines):
diff -r eb5664be6075 -r 875cb7d09831
changeset 4c4b5dfc9944 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=4c4b5dfc9944
description:
style: missed space after switch
diffstat:
0 files changed
diffs (129 lines):
diff -r 875cb7d09831 -r 4c4b5dfc9944 src/arch/mips/dsp.cc
--- a/src/arch/mips/dsp.cc
Why the double negative (!yield_mask != 0)? Seems like this should either
be !yield_mask or yield_mask == 0, but not something in between...
Steve
On Fri, Sep 26, 2008 at 3:29 PM, Nathan Binkert [EMAIL PROTECTED] wrote:
changeset eb5664be6075 in /z/repo/m5
details:
Looking into inorder.diff, the are no changes to files in
src/arch/mips/isa. So if you send those changes, I would be appreciative.
Thanks,
-Rick
Korey Sewell wrote:
Yea, I think there might be a fix for thatin the patch that I sent
did it have any of the arch/mips/isa files?
I did
yeah, I just did this quickly and I guess I missed switch in my
regexp. I just committed fix.
I've been toying around with an emacs script to go through and fix up
some of the basics.
It might be worth spending some effort to get a beautifier working.
There's one that seems to still be getting
I don't disagree. Korey, can you comment? What is correct here?
Nate
2008/9/26 Steve Reinhardt [EMAIL PROTECTED]:
Why the double negative (!yield_mask != 0)? Seems like this should either
be !yield_mask or yield_mask == 0, but not something in between...
Steve
On Fri, Sep 26, 2008 at
I mentioned before that we had a couple of options when it came to
dealing with some of the new warnings that gcc 4.3 generates. Here's
some comments on each of the patches:
I'm leaning towards keeping the parenthesis warnings since the diff to
allow it isn't that big, and it has brought up a
# HG changeset patch
# User Nathan Binkert [EMAIL PROTECTED]
# Date 1222446909 25200
# Node ID 85776a2ac9fa86f739e93a514d9cf5c7a4de467c
# Parent 875cb7d0983158b67edeb7532aa8fa699d0f28a1
gcc: Add extra parens to quell warnings.
Even though we're not incorrect about operator precedence, let's add
# HG changeset patch
# User Nathan Binkert [EMAIL PROTECTED]
# Date 1222446909 25200
# Node ID 996a1882ccc64c4928f81fb3e318c9811bc13240
# Parent 85776a2ac9fa86f739e93a514d9cf5c7a4de467c
gcc: Version 4.3 adds some warnings that we're turning off.
We just can't deal with right now.
diff --git
# HG changeset patch
# User Nathan Binkert [EMAIL PROTECTED]
# Date 1222446909 25200
# Node ID 41322fd94d32ef64ed1b72f3db5ed7cd02bb4d04
# Parent 996a1882ccc64c4928f81fb3e318c9811bc13240
The way this code is written, it can allow an out of bounds array reference.
Since this code is fundamentally
Do you want to hold off on letting a stable out for this stuff? I'd
say that we need to get stable done to allow things into the tree, and
we can try to make this a part of the next stable release which we can
target as 2.0.
Nate
I think high on the list needs to be a clean-up of the
I could go either way. If it forced us to just get it done in 2 or 3
days then it would be worth it. Otherwise, we should push it off, but
not too far.
Ali
On Sep 26, 2008, at 11:51 PM, nathan binkert wrote:
Do you want to hold off on letting a stable out for this stuff? I'd
say that we
Maybe we should start by going through flyspray and determining what
issues still exist and which need to be closed. It would be nice to
get a 2.0 release done, but it seems hard to say that we could do that
in 2 or 3 days. I could put some serious effort into it after ISCA is
over and try to
21 matches
Mail list logo
