From mboxrd@z Thu Jan  1 00:00:00 1970
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Subject: Re: [PATCH] set pv guest default video_memkb to 0
Date: Thu, 20 Nov 2014 14:44:31 -0500
Message-ID: <20141120194431.GB25423@laptop.dumpdata.com>
References: <1416302762.17982.1.camel@citrix.com>
	<1416344228-24164-1-git-send-email-zhigang.x.wang@oracle.com>
	<20141119210845.GF20440@laptop.dumpdata.com>
	<20141119212434.GB27349@zion.uk.xensource.com>
	<1416497391.14429.23.camel@citrix.com>
	<20141120154817.GA31452@zion.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta5.messagelabs.com ([195.245.231.135])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <konrad.wilk@oracle.com>) id 1XrXes-0007c3-Ec
	for xen-devel@lists.xenproject.org; Thu, 20 Nov 2014 19:44:50 +0000
Content-Disposition: inline
In-Reply-To: <20141120154817.GA31452@zion.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Wei Liu <wei.liu2@citrix.com>
Cc: xen-devel@lists.xenproject.org, Zhigang Wang <zhigang.x.wang@oracle.com>, ian.jackson@eu.citrix.com, Ian Campbell <Ian.Campbell@citrix.com>
List-Id: xen-devel@lists.xenproject.org

On Thu, Nov 20, 2014 at 03:48:17PM +0000, Wei Liu wrote:
> On Thu, Nov 20, 2014 at 03:29:51PM +0000, Ian Campbell wrote:
> > On Wed, 2014-11-19 at 21:24 +0000, Wei Liu wrote:
> > > On Wed, Nov 19, 2014 at 04:08:46PM -0500, Konrad Rzeszutek Wilk wrote:
> > > > On Tue, Nov 18, 2014 at 03:57:08PM -0500, Zhigang Wang wrote:
> > > > > Before this patch, pv guest video_memkb is -1, which is an invalid value.
> > > > > And it will cause the xenstore 'memory/targe' calculation wrong:
> > > > > 
> > > > >     memory/target = info->target_memkb - info->video_memkb
> > > > 
> > > > CC-ing the maintainers.
> > > > 
> > > > Is this an regression as compared to Xen 4.4 or is this also in Xen 4.4?
> > > > 
> > > 
> > > I don't think this is a regression, it has been broken for quite a
> > > while.
> > 
> > I think so to.
> > 
> > On the other hand its a pretty clear bug to use video_memkb == -1 and
> > we've seen that it causes real issues. The fix is also fairly obvious.
> > I'm inclined towards suggesting we fix this in 4.5.
> > 
> 
> I concur.

Lets do it then. RElease-Acked-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> 
> Wei.
> 
> > Ian.