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] netfront pv driver building

To: <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: [Xen-devel] netfront pv driver building
From: "Jan Beulich" <jbeulich@xxxxxxxxxx>
Date: Wed, 24 Jan 2007 13:46:17 +0000
Delivery-date: Wed, 24 Jan 2007 05:45:41 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
netfront requires the symbol __supported_pte_mask when PAE is enabled in the
kernel being built for. That symbol, however, isn't being exported, and it 
doesn't
seem likely that mainline would want to see this get exported (after all, the
general assumption is that the page table handling inline functions and macros
are supposed to be used only by the memory management code).

Additionally, both uses are inside a conditional depending upon
!XENFEAT_auto_translated_physmap, which hence can't ever be true if this
code is being built as pv driver. Wouldn't it hence make sense to #ifdef these
code blocks, or to enhance xen/features.h so that checks for those features
that are always on in hvm and return a constant 1 rather than using the
xen_features array.

Thanks, Jan

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

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