From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2115 invoked by alias); 15 Oct 2003 22:58:09 -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 2108 invoked from network); 15 Oct 2003 22:58:08 -0000 Received: from unknown (HELO touchme.toronto.redhat.com) (207.219.125.105) by sources.redhat.com with SMTP; 15 Oct 2003 22:58:08 -0000 Received: from redhat.com (toocool.toronto.redhat.com [172.16.14.72]) by touchme.toronto.redhat.com (Postfix) with ESMTP id 1D5DB8003F5; Wed, 15 Oct 2003 18:58:08 -0400 (EDT) Message-ID: <3F8DD0FF.1020500@redhat.com> Date: Wed, 15 Oct 2003 22:58: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: Kevin Buettner Cc: gdb-patches@sources.redhat.com Subject: Re: RFA: change to ia64-linux-tdep.c References: <3F8DC432.40303@redhat.com> <1031015223519.ZM22998@localhost.localdomain> In-Reply-To: <1031015223519.ZM22998@localhost.localdomain> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-10/txt/msg00523.txt.bz2 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? -- Jeff J.