From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6770 invoked by alias); 14 Nov 2003 21:24:23 -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 6763 invoked from network); 14 Nov 2003 21:24:23 -0000 Received: from unknown (HELO touchme.toronto.redhat.com) (216.129.200.20) by sources.redhat.com with SMTP; 14 Nov 2003 21:24:23 -0000 Received: from redhat.com (toocool.toronto.redhat.com [172.16.14.72]) by touchme.toronto.redhat.com (Postfix) with ESMTP id B6B1D800046; Fri, 14 Nov 2003 16:24:22 -0500 (EST) Message-ID: <3FB54806.6060000@redhat.com> Date: Fri, 14 Nov 2003 21:24:00 -0000 From: "J. Johnston" Organization: Red Hat Inc. User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624 Netscape/7.1 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Andrew Cagney , Kevin Buettner Cc: "J. Johnston" , gdb-patches@sources.redhat.com, Daniel Jacobowitz , Marcel Moolenaar Subject: Re: RFA: ia64 portion of libunwind patch updated References: <3F986E31.8050201@redhat.com> <1031024175718.ZM3475@localhost.localdomain> <3F996D88.9060505@redhat.com> <1031024185625.ZM9827@localhost.localdomain> <3F9F0180.2010702@redhat.com> <20031029012833.GA11070@nevyn.them.org> <3FA043B2.6090401@redhat.com> <3FA7F97B.4090909@redhat.com> <3FAA830F.50607@redhat.com> In-Reply-To: <3FAA830F.50607@redhat.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-11/txt/msg00313.txt.bz2 In light of the recent checked-in patches concerning libunwind and the resolved issues, is the ia64-tdep.c patch ok now? The issue regarding whether libunwind could more efficiently reference the table entries will require an ABI change to libunwind and needs some thought from David. -- Jeff J. J. Johnston wrote: > The attached patch is the updated ia64 libunwind patch addressing a > number of comments. The bfd stuff has been replaced with a simple > target memory read. The syscall stuff has been replaced with a call to > target_read_partial() which has been implemented in another patch. I > have added a few comments per Elena's suggestion. > > Ok to commit? > > -- Jeff J. > > 2003-11-06 Jeff Johnston > David Mosberger > > * ia64-tdep.c: Include elf.h. > [HAVE_LIBUNWIND_IA64_H]: Include libunwind-frame.h and > libunwind-ia64.h. > (ia64_rse_slot_num, ia64_rse_skip_regs): New for libunwind support. > (ia64_gdb2uw_regnum, ia64_uw2gdb_regnum): Ditto. > (ia64_is_fpreg, ia64_access_reg): Ditto. > (ia64_access_fpreg, ia64_access_mem): Ditto. > (get_kernel_table): Ditto. > (ia64_find_unwind_table): Ditto. > (ia64_find_proc_info_x, ia64_put_unwind_info): Ditto. > (ia64_get_dyn_info_list, ia64_libunwind_frame_this_id): Ditto. > (ia64_libunwind_frame_prev_register): Ditto. > (ia64_libunwind_frame_sniffer): Ditto. > (ia64_gdbarch_init)[HAVE_LIBUNWIND_IA64_H]: Add libunwind frame > sniffer. Register libunwind functions needed by generic libunwind > frame > code using libunwind_frame_set_descr().