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-ia64-devel

RE: [Xen-ia64-devel] MAJOR cleanup in patched-from-linux code

To: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>, <xen-ia64-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-ia64-devel] MAJOR cleanup in patched-from-linux code
From: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>
Date: Mon, 2 May 2005 13:36:22 -0700
Delivery-date: Mon, 02 May 2005 20:36:31 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: DIscussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcVPVfAB+miZH566RzKIcJQIiAlWwQAAJWcg
Thread-topic: [Xen-ia64-devel] MAJOR cleanup in patched-from-linux code
I forgot to add... a clean clone is suggested as many
of the patched files changed and some confusion results
from building on top of a bk pull. 

> -----Original Message-----
> From: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx 
> [mailto:xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf 
> Of Magenheimer, Dan (HP Labs Fort Collins)
> Sent: Monday, May 02, 2005 2:32 PM
> To: xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> Subject: [Xen-ia64-devel] MAJOR cleanup in patched-from-linux code
> 
> I just checked in to xen-ia64.bkbits.net://xeno-unstable-ia64.bk
> a major cleanup to the linux patch files.  Many of the patches
> were cluttered with unnecessary additions and some of the files
> appeared to be better off checked in directly rather than on top
> of the linux source.
> 
> (Note that bkbits.net is having some problems in its admin
> interface, so it may take a day or two to push these changes
> to xen.bkbits.net.)
> 
> Here's the results:
> 
> BEFORE:
> 38 patch files
> 1027 lines added
> 160 lines subtracted 
> 
> AFTER:
> 24 patch files
> 348 lines added
> 2 lines subtracted
> 
> Of the 348 lines added, 151 are real code and the rest
> are conditional compile directives (#ifdef/#else/#endif etc).
> All changes are clearly marked with ifdef's, with the
> exception of two lines which are in the middle of a huge
> assembly macro.
> 
> Why not get rid of these entirely and just check in the
> Linux sources?  Because mkbuildtree links in almost 32000
> lines of Linux/ia64 source files and only slightly more
> than 1% change is required to utilize these in Xen/ia64.
> That's leverage!
> 
> 
> _______________________________________________
> Xen-ia64-devel mailing list
> Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ia64-devel
> 

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

<Prev in Thread] Current Thread [Next in Thread>