Perform some cleanups on periodic_cpu_load:

  - fix the startup trace, 'Thread Group C' is incorrectly printed as
    'Thread Group A', PRIO_C as PRIO_B and PERIOD_C as PERIOD_B.

  - remove histogram references, those are never computed.

Signed-off-by: Sebastien Dugue <[EMAIL PROTECTED]>
Acked-by: Darren Hart <[EMAIL PROTECTED]>
Cc: Tim Chavez <[EMAIL PROTECTED]>
---
 .../func/periodic_cpu_load/periodic_cpu_load.c     |   10 +++-------
 1 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/testcases/realtime/func/periodic_cpu_load/periodic_cpu_load.c 
b/testcases/realtime/func/periodic_cpu_load/periodic_cpu_load.c
index 843dc3f..6a6cd54 100644
--- a/testcases/realtime/func/periodic_cpu_load/periodic_cpu_load.c
+++ b/testcases/realtime/func/periodic_cpu_load/periodic_cpu_load.c
@@ -63,8 +63,6 @@
 //#define ITERATIONS 1000 /* min iters for 3 nines */
 // FIXME: need some kind of passing criteria calculation
 //#define PASS_US 100
-// FIXME: consider removing the histograms entirely
-#define HIST_BUCKETS 100
 
 static int run_jvmsim = 0;
 static int ret = 0;
@@ -123,7 +121,6 @@ void *periodic_thread(void *thread)
        void*(*func)(void*) = parg->func;
 
        stats_container_t dat;
-       stats_container_t hist;
        stats_quantiles_t quantiles;
 
        int i = 0;
@@ -132,7 +129,6 @@ void *periodic_thread(void *thread)
        nsec_t exe_start, exe_end, exe_time;
 
        stats_container_init(&dat, ITERATIONS);
-       stats_container_init(&hist, HIST_BUCKETS);
        stats_quantiles_init(&quantiles, 3);
 
        next = rt_gettime();
@@ -188,10 +184,10 @@ int main(int argc, char *argv[])
        printf("  threads: %d\n", THREADS_PER_GROUP);
        printf("  priority: %d\n", PRIO_B);
        printf("  period: %d ms\n", PERIOD_B/NS_PER_MS);
-       printf("Thread Group A:\n");
+       printf("Thread Group C:\n");
        printf("  threads: %d\n", THREADS_PER_GROUP);
-       printf("  priority: %d\n", PRIO_B);
-       printf("  period: %d ms\n", PERIOD_B/NS_PER_MS);
+       printf("  priority: %d\n", PRIO_C);
+       printf("  period: %d ms\n", PERIOD_C/NS_PER_MS);
 
        if (run_jvmsim) {
                printf("jvmsim enabled\n");
-- 
1.5.4.rc2.84.gf85fd-dirty


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to