All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Jan Beulich <JBeulich@novell.com>
Cc: xen-devel@lists.xensource.com, keir.fraser@eu.citrix.com
Subject: Re: [PATCH Xen-unstable] Balloon down memory to achive enough DMA32 memory for PV guests with PCI pass-through to succesfully launch.
Date: Mon, 16 Nov 2009 11:38:27 -0500	[thread overview]
Message-ID: <20091116163827.GB20333@phenom.dumpdata.com> (raw)
In-Reply-To: <4B017E4E020000780001FF0E@vpn.id2.novell.com>

> No, you probably misunderstood (and I probably implied to much in my
> response): On a system with more than 168G, just ballooning out
> arbitrary memory from Dom0 in order to start a 32-bit pv DomU won't
> guarantee that the domain can actually start, as memory beyond the
> 128G boundary is unusable there for such guests.

I see what you mean.
> 
> Conceptually, ballooning out arbitrary amounts of memory to find a
> certain amount below 4G is identical to ballooning out more than the
> amount a guest needs in order to find as much as it needs below 128G.

Perhaps an extra mechanism in the balloon driver in the Dom0 kernel
to only give back to Xen pages below the 4GB mark? And if it
fails return a failure back to Xend?

That would solve the problem of ballooning out more than needed and
would eliminate the "oh-lets balloon out 2 more MBs and see if that
did it".

.. snip ..
> >Since the SWIOTLB size is determined by the 'swiotlb' argument passed to
> >the guest, what if we scanned for that and if it has a number, calculate
> >how much memory that amounts to and use that value? The default still being at 64MB.
> 
> That might be an option, but is very Linux-centric. I think the amount should
> be configurable per guest if something like this is being done at all.

How about both? Have a 'pci_mem' argument to set the default and also
check the swiotlb argument in 'extra.' If there is a 'swiotlb' argument in
'extra' use that value (computed for kilobytes of course).

  reply	other threads:[~2009-11-16 16:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-13 22:16 [PATCH Xen-unstable] Balloon down memory to achive enough DMA32 memory for PV guests with PCI pass-through to succesfully launch Konrad Rzeszutek Wilk
2009-11-16 10:02 ` Jan Beulich
2009-11-16 15:04   ` Konrad Rzeszutek Wilk
2009-11-16 15:31     ` Jan Beulich
2009-11-16 16:38       ` Konrad Rzeszutek Wilk [this message]
2009-11-16 16:47         ` Jan Beulich

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=20091116163827.GB20333@phenom.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=JBeulich@novell.com \
    --cc=keir.fraser@eu.citrix.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.