diff options
author | Keir Fraser <keir.fraser@citrix.com> | 2010-07-02 18:53:37 +0100 |
---|---|---|
committer | Keir Fraser <keir.fraser@citrix.com> | 2010-07-02 18:53:37 +0100 |
commit | d505477d647f79c99e93e61425e8310c54f800b2 (patch) | |
tree | 7f0373837ed418c06a93fd792082538c3aa226b2 /xen/common/trace.c | |
parent | 43b0124af6b67ca65a1b4e82cf6b22b6d56119f8 (diff) | |
download | xen-d505477d647f79c99e93e61425e8310c54f800b2.tar.gz xen-d505477d647f79c99e93e61425e8310c54f800b2.tar.bz2 xen-d505477d647f79c99e93e61425e8310c54f800b2.zip |
trace: adjust printk()s
They should be lower level or rate limited.
Signed-off-by: Jan Beulich <jbeulich@novell.com>
Acked-by: George Dunlap <george.dunlap@eu.citrix.com>
Diffstat (limited to 'xen/common/trace.c')
-rw-r--r-- | xen/common/trace.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/xen/common/trace.c b/xen/common/trace.c index 8d4f02a458..5f26dd0d15 100644 --- a/xen/common/trace.c +++ b/xen/common/trace.c @@ -138,7 +138,7 @@ static int alloc_trace_bufs(void) } t_info->tbuf_size = opt_tbuf_size; - printk("tbuf_size %d\n", t_info->tbuf_size); + printk(XENLOG_INFO "tbuf_size %d\n", t_info->tbuf_size); nr_pages = opt_tbuf_size; order = get_order_from_pages(nr_pages); @@ -195,7 +195,7 @@ static int alloc_trace_bufs(void) /* Write list first, then write per-cpu offset. */ wmb(); t_info->mfn_offset[cpu]=offset; - printk("p%d mfn %"PRIx32" offset %d\n", + printk(XENLOG_INFO "p%d mfn %"PRIx32" offset %d\n", cpu, mfn, offset); offset+=i; } @@ -503,12 +503,13 @@ static inline int __insert_record(struct t_buf *buf, /* Double-check once more that we have enough space. * Don't bugcheck here, in case the userland tool is doing * something stupid. */ - if ( calc_bytes_avail(buf) < rec_size ) + next = calc_bytes_avail(buf); + if ( next < rec_size ) { - printk("%s: %u bytes left (%u - ((%u - %u) %% %u) recsize %u.\n", - __func__, - calc_bytes_avail(buf), - data_size, buf->prod, buf->cons, data_size, rec_size); + if ( printk_ratelimit() ) + printk(XENLOG_WARNING + "%s: avail=%u (size=%08x prod=%08x cons=%08x) rec=%u\n", + __func__, next, data_size, buf->prod, buf->cons, rec_size); return 0; } rmb(); |