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] recent major -unstable changes cause ia64 build to be br

To: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>
Subject: Re: [Xen-devel] recent major -unstable changes cause ia64 build to be broken
From: Hollis Blanchard <hollisb@xxxxxxxxxx>
Date: Tue, 10 May 2005 16:35:24 -0500
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Tue, 10 May 2005 21:37:00 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <516F50407E01324991DD6D07B0531AD542CA2F@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
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: <516F50407E01324991DD6D07B0531AD542CA2F@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla Thunderbird 1.0.2 (X11/20050404)
Magenheimer, Dan (HP Labs Fort Collins) wrote:
> 
> The biggest problem was due to the removal of three
> includes from include/xen/slab.h... this was obscured
> by some of the other changes.
> 
> Could you put these back in where they were before please?
> (If necessary, with a ifdef ia64 for now and we can sort
> it out later.)
> 
> < #include <xen/mm.h>
> < #include <xen/cache.h>
> < #include <xen/types.h> 
> 
> Actually, it appears I only need xen/mm.h if that helps.

I needed to include xen/mm.h in a handful of (common) places with the
new code:
        xen/common/page_alloc.c
        xen/common/schedule.c
        xen/drivers/char/console.c

These files all use things from xen/mm.h, but do not include it. Here I
will repeat my point about relying on implicit includes... :)

-- 
Hollis Blanchard
IBM Linux Technology Center

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