From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23877 invoked by alias); 23 Apr 2004 04:19:04 -0000 Mailing-List: contact gdb-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sources.redhat.com Received: (qmail 23864 invoked from network); 23 Apr 2004 04:19:02 -0000 Received: from unknown (HELO gateway.sf.frob.com) (64.81.54.130) by sources.redhat.com with SMTP; 23 Apr 2004 04:19:02 -0000 Received: from magilla.sf.frob.com (magilla.sf.frob.com [198.49.250.228]) by gateway.sf.frob.com (Postfix) with ESMTP id 87324357B; Thu, 22 Apr 2004 21:19:01 -0700 (PDT) Received: from magilla.sf.frob.com (localhost.localdomain [127.0.0.1]) by magilla.sf.frob.com (8.12.9/8.12.9) with ESMTP id i3N4IvOi005783; Thu, 22 Apr 2004 21:18:57 -0700 Received: (from roland@localhost) by magilla.sf.frob.com (8.12.9/8.12.9/Submit) id i3N4Iuu3005779; Thu, 22 Apr 2004 21:18:56 -0700 Date: Fri, 23 Apr 2004 04:35:00 -0000 Message-Id: <200404230418.i3N4Iuu3005779@magilla.sf.frob.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit From: Roland McGrath To: Jim Blandy Cc: Randolph Chung , gdb@sources.redhat.com Subject: Re: add-symbol-file-from-memory patch breaks non-elf targets? In-Reply-To: Jim Blandy's message of , 22 April 2004 21:03:15 -0500 X-Shopping-List: (1) Ominous bums (2) Quixotic agnostic tables (3) Clairvoyant ignorers (4) Clairvoyant fallacious defusers X-SW-Source: 2004-04/txt/msg00141.txt.bz2 It's unfortunate that this is done in a way that requires a tweak for every specific target. The functionality will work, and thus ought to be available, on any ELF target. The use of symbol_file_add_from_memory from other gdb code to support Linux's vDSO is already needed on more than one Linux target and it is not unlikely that other Linux targets will follow suit in the future. It is crufty and makes maintenance error prone if we have nothing better than copying and updating makefile boilerplate for every specific Linux/CPU target. For example, your patch covers only x86 and powerpc, but already IA64 and x86-64 need this support (and AFAIK not yet powerpc).