From patchwork Sun Dec 29 17:23:36 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg KH X-Patchwork-Id: 182573 Delivered-To: patch@linaro.org Received: by 2002:a92:815a:0:0:0:0:0 with SMTP id e87csp5478004ild; Sun, 29 Dec 2019 10:07:50 -0800 (PST) X-Google-Smtp-Source: APXvYqxr/v8oq/h28O7v0KbDAWZnQoZ1mfYRjG0cljQCFFhc4ijkYiiQ/dh3bMYOou0/ujNLXD/9 X-Received: by 2002:a9d:624e:: with SMTP id i14mr70993143otk.371.1577642869937; Sun, 29 Dec 2019 10:07:49 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1577642869; cv=none; d=google.com; s=arc-20160816; b=oI9Kwpmzwn1bqw1j2roc3QVuh7ZDbMZ347NYsfv+xNhizyhtLNhwdAkabXHKcyzh4b 3s1QltOdDMybhy5tKiR8PpBS886zEVmSMEQyf+BVFQK/qHXZeruhhO1Zd+eBn/ogIp2E MLJ3tK4WIsLcxxBfv/98WDcrXoT3mRruZ/DfipWfD9c5zAuvqqw97IqjnksZWS3LgPaV 6pT0emsK6zkhNcGXK1mULV46HHYOfEGZW3BMZXgYSX9P1ET7+hy7n7XV6+vnFgpxSRwK B3s4zHv4f+Gx5s+EGoJFzzKZT/okKorhlwvceXuM1WRxgsUMvLkOalds39ywhLXuRphG tXig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=u+lkb5ejHcDN1yY1zrZR+XEVc6Gi8a1qnm9U5XTPirE=; b=OSW9USCilkhx6FSPsZCLAT+Ws3EqvljqUjWn3ZlePWCypnb+RRC/3H4DEVy+OJr/FB UQhEBApwsKTywtfGPovcU0Zxgc7KKN0plgCu1v67gW1c2hJIKc8ud+3taFsxXqPOn0x8 VX1EWfeWQQaMZ4PHjnU3+BC+I6LxnSv2O0GfJ7cA98bDJDQCkcSVjBpODRqBEHMvn9WP blBj50XpRP6N28L0eo3KQN+WDJQEPh736Esz2b1x4Fv1JrXUk3f2PSHlK4CqzNLtM3up 4U4vBvyu59k5Rx2bmKyztdQkqPDXVgCtLcYfoRGE6SwpG+/YRpl9UL2s8eGErx8++k22 SB3g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=Ehgtk1xh; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y22si22143636oti.269.2019.12.29.10.07.49; Sun, 29 Dec 2019 10:07:49 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=Ehgtk1xh; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731363AbfL2Rr4 (ORCPT + 27 others); Sun, 29 Dec 2019 12:47:56 -0500 Received: from mail.kernel.org ([198.145.29.99]:58792 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731019AbfL2Rrx (ORCPT ); Sun, 29 Dec 2019 12:47:53 -0500 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (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 466E4206A4; Sun, 29 Dec 2019 17:47:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1577641672; bh=mye650C+sF4IBOpYtJXiGOLwPWidUGX5Vqex7cB2FE4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Ehgtk1xhbWs2mIBPPX8vkYRnLSwPenGUexSg2P1E/rMWnTKToJ9dijwlCuikZP9e/ CLrmLsSz+q3eAXdGQSBZi0t9etTL9Z/3MwYjM8AlHNcvwzN7R7SLfvGX79YMN++0cC 8Ln0uCHfHMg8lGM3sPkZQTL8ugmvC9wpAiPvF9FY= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Leo Yan , Adrian Hunter , Alexander Shishkin , Brajeswar Ghosh , Florian Fainelli , Jiri Olsa , Mark Rutland , Michael Petlan , Namhyung Kim , Peter Zijlstra , Song Liu , Souptick Joarder , Will Deacon , Arnaldo Carvalho de Melo , Sasha Levin Subject: [PATCH 5.4 163/434] perf tests: Disable bp_signal testing for arm64 Date: Sun, 29 Dec 2019 18:23:36 +0100 Message-Id: <20191229172712.608153205@linuxfoundation.org> X-Mailer: git-send-email 2.24.1 In-Reply-To: <20191229172702.393141737@linuxfoundation.org> References: <20191229172702.393141737@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Leo Yan [ Upstream commit 6a5f3d94cb69a185b921cb92c39888dc31009acb ] As there are several discussions for enabling perf breakpoint signal testing on arm64 platform: arm64 needs to rely on single-step to execute the breakpointed instruction and then reinstall the breakpoint exception handler. But if we hook the breakpoint with a signal, the signal handler will do the stepping rather than the breakpointed instruction, this causes infinite loops as below: Kernel space | Userspace ---------------------------------|-------------------------------- | __test_function() -> hit | breakpoint breakpoint_handler() | `-> user_enable_single_step() | do_signal() | | sig_handler() -> Step one | instruction and | trap to kernel single_step_handler() | `-> reinstall_suspended_bps() | | __test_function() -> hit | breakpoint again and | repeat up flow infinitely As Will Deacon mentioned [1]: "that we require the overflow handler to do the stepping on arm/arm64, which is relied upon by GDB/ptrace. The hw_breakpoint code is a complete disaster so my preference would be to rip out the perf part and just implement something directly in ptrace, but it's a pretty horrible job". Though Will commented this on arm architecture, but the comment also can apply on arm64 architecture. For complete information, I searched online and found a few years back, Wang Nan sent one patch 'arm64: Store breakpoint single step state into pstate' [2]; the patch tried to resolve this issue by avoiding single stepping in signal handler and defer to enable the signal stepping when return to __test_function(). The fixing was not merged due to the concern for missing to handle different usage cases. Based on the info, the most feasible way is to skip Perf breakpoint signal testing for arm64 and this could avoid the duplicate investigation efforts when people see the failure. This patch skips this case on arm64 platform, which is same with arm architecture. [1] https://lkml.org/lkml/2018/11/15/205 [2] https://lkml.org/lkml/2015/12/23/477 Signed-off-by: Leo Yan Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Brajeswar Ghosh Cc: Florian Fainelli Cc: Jiri Olsa Cc: Mark Rutland Cc: Michael Petlan Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Song Liu Cc: Souptick Joarder Cc: Will Deacon Link: http://lore.kernel.org/lkml/20191018085531.6348-3-leo.yan@linaro.org Signed-off-by: Arnaldo Carvalho de Melo Signed-off-by: Sasha Levin --- tools/perf/tests/bp_signal.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) -- 2.20.1 diff --git a/tools/perf/tests/bp_signal.c b/tools/perf/tests/bp_signal.c index c1c2c13de254..166f411568a5 100644 --- a/tools/perf/tests/bp_signal.c +++ b/tools/perf/tests/bp_signal.c @@ -49,14 +49,6 @@ asm ( "__test_function:\n" "incq (%rdi)\n" "ret\n"); -#elif defined (__aarch64__) -extern void __test_function(volatile long *ptr); -asm ( - ".globl __test_function\n" - "__test_function:\n" - "str x30, [x0]\n" - "ret\n"); - #else static void __test_function(volatile long *ptr) { @@ -302,10 +294,15 @@ bool test__bp_signal_is_supported(void) * stepping into the SIGIO handler and getting stuck on the * breakpointed instruction. * + * Since arm64 has the same issue with arm for the single-step + * handling, this case also gets suck on the breakpointed + * instruction. + * * Just disable the test for these architectures until these * issues are resolved. */ -#if defined(__powerpc__) || defined(__s390x__) || defined(__arm__) +#if defined(__powerpc__) || defined(__s390x__) || defined(__arm__) || \ + defined(__aarch64__) return false; #else return true;