From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751791AbeAPTWK (ORCPT + 1 other); Tue, 16 Jan 2018 14:22:10 -0500 Received: from mail.efficios.com ([167.114.142.141]:60972 "EHLO mail.efficios.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750892AbeAPTWH (ORCPT ); Tue, 16 Jan 2018 14:22:07 -0500 Date: Tue, 16 Jan 2018 19:22:13 +0000 (UTC) From: Mathieu Desnoyers To: Thomas Gleixner Cc: Ingo Molnar , Peter Zijlstra , linux-kernel , linux-api , Andy Lutomirski , "Paul E. McKenney" , Boqun Feng , Andrew Hunter , maged michael , Avi Kivity , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Dave Watson , "H. Peter Anvin" , Andrea Parri , "Russell King, ARM Linux" , Greg Hackmann , Will Deacon , David Sehr , Linus Torvalds , x86 , linux-arch Message-ID: <1443621141.3981.1516130533202.JavaMail.zimbra@efficios.com> In-Reply-To: References: <20180115191104.12437-1-mathieu.desnoyers@efficios.com> <20180115191104.12437-9-mathieu.desnoyers@efficios.com> Subject: Re: [PATCH for 4.16 08/10] membarrier: x86: Provide core serializing command (v3) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [167.114.142.141] X-Mailer: Zimbra 8.7.11_GA_1854 (ZimbraWebClient - FF52 (Linux)/8.7.11_GA_1854) Thread-Topic: membarrier: x86: Provide core serializing command (v3) Thread-Index: 975NtC0HDdfxzRSo/Hk4EyDWO2a5qQ== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: ----- On Jan 16, 2018, at 1:29 PM, Thomas Gleixner tglx@linutronix.de wrote: > On Mon, 15 Jan 2018, Mathieu Desnoyers wrote: > >> There are two places where core serialization is needed by membarrier: >> >> 1) When returning from the membarrier IPI, >> 2) After scheduler updates curr to a thread with a different mm, before >> going back to user-space, since the curr->mm is used by membarrier to >> check whether it needs to send an IPI to that CPU. > > This wants to be split into x86 and core changes. Ideally you make the core > changes before the previous patch and add the empty inline into > linux/processor.h.... Good point, done. The first commit introducing the new command now also introduces the generic stuff moved from the x86 patches. Thanks, Mathieu > > Thanks, > > tglx -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com