From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:39653)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1aLuAe-0005fa-IL
	for qemu-devel@nongnu.org; Wed, 20 Jan 2016 09:55:41 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1aLuAa-00034E-Rr
	for qemu-devel@nongnu.org; Wed, 20 Jan 2016 09:55:40 -0500
Received: from mx1.redhat.com ([209.132.183.28]:48488)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <kraxel@redhat.com>) id 1aLuAa-000340-M5
	for qemu-devel@nongnu.org; Wed, 20 Jan 2016 09:55:36 -0500
Message-ID: <1453301733.11804.142.camel@redhat.com>
From: Gerd Hoffmann <kraxel@redhat.com>
Date: Wed, 20 Jan 2016 15:55:33 +0100
In-Reply-To: <20160119123739.GA27855@thinpad.lan.raisama.net>
References: <1452257883-19549-1-git-send-email-kraxel@redhat.com>
	<20160119123739.GA27855@thinpad.lan.raisama.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH v2] pc: allow raising low memory via
 max-ram-below-4g option
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Eduardo Habkost <ehabkost@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, qemu-devel@nongnu.org, Richard Henderson <rth@twiddle.net>

  Hi,

> > +     *    qemu -M pc,max-ram-below-4g=3D2G -m 4G     -> 2048M low, 204=
8M high

> I assume max-ram-below-4g > 3.5G was unsupported before, and we
> are breaking compatibility intentionally.

max-ram-below-4g did only reduce memory, so max-ram-below-4g > 3.5G (or
max-ram-below-4g > 3G with gigabyte align) had no effect and therefore
is something pretty pointless.

I'd expect the one case quoted above to be the only case relevant in
practice (i.e. move split from 3G to 2G for more PCI I/O space),
especially given that the option was added after gigabyte alignment
support.

> Because this patch also
> changes the resulting memory layout when
>   3G < max_ram_below_4g < ram_size < 3.5G
> e.g.:
>   qemu -M pc,max-ram-below-4g=3D3200M -m 3328M

Ah, I see.  With the patch applied the gigabyte align option is weighed
higher, so qemu wouldn't give you 3200M lowmem.  I would be highly
surprised to see such a configuration in the wild ...

cheers,
  Gerd