Message ID | 20230112015630.32999-1-kevinb@redhat.com |
---|---|
Headers |
Return-Path: <gdb-patches-bounces+patchwork=sourceware.org@sourceware.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 0938E385840E for <patchwork@sourceware.org>; Thu, 12 Jan 2023 01:57:26 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0938E385840E DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1673488646; bh=PzdIsZkwfV98Nv0gWpCUWR1HlKq7LMQYtDReD1638eM=; h=To:Cc:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=Eck/n4OIFgJitur+yy/Oq8hOdu9ZYsuLmsLhGv4fKxLxEfhQkkM2UoXAHePMXT7tc tu3XOqM8jtSP40uZSI/8VMdM6Lv8j/t4XoghNt7gGW2hfaSDhHmRNsTPlpaRbTkVKe tE5JyWK/zdqOj+J1MmlyeJdOiW2rKvtmjM8EQmWY= X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org 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 329713858D35 for <gdb-patches@sourceware.org>; Thu, 12 Jan 2023 01:57:00 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 329713858D35 Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-111-KRoaxg1LMRWaD1_UrpuNQw-1; Wed, 11 Jan 2023 20:56:56 -0500 X-MC-Unique: KRoaxg1LMRWaD1_UrpuNQw-1 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 615023806630; Thu, 12 Jan 2023 01:56:56 +0000 (UTC) Received: from f34-1.lan (unknown [10.2.17.76]) by smtp.corp.redhat.com (Postfix) with ESMTP id 8DD42492C14; Thu, 12 Jan 2023 01:56:55 +0000 (UTC) To: gdb-patches@sourceware.org Cc: pedro@palves.net, simark@simark.ca, tdevries@suse.de, Kevin Buettner <kevinb@redhat.com> Subject: [PATCH v4 0/8] Fix gdb.base/gdb-sigterm.exp failure/error Date: Wed, 11 Jan 2023 18:56:22 -0700 Message-Id: <20230112015630.32999-1-kevinb@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.9 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=-4.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_NONE, TXREP 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 <gdb-patches.sourceware.org> List-Unsubscribe: <https://sourceware.org/mailman/options/gdb-patches>, <mailto:gdb-patches-request@sourceware.org?subject=unsubscribe> List-Archive: <https://sourceware.org/pipermail/gdb-patches/> List-Post: <mailto:gdb-patches@sourceware.org> List-Help: <mailto:gdb-patches-request@sourceware.org?subject=help> List-Subscribe: <https://sourceware.org/mailman/listinfo/gdb-patches>, <mailto:gdb-patches-request@sourceware.org?subject=subscribe> From: Kevin Buettner via Gdb-patches <gdb-patches@sourceware.org> Reply-To: Kevin Buettner <kevinb@redhat.com> Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org Sender: "Gdb-patches" <gdb-patches-bounces+patchwork=sourceware.org@sourceware.org> |
Series |
Fix gdb.base/gdb-sigterm.exp failure/error
|
|
Message
Kevin Buettner
Jan. 12, 2023, 1:56 a.m. UTC
This series fixes the failure in gdb.base/gdb-sigterm.exp when running in environments with glibc-2.34 or later. It addresses Pedro's concerns regarding the v3 series: https://sourceware.org/pipermail/gdb-patches/2022-February/186150.html Patches 3, 6, and 7 from the earlier series has changed as follows: Patch 3 / "Catch gdb_exception_error instead of gdb_exception (in many places)" no longer contains a change for safe_execute_command(). This is a tricky case; I've reworked it and moved it into its own commit titled "Call quit_force for gdb_exception_forced_quit in safe_execute_command". Patch 6 / "QUIT processing w/ explicit throw for gdb_exception_forced_quit" no longer contains changes to gdb/tui/tui-io.c. These changes have been moved to patch 8 (which was #7 in the v3 series) since the technique used for circumventing the problem with not being able to do a throw are the same for both. Patch 7 (from v3) / "Handle QUIT processing in the scoped_switch_fork_info destructor" is now patch 8 in v4. It's been retitled as "Forced quit cases handled by resetting sync_quit_force_run" It contains changes for the destructor as well as tui_getc(). Patch 6 (v4) / "Call quit_force for gdb_exception_forced_quit in safe_execute_command" is a new patch containing a rework of changes to safe_execute_command that was previously in Patch 3. Kevin Buettner (8): Introduce gdb_exception_forced_quit Handle gdb SIGTERM by throwing / catching gdb_exception_force_quit Catch gdb_exception_error instead of gdb_exception (in many places) Python QUIT processing updates Guile QUIT processing updates Call quit_force for gdb_exception_forced_quit in safe_execute_command QUIT processing w/ explicit throw for gdb_exception_forced_quit Forced quit cases handled by resetting sync_quit_force_run gdb/ada-lang.c | 2 +- gdb/breakpoint.c | 8 ++++---- gdb/cli/cli-interp.c | 11 +++++++++++ gdb/event-top.c | 2 ++ gdb/guile/guile-internal.h | 5 +++++ gdb/guile/scm-pretty-print.c | 5 +++++ gdb/guile/scm-type.c | 5 +++++ gdb/guile/scm-value.c | 5 +++++ gdb/i386-linux-tdep.c | 2 +- gdb/inf-loop.c | 2 +- gdb/infcmd.c | 2 +- gdb/infrun.c | 2 +- gdb/jit.c | 2 +- gdb/linux-fork.c | 13 +++++++++++++ gdb/main.c | 12 ++++++++++++ gdb/mi/mi-cmd-break.c | 2 +- gdb/mi/mi-interp.c | 2 +- gdb/mi/mi-main.c | 4 ++++ gdb/objc-lang.c | 2 +- gdb/parse.c | 2 +- gdb/printcmd.c | 2 +- gdb/python/py-finishbreakpoint.c | 5 +++++ gdb/python/py-gdb-readline.c | 4 ++++ gdb/python/py-symbol.c | 5 +++++ gdb/python/py-utils.c | 3 +++ gdb/python/py-value.c | 5 +++++ gdb/record-btrace.c | 2 +- gdb/record-full.c | 2 +- gdb/remote-fileio.c | 15 ++++++++++----- gdb/solib.c | 2 +- gdb/sparc64-linux-tdep.c | 2 +- gdb/symfile-mem.c | 2 +- gdb/top.h | 2 +- gdb/tui/tui-io.c | 9 +++++++++ gdb/tui/tui.c | 4 ++++ gdb/utils.c | 2 +- gdbsupport/common-exceptions.cc | 14 ++++++++++++++ gdbsupport/common-exceptions.h | 22 +++++++++++++++++++++- 38 files changed, 164 insertions(+), 28 deletions(-)
Comments
On 1/12/23 02:56, Kevin Buettner wrote: > This series fixes the failure in gdb.base/gdb-sigterm.exp when > running in environments with glibc-2.34 or later. > I've tested this (with python and guile enabled) on: - openSUSE Tumbleweed x86_64, where I did see FAIL before, and this series fixes it - openSUSE Leap 15.4 x86_64 No problems found. FWIW, I ran into this when applying: ... $ git am ~/patches/grouped/eml/* Applying: Introduce gdb_exception_forced_quit Applying: Handle gdb SIGTERM by throwing / catching gdb_exception_force_quit .git/rebase-apply/patch:37: indent with spaces. { warning: 1 line adds whitespace errors. Applying: Catch gdb_exception_error instead of gdb_exception (in many places) Applying: Python QUIT processing updates Applying: Guile QUIT processing updates Applying: Call quit_force for gdb_exception_forced_quit in safe_execute_command Applying: QUIT processing w/ explicit throw for gdb_exception_forced_quit .git/rebase-apply/patch:17: indent with spaces. throw; .git/rebase-apply/patch:46: indent with spaces. { .git/rebase-apply/patch:58: indent with spaces. { warning: 3 lines add whitespace errors. Applying: Forced quit cases handled by resetting sync_quit_force_run .git/rebase-apply/patch:41: indent with spaces. to newline, so, for this case, reset the quit flag for warning: 1 line adds whitespace errors. ... Thanks, - Tom > It addresses Pedro's concerns regarding the v3 series: > > https://sourceware.org/pipermail/gdb-patches/2022-February/186150.html > > Patches 3, 6, and 7 from the earlier series has changed as follows: > > Patch 3 / "Catch gdb_exception_error instead of gdb_exception (in many > places)" no longer contains a change for safe_execute_command(). This > is a tricky case; I've reworked it and moved it into its own commit > titled "Call quit_force for gdb_exception_forced_quit in > safe_execute_command". > > Patch 6 / "QUIT processing w/ explicit throw for > gdb_exception_forced_quit" no longer contains changes to > gdb/tui/tui-io.c. These changes have been moved to patch 8 (which > was #7 in the v3 series) since the technique used for circumventing > the problem with not being able to do a throw are the same for both. > > Patch 7 (from v3) / "Handle QUIT processing in the scoped_switch_fork_info > destructor" is now patch 8 in v4. It's been retitled as "Forced quit cases > handled by resetting sync_quit_force_run" It contains changes for the > destructor as well as tui_getc(). > > Patch 6 (v4) / "Call quit_force for gdb_exception_forced_quit in > safe_execute_command" is a new patch containing a rework of changes > to safe_execute_command that was previously in Patch 3. > > Kevin Buettner (8): > Introduce gdb_exception_forced_quit > Handle gdb SIGTERM by throwing / catching gdb_exception_force_quit > Catch gdb_exception_error instead of gdb_exception (in many places) > Python QUIT processing updates > Guile QUIT processing updates > Call quit_force for gdb_exception_forced_quit in safe_execute_command > QUIT processing w/ explicit throw for gdb_exception_forced_quit > Forced quit cases handled by resetting sync_quit_force_run > > gdb/ada-lang.c | 2 +- > gdb/breakpoint.c | 8 ++++---- > gdb/cli/cli-interp.c | 11 +++++++++++ > gdb/event-top.c | 2 ++ > gdb/guile/guile-internal.h | 5 +++++ > gdb/guile/scm-pretty-print.c | 5 +++++ > gdb/guile/scm-type.c | 5 +++++ > gdb/guile/scm-value.c | 5 +++++ > gdb/i386-linux-tdep.c | 2 +- > gdb/inf-loop.c | 2 +- > gdb/infcmd.c | 2 +- > gdb/infrun.c | 2 +- > gdb/jit.c | 2 +- > gdb/linux-fork.c | 13 +++++++++++++ > gdb/main.c | 12 ++++++++++++ > gdb/mi/mi-cmd-break.c | 2 +- > gdb/mi/mi-interp.c | 2 +- > gdb/mi/mi-main.c | 4 ++++ > gdb/objc-lang.c | 2 +- > gdb/parse.c | 2 +- > gdb/printcmd.c | 2 +- > gdb/python/py-finishbreakpoint.c | 5 +++++ > gdb/python/py-gdb-readline.c | 4 ++++ > gdb/python/py-symbol.c | 5 +++++ > gdb/python/py-utils.c | 3 +++ > gdb/python/py-value.c | 5 +++++ > gdb/record-btrace.c | 2 +- > gdb/record-full.c | 2 +- > gdb/remote-fileio.c | 15 ++++++++++----- > gdb/solib.c | 2 +- > gdb/sparc64-linux-tdep.c | 2 +- > gdb/symfile-mem.c | 2 +- > gdb/top.h | 2 +- > gdb/tui/tui-io.c | 9 +++++++++ > gdb/tui/tui.c | 4 ++++ > gdb/utils.c | 2 +- > gdbsupport/common-exceptions.cc | 14 ++++++++++++++ > gdbsupport/common-exceptions.h | 22 +++++++++++++++++++++- > 38 files changed, 164 insertions(+), 28 deletions(-) >
On Thu, 12 Jan 2023 13:37:00 +0100 Tom de Vries <tdevries@suse.de> wrote: > FWIW, I ran into this when applying: > ... > $ git am ~/patches/grouped/eml/* > Applying: Introduce gdb_exception_forced_quit > Applying: Handle gdb SIGTERM by throwing / catching gdb_exception_force_quit > .git/rebase-apply/patch:37: indent with spaces. > { > warning: 1 line adds whitespace errors. > Applying: Catch gdb_exception_error instead of gdb_exception (in many > places) > Applying: Python QUIT processing updates > Applying: Guile QUIT processing updates > Applying: Call quit_force for gdb_exception_forced_quit in > safe_execute_command > Applying: QUIT processing w/ explicit throw for gdb_exception_forced_quit > .git/rebase-apply/patch:17: indent with spaces. > throw; > .git/rebase-apply/patch:46: indent with spaces. > { > .git/rebase-apply/patch:58: indent with spaces. > { > warning: 3 lines add whitespace errors. > Applying: Forced quit cases handled by resetting sync_quit_force_run > .git/rebase-apply/patch:41: indent with spaces. > to newline, so, for this case, reset the quit flag for > warning: 1 line adds whitespace errors. > ... Thanks! I've fixed these whitespace problems in my local sources. I've also added a "Tested-by" attribution for you to each commit message. Kevin