[lkml]   [2017]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 7/7] DWARF: add the config option
On 05/06/2017, 09:19 AM, Ingo Molnar wrote:
> * Linus Torvalds <> wrote:
>> On Fri, May 5, 2017 at 5:22 AM, Jiri Slaby <> wrote:
>>> The DWARF unwinder is in place and ready. So introduce the config option
>>> to allow users to enable it. It is by default off due to missing
>>> assembly annotations.
>> Who actually ends up using this?
> Also, why wasn't Josh Poimboeuf Cc:-ed, who de-facto maintains the x86 unwinding
> code?

I explicitly CCed Josh on 5/7 and 6/7 which touches the code. Besides
that, I assumed he is implicitly CCed via
which is carbon-copied on each of the patches.

> AFAICS this series is just repeating the old mistakes of the old Dwarf unwinders
> of trusting GCC's unwinder data. So NAK for the time being on the whole approach:
> NAcked-by: Ingo Molnar <>

OK, as the series stands now, we indeed do. Noteworthy, we, in SUSE, had
no problems with this reliance for all the time we have been using the

Anyway, objtool is about to vaidate the DWARF data, generate it for
assembly and potentially fix it if problems occur. Could you elaborate
on what else would help you to change your stance?

suse labs

 \ /
  Last update: 2017-05-10 21:21    [W:0.206 / U:16.772 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site