From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1822 invoked by alias); 23 Mar 2008 16:44:23 -0000 Received: (qmail 1813 invoked by uid 22791); 23 Mar 2008 16:44:22 -0000 X-Spam-Check-By: sourceware.org Received: from host0.dyn.jankratochvil.net (HELO host0.dyn.jankratochvil.net) (89.250.240.59) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sun, 23 Mar 2008 16:44:05 +0000 Received: from host0.dyn.jankratochvil.net (localhost [127.0.0.1]) by host0.dyn.jankratochvil.net (8.14.2/8.14.1) with ESMTP id m2NGhXFL001594; Sun, 23 Mar 2008 17:43:33 +0100 Received: (from jkratoch@localhost) by host0.dyn.jankratochvil.net (8.14.2/8.14.2/Submit) id m2NGhVsa001593; Sun, 23 Mar 2008 17:43:31 +0100 Date: Sun, 23 Mar 2008 16:44:00 -0000 From: Jan Kratochvil To: Daniel Jacobowitz Cc: Nick Roberts , gdb-patches@sourceware.org, Chet Ramey Subject: Re: [patch] Fix testsuite annotate-quit race (PR 544) Message-ID: <20080323164331.GA1195@host0.dyn.jankratochvil.net> References: <20080318225436.GA27374@host0.dyn.jankratochvil.net> <20080318231738.GA22432@caradoc.them.org> <20080319081056.GA32510@host0.dyn.jankratochvil.net> <20080321204506.GA5033@host0.dyn.jankratochvil.net> <20080321210323.GA30247@caradoc.them.org> <20080321214448.GB29593@caradoc.them.org> <18406.14411.437345.335844@kahikatea.snap.net.nz> <20080323163027.GB13603@caradoc.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080323163027.GB13603@caradoc.them.org> User-Agent: Mutt/1.5.17 (2007-11-01) X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-03/txt/msg00344.txt.bz2 On Sun, 23 Mar 2008 17:30:27 +0100, Daniel Jacobowitz wrote: ... > I tried running gdb under nohup and it worked fine. I tried it in > Emacs too, and that worked fine: Confirmed the patch broke emacs-22.1-8.fc8.x86_64 (M-x shell ...). readline checks for custom RL_REDISPLAY_FUNCTION and behaves differently in such case. Sorry for suggesting RL_REDISPLAY_FUNCTION - I did not verify it more. Now just to find out how to supply GDB_RL_REDISPLAY simulating the original behavior. latest GDB: rt_sigaction(SIGWINCH, {0x5f9025, [], SA_RESTORER|SA_RESTART, 0x3b98630f30}, {0x4ab21a, [], SA_RESTORER, 0x3b98630f30}, 8) = 0 rt_sigprocmask(SIG_BLOCK, [INT], [CHLD], 8) = 0 rt_sigprocmask(SIG_SETMASK, [CHLD], NULL, 8) = 0 rt_sigprocmask(SIG_BLOCK, NULL, [CHLD], 8) = 0 poll([{fd=3, events=POLLIN}, {fd=0, events=POLLIN, revents=POLLIN}], 2, -1) = 1 rt_sigprocmask(SIG_BLOCK, NULL, [CHLD], 8) = 0 rt_sigprocmask(SIG_BLOCK, NULL, [CHLD], 8) = 0 read(0, "q", 1) = 1 reverted patch: rt_sigaction(SIGWINCH, {0x5f8fb5, [], SA_RESTORER|SA_RESTART, 0x3b98630f30}, {0x4ab1aa, [], SA_RESTORER, 0x3b98630f30}, 8) = 0 write(1, "\n\32\32pre-prompt\n(gdb) \n\32\32prompt\n", 30) = 30 rt_sigprocmask(SIG_BLOCK, NULL, [CHLD], 8) = 0 poll([{fd=3, events=POLLIN}, {fd=0, events=POLLIN, revents=POLLIN}], 2, -1) = 1 rt_sigprocmask(SIG_BLOCK, NULL, [CHLD], 8) = 0 rt_sigprocmask(SIG_BLOCK, NULL, [CHLD], 8) = 0 read(0, "q", 1) = 1 > Does reverting the patch fix the problem? Yes. Regards, Jan