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

Re: [Xen-devel] [TOOLS][RFC] xc_get_pfn_list() and getmemlist.start_pfn

To: Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] [TOOLS][RFC] xc_get_pfn_list() and getmemlist.start_pfn
From: Jimi Xenidis <jimix@xxxxxxxxxxxxxx>
Date: Thu, 19 Oct 2006 09:12:21 -0400
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, XenPPC-devel <xen-ppc-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Thu, 19 Oct 2006 06:12:55 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <C15CE7EC.2BF4%Keir.Fraser@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: <C15CE7EC.2BF4%Keir.Fraser@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

On Oct 19, 2006, at 3:29 AM, Keir Fraser wrote:

On 18/10/06 10:57 pm, "Jimi Xenidis" <jimix@xxxxxxxxxxxxxx> wrote:

It looks like IA64 has introduced a start_pfn for xc_get_pfn_list().
The PPC port requires this as well since it allows us to fill in the
page_array a little bit at a time.

I can either replicate the IA64 ifdef logic (yuk!) or we can formally
add a start_pfn to this interface, since it _is_ a formal member of
the data structure.

I'll take a stab at it, if it is acceptable.

Where is the ifdef logic?
tools/libxc/xc_private.c:265
  #ifndef __ia64__
  int xc_get_pfn_list(int xc_handle,
                    uint32_t domid,
                    xen_pfn_t *pfn_buf,
                    unsigned long max_pfns)

the patch would change this "libxc private" interface to

  int xc_get_pfn_list(int xc_handle,
                    uint32_t domid,
                    xen_pfn_t *pfn_buf,
+                   xen_pfn_t start_pfn,
                    unsigned long max_pfns)

use the start_pfn member of struct xen_domctl_getmemlist
Fix all callers and change xen to pay attention to it, I think I have the x86-xen patch.

-JX

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