WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

[Xen-devel] Re: [PATCH - resend] addition of loglevel for printf in Xen

To: Steven Rostedt <srostedt@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH - resend] addition of loglevel for printf in Xen HV
From: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Date: Thu, 26 Oct 2006 15:19:22 +0100
Cc: chrisw@xxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 26 Oct 2006 07:31:02 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <4540B873.4010906@xxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acb5Cbtb+gMeWmT8EduoQgANk04WTA==
Thread-topic: [PATCH - resend] addition of loglevel for printf in Xen HV
User-agent: Microsoft-Entourage/11.2.5.060620
On 26/10/06 2:30 pm, "Steven Rostedt" <srostedt@xxxxxxxxxx> wrote:

> So anything in between the upper and lower thresholds inclusive would be
> rate limited.  Also I'm not sure of the best way dom0 can communicate
> threshold changes to the hypervisor.  As well as if there should be a
> config to set the defaults.

I think yet-another-little-tool in dom0 is the right way to go. We can add a
sysctl to get/set the parameters. It can pretty-print the current parameters
and also provide a way of changing them. I don't think there's any benefit
to integration with xm -- it's not like this needs to be integrated with
xend or anything like that.

Yes, we should also have a way to set them as a boot parameter. I suggest we
should only enact those explicit parameters when dom0 starts to boot though
(no need to stop Xen being noisy in initial boot). We could have an extra
parameter (e.g., 'quiet') to override that if someone really cares.

It'd be nice to have debug-key support for changing it too, but actually
that's probably not so important if we have the dom0 tool.

Everything you said I agree with, so please do go hack it up. :-)

Another thought... By pushing the rate-limiting into printk() itself, we
lose the ability to print all-or-nothing for related blocks of printk (e.g.,
register dumps). We might want a 'same as previous line' specifier, which
would print/not-print same as previous output line?

 -- Keir

> So what are your thoughts?



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel