From owner-cvs-all@FreeBSD.ORG Thu Sep 16 00:36:30 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 37B2416A4CE for ; Thu, 16 Sep 2004 00:36:30 +0000 (GMT) Received: from duchess.speedfactory.net (duchess.speedfactory.net [66.23.201.84]) by mx1.FreeBSD.org (Postfix) with SMTP id A79D543D54 for ; Thu, 16 Sep 2004 00:36:29 +0000 (GMT) (envelope-from ups@tree.com) Received: (qmail 32313 invoked by uid 89); 16 Sep 2004 00:36:28 -0000 Received: from duchess.speedfactory.net (66.23.201.84) by duchess.speedfactory.net with SMTP; 16 Sep 2004 00:36:28 -0000 Received: (qmail 32289 invoked by uid 89); 16 Sep 2004 00:36:27 -0000 Received: from unknown (HELO palm.tree.com) (66.23.216.49) by duchess.speedfactory.net with SMTP; 16 Sep 2004 00:36:27 -0000 Received: from [127.0.0.1] (localhost.tree.com [127.0.0.1]) by palm.tree.com (8.12.10/8.12.10) with ESMTP id i8G0aQmt026652; Wed, 15 Sep 2004 20:36:27 -0400 (EDT) (envelope-from ups@tree.com) From: Stephan Uphoff To: Brian Fundakowski Feldman In-Reply-To: <20040915233915.GI36708@green.homeunix.org> References: <200409152326.i8FNQnOV009184@repoman.freebsd.org> <20040915233915.GI36708@green.homeunix.org> Content-Type: text/plain Message-Id: <1095294986.25502.16.camel@palm.tree.com> Mime-Version: 1.0 X-Mailer: Ximian Evolution 1.4.6 Date: Wed, 15 Sep 2004 20:36:26 -0400 Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: Julian Elischer cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/i386 gdb_machdep.c src/sys/i386/include db_machdep.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Sep 2004 00:36:30 -0000 Hi, your patch fixes the same problem concerning breakpoints in ddb. However by always using kdb_frame the wrong IP will be read when looking at different threads in ddb. Stephan On Wed, 2004-09-15 at 19:39, Brian Fundakowski Feldman wrote: > On Wed, Sep 15, 2004 at 11:26:49PM +0000, Julian Elischer wrote: > > julian 2004-09-15 23:26:49 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/i386/i386 gdb_machdep.c > > sys/i386/include db_machdep.h > > Log: > > Fix breakpoint handling for i386. > > not sure yet about 5.x... MFC if needed. > > Also fixes small problems with examining some registers and > > some specific gdb transfer problems. > > > > As the patch says: > > This is not a pretty patch and only meant as a temporary > > fix until a better solution is committed. > > > > PR: i386/71715 > > Submitted by: Stephan Uphoff > > MFC after: 1 week > > What about this change I'm still using with KDB so that break points > don't crash my system as immediately? > > --- include/db_machdep.h 10 Jul 2004 23:47:19 -0000 1.18 > +++ include/db_machdep.h 27 Aug 2004 23:14:12 -0000 > @@ -35,7 +35,7 @@ > typedef vm_offset_t db_addr_t; /* address - unsigned */ > typedef int db_expr_t; /* expression - signed */ > > -#define PC_REGS() ((db_addr_t)kdb_thrctx->pcb_eip) > +#define PC_REGS() ((db_addr_t)kdb_frame->tf_eip) > > #define BKPT_INST 0xcc /* breakpoint instruction */ > #define BKPT_SIZE (1) /* size of breakpoint inst */ > > -- > Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ > <> green@FreeBSD.org \ The Power to Serve! \ > Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\ > _______________________________________________ > cvs-src@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/cvs-src > To unsubscribe, send any mail to "cvs-src-unsubscribe@freebsd.org" > >