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] linux: xenbus_be currently doesn't build properl

To: "Keir Fraser" <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] linux: xenbus_be currently doesn't build properly as a module
From: "Jan Beulich" <jbeulich@xxxxxxxxxx>
Date: Thu, 20 Apr 2006 15:33:15 +0200
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 20 Apr 2006 06:34:44 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <88c3dae3cc8a048a507a1a3549475c65@xxxxxxxxxxxx>
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: <44479695.76E4.0078.0@xxxxxxxxxx> <88c3dae3cc8a048a507a1a3549475c65@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>>> Keir Fraser <Keir.Fraser@xxxxxxxxxxxx> 20.04.06 15:08:28 >>>
>
>On 20 Apr 2006, at 13:11, Jan Beulich wrote:
>
>> It depends on the non-exported symbols vmlist and vmlist_lock. As 
>> exporting these doesn't seem reasonable, the patch
>> disables the ability to build this as a module for now.
>>
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
>
>A better fix is for xenbus_map_ring_valloc() to have the prototype:
>struct vm_struct *xenbus_map_ring_valloc(struct xenbus_device *, int);
>
>It then returns the vmarea that is allocated, or ERR_PTR() on error.
>
>Then xenbus_unmap_ring_vfree() takes the vmarea rather than vaddr and 
>we avoid the nasty vmlist scan that cannot be modularised.
>
>There's only one user of these functions right now, so the patch for 
>this proper fix won't be very big.

I specifically didn't do it this proper way because of the comment in the 
function
saying doing it the raw way would be so much simpler - but if that in fact 
doesn't
apply...

Jan


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