From patchwork Thu Oct 18 19:00:26 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Vivier X-Patchwork-Id: 149189 Delivered-To: patch@linaro.org Received: by 2002:a2e:8595:0:0:0:0:0 with SMTP id b21-v6csp2365370lji; Thu, 18 Oct 2018 12:08:15 -0700 (PDT) X-Google-Smtp-Source: ACcGV63ETicjfr7C/Rbpd84VK2585I01/btWVxuTD0mpwFCfTTz5a+3sDZgulBT0FtV1HgEWk0ke X-Received: by 2002:a37:ab12:: with SMTP id u18-v6mr29436835qke.305.1539889695862; Thu, 18 Oct 2018 12:08:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1539889695; cv=none; d=google.com; s=arc-20160816; b=j1bb2rk7EoBDKrqHitQNH0UYJ2gKHMIXMeIQ5l798bkwWIAu989uuwVBW3BF/RCp/T txYDS48+eiQ4azeOZmQPU7F/y6kItU3UzpMcxeKGYkKkZrvXG+srjb0YPn3EfMAud4wm kjtSA1U2TtWaOhESh7ByhXUqyqkkecB/XByADkdZ2BMmgoJrflLynyGOhwdnnmYtx/fX EA0gDHb4AbiHg7/w4jmUpM6oLMUQrj12kyhAKuMRbpCqIkstX7uipc0XNI2efEPhgVUd SWXutOZULF7OS7O5zsKP3KAHw9p0L/BFnRyb1LqhtDzod3xoDJmu2R/rBoB46Rszq/g4 bwfA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:to:from; bh=1RE2WpzXcE5pNNtbde+Ss5GzLC6R5/wTWZzbs+tttV0=; b=RyG3c1MwDRAK3c2+z29qqoPRVrxlK6X+KQuSAhs+Xvsxh73xfDKDdQ8omCYC9ORNbK F/rqUWXL85fbwp27T840kMzCFzId27uM4Vkc8b7Cr27yVuTReN6xP83LwEjaC9SKw27D ufRY9qLCOBoDXOQsvbGssBnjSQ6sTcNl34RRytwycpRxuyuxLqeo2ySUFG3SPfHiAJL5 bLBkIvJaqWgIiHnvaIvtH5TdqN2hVGRWJ5hq7mLyIlIFc7RJwrDqgtbMMtFvjCUZ57c6 gSiJNJBFqMYc4sdhhpk0pNX01o3/+nIqpFau/swYhBXla2cEEG3JaApq8pHsWwSiz5LR lthg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 2001:4830:134:3::11 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org" Return-Path: Received: from lists.gnu.org (lists.gnu.org. [2001:4830:134:3::11]) by mx.google.com with ESMTPS id r17si2355000qvj.164.2018.10.18.12.08.15 for (version=TLS1 cipher=AES128-SHA bits=128/128); Thu, 18 Oct 2018 12:08:15 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 2001:4830:134:3::11 as permitted sender) client-ip=2001:4830:134:3::11; Authentication-Results: mx.google.com; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 2001:4830:134:3::11 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org" Received: from localhost ([::1]:43999 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gDDeZ-0001RK-3Q for patch@linaro.org; Thu, 18 Oct 2018 15:08:15 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47800) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gDDXK-0004CA-RV for qemu-devel@nongnu.org; Thu, 18 Oct 2018 15:00:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gDDXG-00007n-QF for qemu-devel@nongnu.org; Thu, 18 Oct 2018 15:00:46 -0400 Received: from mout.kundenserver.de ([217.72.192.73]:34745) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gDDXG-000057-FQ for qemu-devel@nongnu.org; Thu, 18 Oct 2018 15:00:42 -0400 Received: from localhost.localdomain ([78.238.229.36]) by mrelayeu.kundenserver.de (mreue108 [212.227.15.183]) with ESMTPSA (Nemesis) id 1Mirb8-1fjMxQ0668-00ex0R; Thu, 18 Oct 2018 21:00:32 +0200 Received: from localhost.localdomain ([78.238.229.36]) by mrelayeu.kundenserver.de (mreue108 [212.227.15.183]) with ESMTPSA (Nemesis) id 1Mirb8-1fjMxQ0668-00ex0R; Thu, 18 Oct 2018 21:00:32 +0200 From: Laurent Vivier To: qemu-devel@nongnu.org Date: Thu, 18 Oct 2018 21:00:26 +0200 Message-Id: <20181018190026.9494-3-laurent@vivier.eu> X-Mailer: git-send-email 2.17.2 In-Reply-To: <20181018190026.9494-1-laurent@vivier.eu> References: <20181018190026.9494-1-laurent@vivier.eu> X-Provags-ID: V03:K1:YwyU+Bw/KbiaikvW1DWuHd9VpdZr8UD1gGXGoUyuYP8ckuEZk5+ Pb3l8bbYros03nWaBtrNGvtCdKxET4oabo20tAzh6wphnxHD1IP8loSDKi68T27+Eo7U9kG +pq2RWCOOzszQO/3QZfb+atvvSEbiAtBHpJINaCTVoHMMZpseBexrJQUWXR04nEWpJ2KDs0 e9mnGObwbbSiCNCTWgvcQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:lDYoMkibq2o=:1dySA6yYX6itqIEuDBElwO l1X1YMMMbthw5RsvjIDT2loxeUGORdnsWG35oBzX9XHK+1VrwZG65nmu9AuKCHjExWxTz0k0h deL5tdKeQcfRVlc1fYKaO3sIv57DJM+4b8YwA5QS4MagKLQQZx13mcJ5aKL0CsWCm64HRTemL cJATPXmRT5DDWhBosqx0zrqoI06ksPqozjMardAF2isNuV4PO1HR/mPa8I5aWsQrFjqBAL72O 2NTIuAYj0pI177cqNNq0c5RosNptUBU7ut4kgSumfBU3iWooaK6YmBTvKQzrIRAzR0JQhLsLb Ej/6i7BriwFVWKjgW/bqhbBNutAW6ztmcLcA6mq7JlVtpYQXFYpsIh4BkzlO8qDPKNiqCD7qX WIuceyn117j/LBAKcGJIJq1AvU+UbNUjxD5Fq9P9S+rOwJr8/1M1xBh4cic1RuK1/DQu+KhzY qssm0sOTwEN/Ua5Nj2uKOE5uRY4u7Bw2T5WHVI3QuWR+vCOKEsk0vi70CBhgNoi6uj/FfR4Vb 9ofjN15LvKC/qhOkyVzy/Foq0zkNxRZCpoIV/9pN77EkllkuiqiMOpaOmfvxu8Gd9RKEjc3FC xGYjUnANxUqq9UFhb3w7KuOCdtZCLbrfshD1I08QzT6cNay2gpK5YEzzZxQ4ytZCJPw6WCRJa Q0GqQxvzRDKZx8HNVjvl3TTkgOczWEo1NjCDS+X9mwuOb17WCv7mjwYOJgk676xgNlQo= X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 217.72.192.73 Subject: [Qemu-devel] [2 2/2] linux-user/sparc/signal.c: Remove unnecessary comment X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Riku Voipio , Laurent Vivier Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" From: Peter Maydell Remove a comment suggesting that we need to call tb_flush() after writing the SPARC signal frame trampoline insns. This isn't necessary in QEMU, because (even if the guest architecture requires explicit icache maintenance) we ensure that memory writes result in invalidation of translated code from that memory. Signed-off-by: Peter Maydell Message-Id: <20181009184017.15675-1-peter.maydell@linaro.org> Signed-off-by: Laurent Vivier --- linux-user/sparc/signal.c | 4 ---- 1 file changed, 4 deletions(-) -- 2.17.2 diff --git a/linux-user/sparc/signal.c b/linux-user/sparc/signal.c index b4c60aa446..e44e99993c 100644 --- a/linux-user/sparc/signal.c +++ b/linux-user/sparc/signal.c @@ -258,10 +258,6 @@ void setup_frame(int sig, struct target_sigaction *ka, __put_user(val32, &sf->insns[1]); if (err) goto sigsegv; - - /* Flush instruction space. */ - // flush_sig_insns(current->mm, (unsigned long) &(sf->insns[0])); - // tb_flush(env); } unlock_user(sf, sf_addr, sizeof(struct target_signal_frame)); return;