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

[Xen-devel] Re: [PATCH] remove fs/Kconfig from sparse tree in favor of a

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] Re: [PATCH] remove fs/Kconfig from sparse tree in favor of a 1 line patch
From: Chris Wright <chrisw@xxxxxxxxxxxx>
Date: Fri, 9 Jun 2006 21:31:15 -0700
Delivery-date: Fri, 09 Jun 2006 21:28:51 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20060610021411.GP14641@xxxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <20060609222922.GO14641@xxxxxxxxxxxxxxx> <20060609230447.GO2724@xxxxxxxxxxxxxxxxx> <20060610021411.GP14641@xxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
* Sean Dague (japh@xxxxxxxxxx) wrote:
[snipped examples of the one-line diffs]

Hey, it's getting better, there's been files in there with no diffs ;-)

> I'd really argue that while the sparse tree is needed for some of the
> extensive code changes, using it to turn off single features in Kconfig in
> directories where no code is changed, is verging on abuse. ;)

It is a good example of the sparse tree not being very compact
representation of those diffs.  But it's valid in the sense that files
that need Xen specific changes are patched and tracked as source in the
sparse tree rather than diffs.  The latter gets to be a pain when you
want to make changes, interdiff isn't that nice.  And if you want to
track all this stuff in another tree (distro tree, for example) having
bits in sparse and bits in patches/ is just more of a headache.

> This is specifically causing me pain trying to apply add squashfs support,
> which otherwise would have been dropping 1 patch in place, and tweaking my
> build config.  In the current infrastructure it requires a patch + sparse
> tree changes, which is really unfortunate, and I think completely
> unnessesary, given the very small changes that Xen needs in fs/Kconfig.

The part I'm missing is why you can't patch squashfs to the prepped tree.
Patch should apply fine (albeit a bit of offset in the fs/Kconfig).
I typically keep a stash of patches on top of the unpacked patches/ +
sparse, seems to work well (except for the minor annoyance that patches/
is the default patch dir for quilt).

thanks,
-chris

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