From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1422761AbbE2VkG (ORCPT ); Fri, 29 May 2015 17:40:06 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60218 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756266AbbE2Vj7 (ORCPT ); Fri, 29 May 2015 17:39:59 -0400 Date: Fri, 29 May 2015 17:39:56 -0400 From: "Frank Ch. Eigler" To: Josh Poimboeuf Cc: Andy Lutomirski , Ingo Molnar , Jan Beulich , Borislav Petkov , Peter Zijlstra , Ingo Molnar , Brian Gerst , =?iso-8859-1?Q?Fr=E9d=E9ric?= Weisbecker , Thomas Gleixner , Linus Torvalds , Denys Vlasenko , "linux-kernel@vger.kernel.org" , "H. Peter Anvin" , Masami Hiramatsu , Dave Anderson , x86@kernel.org Subject: Re: [PATCH] x86/debug: Remove perpetually broken, unmaintainable dwarf annotations Message-ID: <20150529213956.GA9869@redhat.com> References: <5566EBE7020000780007E659@mail.emea.novell.com> <20150528090133.GA469@gmail.com> <5566FFDD020000780007E711@mail.emea.novell.com> <20150528112017.GA28196@gmail.com> <55671D63020000780007E885@mail.emea.novell.com> <20150528131743.GA9496@gmail.com> <20150529202716.GA24561@treble.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150529202716.GA24561@treble.redhat.com> User-Agent: Mutt/1.4.2.2i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi - On Fri, May 29, 2015 at 03:27:16PM -0500, Josh Poimboeuf wrote: > [...] > > > Also, with the feature missing completely, maybe someone finds a method to > > > introduce it in a maintainable fashion, while with the feature included upstream > > > there's very little pressure to do that. As a bonus we'd also win a workable dwarf > > > unwinder. > > > > Before doing something drastic like this, I think we should get Josh's > > opinion, since I think he's working on a new (?) unwinder. > > I'd definitely like to replace all the asm DWARF CFI annotations with > something more automated and robust. So it doesn't really affect me > whether they're ripped out now or replaced later. > [...] > Then again, I'm not sure how useful or reliable the existing annotations > are anyway, so maybe it doesn't matter much. In our experience as consumers of this CFI information for years in systemtap, the annotations have been generally correct and reliable. Their presence allows reliable, correct, and efficient kernel->userspace backtracing as used in important systemtap scripts. If the current complaint is primarily about testability, it would be easy to add simple stap-based tests to the kernel to exercise the code and confirm its operation. Perhaps we could extract a specialized self-contained test case (containing an unwinder). I'm not in a position to judge the purported cost savings of removing this code, but there is definitely a negative benefit as a loss of useful functionality, esp. with no replacement in sight. - FChE