All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: "David Gibson" <david@gibson.dropbear.id.au>,
	"Cédric Le Goater" <clg@kaod.org>
Cc: qemu-devel@nongnu.org, qemu-ppc@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 1/2] ppc: Add proper real mode translation support
Date: Wed, 29 Jun 2016 12:59:05 +1000	[thread overview]
Message-ID: <1467169145.20278.203.camel@kernel.crashing.org> (raw)
In-Reply-To: <20160629024125.GF8885@voom.fritz.box>

On Wed, 2016-06-29 at 12:41 +1000, David Gibson wrote:
> > +        /* Actually we don't support unbounded RMA anymore since
> we
> > +         * added proper emulation of HV mode. The max we can get
> is
> > +         * 16G which also happens to be what we configure for PAPR
> > +         * mode so make sure we don't do anything bigger than that
> > +         */
> > +        spapr->rma_size = MIN(spapr->rma_size, 0x400000000ull);
> 
> #1 - Instead of the various KVM / non-KVM cases here, it might be
> simpler to just always clamp the RMA to 256MiB.

That would be sad ... we benefit from having a larger RMA..

Cheers,
Ben.


  reply	other threads:[~2016-06-29  2:59 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-28  6:48 [Qemu-devel] [PATCH 0/2] pnv: handle real mode addressing in HV mode Cédric Le Goater
2016-06-28  6:48 ` [Qemu-devel] [PATCH 1/2] ppc: Add proper real mode translation support Cédric Le Goater
2016-06-29  2:41   ` David Gibson
2016-06-29  2:59     ` Benjamin Herrenschmidt [this message]
2016-06-29  3:05       ` David Gibson
2016-06-28  6:48 ` [Qemu-devel] [PATCH 2/2] ppc: Fix 64K pages support in full emulation Cédric Le Goater
2016-06-29  2:22   ` David Gibson
2016-06-30 10:56   ` Anton Blanchard
2016-06-30 11:08     ` Benjamin Herrenschmidt
2016-06-30 16:01     ` Cédric Le Goater
2016-06-30 22:13       ` Benjamin Herrenschmidt
2016-06-30 23:56         ` David Gibson
2016-07-01  6:06         ` Cédric Le Goater

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=1467169145.20278.203.camel@kernel.crashing.org \
    --to=benh@kernel.crashing.org \
    --cc=clg@kaod.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    /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.