On Tue, 2009-07-07 at 12:11 +0530, Sripathi Kodi wrote: > On Monday 06 July 2009 21:20:49 Subrata Modak wrote: > > On Fri, 2009-07-03 at 12:35 +0530, Sripathi Kodi wrote: > > > Hi, > > > > > > The pass criterion in pi_perf test case is wrong. It compares the minimum > > > amount of time taken by the low priority thread with the maximum amount > > > of time taken by the high priority thread to calculate the PI delay. > > > Obviously, these min and max don't necessarily happen in the same > > > iteration, resulting in a number of false failures. The correct way is to > > > compare the time taken by low and high priority threads in each iteration > > > and then find the maximum delay experienced by high priority thread > > > across the iterations. This patch implements the change. Additionally, > > > this patch removes lock_wait_dat array, which is not needed anymore as > > > well as makes a couple of messages easier to understand. > > > > > > This patch changes the messages displayed by this test case slightly. > > > They look like the following now: > > > > > > Low prio thread started > > > High prio thread started > > > Busy 0 started > > > Busy 1 started > > > Busy 2 started > > > Busy 3 started > > > Time taken for high prio thread to get the lock once released by low prio > > > thread > > > Min delay = 10 us > > > Max delay = 36 us > > > Average delay = 17.06 us > > > Standard Deviation = 8.11 us > > > Quantiles: > > > 99.0% < 36 > > > Criteria: High prio lock wait time < (Low prio lock held time + 200 us) > > > Result: PASS > > > > > > I have tested this by running 1000 iterations of pi_perf test on a couple > > > of machines. > > > > > > Thanks, > > > Sripathi. > > > > > > --- > > > > > > realtime: Fix the pass criterion of pi_perf test case. > > > > > > Signed-off-by: Sripathi Kodi <[email protected]> > > > > Hi Sri, > > > > Is this based on the latest snapshot. I get the following failures: > > > > patching file testcases/realtime/func/pi_perf/pi_perf.c > > Hunk #1 FAILED at 1. > > > > > > Can you please resend ? > > Sorry Subrata, the patch seems to be fine on my machine, but is going bad in > my > mail client. If you don't mind, I have attached the patch to this mail for > now. > > Thanks, > Sripathi.
Thanks. Regards-- Subrata > > -- > > realtime: Fix the pass criterion of pi_perf test case. > > Signed-off-by: Sripathi Kodi <[email protected]> > Acked-by: Will Schmidt <[email protected]> > Acked-by: Gowrishankar <[email protected]> > > <snip> ------------------------------------------------------------------------------ Enter the BlackBerry Developer Challenge This is your chance to win up to $100,000 in prizes! For a limited time, vendors submitting new applications to BlackBerry App World(TM) will have the opportunity to enter the BlackBerry Developer Challenge. See full prize details at: http://p.sf.net/sfu/blackberry _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
