All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olaf Hering <olaf@aepfle.de>
To: Keir Fraser <keir.xen@gmail.com>
Cc: xen-devel@lists.xensource.com, Jan Beulich <JBeulich@novell.com>
Subject: Re: [PATCH] xen: increase static dmesg buffer to 64K
Date: Mon, 18 Jul 2011 10:45:12 +0200	[thread overview]
Message-ID: <20110718084512.GA23061@aepfle.de> (raw)
In-Reply-To: <CA49AE62.1E119%keir.xen@gmail.com>

On Mon, Jul 18, Keir Fraser wrote:

> On 18/07/2011 09:16, "Olaf Hering" <olaf@aepfle.de> wrote:
> > 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.

Yes, I was booting with conring_size=, but the bulk of messages were
already printed at that point. My 80 cpu testbox had 24K in dmesg buffer
with my patch.

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

I will put that on my TODO list, and look at your and Jans suggestion to
move the dmesg buffer allocation earlier.

Olaf

      reply	other threads:[~2011-07-18  8:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-17 15:43 [PATCH] xen: increase static dmesg buffer to 64K Olaf Hering
2011-07-18  7:30 ` Jan Beulich
2011-07-18  8:16   ` Olaf Hering
2011-07-18  8:31     ` Jan Beulich
2011-07-18  8:33     ` Keir Fraser
2011-07-18  8:45       ` Olaf Hering [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110718084512.GA23061@aepfle.de \
    --to=olaf@aepfle.de \
    --cc=JBeulich@novell.com \
    --cc=keir.xen@gmail.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.