From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18311 invoked by alias); 28 Jun 2004 14:47:54 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 18268 invoked from network); 28 Jun 2004 14:47:53 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sourceware.org with SMTP; 28 Jun 2004 14:47:53 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.12.10/8.12.10) with ESMTP id i5SElre3016982 for ; Mon, 28 Jun 2004 10:47:53 -0400 Received: from localhost.redhat.com (porkchop.devel.redhat.com [172.16.58.2]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id i5SElq022234; Mon, 28 Jun 2004 10:47:52 -0400 Received: from gnu.org (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id E603E2B9D; Mon, 28 Jun 2004 10:47:47 -0400 (EDT) Message-ID: <40E02F93.7040500@gnu.org> Date: Mon, 28 Jun 2004 14:47:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-GB; rv:1.4.1) Gecko/20040217 MIME-Version: 1.0 To: Paul Hilfinger Cc: gdb-patches@sources.redhat.com Subject: Re: [PATCH]: Further updates to ada-* files References: <20040608090758.C59CAF2940@nile.gnat.com> <20040616083422.C3B5CF2C03@nile.gnat.com> <40DE1064.9020900@gnu.org> <20040627085653.31703F2E19@nile.gnat.com> In-Reply-To: <20040627085653.31703F2E19@nile.gnat.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2004-06/txt/msg00636.txt.bz2 > Andrew, > > >>> Lets start with the obvious, can you fix the ARI problems (Run >>> 'gdb_ari.sh -Wari' from http://sources.redhat.com/gdb/ari/)? > > > OK > > >>> Looking over the code, ada-tasks.c is going to be a problem. Can we >>> ignore that for now (removing it?) and just focus on getting basic >>> language support enabled? Once that's done, we can start working >>> through the underlying issues that prompted ACT to add ada-tasks.c. > > > Right. I disabled it (i.e., it's not compiled) in the latest version > of the patch I sent, and do not expect it to be reviewed. We've > decided that now is as good a time as any to do a much-needed major > revision of ada-tasks.c; I'm not sure right now if I want to remove > the source entirely from the repository. Depends on how major a > change I need. Lets remove it. The question is how big a thread model change GDB is going to get (and regardless of Ada's task model) and how much having that dead code would confuse things. > The "underlying issues" are that Ada has tasks and they are (in general) > built on top of, but in principle distinct from threads. As for why to > put it in a separate file---well it's messy, due to the fact that > tasks map onto threads differently on every single confounded platform, > it seems. It's messy because the underlying thread model is bug---d. >>> Did that previous ChangeLog problem get fixed? > > > Not yet, but I haven't forgotten it. It must go in, but I fear it will be > of no help for the current task. Andrew