From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8027 invoked by alias); 16 Oct 2003 00:43:15 -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 8017 invoked from network); 16 Oct 2003 00:43:15 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 16 Oct 2003 00:43:15 -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 h9G0hEM24281 for ; Wed, 15 Oct 2003 20:43:14 -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 h9G0hDr10031; Wed, 15 Oct 2003 20:43:13 -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 h9G0hCwC019938; Wed, 15 Oct 2003 20:43:12 -0400 Received: (from kev@localhost) by localhost.localdomain (8.11.6/8.11.6) id h9G0h7i23942; Wed, 15 Oct 2003 17:43:07 -0700 Date: Thu, 16 Oct 2003 00:43:00 -0000 From: Kevin Buettner Message-Id: <1031016004306.ZM23941@localhost.localdomain> In-Reply-To: "J. Johnston" "Re: RFA: change to ia64-linux-tdep.c" (Oct 15, 6:58pm) References: <3F8DC432.40303@redhat.com> <1031015223519.ZM22998@localhost.localdomain> <3F8DD0FF.1020500@redhat.com> To: "J. Johnston" , Kevin Buettner Subject: Re: RFA: change to ia64-linux-tdep.c Cc: gdb-patches@sources.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2003-10/txt/msg00529.txt.bz2 On Oct 15, 6:58pm, J. Johnston wrote: > Kevin Buettner wrote: > > 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 > > > > Patch checked into mainline. Should patch be checked into gdb 6.0? It can be if you want it to be. (6.0 is released, so it's not clear if there's much value in checking it into the branch...) Kevin