WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Introduce dom0-min-space configuration option

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Introduce dom0-min-space configuration option
From: Michal Novotny <minovotn@xxxxxxxxxx>
Date: Thu, 22 Jul 2010 09:50:47 +0200
Cc: "'xen-devel@xxxxxxxxxxxxxxxxxxx'" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 22 Jul 2010 00:52:04 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19526.62789.227136.192517@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4C3B59FE.2060201@xxxxxxxxxx> <19516.43557.723062.885082@xxxxxxxxxxxxxxxxxxxxxxxx> <4C3D9E22.3030302@xxxxxxxxxx> <19526.62789.227136.192517@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.9) Gecko/20100430 Fedora/3.0.4-3.fc13 Thunderbird/3.0.4
On 07/21/2010 03:25 PM, Ian Jackson wrote:
Michal Novotny writes ("Re: [Xen-devel] [PATCH] Introduce dom0-min-space 
configuration option"):
Ian, the problem is when pygrub extracts the vmlinuz and initrd for PV
guests but there's insufficient space on the dom0, there's no error
message but the error is being raised from Xend itself, [...]
So the correct fix is to make sure the error propagation works
correctly.


Well, the patching in the all the parts of the code may be better but there may be many issues caused there by dom0 being out of space and not all of them have to be xen related - nevertheless users can think they're xen related until they find out the dom0 is out of space and therefore although Xen may *not* be responsible they can blame Xen for that.

  libxc to be precise since there's the zStream inflation code but
since the input data are not valid (i.e. they're just partial,
That suggests that not only is the error being bungled somewhere, it's
being outright ignored and something tries to decompress the truncated
file.

Well, there's no other way to fix it than to implement a check for enough space on dom0 for extracting vmlinuz and initrd files from the image file. If it's partially extracted it's understandable that the data are not valid input for zStream processing and therefore it runs into Z_BUF_ERROR (-5) during the process.

These bugs should be fixed.

But I think your patch is entirely wrong, I'm afraid.

Ian.

Why do you think my patch is entirely wrong? If you're talking about functionality it's been tested on x86_64 RHEL-6 guest with upstream Xen installed.

Michal

--
Michal Novotny<minovotn@xxxxxxxxxx>, RHCE
Virtualization Team (xen userspace), Red Hat


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel