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

Re: [Xen-devel] [PATCH] xen: increase static dmesg buffer to 64K

To: Olaf Hering <olaf@xxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] xen: increase static dmesg buffer to 64K
From: Keir Fraser <keir.xen@xxxxxxxxx>
Date: Mon, 18 Jul 2011 09:33:22 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 18 Jul 2011 01:34:41 -0700
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=user-agent:date:subject:from:to:cc:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; bh=AncbnkP5sfhCp2Sc+ErXuwjF4ekvAxp+dk2pymXKElY=; b=jBEVyzHmmBLM0vD2+4ZQE45AZzVT/YoXmnQ5mvxUrslgTAQOLlUUWYFzD/5h2jA4gj Pxd5QnIEeLjHYZD36YQR9ZwqiqbBDUxAqNIRafROjZatYWEK0Cpv0xFflXeSX5Osctud +j5tWN5Z04zPeW1ZGH94H3S3srDPJTz8UAs4k=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20110718081628.GA22448@xxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcxFJVprNS9/82NFnEep6COJC8R40g==
Thread-topic: [Xen-devel] [PATCH] xen: increase static dmesg buffer to 64K
User-agent: Microsoft-Entourage/12.30.0.110427
On 18/07/2011 09:16, "Olaf Hering" <olaf@xxxxxxxxx> wrote:

> On Mon, Jul 18, Jan Beulich wrote:
> 
>>>>> On 17.07.11 at 17:43, Olaf Hering <olaf@xxxxxxxxx> wrote:
>>> # HG changeset patch
>>> # User Olaf Hering <olaf@xxxxxxxxx>
>>> # Date 1310917380 -7200
>>> # Node ID c6cade90d47f32e19f529930ba9f9acfa69f065f
>>> # Parent  31dd84463eece20bd01c7aee22b52a0c06c67545
>>> xen: increase static dmesg buffer to 64K
>>> 
>>> On large systems the static dmesg buffer will overflow the 16K buffer, early
>>> messages are lost. Increase the size to 64K to capture all lines on systems
>>> without serial console.
>> 
>> Please don't - on small systems it's a waste, and on even larger
>> systems it still won't help. If anything, the dynamic allocation may
>> need to happen earlier. As you probably saw, console_init_postirq()
>> already sizes the buffer dependent on the number of CPUs in the
>> system.
> 
> I think conring_size= should be evaluated very early. Is there a way to
> allocate a range of mfns very early which can then be used for the dmesg
> buffer?

Are you actually losing messages because you can't allocate the larger
console ring early enough? It seems a bit unlikely, but possible I suppose.
Anyhow, the normal memory allocator is available plenty early enough. You'd
just need to do the allocation earlier than console_init_postirq(). Sometime
soon after the call to end_boot_allocator() would work nicely.

 -- Keir

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



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