Hi, here is my two cents on some of these issues: 1. I agree that default for debug should be off
2. I think the oldsimulator should stay available. The new simulator needs some configuration to do, which could be complex for a newcomer. But I think the newsimulator could have a fancy name, which expresses the major benefits it provides, e.g. dynamic-simulator or configurable-simulator. Maybe we should rename the old simulator to simple-simulator? 3. I would prefer the possibility to have separate logfiles if you activate two plugins. Cheers, Uli > -----Original Message----- > From: ext Bishop, Michael (ISB Linux/Telco) > [mailto:[email protected]] > Sent: Wednesday, May 26, 2010 5:55 PM > To: '[email protected]'; Lars Wetzel > Cc: Devarajulu, Mohanasundaram > Subject: Re: [Openhpi-devel] New simulator testing > > Lars - > > I agree with Mohan that debug messages should not be turned > on by default. > > I'm not sure the old simulator should be removed from > openhpi.conf. Perhaps it could be commented out - and the > new simulator used as the default plug-in. Does anyone else > have an opinion about this? If the new simulator is going to > be the default plug-in - it needs to be rock solid. > > Please investigate Mohan's comment about the log file (#3 > below). We do not want the new simulator to be confusing to > OpenHPI users, especially if it becomes the default plug-in. > > Also - please investigate 4 and 5 and let us know of any > changes you decide to make. > > Thanks! > > --michael > > Regards, > Michael Bishop > Enterprise Servers and Storage (ISB Linux/Telco) > Hewlett-Packard Company > 3404 E. Harmony Rd. Bldg. 5L, Post B7, Mailstop 42 > Fort Collins, CO 80528-9599 > Phone: 970-898-4393 > E-Mail: [email protected] > > > > -----Original Message----- > > From: mo...@fc [mailto:[email protected]] > > Sent: Tuesday, May 25, 2010 5:06 PM > > To: Lars Wetzel > > Cc: [email protected] > > Subject: [Openhpi-devel] New simulator testing > > > > > > Hi Lars, > > > > New simulator is having better results as for as hpib-test is > > concerned. > > Here are some of my observations with the testing. > > > > 1. Looks like the debug option is turned on by default in the new > > simulator plugin. This could be off. > > > > 2. Once the new_simulator code is merged the old simulator > information > > could be removed from the openhpi.conf > > > > 3. I do not understand the need for the log file in the > openhpi.conf. > > More over even if I define two logfiles for 2 new_simulator > > entries, it > > takes only one. > > > > 4. The default simulation.data needs to have complete > entity path for > > every entity. But there are few that do not have the base > > path now. They > > are mis-interpreted now, based on the path defined in the > > openhpi.conf. > > They are {SYSTEM_BOARD,1}, {DISK_DRIVE,2} in the > > ANNUNCIATOR/RDR_DETAIL. > > They get converted to {SYSTEM_BOARD,X}, {DISK_DRIVE,X} (X is 9 in my > > test case) as {SYSTEM_CHASSIS,1} is prefixed to these. > These could be > > seen in the output of hpigensimdata > > > > 5. timestamps are changed in the hpigensimdata output too. > > > > I will be doing more testing. Will report the results to > you as I find > > out more. > > > > Thanks > > Mohan > > > > -- > > mo...@fc <[email protected]> > > > > > > -------------------------------------------------------------- > > ---------------- > > > > _______________________________________________ > > Openhpi-devel mailing list > > [email protected] > > https://lists.sourceforge.net/lists/listinfo/openhpi-devel > > > > -------------------------------------------------------------- > ---------------- > > _______________________________________________ > Openhpi-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/openhpi-devel > ------------------------------------------------------------------------------ _______________________________________________ Openhpi-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openhpi-devel
