From patchwork Thu Sep 29 11:42:19 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Alex_Benn=C3=A9e?= X-Patchwork-Id: 610613 Delivered-To: patch@linaro.org Received: by 2002:a17:522:c983:b0:460:3032:e3c4 with SMTP id kr3csp924989pvb; Thu, 29 Sep 2022 07:25:05 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5wqottFhqTUrWZqIa92cvqIOnyClqiow2f8srjYy18mTdx989muN9o9OhBut8tjtVe0h1e X-Received: by 2002:ac8:5755:0:b0:35b:ef17:fe08 with SMTP id 21-20020ac85755000000b0035bef17fe08mr2539364qtx.126.1664461505053; Thu, 29 Sep 2022 07:25:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1664461505; cv=none; d=google.com; s=arc-20160816; b=n7+pI/sQMfRbWj9VZ84WSr0fHt3jQhXmR2UgcoAsdc6VYf00OP9SQGB9g7sxSOk/JX zdXTD7TbDJlHUcCTr5FCOCNA7D6/fLFHuSigX/nrfElfl3++RgPXSvVdIEjZ4NneHilY NjhZSYWpKCw0erl+sg47R3GRlNQfmH09YckE8bEiLvco9FGPm0QKLmXP3opg8wwz8bl5 mloK0dhzzBwlCXII0fkB6B4bywFggZCo071QFKNISjTawlaVgGvDFUGleeQt7eIKEIm3 F0Qr4gkho2ZJwgxmHtu+lzMb1o+/tuz0mURQHCF3VIi+Kyf2BHUMwmUPiCVNTNx8Q94y wtTg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=XomK1kpXDqBk+Ebyq0kQyqmYRo/vpKvPg6SHFpAPqMU=; b=LA3VIsHgBuD2wZpWxhVW6Ovtv7ksdfnNJrPNh4bXbYV8D3YPcO/+tzWNHLN1jK+otZ +p6C1hkZZePV8F9mmuBeOk56uaV9JMKfW7vBIzwvJSqULhBx/AwK5MVCpGSjI9DqnAv4 zqF8QwKcVy5svbDGnZS2zOAnbRhCAz2y8rhzB33gMl2Efbs5xueY47wAmIGz7L+jfE/f QIY4P3aTfgfRdIlUjhLIHWrzXEZkQobHC8KoI2LITb4FYeRC8M3mn+cCp6Z9IRjLMGJE 2l+dd570RWpNhPfcrjt2T8+BtV5DhFgwLveXc82adWCQh6EFEllxeEdzJCYzq3FEr+wB Fg7w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=tIgI4mtO; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id f2-20020a0562141d2200b004aa9e041c9esi4939563qvd.186.2022.09.29.07.25.04 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Thu, 29 Sep 2022 07:25:05 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=tIgI4mtO; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1]:35820 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oduTI-0008Qv-I8 for patch@linaro.org; Thu, 29 Sep 2022 10:25:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:37756) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1odsD0-0007TH-89 for qemu-devel@nongnu.org; Thu, 29 Sep 2022 08:00:06 -0400 Received: from mail-wr1-x42e.google.com ([2a00:1450:4864:20::42e]:41516) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1odsCu-0004NU-Mq for qemu-devel@nongnu.org; Thu, 29 Sep 2022 08:00:05 -0400 Received: by mail-wr1-x42e.google.com with SMTP id r6so1814629wru.8 for ; Thu, 29 Sep 2022 04:59:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date; bh=XomK1kpXDqBk+Ebyq0kQyqmYRo/vpKvPg6SHFpAPqMU=; b=tIgI4mtOBVpLyM5P7GpZ8WrF7OVzbQbi1d4wBSIQs036078Y+fh68AvJm9shzXsemj /lKYWBWU0l1YNcUWWkXQUXztMjru1GXzQf/PYXGtQSxVlT9OZE8gII5m7eyjTbhmXDjY qHf5rcQ0JSgjBclL3nPgKsFj/v6rKUjNddTh9OlVPjGz+HdFdmLw8YACwkY/iNvHr7Ti ADghZWKhFnCg8WfveggGQrp2ZAV2NwZjJsacPXnoR4WZ5aLQM1g2sqqVPPHP5ESiabeU XQxc4aSEcJWufi+jxgQNGoR6p12pDDAyapcv0vliiJ53MWD9Mwyo+5a4B2GuiNLaqVsS XWnQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date; bh=XomK1kpXDqBk+Ebyq0kQyqmYRo/vpKvPg6SHFpAPqMU=; b=76dWCqfvj7VaEEpmQutsia50F29YnLz5j3qS8cXqndJvdbD2QFfPihs0dITZovKGih 5pO99RCRV6iQOuDq5XkEk1uuevprUyS89cAPvFVf2OhVeIdEo/88yahLjKVMeSl00RcK YkCmfmw7n1bcYdj4qMBqVx7g7bP9rlscwwkX1MTBoby4a+es7MTHQEcKGyXygsjf3JTi rLm9G1/zAzGb7q1t2BNqWk+PbyBFWiIAz8TVLGkPghKg/m3wRDPN2RfpBpZ52Ts2NONb kLT30kzrdbD27w5C+gMSBgFwL3QkLGr1/RBJdKOdf2jh20HW8OiaxnfGsXZkAQMBEWxp AwNQ== X-Gm-Message-State: ACrzQf1l/vpK/ZjIjabgdw+ZsZgvytrtXxRfmE/r11G6bx7nMr7q3lIQ vmz8gkw/NYXD69+QfA2Tcy2KcA== X-Received: by 2002:a05:6000:1f09:b0:22c:c6d9:5f42 with SMTP id bv9-20020a0560001f0900b0022cc6d95f42mr2065581wrb.84.1664452798085; Thu, 29 Sep 2022 04:59:58 -0700 (PDT) Received: from zen.linaroharston ([185.81.254.11]) by smtp.gmail.com with ESMTPSA id p14-20020adfce0e000000b0022af6c93340sm6579624wrn.17.2022.09.29.04.59.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Sep 2022 04:59:54 -0700 (PDT) Received: from zen.lan (localhost [127.0.0.1]) by zen.linaroharston (Postfix) with ESMTP id AD2371FFD8; Thu, 29 Sep 2022 12:42:35 +0100 (BST) From: =?utf-8?q?Alex_Benn=C3=A9e?= To: qemu-devel@nongnu.org Cc: fam@euphon.net, berrange@redhat.com, f4bug@amsat.org, aurelien@aurel32.net, pbonzini@redhat.com, stefanha@redhat.com, crosa@redhat.com, minyihh@uci.edu, ma.mandourr@gmail.com, Luke.Craig@ll.mit.edu, cota@braap.org, aaron@os.amperecomputing.com, kuhn.chenqun@huawei.com, robhenry@microsoft.com, mahmoudabdalghany@outlook.com, =?utf-8?q?Alex_Benn=C3=A9e?= , Alexandre Iooss , Richard Henderson Subject: [PATCH v1 39/51] contrib/plugins: reset skip when matching in execlog Date: Thu, 29 Sep 2022 12:42:19 +0100 Message-Id: <20220929114231.583801-40-alex.bennee@linaro.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220929114231.583801-1-alex.bennee@linaro.org> References: <20220929114231.583801-1-alex.bennee@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::42e; envelope-from=alex.bennee@linaro.org; helo=mail-wr1-x42e.google.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" The purpose of the matches was to only track the execution of instructions we care about. Without resetting skip to the value at the start of the block we end up dumping all instructions after the match with the consequent load on the instrumentation. Signed-off-by: Alex Bennée Cc: Alexandre Iooss Reviewed-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20220921160801.1490125-10-alex.bennee@linaro.org> --- v2 - drop the ? bool ternary for skip --- contrib/plugins/execlog.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/contrib/plugins/execlog.c b/contrib/plugins/execlog.c index e659ac9cbb..1b3bb7ebba 100644 --- a/contrib/plugins/execlog.c +++ b/contrib/plugins/execlog.c @@ -88,7 +88,7 @@ static void vcpu_insn_exec(unsigned int cpu_index, void *udata) static void vcpu_tb_trans(qemu_plugin_id_t id, struct qemu_plugin_tb *tb) { struct qemu_plugin_insn *insn; - bool skip = (imatches || amatches) ? true : false; + bool skip = (imatches || amatches); size_t n = qemu_plugin_tb_n_insns(tb); for (size_t i = 0; i < n; i++) { @@ -147,6 +147,9 @@ static void vcpu_tb_trans(qemu_plugin_id_t id, struct qemu_plugin_tb *tb) /* Register callback on instruction */ qemu_plugin_register_vcpu_insn_exec_cb(insn, vcpu_insn_exec, QEMU_PLUGIN_CB_NO_REGS, output); + + /* reset skip */ + skip = (imatches || amatches); } }