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-devel] [PATCH 1/5 TAKE 2] xenoprof: split xen x86 xenoprof code

To: "Isaku Yamahata" <yamahata@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH 1/5 TAKE 2] xenoprof: split xen x86 xenoprof code
From: "Santos, Jose Renato G" <joserenato.santos@xxxxxx>
Date: Mon, 20 Nov 2006 13:17:41 -0600
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Mon, 20 Nov 2006 11:17:50 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20061118024659.GA2090%yamahata@xxxxxxxxxxxxx>
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
Thread-index: AccKu9tGLTw/wdrpSni3uhNs9CrD3QCGrzUQ
Thread-topic: [Xen-devel] [PATCH 1/5 TAKE 2] xenoprof: split xen x86 xenoprof code
 

> -----Original Message-----
> From: Isaku Yamahata [mailto:yamahata@xxxxxxxxxxxxx] 
> Sent: Friday, November 17, 2006 6:47 PM
> To: Santos, Jose Renato G
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxx; xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [PATCH 1/5 TAKE 2] xenoprof: split 
> xen x86 xenoprof code
> 
> On Fri, Nov 17, 2006 at 01:26:04PM -0600, Santos, Jose Renato G wrote:
> 
> > > > QUESTIONS for Isaku:
> > > > 
> > > > patch 4/5:
> > > > ==========
> > > > 
> > > > > +struct xenoprof_shared_buffer {
> > > > > +     char                                    *buffer;
> > > > > +     struct xenoprof_arch_shared_buffer      arch;
> > > > > +};
> > > > 
> > > > The arch field has no extra info for x86. Why do you need this?
> > > 
> > > On IA64 it is defined as
> > > struct xenoprof_arch_shared_buffer {
> > >         struct resource*        res;
> > > };
> > > 
> > > On IA64, auto translated mode feature is enabled.
> > > It means that page mapping from xen is based on pseudo physical 
> > > address, not virtual address. Pseudo physical address used to map 
> > > pages must be recorded somewhere in order to unmap.
> > > I guess, if xenoprof/x86 supported auto translated guest mode 
> > > something similar structure would be necessary.
> > > 
> > > 
> >   Isaku
> > 
> >  I am still not convinced that we need this. Since this in 
> the kernel
> > you can always get the pseudo physical address from the 
> virtual address,
> > right? Am I missing something?
> 
> Yes, the pseudo physical address itself can be get.
> The point is that the pseudo physical addresses aren't always
> backed by struct page.
> In such a case alloc_pages()/free_pages() can't be used, 
> another structure
> is necessary.
> In fact Xen/IA64 allocates the pseudo physical address resion by
> allocate_resource(&iomem_resource, ...). Not by alloc_pages().
> Why doesn't Xen/IA64 use alloc_pages()?
> Because it is also used for grant table foreign domain page mapping
> and its size is very large sometimes. Using alloc_pages() 
> results in OOM.
> Xen/PPC seems to adopt another approach, though.
> 

  OK, I understand now.
  You want to keep the mem resource info and the buffer virtual address
in the same struct which you pass to the arch specific code for mapping
and unmapping xenoprof pages. I think that is fine.

  Regards

  Renato

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

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