On Thu, Sep 16, 2010 at 4:12 PM, Joel Hestness <hestn...@cs.utexas.edu> wrote: > Hi guys, > I am using swig 1.3.40, so per Nate's note in the last changeset, I don't > think that should be an issue. > @Steve: Thanks for the pointer to 'bisect'. Also, in debugging this > problem, I ran into the same MC146818 checkpointing/drain bug from before. > Previously, I had written a patch to fix the problem. However, I just > talked to Brad about it, and he mentioned that you were thinking about > backing out that changeset (7559). Based on what I see in the code, it's > unclear whether my patch does the right thing and I think the previous code > is certainly more correct. I backed out that changeset locally, and it > fixed the MC146818 problem, so my recommendation would be to back out that > changeset in the repo. If we're in agreement, should I submit that patch > for review?
No, I don't think there's a need for review; I'll just commit it as I told Brad I would do a while ago. Thanks for the reminder. Sorry for not doing it sooner and causing you to have to spend time on it. Steve _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
