From nobody Sat Feb 7 12:11:39 2026 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=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1661935469; cv=none; d=zohomail.com; s=zohoarc; b=gLyGIKFqjNXx9C3OY7Om4beU6MRe4PBqwyBSj+KMyARyKnivVEbj736hzmPhLlUhV7FljYCqdVMajis7uJ4KrLKqU/RFpoColeZS5gHOW72e4wAd8JrdOP9KPY9V2tOd4BeI29hx1Hw4fs8EvobPzhE6MvAsFlrARtTtjOvGPQU= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1661935469; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=3a8deZn4sRTDuQ2LfrIKu69B0XmGRcZFz5ntLrrTeuM=; b=EVn3JR4R7i+sG60GjaSNJ5x0jdihysDHxoOH7m7BtoabRyiFP5DrdY2wPyMSbQp0MuFqKedlCMVnlkZkDJp19JJpSzduhPNfiR5SrUq60XnBnd1BgAbQirbOoqje0FmFIPCYXv++i+Dz2FMkp1RIy0mMdMvgiHmn7y41W6E2wQY= 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 1661935469149487.5650544264206; Wed, 31 Aug 2022 01:44:29 -0700 (PDT) Received: from localhost ([::1]:39986 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oTJKk-0004xR-8x for importer@patchew.org; Wed, 31 Aug 2022 04:44:26 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:38582) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oTJJQ-0003aE-Nd for qemu-devel@nongnu.org; Wed, 31 Aug 2022 04:43:04 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]:37963) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oTJJN-0007tj-Si for qemu-devel@nongnu.org; Wed, 31 Aug 2022 04:43:04 -0400 Received: from mail-wm1-f69.google.com (mail-wm1-f69.google.com [209.85.128.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-7-QO1IcIHhPSqVBMriAEkprg-1; Wed, 31 Aug 2022 04:41:30 -0400 Received: by mail-wm1-f69.google.com with SMTP id b16-20020a05600c4e1000b003a5a47762c3so7967015wmq.9 for ; Wed, 31 Aug 2022 01:41:30 -0700 (PDT) Received: from localhost ([31.111.84.229]) by smtp.gmail.com with ESMTPSA id f6-20020a7bc8c6000000b003a35ec4bf4fsm1398779wml.20.2022.08.31.01.41.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Aug 2022 01:41:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1661935381; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=3a8deZn4sRTDuQ2LfrIKu69B0XmGRcZFz5ntLrrTeuM=; b=NIn5cmYa03jZd6YqFGIdhYFwKscYZ/PmCLA8eCo5XP9jZIbPvK5+cEvx+YO3QB915M8pZK W8DTTghglUIuRSY0tKkbpuopADXLxxMTiC/F2erWpYCDgosLlKgaM9U9haSkEmgvArVbQv wZ+v7lWu+uCLy8rq0kfykQjl51wZBYk= X-MC-Unique: QO1IcIHhPSqVBMriAEkprg-1 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=3a8deZn4sRTDuQ2LfrIKu69B0XmGRcZFz5ntLrrTeuM=; b=S9LFO+I999ueF8Dk2NolRobEo1FQB0IECuVPL3IvqPB/n4rLlwyei8nayMb2WNnhkg FxyDyJrvPgAwdn4chiMY7mfGGoW2ftwouRxlzD9FHZ344ixCRtZU2Pd176RTBsL4A0z/ gkFSKQTZcJOG+3K+LMusryy4d4jTq4IdccnqgX6KjmfLsAyNuDlSC/W4hsv4OVdzyj2y 1NerhP+NGxdSHX+rfUVfd0+K62BM3907KKojyHMV+RZYsf95FbVUBkG4Ph6m4TCpKKQV kZ98E8GZQqhr56UGzn75QGnz5sBQ49yZrETN1KalhJ7o6jk1w9TYHiPEK0k4zuhKJnsc nVrQ== X-Gm-Message-State: ACgBeo12Y7LPKBfeOkO9PmRW/l3AmTbw+oVPIWHHfzpiMw4qY4cCAJ6I GXiyPyoAF1xQm3jf3ndoHKenlcJ3NmcFPam/cD0XUDAoYkCLh3zhInfvYGv393rGgo+Eisry2Nm EeMfQwhbjZzWvWqHxxfJNyFLJAQgTX6fl9qp0D+6zZRv0ao+BkArHf6rTrOwKY6P87Gs= X-Received: by 2002:a5d:510e:0:b0:226:e949:8baf with SMTP id s14-20020a5d510e000000b00226e9498bafmr1812287wrt.204.1661935289166; Wed, 31 Aug 2022 01:41:29 -0700 (PDT) X-Google-Smtp-Source: AA6agR62+Y85lLKovUBzVAqo0xA9lmNPA+o1NPrEEFyJDB82+ROCDUA6I/RgtHAhN8L8NmLSfGpdnw== X-Received: by 2002:a5d:510e:0:b0:226:e949:8baf with SMTP id s14-20020a5d510e000000b00226e9498bafmr1812268wrt.204.1661935288772; Wed, 31 Aug 2022 01:41:28 -0700 (PDT) From: Andrew Burgess To: qemu-devel@nongnu.org Cc: Andrew Burgess Subject: [PATCH 1/2] target/riscv: remove fflags, frm, and fcsr from riscv-*-fpu.xml Date: Wed, 31 Aug 2022 09:41:22 +0100 Message-Id: <0fbf2a5b12e3210ff3867d5cf7022b3f3462c9c8.1661934573.git.aburgess@redhat.com> X-Mailer: git-send-email 2.25.4 In-Reply-To: References: 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=170.10.129.124; envelope-from=aburgess@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 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+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1661935470485100001 Content-Type: text/plain; charset="utf-8" While testing some changes to GDB's handling for the RISC-V registers fcsr, fflags, and frm, I spotted that QEMU includes these registers twice in the target description it sends to GDB, once in the fpu feature, and once in the csr feature. Right now things basically work OK, QEMU maps these registers onto two different register numbers, e.g. fcsr maps to both 68 and 73, and GDB can use either of these to access the register. However, GDB's target descriptions don't really work this way, each register should appear just once in a target description, mapping the register name onto the number GDB should use when accessing the register on the target. Duplicate register names actually result in duplicate registers on the GDB side, however, as the registers have the same name, the user can only access one of these registers. Currently GDB has a hack in place, specifically for RISC-V, to spot the duplicate copies of these three registers, and hide them from the user, ensuring the user only ever sees a single copy of each. In this commit I propose fixing this issue on the QEMU side, and in the process, simplify the fpu register handling a little. I think we should, remove fflags, frm, and fcsr from the two (32-bit and 64-bit) fpu feature xml files. These files will only contain the 32 core floating point register f0 to f31. The fflags, frm, and fcsr registers will continue to be advertised in the csr feature as they currently are. With that change made, I will simplify riscv_gdb_get_fpu and riscv_gdb_set_fpu, removing the extra handling for the 3 status registers. Signed-off-by: Andrew Burgess Reviewed-by: Alistair Francis --- gdb-xml/riscv-32bit-fpu.xml | 4 ---- gdb-xml/riscv-64bit-fpu.xml | 4 ---- target/riscv/gdbstub.c | 32 ++------------------------------ 3 files changed, 2 insertions(+), 38 deletions(-) diff --git a/gdb-xml/riscv-32bit-fpu.xml b/gdb-xml/riscv-32bit-fpu.xml index 1eaae9119e..84a44ba8df 100644 --- a/gdb-xml/riscv-32bit-fpu.xml +++ b/gdb-xml/riscv-32bit-fpu.xml @@ -43,8 +43,4 @@ - - - - diff --git a/gdb-xml/riscv-64bit-fpu.xml b/gdb-xml/riscv-64bit-fpu.xml index 794854cc01..9856a9d1d3 100644 --- a/gdb-xml/riscv-64bit-fpu.xml +++ b/gdb-xml/riscv-64bit-fpu.xml @@ -49,8 +49,4 @@ - - - - diff --git a/target/riscv/gdbstub.c b/target/riscv/gdbstub.c index 9ed049c29e..9974b7aac6 100644 --- a/target/riscv/gdbstub.c +++ b/target/riscv/gdbstub.c @@ -114,20 +114,6 @@ static int riscv_gdb_get_fpu(CPURISCVState *env, GByte= Array *buf, int n) if (env->misa_ext & RVF) { return gdb_get_reg32(buf, env->fpr[n]); } - /* there is hole between ft11 and fflags in fpu.xml */ - } else if (n < 36 && n > 32) { - target_ulong val =3D 0; - int result; - /* - * CSR_FFLAGS is at index 1 in csr_register, and gdb says it is FP - * register 33, so we recalculate the map index. - * This also works for CSR_FRM and CSR_FCSR. - */ - result =3D riscv_csrrw_debug(env, n - 32, &val, - 0, 0); - if (result =3D=3D RISCV_EXCP_NONE) { - return gdb_get_regl(buf, val); - } } return 0; } @@ -137,20 +123,6 @@ static int riscv_gdb_set_fpu(CPURISCVState *env, uint8= _t *mem_buf, int n) if (n < 32) { env->fpr[n] =3D ldq_p(mem_buf); /* always 64-bit */ return sizeof(uint64_t); - /* there is hole between ft11 and fflags in fpu.xml */ - } else if (n < 36 && n > 32) { - target_ulong val =3D ldtul_p(mem_buf); - int result; - /* - * CSR_FFLAGS is at index 1 in csr_register, and gdb says it is FP - * register 33, so we recalculate the map index. - * This also works for CSR_FRM and CSR_FCSR. - */ - result =3D riscv_csrrw_debug(env, n - 32, NULL, - val, -1); - if (result =3D=3D RISCV_EXCP_NONE) { - return sizeof(target_ulong); - } } return 0; } @@ -404,10 +376,10 @@ void riscv_cpu_register_gdb_regs_for_features(CPUStat= e *cs) CPURISCVState *env =3D &cpu->env; if (env->misa_ext & RVD) { gdb_register_coprocessor(cs, riscv_gdb_get_fpu, riscv_gdb_set_fpu, - 36, "riscv-64bit-fpu.xml", 0); + 32, "riscv-64bit-fpu.xml", 0); } else if (env->misa_ext & RVF) { gdb_register_coprocessor(cs, riscv_gdb_get_fpu, riscv_gdb_set_fpu, - 36, "riscv-32bit-fpu.xml", 0); + 32, "riscv-32bit-fpu.xml", 0); } if (env->misa_ext & RVV) { gdb_register_coprocessor(cs, riscv_gdb_get_vector, riscv_gdb_set_v= ector, --=20 2.25.4 From nobody Sat Feb 7 12:11:39 2026 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=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1661935646; cv=none; d=zohomail.com; s=zohoarc; b=ETvUZVCGYF7prl5TKVwxOhPRzD4/Sldf0Xzy85t2vgeUiOy+Gd7RMHNSvNFno1FIqAkp8WHReEsZ1YYOsO1PiViAqTSNrYGdEiP+iz/aPMnSlTFwO14xJgGB1Ttgvy9tb4mByXUcXFFJtSyw4ZpvGD1P4jLqMnuxpCv5f2+9tHA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1661935646; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=xuN1AbMHxaL8zO8+2+jI/oXQZ565O/pfjes8Gp/0XuM=; b=VQA2Zj76d495zErTdlLLcMzCCClT4JlMHFWnS/xrwsXXf0+0vZLj1NLCkmgDxCUES3DSeVLETVwq8Ve7giOobB2zQJy3AXyhYyooDTmWJhw4K2h/ZQHprRnVRmrHn0XTkLiqifN+CT4/rAEMdNNosFfnhkCNluGPgIfHApVQcBI= 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 1661935646625668.2275829789609; Wed, 31 Aug 2022 01:47:26 -0700 (PDT) Received: from localhost ([::1]:38406 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oTJNd-0007Df-Hd for importer@patchew.org; Wed, 31 Aug 2022 04:47:25 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:38580) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oTJJQ-0003Zy-2f for qemu-devel@nongnu.org; Wed, 31 Aug 2022 04:43:04 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]:32253) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oTJJN-0007tk-Qg for qemu-devel@nongnu.org; Wed, 31 Aug 2022 04:43:03 -0400 Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-358-MpAIe5g9OLO4GZhI8-4-YA-1; Wed, 31 Aug 2022 04:41:31 -0400 Received: by mail-wm1-f71.google.com with SMTP id c66-20020a1c3545000000b003a5f6dd6a25so11425013wma.1 for ; Wed, 31 Aug 2022 01:41:31 -0700 (PDT) Received: from localhost ([31.111.84.229]) by smtp.gmail.com with ESMTPSA id ay21-20020a05600c1e1500b003a536d5aa2esm1464030wmb.11.2022.08.31.01.41.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Aug 2022 01:41:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1661935381; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xuN1AbMHxaL8zO8+2+jI/oXQZ565O/pfjes8Gp/0XuM=; b=Ob7eBuoWFxpd5AwRwU5DIikIO5NaImYj5AFp2smr3in2UTxOmUwTTnWsgtKH4rDse0/eD5 az/jLM3kdhPmK3nkw5+PQUU3v/QLO8i288sc6MyUI0ha65htYN2eliJZ+pYRZX5E0CSyNu yduE7U2wUCNnwvXogLTf4JewJ0xOwgs= X-MC-Unique: MpAIe5g9OLO4GZhI8-4-YA-1 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=xuN1AbMHxaL8zO8+2+jI/oXQZ565O/pfjes8Gp/0XuM=; b=QL7Sik6yrUnPIIjp7W9oxi4HeQqHg4Ts0VYol1Z7Zu38xKWO6GDaY9Kfp4lc31PDfQ N/iOUrniHbyQi4olPdZVK087NQ7ruAfaPM3+2cyt5Uwd/kk8c6i8iMnRTAiWQ+vS4pP8 Ce6jXhj6Y6UdKOgNt1lE7P0AuR7PtqWkyn3bX4IqxbRXKOQRA8+PY8Bv+4TGUIN6MeYA jRDIAQ5XL4+YUcUyHTSbAOuwXrsEeZXAH426C2JkfKgRIntOu4B1CEJHCYMxL7SkR9gb S5MRVYQKxhoJuNl/OnM+wyuUB61NAKuXNKmLET+vVNnaoh15kBT/PorXARsGJC2bLX/o i6Sw== X-Gm-Message-State: ACgBeo2zQxTvDOhBbnQ/KmP469EtFOGKE7fR3DIb/pKVkF0BRwemn0vz picRb14Xss54Iyld6wQPA9AQOQ4j8AYC9J1fPRSis90qHFGv6yYM5zHhFqtRU6fw+3RwQoO++Rp PyzX8DGZ9Hrt6KS3baucd0Taxb+dcIBE3RbGk32a8CIXMyqaF5jKyIkWa1VKwybK2Bv4= X-Received: by 2002:a05:6000:54d:b0:225:4d5c:4615 with SMTP id b13-20020a056000054d00b002254d5c4615mr10385489wrf.107.1661935290436; Wed, 31 Aug 2022 01:41:30 -0700 (PDT) X-Google-Smtp-Source: AA6agR4+4HvfXdz5482ZCK5FAZC9mCEmXeh0wryNV6+N5dD9bcr+dH0SJCzfbmKC/+7So0lyjRoUBQ== X-Received: by 2002:a05:6000:54d:b0:225:4d5c:4615 with SMTP id b13-20020a056000054d00b002254d5c4615mr10385474wrf.107.1661935290056; Wed, 31 Aug 2022 01:41:30 -0700 (PDT) From: Andrew Burgess To: qemu-devel@nongnu.org Cc: Andrew Burgess Subject: [PATCH 2/2] target/riscv: remove fixed numbering from GDB xml feature files Date: Wed, 31 Aug 2022 09:41:23 +0100 Message-Id: <6069395f90e6fc24dac92197be815fedf42f5974.1661934573.git.aburgess@redhat.com> X-Mailer: git-send-email 2.25.4 In-Reply-To: References: 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=170.10.129.124; envelope-from=aburgess@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 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+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1661935647592100001 Content-Type: text/plain; charset="utf-8" The fixed register numbering in the various GDB feature files for RISC-V only exists because these files were originally copied from the GDB source tree. However, the fixed numbering only exists in the GDB source tree so that GDB, when it connects to a target that doesn't provide a target description, will use a specific numbering scheme. That numbering scheme is designed to be compatible with the first versions of QEMU (for RISC-V), that didn't send a target description, and relied on a fixed numbering scheme. Because of the way that QEMU manages its target descriptions, recording the number of registers in each feature, and just relying on GDB's numbering starting from 0, then I propose that we remove all the fixed numbering from the RISC-V feature xml files, and just rely on the standard numbering scheme. Plenty of other targets manage their xml files this way, e.g. ARM, AArch64, Loongarch, m68k, rx, and s390. Signed-off-by: Andrew Burgess Acked-by: Alistair Francis Reviewed-by: Palmer Dabbelt --- gdb-xml/riscv-32bit-cpu.xml | 6 +----- gdb-xml/riscv-32bit-fpu.xml | 6 +----- gdb-xml/riscv-64bit-cpu.xml | 6 +----- gdb-xml/riscv-64bit-fpu.xml | 6 +----- 4 files changed, 4 insertions(+), 20 deletions(-) diff --git a/gdb-xml/riscv-32bit-cpu.xml b/gdb-xml/riscv-32bit-cpu.xml index 0d07aaec85..466f2c0648 100644 --- a/gdb-xml/riscv-32bit-cpu.xml +++ b/gdb-xml/riscv-32bit-cpu.xml @@ -5,13 +5,9 @@ are permitted in any medium without royalty provided the copyright notice and this notice are preserved. --> =20 - - - + diff --git a/gdb-xml/riscv-32bit-fpu.xml b/gdb-xml/riscv-32bit-fpu.xml index 84a44ba8df..24aa087031 100644 --- a/gdb-xml/riscv-32bit-fpu.xml +++ b/gdb-xml/riscv-32bit-fpu.xml @@ -5,13 +5,9 @@ are permitted in any medium without royalty provided the copyright notice and this notice are preserved. --> =20 - - - + diff --git a/gdb-xml/riscv-64bit-cpu.xml b/gdb-xml/riscv-64bit-cpu.xml index b8aa424ae4..c4d83de09b 100644 --- a/gdb-xml/riscv-64bit-cpu.xml +++ b/gdb-xml/riscv-64bit-cpu.xml @@ -5,13 +5,9 @@ are permitted in any medium without royalty provided the copyright notice and this notice are preserved. --> =20 - - - + diff --git a/gdb-xml/riscv-64bit-fpu.xml b/gdb-xml/riscv-64bit-fpu.xml index 9856a9d1d3..d0f17f9984 100644 --- a/gdb-xml/riscv-64bit-fpu.xml +++ b/gdb-xml/riscv-64bit-fpu.xml @@ -5,10 +5,6 @@ are permitted in any medium without royalty provided the copyright notice and this notice are preserved. --> =20 - - =20 @@ -17,7 +13,7 @@ =20 - + --=20 2.25.4