From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17844 invoked by alias); 15 Oct 2003 22:35:33 -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 17786 invoked from network); 15 Oct 2003 22:35:30 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 15 Oct 2003 22:35:30 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.11.6/8.11.6) with ESMTP id h9FMZUM25859 for ; Wed, 15 Oct 2003 18:35:30 -0400 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [172.16.52.156]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id h9FMZQr09015; Wed, 15 Oct 2003 18:35:26 -0400 Received: from localhost.localdomain (vpn50-39.rdu.redhat.com [172.16.50.39]) by pobox.corp.redhat.com (8.12.8/8.12.8) with ESMTP id h9FMZPwC009310; Wed, 15 Oct 2003 18:35:25 -0400 Received: (from kev@localhost) by localhost.localdomain (8.11.6/8.11.6) id h9FMZJX22999; Wed, 15 Oct 2003 15:35:19 -0700 Date: Wed, 15 Oct 2003 22:35:00 -0000 From: Kevin Buettner Message-Id: <1031015223519.ZM22998@localhost.localdomain> In-Reply-To: "J. Johnston" "RFA: change to ia64-linux-tdep.c" (Oct 15, 6:03pm) References: <3F8DC432.40303@redhat.com> To: "J. Johnston" , gdb-patches@sources.redhat.com Subject: Re: RFA: change to ia64-linux-tdep.c MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2003-10/txt/msg00516.txt.bz2 On Oct 15, 6:03pm, J. Johnston wrote: > The following changes the code in ia64_linux_sigcontext_register_address to use > the fact that the sigframe has the address of the sigcontext area stored at > offset 16 from the stack pointer. This change avoids using a magic constant to > find the start of the sigcontext area which may change in various kernel > versions as fields are added or subtracted to the sigframe. > > Code has been tested with signal handling test cases. > > The following is an excerpt of the linux kernel sigframe.h code: > > struct sigframe { > /* > * Place signal handler args where user-level unwinder can find them > easily. > * DO NOT MOVE THESE. They are part of the IA-64 Linux ABI and there > is * user-level code that depends on their presence! > */ > unsigned long arg0; /* signum */ > unsigned long arg1; /* siginfo pointer */ > unsigned long arg2; /* sigcontext pointer */ > /* > * End of architected state. > */ > > Ok to Commit? Yes. Definitely! Thanks for doing this. Kevin