[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH 5 of 6] xen: sched_sedf: beautify statisics in SEDF
On 23/10/12 17:33, Dario Faggioli wrote:
#ifdef SEDF_STATS
- if ( EDOM_INFO(d)->block_time_tot != 0 )
- printk(" pen=%"PRIu64"%%", (EDOM_INFO(d)->penalty_time_tot * 100) /
- EDOM_INFO(d)->block_time_tot);
- if ( EDOM_INFO(d)->block_tot != 0 )
+ if ( EDOM_INFO(d)->stats.block_time_tot != 0 )
+ printk(" pen=%"PRIu64"%%", (EDOM_INFO(d)->stats.penalty_time_tot *
100) /
+ EDOM_INFO(d)->stats.block_time_tot);
+ if ( EDOM_INFO(d)->stats.block_tot != 0 )
printk("\n blks=%u sh=%u (%u%%) (shc=%u (%u%%) shex=%i "\
"shexsl=%i) l=%u (%u%%) avg: b=%"PRIu64" p=%"PRIu64"",
- EDOM_INFO(d)->block_tot, EDOM_INFO(d)->short_block_tot,
- (EDOM_INFO(d)->short_block_tot * 100)
- / EDOM_INFO(d)->block_tot, EDOM_INFO(d)->short_cont,
- (EDOM_INFO(d)->short_cont * 100) / EDOM_INFO(d)->block_tot,
- EDOM_INFO(d)->pen_extra_blocks,
- EDOM_INFO(d)->pen_extra_slices,
- EDOM_INFO(d)->long_block_tot,
- (EDOM_INFO(d)->long_block_tot * 100) / EDOM_INFO(d)->block_tot,
- (EDOM_INFO(d)->block_time_tot) / EDOM_INFO(d)->block_tot,
- (EDOM_INFO(d)->penalty_time_tot) / EDOM_INFO(d)->block_tot);
+ EDOM_INFO(d)->stats.block_tot,
EDOM_INFO(d)->stats.short_block_tot,
+ (EDOM_INFO(d)->stats.short_block_tot * 100)
+ / EDOM_INFO(d)->stats.block_tot, EDOM_INFO(d)->stats.short_cont,
+ (EDOM_INFO(d)->stats.short_cont * 100) /
EDOM_INFO(d)->stats.block_tot,
+ EDOM_INFO(d)->stats.pen_extra_blocks,
+ EDOM_INFO(d)->stats.pen_extra_slices,
+ EDOM_INFO(d)->stats.long_block_tot,
+ (EDOM_INFO(d)->stats.long_block_tot * 100) /
EDOM_INFO(d)->stats.block_tot,
+ (EDOM_INFO(d)->stats.block_time_tot) /
EDOM_INFO(d)->stats.block_tot,
+ (EDOM_INFO(d)->stats.penalty_time_tot) /
EDOM_INFO(d)->stats.block_tot);
...wouldn't it be even more beautiful to have a macro for reading stats
as well?
Like I said, it's fine as it is, but since you're looking for beauty, I
figured I'd point it out. :-)
I see what you mean. Again, as you wish.
This code will need some (and quite a bit actually) of attention as soon
as I or someone else get the time to work on it. If you're fine about
taking this as is, I'll make a note to self about the macro (as I agree
it would be nice).
OK -- in that case:
Acked-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
