On Thursday 10 January 2008 03:12:33 Sebastien Dugue wrote:
>   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: 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");



-- 
Darren Hart
IBM Linux Technology Center
Real-Time Linux Team

-------------------------------------------------------------------------
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