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] libfsimage: zfs build fix

To: Mark Johnson <johnson.nh@xxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libfsimage: zfs build fix
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Thu, 15 Apr 2010 17:31:24 +0100
Cc: Christoph Egger <Christoph.Egger@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 15 Apr 2010 09:34:29 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <r2o521a4d121004150851h3c92158bsbbe42c4d94bcf8f3@xxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acrcs55IOhTsobLBRA2WFgorjGp/YAABXhVl
Thread-topic: [Xen-devel] [PATCH] libfsimage: zfs build fix
User-agent: Microsoft-Entourage/12.24.0.100205
On 15/04/2010 16:51, "Mark Johnson" <johnson.nh@xxxxxxxxx> wrote:

> On Thu, Apr 15, 2010 at 10:10 AM, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> wrote:
>> Christoph Egger writes ("[Xen-devel] [PATCH] libfsimage: zfs build fix"):
>>> Attached patch makes libfsimage build again on NetBSD.
> 
> The fsimage zfs patch was changed around to make it simpler
> to keep in sync with the grub changes.  fsys_zfs.c, etc are exact copies
> of what's in grub  e.g.
>     
> http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/grub/grub-0.97/s
> tage2/fsys_zfs.c
> 
> fsi_zfs.* are kludges to get the grub code to fit into libfsimage and
> is the only code which is different from the grub code.
> 
> It would be easier to keep in sync if this would be continued.

Unfortunately with -Wall the zfs sources spewed a quite unreasonable number
of warnings. Enough for me to get annoyed, go clean them all up, and add
-Werror.

I don't know how important it is to keep the files identical? It's not like
interfaces or anything have been changed: all the modifications are trivial
one-liners. Basically, it should still be possible to dump a new version of
the zfs code into our tree sometime down the line, and then go xfix up the
build warnings again (it didn't take that long). Doesn't that suffice? You
could even try getting GRUB guys to accept our build cleanups upstream (fat
chance given my previous experiences with them, I'll admit). ;-)

 -- Keir



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