From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32257 invoked by alias); 27 Nov 2012 14:42:06 -0000 Received: (qmail 32242 invoked by uid 22791); 27 Nov 2012 14:42:05 -0000 X-SWARE-Spam-Status: No, hits=-3.2 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from sibelius.xs4all.nl (HELO glazunov.sibelius.xs4all.nl) (83.163.83.176) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 27 Nov 2012 14:41:58 +0000 Received: from glazunov.sibelius.xs4all.nl (kettenis@localhost [127.0.0.1]) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3) with ESMTP id qAREfmDL011798; Tue, 27 Nov 2012 15:41:48 +0100 (CET) Received: (from kettenis@localhost) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3/Submit) id qAREfltJ031926; Tue, 27 Nov 2012 15:41:47 +0100 (CET) Date: Tue, 27 Nov 2012 14:42:00 -0000 Message-Id: <201211271441.qAREfltJ031926@glazunov.sibelius.xs4all.nl> From: Mark Kettenis To: markus.t.metzger@intel.com CC: gdb-patches@sourceware.org, markus.t.metzger@gmail.com, jan.kratochvil@redhat.com, palves@redhat.com, tromey@redhat.com In-reply-to: (markus.t.metzger@intel.com) Subject: Re: [patch v4 13/13] btrace, x86: restrict to Atom References: <1354013351-14791-1-git-send-email-markus.t.metzger@intel.com> <1354013351-14791-14-git-send-email-markus.t.metzger@intel.com> <201211271119.qARBJ7KS012770@glazunov.sibelius.xs4all.nl> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-11/txt/msg00738.txt.bz2 > From: "Metzger, Markus T" > Date: Tue, 27 Nov 2012 11:49:09 +0000 > > > > From: > > > Date: Tue, 27 Nov 2012 11:49:11 +0100 > > > > > > From: Markus Metzger > > > > > > Restrict branch tracing support to Atom processors. > > > > > > 2012-11-27 Markus Metzger > > > > > > * amd64-linux-nat.c (amd64_linux_supports_btrace): New. > > > (_initialize_amd64_linux_nat): Change supports_btrace method. > > > * i386-linux-nat.c (i386_linux_supports_btrace): New. > > > (_initialize_i386_linux_nat): Change supports_btrace method. > > > > > > gdbserver/ > > > * linux-low.h (linux_target_ops): Add supports_btrace method. > > > * linux-low.c (linux_low_supports_btrace): New. > > > (linux_target_ops): Change supports_btrace method. > > > * linux-x86-low.c (x86_supports_btrace): New. > > > (the_low_target): Add supports_btrace method. > > > > This is still unacceptable. This needs to be fixed in the Linux kernel. > > We will still need this patch in gdb so we handle kernels that do > not contain the fix correctly. Frankly I don't see why. I just consider this to be a bug in a recently introduced new kernel feature. People should just upgrade to a kernel that has this bug fixed. We can't add workarounds for every Linux kernel bug. And in this case it is especially bad since you're restricting GDB to supporting this feature on specific CPU models even on systems that support the feature on other CPU types. I feel (rather strongly) that this is bad.