From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH RFC 25/35] arm: acpi add helper functions to map memory regions Date: Thu, 05 Feb 2015 04:03:58 +0000 Message-ID: <54D2EBAE.3080909@linaro.org> References: <1423058539-26403-1-git-send-email-parth.dixit@linaro.org> <1423058539-26403-26-git-send-email-parth.dixit@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1423058539-26403-26-git-send-email-parth.dixit@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: parth.dixit@linaro.org, xen-devel@lists.xen.org Cc: christoffer.dall@linaro.org, tim@xen.org, stefano.stabellini@citrix.com, ian.campbell@citrix.com, jbeulich@suse.com List-Id: xen-devel@lists.xenproject.org Hi parth, Title: this is not acpi specific. On 04/02/2015 14:02, parth.dixit@linaro.org wrote: > From: Parth Dixit > > For passing ACPI tables to dom0, UEFI memory needs to be mapped > by xen in dom0 address space. This patch adds helper functions for mapping. I believe that this is not ACPI/RAM specific. Any cached MMIO regions will have same issue. This because Device memory is too strong and disallow unaligned access. > Signed-off-by: Parth Dixit > --- > xen/arch/arm/p2m.c | 24 ++++++++++++++++++++++++ > xen/include/asm-arm/p2m.h | 10 ++++++++++ > 2 files changed, 34 insertions(+) > > diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c > index 8809f5a..5593a91 100644 > --- a/xen/arch/arm/p2m.c > +++ b/xen/arch/arm/p2m.c > @@ -943,6 +943,30 @@ int p2m_populate_ram(struct domain *d, > 0, MATTR_MEM, p2m_ram_rw); > } > > +int map_ram_regions(struct domain *d, > + unsigned long start_gfn, > + unsigned long nr, > + unsigned long mfn) I don't like the name of the function. It gives the impression that we map any RAM region to the guest via this function. Which is obviously wrong and should never be done. Regards, -- Julien Grall