From patchwork Tue Nov 27 11:13:05 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Burgess X-Patchwork-Id: 30315 Received: (qmail 69001 invoked by alias); 27 Nov 2018 11:13:21 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 68869 invoked by uid 89); 27 Nov 2018 11:13:20 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.2 spammy=HX-HELO:sk:mail-wm, HX-Google-DKIM-Signature:6WVQ X-HELO: mail-wm1-f44.google.com Received: from mail-wm1-f44.google.com (HELO mail-wm1-f44.google.com) (209.85.128.44) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 27 Nov 2018 11:13:18 +0000 Received: by mail-wm1-f44.google.com with SMTP id q26so21657659wmf.5 for ; Tue, 27 Nov 2018 03:13:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=embecosm.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :in-reply-to:references; bh=lfkoiw1tjdZ8J3NkfVbrywo2bZZ98M9X0AAQE3zj8mM=; b=b2isGO9mjr2FHXHzDYcqQZUpUlf4AbTFjPk5r95A0pvvQiXXhq6NKqcXeDm3EbdbZj /egjvOvQAGZcnaMWk6MyFIDTQE1m6nSDuavzSxOpoM6I+Rv07AznR1gwOrhVA17puMOJ 6APyakXbRgjitQVz85H9aG7BmoB2AZ11W6PgHWnHSZvJGvRvqsxdSw1fJrwW4XBBMydv /GQj3rjYOoljEPCRCj+eJIua66vvL9FAlOvad9Ck0l8foodeQs4UohL9S3l5SkbNIODe a6gXu7dT+ndrj1OG0gzfl7FXL4ijR0QW43dwAN2yunHhaoUrH8n+YE8XoUqL0su9hGf9 DdQA== Return-Path: Received: from localhost (host86-156-236-171.range86-156.btcentralplus.com. [86.156.236.171]) by smtp.gmail.com with ESMTPSA id e8-v6sm7971161wmf.22.2018.11.27.03.13.14 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 27 Nov 2018 03:13:14 -0800 (PST) From: Andrew Burgess To: gdb-patches@sourceware.org Cc: Simon Marchi , Andrew Burgess Subject: [PATCHv2 2/3] gdb/regcache: When saving, ignore registers that can't be read Date: Tue, 27 Nov 2018 11:13:05 +0000 Message-Id: <1fdb87a3328423d35fda3d45bdf54fa11bb8d82c.1543317060.git.andrew.burgess@embecosm.com> In-Reply-To: References: In-Reply-To: References: X-IsSubscribed: yes The previous commit addressed an assertion that could trigger if a target threw an error while saving state ahead of an inferior function call. The specific case that highlighted this issue was a RISC-V target that claimed to support floating point registers, but when GDB tried to read a floating point register the remote sent back an error. With the previous commit we no longer see an assertion for this target, now GDB abandons the inferior function call. Although this is slightly better, it feels like for this specific case GDB could do even better. If during a call to reg_buffer::save GDB encounters an error trying to read a register then GDB should simply mark the register as unavailable and carry on. The consequence of marking the register unavailable is that GDB will not then try to restore the register once the inferior function call is complete. What I haven't done in this commit is provide any user feedback that GDB would like to backup a particular register, but can't. Right now I figure that if the user cares about this they would probably try 'p $reg_name' themselves, at which point it becomes obvious that the register can't be read. That said, I'm open to adding a warning that the register failed to save if that is thought important. I've tested this using on X86-64/Linux native, and for native-gdbserver with no regressions. Against my miss-behaving target I can now make inferior calls without any problems. gdb/ChangeLog: * regcache.c (reg_buffer::save): When saving the current register state, ignore registers that can't be read. --- gdb/ChangeLog | 5 +++++ gdb/regcache.c | 12 +++++++++++- 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/gdb/regcache.c b/gdb/regcache.c index 6e0e8c3e7e0..c9503295f59 100644 --- a/gdb/regcache.c +++ b/gdb/regcache.c @@ -277,7 +277,17 @@ reg_buffer::save (register_read_ftype cooked_read) if (gdbarch_register_reggroup_p (gdbarch, regnum, save_reggroup)) { gdb_byte *dst_buf = register_buffer (regnum); - enum register_status status = cooked_read (regnum, dst_buf); + enum register_status status; + + TRY + { + status = cooked_read (regnum, dst_buf); + } + CATCH (ex, RETURN_MASK_ERROR) + { + status = REG_UNAVAILABLE; + } + END_CATCH gdb_assert (status != REG_UNKNOWN);