From nobody Thu May 16 04:50:59 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=linaro.org ARC-Seal: i=1; a=rsa-sha256; t=1633978478; cv=none; d=zohomail.com; s=zohoarc; b=aOWkP2IoSHyGYVNXyvQh0Ahu++tfw9IV1pSaly01vtTHlmR7F6X5QY6ykIgCFt9dz00BRELqKubsi7Ie+XsZQuoFQEW2sreg+BKQn3KfaThC/CKJn+n6klEvxyIIA80QdaTSl/P7igKaqpNdGHAFy0w5xdLqt7ijGoOmqrJtGwc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1633978478; h=Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=JyzFYnmTTZxRUyEWhU0BXDzTpJ0CMo8xjc8mo8AYFj0=; b=aYGfnYbsD0dgjxHp0Vn8en8zbTc9C8VBy7dHKt2cZ+GEp2a2HeDFgXmiDviLwk9wxaCaq6M0xjcapTSbrzd4bBMfjOEbdoxxhx+hA8OIZjzZdgeMGNd7SejB5HdPI69mLgtYO+gpNqLMrjpPFTCA7iuH+FdRLfMPUCg+Uht3wi8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1633978478482318.2636613321423; Mon, 11 Oct 2021 11:54:38 -0700 (PDT) Received: from localhost ([::1]:56308 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ma0RZ-0004iW-4J for importer@patchew.org; Mon, 11 Oct 2021 14:54:37 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42162) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ma0Qc-0003i7-FH for qemu-devel@nongnu.org; Mon, 11 Oct 2021 14:53:38 -0400 Received: from mail-pj1-x102f.google.com ([2607:f8b0:4864:20::102f]:43630) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1ma0Qa-0004un-F6 for qemu-devel@nongnu.org; Mon, 11 Oct 2021 14:53:37 -0400 Received: by mail-pj1-x102f.google.com with SMTP id k23-20020a17090a591700b001976d2db364so670345pji.2 for ; Mon, 11 Oct 2021 11:53:35 -0700 (PDT) Received: from localhost.localdomain ([71.212.134.125]) by smtp.gmail.com with ESMTPSA id 130sm8312256pfz.77.2021.10.11.11.53.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Oct 2021 11:53:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=JyzFYnmTTZxRUyEWhU0BXDzTpJ0CMo8xjc8mo8AYFj0=; b=X803CuRAk/0LB3yHJ4riOFOXqCQgf8lrfa5ogRwZgM8qYJEPzg5cuNFX7MxreHXk0R zUfMMLlMnYkLZjt7QOnthqd3dwYfkpa9TGSEmBTNfTVaJrazuunWxkgtLq1zPE3IF3R0 78YL6TR1YBlGKM8XsvoRmILWI4F2PUulnrXBl1kHacjLoK8130VB9pjO3NQFR0pxZ17Z r6bPdZFZZdkWslvTGzeuQkTzc2auNMmMHgOIkTekk0s9auEcFuP0Yp+fuZfSjuKRN3jn VGeOY8EyScxCPHDsWvdD7aJVAhuzhDhgwFF4Wz/oIW26j256y5PJQ7SKwCbBCDXe7O6l ABdQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=JyzFYnmTTZxRUyEWhU0BXDzTpJ0CMo8xjc8mo8AYFj0=; b=GHzS11r8LDXAZtlW7z+X7GppKIXBfNWmydpmFvnEOO0QjzLbrMKGe8ZY3gFHHs8yVY saXOnfM2AwcLitKReY0c/mxFMgCdYTQQEKQkrfZ2sgsmnIVcYTqUiTzQ2FA6srTvt0iC zc/2UKT/qXdxPgi1PaKQc8LtNp3bxB5cQFW5TX2ranFSIeqPwHGYp8fAJkjoWW2Afk2s VYHf77vhMkRy46jStJJUqoybNmPCMOu09i85itw+T1ZNrct5l7A2UFsv4LP2yfq5zoP7 FQEwU3zbyMVfYGGnX7GP3Y7iI5ZolaKItQYFvhLwkj/zS40WM71sWQ43a9e1v5QWovRd +4lA== X-Gm-Message-State: AOAM533BVFSnJ7tOgWRki2psL3aofjPgRlQQecQO8tcIdihCPw0/l3ej UWlPKNQ3CwhS2LHoYxR1An8IohTD8AQtmw== X-Google-Smtp-Source: ABdhPJy6I1Ru5PWSNllCDhRkV32N7AS5D1tVRK77P6l64lIjoTSzxzUps9AGpalDI35XRRXspq6RGQ== X-Received: by 2002:a17:90b:4c0f:: with SMTP id na15mr792389pjb.200.1633978414588; Mon, 11 Oct 2021 11:53:34 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Subject: [PATCH] target/s390x: move tcg_gen_insn_start to s390x_tr_insn_start Date: Mon, 11 Oct 2021 11:53:32 -0700 Message-Id: <20211011185332.166763-1-richard.henderson@linaro.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2607:f8b0:4864:20::102f; envelope-from=richard.henderson@linaro.org; helo=mail-pj1-x102f.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.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alex.bennee@linaro.org Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @linaro.org) X-ZM-MESSAGEID: 1633978480457100001 Content-Type: text/plain; charset="utf-8" We use INDEX_op_insn_start to make the start of instruction boundaries. If we don't do it in the .insn_start hook things get confused especially now plugins want to use that marking to identify the start of instructions and will bomb out if it sees instrumented ops before the first instruction boundary. Signed-off-by: Richard Henderson Reviewed-by: Alex Benn=C3=A9e --- target/s390x/tcg/translate.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/target/s390x/tcg/translate.c b/target/s390x/tcg/translate.c index f284870cd2..a2d6fa5cca 100644 --- a/target/s390x/tcg/translate.c +++ b/target/s390x/tcg/translate.c @@ -138,6 +138,7 @@ struct DisasFields { struct DisasContext { DisasContextBase base; const DisasInsn *insn; + TCGOp *insn_start; DisasFields fields; uint64_t ex_value; /* @@ -6380,8 +6381,8 @@ static DisasJumpType translate_one(CPUS390XState *env= , DisasContext *s) /* Search for the insn in the table. */ insn =3D extract_insn(env, s); =20 - /* Emit insn_start now that we know the ILEN. */ - tcg_gen_insn_start(s->base.pc_next, s->cc_op, s->ilen); + /* Update insn_start now that we know the ILEN. */ + tcg_set_insn_start_param(s->insn_start, 2, s->ilen); =20 /* Not found means unimplemented/illegal opcode. */ if (insn =3D=3D NULL) { @@ -6552,6 +6553,11 @@ static void s390x_tr_tb_start(DisasContextBase *db, = CPUState *cs) =20 static void s390x_tr_insn_start(DisasContextBase *dcbase, CPUState *cs) { + DisasContext *dc =3D container_of(dcbase, DisasContext, base); + + /* Delay the set of ilen until we've read the insn. */ + tcg_gen_insn_start(dc->base.pc_next, dc->cc_op, 0); + dc->insn_start =3D tcg_last_op(); } =20 static void s390x_tr_translate_insn(DisasContextBase *dcbase, CPUState *cs) --=20 2.25.1