Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752711AbaKGOk4 (ORCPT ); Fri, 7 Nov 2014 09:40:56 -0500 Received: from smtp.citrix.com ([66.165.176.89]:16088 "EHLO SMTP.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751880AbaKGOky (ORCPT ); Fri, 7 Nov 2014 09:40:54 -0500 X-IronPort-AV: E=Sophos;i="5.07,332,1413244800"; d="scan'208";a="189146698" Message-ID: <545CD9F2.80208@citrix.com> Date: Fri, 7 Nov 2014 14:40:50 +0000 From: David Vrabel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.5.0 MIME-Version: 1.0 To: Juergen Gross , , , , , , , , Subject: Re: [Xen-devel] [PATCH V2 5/5] Xen: switch to linear virtual mapped sparse p2m list References: <1415252853-7106-1-git-send-email-jgross@suse.com> <1415252853-7106-6-git-send-email-jgross@suse.com> <545CCF15.4070402@citrix.com> <545CD306.90001@suse.com> In-Reply-To: <545CD306.90001@suse.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-DLP: MIA2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/11/14 14:11, Juergen Gross wrote: > On 11/07/2014 02:54 PM, David Vrabel wrote: >> On 06/11/14 05:47, Juergen Gross wrote: >> [...] >>> @@ -526,23 +411,83 @@ unsigned long get_phys_to_machine(unsigned long >>> pfn) >>> return IDENTITY_FRAME(pfn); >>> } >>> >>> - topidx = p2m_top_index(pfn); >>> - mididx = p2m_mid_index(pfn); >>> - idx = p2m_index(pfn); >>> + ptep = lookup_address((unsigned long)(xen_p2m_addr + pfn), &level); >>> + BUG_ON(!ptep || level != PG_LEVEL_4K); >>> >>> /* >>> * The INVALID_P2M_ENTRY is filled in both p2m_*identity >>> * and in p2m_*missing, so returning the INVALID_P2M_ENTRY >>> * would be wrong. >>> */ >>> - if (p2m_top[topidx][mididx] == p2m_identity) >>> + if (pte_pfn(*ptep) == PFN_DOWN(__pa(p2m_identity))) >>> return IDENTITY_FRAME(pfn); >>> >>> - return p2m_top[topidx][mididx][idx]; >>> + return xen_p2m_addr[pfn]; >> >> You should test xen_p2m_addr[pfn] == INVALID_P2M_ENTRY before checking >> if it's an identity entry. This should skip the more expensive >> lookup_address() in the common case. > > I do. The check is in __pfn_to_mfn(). get_phys_to_machine() is called in > this case only. So you do. I missed that. >>> bool __set_phys_to_machine(unsigned long pfn, unsigned long mfn) >> >> I think you should map p2m_missing and p2m_identity as read-only and do >> the new page allocation on a write fault. >> >> set_phys_to_machine() is used every grant map and unmap and in the >> common case (already allocated array page) it must be a fast and simple: >> >> xen_p2m_addr[pfn] = mfn; > > Nice idea. I'll try it. You probably want to try this with a custom exception fixup (or abuse put_user()). David -- 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/