Hi Michael and Mohan, sorry I forget it. Both is fixed.
Regards Lars On Friday, 4. June 2010 23:33, Bishop, Michael (ISB Linux/Telco) wrote: > Lars - > > I downloaded your latest branch (7091) and ran testing (hpitest 3.1.1) on > the dynamic simulator. > > I'm getting 8 failures out of 711 test cases - using hpitest-3.1.1. > > A couple of comments: > > First - thanks for all the work. > > Second, in the openhpi.conf file - you still refer to the simulator as the > new_simulator in your comments for your stanza. Might want to change that. > > Secondly, it does not appear that the IDR area count issue has been fixed. > This is the issue that Mohan had described earlier. The area count and the > number of actual IDR areas should always match. > > If you start up your dynamic_simulator plugin, then run > /usr/bin/hpigensimdata and capture the output. > > The run the test: > > ./hpitest Inventory/saHpiIdrInfoGet/UpdateCounter > > Then run /usr/bin/hpigensimdata again, and compare the output to the first > run of hpigensimdata - you will see the discrepancy. In second output of > hpigensimdata, the area count is set at 2 - but the actual number of areas > present in the data file is only 1. > > /usr/bin/hpigensimdata > mdata1.txt > ./hpitest Inventory/saHpiIdrInfoGet/UpdateCounter > /usr/bin/hpigensimdata > mdata2.txt > diff mdata1.txt mdata2.txt > 1715c1715 > < UpdateCount=0 > --- > > > UpdateCount=2 > > 1717c1717 > < NumAreas=1 > --- > > > NumAreas=2 > > --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: Lars Wetzel [mailto:[email protected]] > > Sent: Thursday, June 03, 2010 2:52 PM > > To: [email protected] > > Cc: Bishop, Michael (ISB Linux/Telco); Devarajulu, Mohanasundaram; > > Kleber, Ulrich (NSN - DE/Munich) > > Subject: Re: [Openhpi-devel] New simulator testing > > > > > > Hi Michael, Mohan and Uli, > > > > ok following changes were made: > > > > 1. and 3. debug and logfile > > The logfile generation is now deactivted by default. > > > > 4. ep of announcements > > Fix of simulation.data.example > > > > 6. naming > > Change the naming to dynamic_simulator (library libdyn_simulator) > > It works on my machine, but due to some problems with svn (I > > had to commit in > > 3 steps) I hope it works also with the data inside the repository. > > Furthermore it is and was the most critical part of the changes. > > > > 2. new <-> old > > it's still your part ;-) > > > > 5. timestamp of announcement > > fault or not fault, it has to be discussed if it is really > > necessary to work > > with original timestamp in the announcements. > > > > soa_plugin: > > Testcases > > Sensor/saHpiSensorReadingGet/GetAndVerify > > Sensor/saHpiSensorThresholdsGet/GetThresholds > > Will now pass. > > The reason was a inconsistence of the data. I enhanced the > > hpigensimdata > > client to overcome it. (DataFormat.IsSupported was set on > > SAHPI_TRUE but > > Reading.IsSupported returns SAHPI_FALSE) > > I don't understand how the test cases can pass in your > > system. Maybe it was a > > temporarily bug? But nevertheless the client will throw a > > warning if it > > happens again. > > > > Sensor/saHpiSensorEventMasksSet/SensorEnableChangeEvent > > Wasn't supported by old and new simulator. > > Now it is supported in the new simulator and had a small bug. - Fixed > > > > Inventory/saHpiIdrAreaHeaderGet/TraversalByAreaType > > Was a bug -> fixed > > > > Inventory/saHpiIdrFieldSet/SetField > > {COOLING_DEVICE,x} and FANs > > Fails due to wrong data -> Field has wrong Type entry > > Can you please verify it with original hardware > > > > Inventory/saHpiIdrAreaAddById/AddAreaById > > SAHPI_FIRST_ENTRY is a valid AreaId and can be used with this > > function, > > provided the IDR does not have a pre-existing Area with that AreaId. > > In the data the areas starts with the Id 0 == > > SAHPI_FIRST_ENTRY. So I think > > the issue is more a problem of the test case?! > > > > HotSwap/saHpiAutoExtractTimeoutGet/GetTimeout > > Was a Typo, is now fixed > > > > atca_data: > > Sorry is still open: > > POWER_CYCLE -> Event generation is missing > > Textbuffer doesn't support binary data > > > > Hope I didn't bring in some new faults. > > > > Best Regards > > Lars > > --------------------------------------------------------------------------- >--- ThinkGeek and WIRED's GeekDad team up for the Ultimate > GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the > lucky parental unit. See the prize list and enter to win: > http://p.sf.net/sfu/thinkgeek-promo > _______________________________________________ > Openhpi-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/openhpi-devel -- ------------------------------- Dipl. Wi.ing. Lars Wetzel Uttinger Str. 13 86938 Schondorf a. Ammersee Tel.: 0179-2096845 Mail: [email protected] USt-IdNr.: DE181396006 ------------------------------------------------------------------------------ ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo _______________________________________________ Openhpi-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openhpi-devel
