From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.5 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1FF62C3A5A0 for ; Tue, 20 Aug 2019 01:50:02 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [203.11.71.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 545B722CE8 for ; Tue, 20 Aug 2019 01:50:01 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=fossix-org.20150623.gappssmtp.com header.i=@fossix-org.20150623.gappssmtp.com header.b="xDgOvr7d" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 545B722CE8 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=fossix.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Received: from bilbo.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 46CDJ56yFMzDqd3 for ; Tue, 20 Aug 2019 11:49:57 +1000 (AEST) Authentication-Results: lists.ozlabs.org; spf=none (mailfrom) smtp.mailfrom=fossix.org (client-ip=2607:f8b0:4864:20::443; helo=mail-pf1-x443.google.com; envelope-from=santosh@fossix.org; receiver=) Authentication-Results: lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=fossix.org Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=fossix-org.20150623.gappssmtp.com header.i=@fossix-org.20150623.gappssmtp.com header.b="xDgOvr7d"; dkim-atps=neutral Received: from mail-pf1-x443.google.com (mail-pf1-x443.google.com [IPv6:2607:f8b0:4864:20::443]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 46CDFx0C1zzDqd3 for ; Tue, 20 Aug 2019 11:48:03 +1000 (AEST) Received: by mail-pf1-x443.google.com with SMTP id f17so2306119pfn.6 for ; Mon, 19 Aug 2019 18:48:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fossix-org.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:in-reply-to:references:date:message-id :mime-version; bh=RfLs+sRfBSDgb+tBIhChiANAEUhKiMw/rtbpKhN6zaY=; b=xDgOvr7dIvIKX4kULkDv9+SPhMsQwKzSH0sKoqyXx/wpx5DHVoyuaCtOBzG7HwuRYp jh8X8OfTPK5SowqYmxTCsg/xKHv5iG8CBBSXHn+pHgPpPDEBO3xZvClGHHr9/V5Osy+E FUAIaGka/yjKUD9VxaCbcIDwAuVdeaZdXfPqg/23hXjLcYhNGoTDXJEioXK2eF/wdcSj vINbNRuNtkq3oXEXtMgq/SXF3lXNzxo/PJrxwQUVM68i62c44Ofjb2IwbBDn0ZAJb/Mk vBUPUOUVATwFGL18ru1BrsD/w8Fnpivid49hcQhPvzboCRhk+cL4mfmFKt39vO552iHQ QkCw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=RfLs+sRfBSDgb+tBIhChiANAEUhKiMw/rtbpKhN6zaY=; b=qiwvOVxK4Pik7ctC/zp9uatOFmInDnyugb1jvImKkQsJO4CQ6AZUT+Fjcf3lGiiX9A vegkiTczl++sh6g+lyzkkoyrsM24sHXcHHEJ3UI/CcL4qROTpEXzaQb5q+ZDBFKjrEtH sG4AVUD1Ha4xjIywpD65KFcvDLw1agxTQCYt9Hzhq127tOmrMqGFSVRp5CGAoiECcKI8 imyRpAeYRXd71g9GrL2mzX7ev+zPTLgG8cpPsJH13JQYELG1Lbos+aOWVDHyK0vsOF1B Nv8ttuJvxkPAAktyuVPeGeyrYGvdJSb9PiugjsTUK3lmh8U2KZoL7r4CaAlOtYruu7Cl kMXQ== X-Gm-Message-State: APjAAAXI3gLeRvOQbnQIq4IQgqtF+bJfIEAU2G+16jOKy+t1OL7cFEh2 plW7Q1bAhubFoR0bXZYyM3fGEg== X-Google-Smtp-Source: APXvYqwVegiYT4hHAvpHnbXs+lPZw+l1k9sXk0MQ52n1ouVNqGK5erNtO31mcO6BFgTZIuKjY+o1eQ== X-Received: by 2002:a65:4841:: with SMTP id i1mr22588419pgs.316.1566265679652; Mon, 19 Aug 2019 18:47:59 -0700 (PDT) Received: from localhost ([49.205.218.65]) by smtp.gmail.com with ESMTPSA id s20sm16948435pfe.169.2019.08.19.18.47.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 19 Aug 2019 18:47:59 -0700 (PDT) From: Santosh Sivaraj To: Nicholas Piggin , Linux Kernel , linuxppc-dev Subject: Re: [PATCH v10 2/7] powerpc/mce: Fix MCE handling for huge pages In-Reply-To: <1566223931.kpuwkor3n7.astroid@bobo.none> References: <20190815003941.18655-1-santosh@fossix.org> <20190815003941.18655-3-santosh@fossix.org> <1566223931.kpuwkor3n7.astroid@bobo.none> Date: Tue, 20 Aug 2019 07:17:56 +0530 Message-ID: <87ef1gppyr.fsf@santosiv.in.ibm.com> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "Aneesh Kumar K.V" , Mahesh Salgaonkar , stable@vger.kernel.org, Chandan Rajendra , Mahesh Salgaonkar , Reza Arbab Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" Hi Nick, Nicholas Piggin writes: > Santosh Sivaraj's on August 15, 2019 10:39 am: >> From: Balbir Singh >> >> The current code would fail on huge pages addresses, since the shift would >> be incorrect. Use the correct page shift value returned by >> __find_linux_pte() to get the correct physical address. The code is more >> generic and can handle both regular and compound pages. >> >> Fixes: ba41e1e1ccb9 ("powerpc/mce: Hookup derror (load/store) UE errors") >> Signed-off-by: Balbir Singh >> [arbab@linux.ibm.com: Fixup pseries_do_memory_failure()] >> Signed-off-by: Reza Arbab >> Co-developed-by: Santosh Sivaraj >> Signed-off-by: Santosh Sivaraj >> Tested-by: Mahesh Salgaonkar >> Cc: stable@vger.kernel.org # v4.15+ >> --- >> arch/powerpc/include/asm/mce.h | 2 +- >> arch/powerpc/kernel/mce_power.c | 55 ++++++++++++++-------------- >> arch/powerpc/platforms/pseries/ras.c | 9 ++--- >> 3 files changed, 32 insertions(+), 34 deletions(-) >> >> diff --git a/arch/powerpc/include/asm/mce.h b/arch/powerpc/include/asm/mce.h >> index a4c6a74ad2fb..f3a6036b6bc0 100644 >> --- a/arch/powerpc/include/asm/mce.h >> +++ b/arch/powerpc/include/asm/mce.h >> @@ -209,7 +209,7 @@ extern void release_mce_event(void); >> extern void machine_check_queue_event(void); >> extern void machine_check_print_event_info(struct machine_check_event *evt, >> bool user_mode, bool in_guest); >> -unsigned long addr_to_pfn(struct pt_regs *regs, unsigned long addr); >> +unsigned long addr_to_phys(struct pt_regs *regs, unsigned long addr); >> #ifdef CONFIG_PPC_BOOK3S_64 >> void flush_and_reload_slb(void); >> #endif /* CONFIG_PPC_BOOK3S_64 */ >> diff --git a/arch/powerpc/kernel/mce_power.c b/arch/powerpc/kernel/mce_power.c >> index a814d2dfb5b0..e74816f045f8 100644 >> --- a/arch/powerpc/kernel/mce_power.c >> +++ b/arch/powerpc/kernel/mce_power.c >> @@ -20,13 +20,14 @@ >> #include >> >> /* >> - * Convert an address related to an mm to a PFN. NOTE: we are in real >> - * mode, we could potentially race with page table updates. >> + * Convert an address related to an mm to a physical address. >> + * NOTE: we are in real mode, we could potentially race with page table updates. >> */ >> -unsigned long addr_to_pfn(struct pt_regs *regs, unsigned long addr) >> +unsigned long addr_to_phys(struct pt_regs *regs, unsigned long addr) >> { >> - pte_t *ptep; >> - unsigned long flags; >> + pte_t *ptep, pte; >> + unsigned int shift; >> + unsigned long flags, phys_addr; >> struct mm_struct *mm; >> >> if (user_mode(regs)) >> @@ -35,14 +36,21 @@ unsigned long addr_to_pfn(struct pt_regs *regs, unsigned long addr) >> mm = &init_mm; >> >> local_irq_save(flags); >> - if (mm == current->mm) >> - ptep = find_current_mm_pte(mm->pgd, addr, NULL, NULL); >> - else >> - ptep = find_init_mm_pte(addr, NULL); >> + ptep = __find_linux_pte(mm->pgd, addr, NULL, &shift); >> local_irq_restore(flags); >> + >> if (!ptep || pte_special(*ptep)) >> return ULONG_MAX; >> - return pte_pfn(*ptep); >> + >> + pte = *ptep; >> + if (shift > PAGE_SHIFT) { >> + unsigned long rpnmask = (1ul << shift) - PAGE_SIZE; >> + >> + pte = __pte(pte_val(pte) | (addr & rpnmask)); >> + } >> + phys_addr = pte_pfn(pte) << PAGE_SHIFT; >> + >> + return phys_addr; >> } > > This should remain addr_to_pfn I think. None of the callers care what > size page the EA was mapped with. 'pfn' is referring to the Linux pfn, > which is the small page number. > > if (shift > PAGE_SHIFT) > return (pte_pfn(*ptep) | ((addr & ((1UL << shift) - 1)) >> PAGE_SHIFT); > else > return pte_pfn(*ptep); > > Something roughly like that, then you don't have to change any callers > or am I missing something? Here[1] you asked to return the real address rather than pfn, which all callers care about. So made the changes accordingly. [1] https://www.spinics.net/lists/kernel/msg3187658.html Thanks, Santosh > > Thanks, > Nick