From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31334 invoked by alias); 6 Apr 2008 06:42:13 -0000 Received: (qmail 31326 invoked by uid 22791); 6 Apr 2008 06:42:12 -0000 X-Spam-Check-By: sourceware.org Received: from mx2.snap.net.nz (HELO viper.snap.net.nz) (202.37.101.25) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sun, 06 Apr 2008 06:41:45 +0000 Received: from kahikatea.snap.net.nz (60.60.255.123.dynamic.snap.net.nz [123.255.60.60]) by viper.snap.net.nz (Postfix) with ESMTP id 8DA823DA10F; Sun, 6 Apr 2008 18:41:41 +1200 (NZST) Received: by kahikatea.snap.net.nz (Postfix, from userid 1000) id 6D9118FC6D; Sun, 6 Apr 2008 18:41:39 +1200 (NZST) From: Nick Roberts MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <18424.28834.573873.54124@kahikatea.snap.net.nz> Date: Mon, 07 Apr 2008 00:06:00 -0000 To: Vladimir Prus Cc: gdb-patches@sources.redhat.com Subject: Re: linux native async mode support In-Reply-To: <200804051328.43973.ghost@cs.msu.su> References: <200803140810.22883.pedro@codesourcery.com> <200804041546.41673.ghost@cs.msu.su> <18423.16925.181684.296006@kahikatea.snap.net.nz> <200804051328.43973.ghost@cs.msu.su> X-Mailer: VM 7.19 under Emacs 22.2.50.1 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-04/txt/msg00121.txt.bz2 > > There's an extra "^done" because this is a CLI command entered in MI (and > > therfore case CLI_COMMAND: of captured_mi_execute_command). There are many > > issues here like should we diallow immediate use of CLI commands now and > > require explicit use of "-interpreter-exec console"? > > Probably not -- I don't see any immediate advantage from that. The node "GDB/MI Compatibility with CLI" explains why. > > Then "-interpreter-exec > > console next" doesn't emit "(gdb)/n" after "^running" when in asynchronous > > mode, so should we remove it from synchronous mode too (as you have suggested)? > > This prompting is fairly confusing, so it could be that I've confused myself > as well, but I think "-interpreter-exec console next" actually *should* print > the prompt if the target is async-capable. The reason is that for MI mode, > we don't really care about terminal ownership issues, and then should not > care about sync_execution, and then this bit of code: > > else if (sync_execution) > { > /* Don't print the prompt. We are executing the target in > synchronous mode. */ > args->action = EXECUTE_COMMAND_SUPPRESS_PROMPT; > return; > } > > in captured_mi_execute_command should be removed. In fact, I have a > patch-in-progress to do exactly that, and hopefully will be posting it later > today. OK. > > But these are all MI issues and this test is meant to just be a mark in the > > sand for asynchronous mode, and one that I had lying around. For the moment, I > > don't really want to work on it further > > Okay :-) Then, will you mind if I apply this patch either when I'm done with > another round of MI testsuite cleanups I'm doing today, or after the ^done > is gone, whichever proves more convenient? I'll commit this as is then. Then, of course, you can decide what changes you wish to make. Meanwhile I'll try to come up with some tests for floating variable objects which will be more efficient use of my time. > > > BTW, I'm not even sure that "^done" vs. "^running" + "^done" is so big > > > difference that a helper function cannot be introduced. > > > > The comment suggests that mi_gdb_test will not work but I have not revisited > > that since writing the test 18 months or so ago. If true, that could also > > make helper functions difficult. > > Do you happen to remember what race condition the comment mentions? I must have copied the comment from mi-simplerun.exp and I think I found it necessary with my asynchronous branch, although I can't be sure, and it may not be be necessary with Pedro's version (.i.e. mainline). -- Nick http://www.inet.net.nz/~nickrob