Re: [m5-dev] [PATCH] imported patch inorder-alpha-port

2009-02-22 Thread Korey Sewell
I'm a little unsure of what you are saying here Nate. I am trying to add a mt.hh for ALPHA (so it can use the InOrder model) but through some fault of mines the alpha version of the mt.hh file got omitted. Eventually, any ISA wanting to use InOrder would need to at the very least have some bare

Re: [m5-dev] [PATCH] imported patch inorder-alpha-port

2009-02-21 Thread nathan binkert
1) not sure why the mt.hh file didnt get added to the patch, but it should be there I think the real problem is that you're adding an mt.hh, but only for mips. That doesn't work so well with the switching directory code. ___ m5-dev mailing list

Re: [m5-dev] [PATCH] imported patch inorder-alpha-port

2009-02-21 Thread Korey Sewell
oooph...32/4 =8...8 bits in a byte...I plead the 5th! Originally, that code came from trying to align with some of the SPARC generic constants (SingleWidth) and I got a little too cute trying to define a SingleBytes. And since no ALPHA or MIPS code depends on SingleBytes that's probably why that

[m5-dev] [PATCH] imported patch inorder-alpha-port

2009-02-20 Thread Korey Sewell
# HG changeset patch # User Korey Sewell ksew...@umich.edu # Date 1235138771 18000 # Node ID b6e4240c46e429bf99fefd0d61fef1465de86e49 # Parent 7a74edaa8741dd7fb541ef6d404dac3a9ebc86f9 imported patch inorder-alpha-port diff -r 7a74edaa8741 -r b6e4240c46e4 src/arch/SConscript ---

Re: [m5-dev] [PATCH] imported patch inorder-alpha-port

2009-02-20 Thread Korey Sewell
Before this ALPHA port for the InOrder model can make it to a changeset, there are few issues that need to be resolved in this patch: - Adding member get/setReg functions to FloatRegFile: The inorder model would like to instantiate multiple int/float register files and just one miscellaneous

Re: [m5-dev] [PATCH] imported patch inorder-alpha-port

2009-02-20 Thread Gabriel Michael Black
I haven't looked at this incredibly closely yet, but I have a few questions. First, what's this mt.hh file? You made it an ISA switched header, but I don't see anything by that name being added. What does it do? Second, you shouldn't add a comment out include of cpu/inorder/cpu.hh. Third,