From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23871 invoked by alias); 6 Jul 2009 16:49:27 -0000 Received: (qmail 23863 invoked by uid 22791); 6 Jul 2009 16:49:26 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,J_CHICKENPOX_93,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 06 Jul 2009 16:49:20 +0000 Received: (qmail 8185 invoked from network); 6 Jul 2009 16:49:18 -0000 Received: from unknown (HELO orlando.local) (pedro@127.0.0.2) by mail.codesourcery.com with ESMTPA; 6 Jul 2009 16:49:18 -0000 From: Pedro Alves To: Aleksandar Ristovski Subject: Re: [patch] gdbserver: Add qnx target Date: Mon, 06 Jul 2009 16:49:00 -0000 User-Agent: KMail/1.9.10 Cc: gdb-patches@sourceware.org References: <200906301312.42665.pedro@codesourcery.com> <4A521FD6.7040700@qnx.com> In-Reply-To: <4A521FD6.7040700@qnx.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200907061750.05522.pedro@codesourcery.com> 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: 2009-07/txt/msg00140.txt.bz2 On Monday 06 July 2009 17:01:26, Aleksandar Ristovski wrote: > Pedro Alves wrote: > > Are ldd.c's sources available? :-) If not, could you paste > > that comment there too? > > Comment from ldd.c pasted. Thank you! Looks nicer now. > > On Wednesday 24 June 2009 19:50:39, Aleksandar Ristovski wrote > >> + i[34567]86-*-nto*) srv_regobj=reg-i386.o > >> + srv_tgtobj="nto-low.o nto-x86-low.o" > >> + srv_qnx_LIBS=-lsocket > >> + srv_qnx="yes" > > > > Do you think the set of libs will change depending on the qnx > > arch? In other words, if you're adding a $srv_qnx, do you really > > need a new srv_qnx_LIBS variable? > > Unlikely. I just thought it was nicer. But in this patch I > removed it and -lsocket is now "hard-coded" in > configure.ac/configure. Thanks. I think that either your original patch (which I had okayed), or the new one is fine --- while that version was middle ground without the benefits of either. > nto_inferior.exit_signo. Current code in nto-procfs.c > doesn't do that but it's a bug that results gdb reporting > "program exited normally" after program dies due to > unhandled signal. Yeah. You just might find out that the suggestion I had given before would fix this cleanly. :-) > ChangeLog: > Adding Neutrino gdbserver. > * configure: Regenerated. > * configure.ac: Add case for srv_qnx and set LIBS > accordingly.. ^ spurious '.' > * configure.srv (i[34567]86-*-nto*): New target. > * nto-low.c, nto-low.h, nto-x86-low.c: New files. > * remote-utils.c [__QNX__]: Include sys/iomgr.h ^ missing '.' > (nto_comctrl) [__QNX__]: New function. > (enable_async_io, disable_async_io) [__QNX__]: Call > nto_comctrl. Ok. Please commit. -- Pedro Alves