From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yh0-x22b.google.com (mail-yh0-x22b.google.com [IPv6:2607:f8b0:4002:c01::22b]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 8A0AA140125 for ; Thu, 3 Apr 2014 13:37:01 +1100 (EST) Received: by mail-yh0-f43.google.com with SMTP id b6so1101688yha.2 for ; Wed, 02 Apr 2014 19:36:56 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1390292129-15871-1-git-send-email-pingfank@linux.vnet.ibm.com> <8761pdk6x5.fsf@linux.vnet.ibm.com> Date: Thu, 3 Apr 2014 10:36:55 +0800 Message-ID: Subject: Re: [PATCH v2] powernv: kvm: make _PAGE_NUMA take effect From: Liu ping fan To: Alexander Graf Content-Type: text/plain; charset=UTF-8 Cc: kvm-devel , kvm-ppc , Paul Mackerras , "Aneesh Kumar K.V" , linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Alex, could you help to pick up this patch? since v3.14 kernel can enable numa fault for powerpc. Thx, Fan On Mon, Jan 27, 2014 at 5:11 PM, Alexander Graf wrote: > > On 21.01.2014, at 10:42, Aneesh Kumar K.V wrote: > >> Liu Ping Fan writes: >> >>> To make sure that on host, the pages marked with _PAGE_NUMA result in a fault >>> when guest access them, we should force the checking when guest uses hypercall >>> to setup hpte. >>> >>> Signed-off-by: Liu Ping Fan >> >> Reviewed-by: Aneesh Kumar K.V >> >> When we mark pte with _PAGE_NUMA we already call mmu_notifier_invalidate_range_start and >> mmu_notifier_invalidate_range_end, which will mark existing guest hpte >> entry as HPTE_V_ABSENT. Now we need to do that when we are inserting new >> guest hpte entries. This patch does that. > > So what happens next? We insert a page into the HTAB without HPTE_V_VALID set, so the guest will fail to use it. If the guest does an H_READ on it it will suddenly turn to V_VALID though? > > I might need a crash course in the use of HPTE_V_ABSENT. > > > Alex >