Nilay,

Please be more professional when responding to questions.  If you have a 
particular idea on how to fix the problem, we would appreciate if you described 
it rather than leaving us a trail of breadcrumbs.

Please note that your patch, 10311, is the one that breaks the tenet that 
ClockedObjects, which include ports, are MemObjects.  It is unfortunate that 
our regression tester does not test config.ini loading with ruby and did not 
realize that the SimpleNetwork is different than all other ClockedObjects.  
However, I do not believe revision 10458 is at fault here.  Rather it is 
SimpleNetwork's unconventional use of ports.

We appreciate your help on this matter.  Important work is currently blocked on 
this bug.

Thank you,

Brad




-----Original Message-----
From: gem5-dev [mailto:gem5-dev-boun...@gem5.org] On Behalf Of Nilay Vaish
Sent: Tuesday, June 23, 2015 3:10 PM
To: gem5 Developer List
Subject: Re: [gem5-dev] Problem Loading SimpleNetwork from INI Config File

On Tue, 23 Jun 2015, Gross, Joe wrote:

> Hi Nilay,
>
> I've looked through these but don't see any obvious special behaviors 
> that would apply to the SimpleNetwork deserialization. However, 
> neither of these have changed significantly in quite a while, so I 
> think whatever broke was in Network or SimpleNetwork.


Joe, src/sim/cxx_manager.cc did not used to exist until revision 10458. 
So, you are completely incorrect on whatever you think is broken.  Compare the 
functions src/python/swig/pyobject.cc:connectPorts() and 
CxxConfigManager::bindPort().

>
> Since you're the author of a variety of changes to Network and 
> SimpleNetwork between when this was working and now, I thought you would 
> know fairly quickly what changed and why SimpleNetwork no longer works 
> in this framework. If you can offer something specific (or better yet, a 
> patch), that would help resolve this bug. Thanks!
>

I would really prefer that you learn how to figure out what a particular 
piece of code might be doing.  Earlier I told you what files to compare. 
I have now told you what functions to compare.  If you are still not able 
to figure out, I'll suggest another step towards the solution that I have 
in mind.


--
Nilay
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to