From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id 67AgKq3SmGP7NAAAWB0awg (envelope-from ) for ; Tue, 13 Dec 2022 14:29:49 -0500 Received: by simark.ca (Postfix, from userid 112) id A2B9D1E126; Tue, 13 Dec 2022 14:29:49 -0500 (EST) Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=vh0KP865; dkim-atps=neutral X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-5.3 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,NICE_REPLY_A, RCVD_IN_DNSWL_MED,RDNS_DYNAMIC,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from sourceware.org (ip-8-43-85-97.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 1CB4F1E0CD for ; Tue, 13 Dec 2022 14:29:49 -0500 (EST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id C3AAA3877203 for ; Tue, 13 Dec 2022 19:29:47 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C3AAA3877203 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1670959787; bh=zKpjlQ0tXWeYzoCNOkgoLgYNkwbeuHImsyVBJ/7NQKQ=; h=Date:Subject:To:Cc:References:In-Reply-To:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=vh0KP865NILbqJcVyIfp7w0ufTUqFg7WXY5QTxrXHJDZjDhjP1kXiwZx2EVUEVN9U AGZGLDYEbp27X/zTrxKr/hw43Zmp889U2HkBJouSDxzJXWIElyc3cXkBEe7rYiNYRb jzZGQ/BOePmjO+OAJ32q+/+eUNu15JD+vFSMn0Aw= Received: from simark.ca (simark.ca [158.69.221.121]) by sourceware.org (Postfix) with ESMTPS id DA770383D8FE for ; Tue, 13 Dec 2022 19:29:26 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org DA770383D8FE Received: from [172.16.0.64] (192-222-180-24.qc.cable.ebox.net [192.222.180.24]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPSA id 2AB841E0CD; Tue, 13 Dec 2022 14:29:26 -0500 (EST) Message-ID: <4dc13e01-2fd8-a63e-24f2-a1f7c7650d3b@simark.ca> Date: Tue, 13 Dec 2022 14:29:25 -0500 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.1 Subject: Re: [PATCH] gdb: fix command lookup in execute_command () Content-Language: fr To: =?UTF-8?Q?Jan_Vran=c3=bd?= , "gdb-patches@sourceware.org" , "tom@tromey.com" Cc: "luis.machado@arm.com" References: <1a96057b-afd6-1e90-15c9-ee106a2998c7@arm.com> <20221213112241.621889-1-jan.vrany@labware.com> <878rjbfinm.fsf@tromey.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: , From: Simon Marchi via Gdb-patches Reply-To: Simon Marchi Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" On 12/13/22 13:48, Jan Vraný wrote: > On Tue, 2022-12-13 at 08:05 -0700, Tom Tromey wrote: >>>>>>> "Jan" == Jan Vrany via Gdb-patches writes: >> >> Jan> The below patch should fix this case - with it on my system >> Jan> gdb.base/define.exp passes. >> >> Jan> Alternatively I may just revert the commit until better solution >> Jan> is found. >> >> This patch looks ok to me, but could you say whether you ran all the >> tests or just define.exp? Given the history I think a full regression >> test is warranted. > > I did use try-build which succeeded (build #28) but it seems to me it > runs only very limited number of tests. > > I did run all of gdb.base and did not spot any new regression compared > to master with my (previous, broken) patch reverted. I did not run more > than gdb.base mainly because just gdb.base takes about an hour on my machine. > Also I get weird intermittent failures when running testsuite. > > Jan Here are the unexpected failures I saw: UNRESOLVED: gdb.base/bp-cmds-execution-x-script.exp: run to end UNRESOLVED: gdb.base/bp-cmds-run-with-ex.exp: execute bp commands UNRESOLVED: gdb.base/bp-cmds-sourced-script.exp: source the script UNRESOLVED: gdb.base/bp-cmds-sourced-script.exp: continue until exit UNRESOLVED: gdb.base/commands.exp: deprecated_command_test: source file containing xxx_yyy command and its alias UNRESOLVED: gdb.base/commands.exp: deprecated_command_test: deprecated alias with prefix give a warning UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: define real_command: input 1: define real_command UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: alias alias_command = real_command UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: alias alias_with_args_command = real_command 123 UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: help real_command, before UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: maintenance deprecate alias_command UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: maintenance deprecate alias_with_args_command UNRESOLVED: gdb.base/commands.exp: deprecated_command_alias_help_test: help real_command, after UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: delete all breakpoints in delete_breakpoints UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: break factorial UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: begin commands UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: add silent command UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: add clear command UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: add printf command UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: add cont command UNRESOLVED: gdb.base/commands.exp: bp_deleted_in_command_test: end commands UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: delete all breakpoints in delete_breakpoints UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: breakpoint UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: begin commands in bp_deleted_in_command_test UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: add silent tbreak command UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: add printf tbreak command UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: add cont tbreak command UNRESOLVED: gdb.base/commands.exp: temporary_breakpoint_commands: end tbreak commands UNRESOLVED: gdb.base/commands.exp: stray_arg0_test: #1 UNRESOLVED: gdb.base/commands.exp: stray_arg0_test: #2 UNRESOLVED: gdb.base/commands.exp: stray_arg0_test: #3 UNRESOLVED: gdb.base/commands.exp: stray_arg0_test: #4 UNRESOLVED: gdb.base/commands.exp: source_file_with_indented_comment: source file UNRESOLVED: gdb.base/commands.exp: recursive_source_test: source file UNRESOLVED: gdb.base/commands.exp: if_commands_test: set $tem UNRESOLVED: gdb.base/commands.exp: if_commands_test: if $tem == 2 - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: break -q main - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: else - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: break factorial - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: commands - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: silent - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: set $tem = 3 - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: continue - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: first end - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: second end - if_commands_test 1 UNRESOLVED: gdb.base/commands.exp: if_commands_test: if $tem == 1 - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: break -q main - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: else - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: break factorial - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: commands - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: silent - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: set $tem = 3 - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: continue - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: first end - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: if_commands_test: second end - if_commands_test 2 UNRESOLVED: gdb.base/commands.exp: error_clears_commands_left: hook-stop 1 UNRESOLVED: gdb.base/commands.exp: error_clears_commands_left: hook-stop 1a UNRESOLVED: gdb.base/commands.exp: error_clears_commands_left: hook-stop 1b UNRESOLVED: gdb.base/commands.exp: error_clears_commands_left: delete all breakpoints in delete_breakpoints UNRESOLVED: gdb.base/dprintf-execution-x-script.exp: load and run script with -x UNRESOLVED: gdb.base/dprintf-execution-x-script.exp: load and run script using source command UNRESOLVED: gdb.base/dprintf-execution-x-script.exp: run again UNRESOLVED: gdb.base/save-bp.exp: source bps UNRESOLVED: gdb.base/save-bp.exp: info break (pattern 1) (timeout) UNRESOLVED: gdb.base/trace-commands.exp: source -v (pattern 4) (timeout) UNRESOLVED: gdb.base/trace-commands.exp: set trace-commands UNRESOLVED: gdb.base/trace-commands.exp: show trace-commands says on UNRESOLVED: gdb.base/trace-commands.exp: simple trace-commands test UNRESOLVED: gdb.base/trace-commands.exp: nested trace-commands test (pattern 1) (timeout) UNRESOLVED: gdb.base/trace-commands.exp: define user command (pattern 1) (timeout) UNRESOLVED: gdb.base/trace-commands.exp: nested trace-commands test with source (pattern 1) (timeout) UNRESOLVED: gdb.base/trace-commands.exp: depth resets on error part 1 (pattern 1) (timeout) UNRESOLVED: gdb.base/trace-commands.exp: depth resets on error part 2 UNRESOLVED: gdb.trace/save-trace.exp: relative: read back saved tracepoints UNRESOLVED: gdb.trace/save-trace.exp: relative: verify recovered tracepoints DUPLICATE: gdb.trace/save-trace.exp: relative: verify recovered tracepoints UNRESOLVED: gdb.trace/save-trace.exp: relative: verify default-collect UNRESOLVED: gdb.trace/save-trace.exp: absolute: save tracepoint definitions UNRESOLVED: gdb.trace/save-trace.exp: absolute: clear default-collect UNRESOLVED: gdb.trace/save-trace.exp: absolute: delete tracepoints UNRESOLVED: gdb.trace/save-trace.exp: absolute: read back saved tracepoints UNRESOLVED: gdb.trace/save-trace.exp: absolute: verify recovered tracepoints DUPLICATE: gdb.trace/save-trace.exp: absolute: verify recovered tracepoints UNRESOLVED: gdb.trace/save-trace.exp: absolute: verify default-collect UNRESOLVED: gdb.trace/save-trace.exp: verify help save tracepoints Simon