Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753441AbbGPPIl (ORCPT ); Thu, 16 Jul 2015 11:08:41 -0400 Received: from smtp02.citrix.com ([66.165.176.63]:49449 "EHLO SMTP02.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752314AbbGPPIj (ORCPT ); Thu, 16 Jul 2015 11:08:39 -0400 X-IronPort-AV: E=Sophos;i="5.15,488,1432598400"; d="scan'208";a="284749957" Date: Thu, 16 Jul 2015 16:05:12 +0100 From: Stefano Stabellini X-X-Sender: sstabellini@kaball.uk.xensource.com To: Julien Grall CC: , , , , , Julien Grall , Konrad Rzeszutek Wilk , Boris Ostrovsky , David Vrabel Subject: Re: [PATCH v2 04/20] xen/grant: Add helper gnttab_page_grant_foreign_access_ref In-Reply-To: <1436474552-31789-5-git-send-email-julien.grall@citrix.com> Message-ID: References: <1436474552-31789-1-git-send-email-julien.grall@citrix.com> <1436474552-31789-5-git-send-email-julien.grall@citrix.com> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" X-DLP: MIA1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2950 Lines: 85 On Thu, 9 Jul 2015, Julien Grall wrote: > Many PV drivers contain the idiom: > > pfn = page_to_mfn(...) /* Or similar */ > gnttab_grant_foreign_access_ref > > Replace it by a new helper. Note that when Linux is using a different > page granularity than Xen, the helper only gives access to the first 4KB > grant. > > This is useful where drivers are allocating a full Linux page for each > grant. > > Also include xen/interface/grant_table.h rather than xen/grant_table.h in > asm/page.h for x86 to fix a compilation issue [1]. Only the former is > useful in order to get the structure definition. > > [1] Interpendency between asm/page.h and xen/grant_table.h which result > to page_mfn not being defined when necessary. > > Signed-off-by: Julien Grall > Cc: Konrad Rzeszutek Wilk > Cc: Boris Ostrovsky > Cc: David Vrabel > --- > Changes in v2: > - Patch added > --- > arch/x86/include/asm/xen/page.h | 2 +- > include/xen/grant_table.h | 9 +++++++++ > 2 files changed, 10 insertions(+), 1 deletion(-) > > diff --git a/arch/x86/include/asm/xen/page.h b/arch/x86/include/asm/xen/page.h > index c44a5d5..fb2e037 100644 > --- a/arch/x86/include/asm/xen/page.h > +++ b/arch/x86/include/asm/xen/page.h > @@ -12,7 +12,7 @@ > #include > > #include > -#include > +#include > #include > > /* Xen machine address */ > diff --git a/include/xen/grant_table.h b/include/xen/grant_table.h > index 6f77378..6a1ef86 100644 > --- a/include/xen/grant_table.h > +++ b/include/xen/grant_table.h > @@ -131,6 +131,15 @@ void gnttab_cancel_free_callback(struct gnttab_free_callback *callback); > void gnttab_grant_foreign_access_ref(grant_ref_t ref, domid_t domid, > unsigned long frame, int readonly); > > +/* Give access to the first 4K of the page */ > +static inline void gnttab_page_grant_foreign_access_ref( > + grant_ref_t ref, domid_t domid, > + struct page *page, int readonly) > +{ I like this. I think it might make sense to call it gnttab_page_grant_foreign_access_ref_one to make clear that it is only granting the first 4K. In the future we could introduce a new function, called gnttab_page_grant_foreign_access_ref, that grants all 4K in the page. In any case Reviewed-by: Stefano Stabellini > + gnttab_grant_foreign_access_ref(ref, domid, page_to_mfn(page), > + readonly); > +} > + > void gnttab_grant_foreign_transfer_ref(grant_ref_t, domid_t domid, > unsigned long pfn); > > -- > 2.1.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/