From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 88BCE384D1B5 for ; Mon, 13 Jun 2022 16:16:06 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 88BCE384D1B5 Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-287-L4Fce1KkOtaKAZXXUsV-vw-1; Mon, 13 Jun 2022 12:16:05 -0400 X-MC-Unique: L4Fce1KkOtaKAZXXUsV-vw-1 Received: by mail-qt1-f198.google.com with SMTP id r9-20020a05622a034900b00304ea5d41ddso4680236qtw.20 for ; Mon, 13 Jun 2022 09:16:05 -0700 (PDT) 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:in-reply-to :references:mime-version:content-transfer-encoding; bh=LqlomOlwLLcfhl7Z16ZrZ6qlNSyGXnZjFUUJNGrtTls=; b=UPRa1NZhIViTZpnsKx5ACYjmtlYJMCrPG1kJm8XEhMyeUL07+VlGBDY1679eYJ6r0Z VypvxgH1DCQvLDxxzlxcw+tbOJxR8AJyFwSENc0JpA//ayQ564nptLXyPsjAg7IrgNGS dpLviFMRph3XmxyVJAo/1M1WWLaQaDh94/VNT2KgcCXHcI7kgV3vq7gsB1Vpj8cBJOEm xCsCfgfq5aBxWQ9Z4j6dpgtabLV2Dd8JWH41rmvqY0Arw5o2XDs4BPx4Alu9O80SQL8u kPY/gZKgPUxnd8WAtG5iamz0Jm59Ljio0WUWb4quEs/m/sX0HGHmN1aE6AJ6AA3NZYEj 5JJQ== X-Gm-Message-State: AOAM530HNjEBa1e7rsm9z79jSQ0bY8F/cR79gOW/L8mwwsVC2lseZ8Gs /Ug2D6XZvn1HJMDsT6kevqqs/w/pCRBc3KxMCuOQkWkKrYN7z0QQ9kBtAGMqD0pjMRD8XIdzPoG Qmp8om023J3sK62ad8yt6PVTK3QfYn6QBbWe6NwbYkOxh1NCBby7QfAqdruXkJ0Ak5GXjrhCf3w == X-Received: by 2002:a05:620a:468f:b0:6a7:24a0:84f4 with SMTP id bq15-20020a05620a468f00b006a724a084f4mr546590qkb.285.1655136964149; Mon, 13 Jun 2022 09:16:04 -0700 (PDT) X-Google-Smtp-Source: ABdhPJymoBOGemk5tow4LYZtCR3as+ZIrAkyVN5e8/762iMKMTjV+5YPFdHtW7VA9E7SOEqi5yI6/A== X-Received: by 2002:a05:620a:468f:b0:6a7:24a0:84f4 with SMTP id bq15-20020a05620a468f00b006a724a084f4mr546552qkb.285.1655136963618; Mon, 13 Jun 2022 09:16:03 -0700 (PDT) Received: from localhost (host109-154-20-145.range109-154.btcentralplus.com. [109.154.20.145]) by smtp.gmail.com with ESMTPSA id p17-20020a05622a13d100b00304e95ec0fbsm5359574qtk.89.2022.06.13.09.16.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Jun 2022 09:16:03 -0700 (PDT) From: Andrew Burgess To: gdb-patches@sourceware.org Cc: Luis Machado , Andrew Burgess Subject: [PATCHv3 2/6] gdb/mips: rewrite show_mask_address Date: Mon, 13 Jun 2022 17:15:50 +0100 Message-Id: <9d760932018624c0d141b15a6cb286bf6740b61a.1655136816.git.aburgess@redhat.com> X-Mailer: git-send-email 2.25.4 In-Reply-To: References: MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII"; x-default=true X-Spam-Status: No, score=-12.5 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 13 Jun 2022 16:16:07 -0000 This commit is similar to the previous commit, but in this case GDB is actually relying on undefined behaviour. Consider building GDB for all targets on x86-64/GNU-Linux, then doing this: (gdb) show mips mask-address Zeroing of upper 32 bits of 64-bit addresses is auto. The 32 bit address mask is set automatically. Currently disabled (gdb) The 'show mips mask-address' command ends up in show_mask_address in mips-tdep.c, and this function does this: mips_gdbarch_tdep *tdep = (mips_gdbarch_tdep *) gdbarch_tdep (target_gdbarch ()); Later we might pass TDEP to mips_mask_address_p. However, in my example above, on an x86-64 native target, the current target architecture will be an x86-64 gdbarch, and the tdep field within the gdbarch will be of type i386_gdbarch_tdep, not of type mips_gdbarch_tdep, as a result the cast above was incorrect, and TDEP is not pointing at what it thinks it is. I also think the current output is a little confusing, we appear to have two lines that show the same information, but using different words. The first line comes from calling deprecated_show_value_hack, while the second line is printed directly from show_mask_address. However, both of these lines are printing the same mask_address_var value. I don't think the two lines actually adds any value here. Finally, none of the text in this function is passed through the internationalisation mechanism. It would be nice to remove another use of deprecated_show_value_hack if possible, so this commit does a complete rewrite of show_mask_address. After this commit the output of the above example command, still on my x86-64 native target is: (gdb) show mips mask-address Zeroing of upper 32 bits of 64-bit addresses is "auto" (current architecture is not MIPS). The 'current architecture is not MIPS' text is only displayed when the current architecture is not MIPS. If the architecture is mips then we get the more commonly seen 'currently "on"' or 'currently "off"', like this: (gdb) set architecture mips The target architecture is set to "mips". (gdb) show mips mask-address Zeroing of upper 32 bits of 64-bit addresses is "auto" (currently "off"). (gdb) All the text is passed through the internationalisation mechanism, and we only call gdbarch_tdep when we know the gdbarch architecture is bfd_arch_mips. --- gdb/mips-tdep.c | 37 +++++++++++++++++-------------------- 1 file changed, 17 insertions(+), 20 deletions(-) diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c index 805c5beba59..fc1cb46de84 100644 --- a/gdb/mips-tdep.c +++ b/gdb/mips-tdep.c @@ -1183,28 +1183,25 @@ static void show_mask_address (struct ui_file *file, int from_tty, struct cmd_list_element *c, const char *value) { - mips_gdbarch_tdep *tdep - = (mips_gdbarch_tdep *) gdbarch_tdep (target_gdbarch ()); - - deprecated_show_value_hack (file, from_tty, c, value); - switch (mask_address_var) + const char *additional_text = ""; + if (mask_address_var == AUTO_BOOLEAN_AUTO) { - case AUTO_BOOLEAN_TRUE: - gdb_printf (file, "The 32 bit mips address mask is enabled\n"); - break; - case AUTO_BOOLEAN_FALSE: - gdb_printf (file, "The 32 bit mips address mask is disabled\n"); - break; - case AUTO_BOOLEAN_AUTO: - gdb_printf - (file, - "The 32 bit address mask is set automatically. Currently %s\n", - mips_mask_address_p (tdep) ? "enabled" : "disabled"); - break; - default: - internal_error (__FILE__, __LINE__, _("show_mask_address: bad switch")); - break; + if (gdbarch_bfd_arch_info (target_gdbarch ())->arch != bfd_arch_mips) + additional_text = _(" (current architecture is not MIPS)"); + else + { + mips_gdbarch_tdep *tdep + = (mips_gdbarch_tdep *) gdbarch_tdep (target_gdbarch ()); + + if (mips_mask_address_p (tdep)) + additional_text = _(" (currently \"on\")"); + else + additional_text = _(" (currently \"off\")"); + } } + + gdb_printf (file, _("Zeroing of upper 32 bits of 64-bit addresses is \"%s\"%s.\n"), + value, additional_text); } /* Tell if the program counter value in MEMADDR is in a standard ISA -- 2.25.4