From mboxrd@z Thu Jan 1 00:00:00 1970 From: Will Deacon Subject: Re: [PATCH 4/7] arm64: tlbflush.h: add __tlbi() macro Date: Wed, 28 Sep 2016 13:48:07 +0100 Message-ID: <20160928124806.GI24658@arm.com> References: <1473761769-30572-1-git-send-email-punit.agrawal@arm.com> <1473761769-30572-5-git-send-email-punit.agrawal@arm.com> <20160923160710.GF6397@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id A4FBF40C9A for ; Wed, 28 Sep 2016 08:38:42 -0400 (EDT) Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id mx41Y4YlYswf for ; Wed, 28 Sep 2016 08:38:38 -0400 (EDT) Received: from foss.arm.com (foss.arm.com [217.140.101.70]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 5AE1D412ED for ; Wed, 28 Sep 2016 08:38:38 -0400 (EDT) Content-Disposition: inline In-Reply-To: <20160923160710.GF6397@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu To: Punit Agrawal Cc: kvm@vger.kernel.org, Marc Zyngier , Catalin Marinas , linux-kernel@vger.kernel.org, Steven Rostedt , Ingo Molnar , kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org List-Id: kvmarm@lists.cs.columbia.edu On Fri, Sep 23, 2016 at 05:07:10PM +0100, Will Deacon wrote: > On Tue, Sep 13, 2016 at 11:16:06AM +0100, Punit Agrawal wrote: > > From: Mark Rutland > > > > As with dsb() and isb(), add a __tlbi() helper so that we can avoid > > distracting asm boilerplate every time we want a TLBI. As some TLBI > > operations take an argument while others do not, some pre-processor is > > used to handle these two cases with different assembly blocks. > > > > The existing tlbflush.h code is moved over to use the helper. > > > > Signed-off-by: Mark Rutland > > Cc: Catalin Marinas > > Cc: Marc Zyngier > > [ rename helper to __tlbi, update comment and commit log ] > > Signed-off-by: Punit Agrawal > > Reviewed-by: Will Deacon > > --- > > arch/arm64/include/asm/tlbflush.h | 34 ++++++++++++++++++++++++++-------- > > 1 file changed, 26 insertions(+), 8 deletions(-) > > Given that this series seems to have stalled, I'm inclined to pick this > patch (and only this patch) up in the arm64 tree. It's a standalone tidy > up and means one fewer dependency next time round. Ok, I've picked this one up. Please re-post the rest of the series after the merge window. Will