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=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_2 autolearn=ham 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 73D8AC54E49 for ; Fri, 8 May 2020 18:58:29 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 399A221974 for ; Fri, 8 May 2020 18:58:29 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 399A221974 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=goodmis.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id A6DF9900017; Fri, 8 May 2020 14:58:28 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A1D8A900004; Fri, 8 May 2020 14:58:28 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 95A25900017; Fri, 8 May 2020 14:58:28 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0243.hostedemail.com [216.40.44.243]) by kanga.kvack.org (Postfix) with ESMTP id 7EF25900004 for ; Fri, 8 May 2020 14:58:28 -0400 (EDT) Received: from smtpin24.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 4968E180AD806 for ; Fri, 8 May 2020 18:58:28 +0000 (UTC) X-FDA: 76794462696.24.pin75_2cd0d8aa0120d X-HE-Tag: pin75_2cd0d8aa0120d X-Filterd-Recvd-Size: 2969 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf19.hostedemail.com (Postfix) with ESMTP for ; Fri, 8 May 2020 18:58:27 +0000 (UTC) Received: from gandalf.local.home (cpe-66-24-58-225.stny.res.rr.com [66.24.58.225]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 8924D2083B; Fri, 8 May 2020 18:58:25 +0000 (UTC) Date: Fri, 8 May 2020 14:58:22 -0400 From: Steven Rostedt To: Joerg Roedel Cc: x86@kernel.org, hpa@zytor.com, Dave Hansen , Andy Lutomirski , Peter Zijlstra , rjw@rjwysocki.net, Arnd Bergmann , Andrew Morton , Vlastimil Babka , Michal Hocko , Joerg Roedel , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, linux-arch@vger.kernel.org, linux-mm@kvack.org Subject: Re: [RFC PATCH 6/7] mm: Remove vmalloc_sync_(un)mappings() Message-ID: <20200508145822.07fcc32b@gandalf.local.home> In-Reply-To: <20200508144043.13893-7-joro@8bytes.org> References: <20200508144043.13893-1-joro@8bytes.org> <20200508144043.13893-7-joro@8bytes.org> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Fri, 8 May 2020 16:40:42 +0200 Joerg Roedel wrote: > From: Joerg Roedel > > These functions are not needed anymore because the vmalloc and ioremap > mappings are now synchronized when they are created or teared down. > > Remove all callers and function definitions. > > Signed-off-by: Joerg Roedel > You'll need to fold this into this patch, as my patch has already hit Linus's tree. But I applied your whole series and I'm not able to reproduce the bug. Tested-by: Steven Rostedt (VMware) -- Steve diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 29615f15a820..1424a89193c6 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -8526,19 +8526,6 @@ static int allocate_trace_buffers(struct trace_array *tr, int size) */ allocate_snapshot = false; #endif - - /* - * Because of some magic with the way alloc_percpu() works on - * x86_64, we need to synchronize the pgd of all the tables, - * otherwise the trace events that happen in x86_64 page fault - * handlers can't cope with accessing the chance that a - * alloc_percpu()'d memory might be touched in the page fault trace - * event. Oh, and we need to audit all other alloc_percpu() and vmalloc() - * calls in tracing, because something might get triggered within a - * page fault trace event! - */ - vmalloc_sync_mappings(); - return 0; }