Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751302AbdH2SyY (ORCPT ); Tue, 29 Aug 2017 14:54:24 -0400 Received: from mail.efficios.com ([167.114.142.141]:54243 "EHLO mail.efficios.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751186AbdH2SyX (ORCPT ); Tue, 29 Aug 2017 14:54:23 -0400 Date: Tue, 29 Aug 2017 18:55:37 +0000 (UTC) From: Mathieu Desnoyers To: Max Filippov Cc: "Paul E. McKenney" , linux-kernel , Peter Zijlstra , Chris Zankel , linux-xtensa@linux-xtensa.org Message-ID: <1616508443.22335.1504032937518.JavaMail.zimbra@efficios.com> In-Reply-To: References: <20170827213659.29210-1-mathieu.desnoyers@efficios.com> Subject: Re: [PATCH] Fix: xtensa: add missing sync_core 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: xtensa: add missing sync_core Thread-Index: vkIyR2lMKEWtHBuJjfTzDtwytYX4bQ== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1039 Lines: 39 ----- On Aug 28, 2017, at 1:12 PM, Max Filippov jcmvbkbc@gmail.com wrote: > Hi Mathieu, > > On Mon, Aug 28, 2017 at 12:36 AM, Mathieu Desnoyers > wrote: >> The membarrier system call now requires all architectures to implement >> sync_core(). On Xtensa, it is provided by the EXTW instruction. >> >> [ Completely untested! Can someone on the xtensa side confirm whether >> EXTW is the right way to serialize core execution and try it out ? ] > > Thanks for the patch. I'm currently travelling, I'll give it a try next week > once I'm back at work. Hi Max, I think we may need to flush the icache to make it consistent with the dcache too on xtensa, in addition to the EXTW. The goal here is to allow JIT engines to reclaim and re-use memory after they discard dynamically generated code. This is similar to what we'd need to do on arm32, where they have inconsistent d/i-caches. Thoughts ? Thanks, Mathieu > > -- > Thanks. > -- Max -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com