From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9575 invoked by alias); 8 May 2002 02:15:44 -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 9565 invoked from network); 8 May 2002 02:15:38 -0000 Received: from unknown (HELO cygnus.com) (205.180.83.203) by sources.redhat.com with SMTP; 8 May 2002 02:15:38 -0000 Received: from redhat.com (reddwarf.sfbay.redhat.com [172.16.24.50]) by runyon.cygnus.com (8.8.7-cygnus/8.8.7) with ESMTP id TAA11736; Tue, 7 May 2002 19:15:34 -0700 (PDT) Message-ID: <3CD88738.2E9B1BC4@redhat.com> Date: Tue, 07 May 2002 19:15:00 -0000 From: Michael Snyder Organization: Red Hat, Inc. X-Accept-Language: en MIME-Version: 1.0 To: Andrew Cagney CC: Daniel Jacobowitz , Michael Snyder , gdb-patches@sources.redhat.com Subject: Re: [RFA/RFC] Tweak for a gdb.mi test. References: <200205080109.g4819B821604@reddwarf.sfbay.redhat.com> <20020508013041.GA29600@nevyn.them.org> <3CD887A4.6010605@cygnus.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-SW-Source: 2002-05/txt/msg00194.txt.bz2 Andrew Cagney wrote: > > > On Tue, May 07, 2002 at 06:09:11PM -0700, Michael Snyder wrote: > > > >> > >> I'm gonna ask for a second pair of eyes, since I don't know MI > >> very well. > >> > >> What this is -- the test is examining the stack, but it is > >> assuming that main is the last frame. My change allows for > >> one extra frame below main (eg. for '_start'). > >> > >> OK to check in? > > > > > > Before you check this in, I would prefer to have a policy decision > > in place about whether we should show that frame or not. The relevant > > macro is FRAME_CHAIN_VALID; I believe we should universally (or almost > > universally) change this to stop at main. I think that's > > func_frame_chain_valid but don't trust my memory. > > (don't remember which function either, but) > Yes, I don't think the backtrace should go past main so I think the > change is wrong. > > I remember much debate about the test at the time (it was Cygnus > internal unfortunatly). The thing that clinched the deal was the > obeservation (made by a human factors person) that the behavour had to > be consistent across platforms. 1) It may have to be, but it isn't. 2) Why does it have to be? Lots of behaviors differ between platforms. 3) My real reason for submitting this fix is that it breaks with the "needs_status_wrapper" change that I submitted the other day. Of course, technically it would not break if gdb always stopped the trace at main, but it doesn't... > For a given OS (e.g. eCos, GNU/Linux, > ...) the backtrace should look identical, regardless of the ISA. > > Having each ISA making independant, and somewhat arbitrary, decisions is > wrong. Whether that's so or not, they do. Do you want to have this always fail for current gdb ports? > From memory, a suggestion was to let people select the back-trace > policy independant of the current architecture. I thought we also had a policy of not inserting tests that we knew would fail on some targets? Something about this being a regression test... Sorry, don't mean to be snippy...