Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1695531

Re: [PATCH v4 2/2] x86/kconfig: make it easier to switch to the new ORC unwinder

Show all headers | View raw

* Josh Poimboeuf <jpoimboe@redhat.com> wrote:

> A couple of Kconfig changes which make it much easier to switch to the
> new CONFIG_ORC_UNWINDER:
> 
> 1) Remove x86 dependencies on CONFIG_FRAME_POINTER for lockdep,
>    latencytop, and fault injection.  x86 has a 'guess' unwinder which
>    just scans the stack for kernel text addresses.  It's not 100%
>    accurate but in many cases it's good enough.  This allows those users
>    who don't want the text overhead of the frame pointer or ORC
>    unwinders to still use these features.  More importantly, this also
>    makes it much more straightforward to disable frame pointers.
> 
> 2) Make CONFIG_ORC_UNWINDER depend on !CONFIG_FRAME_POINTER.  While it
>    would be possible to have both enabled, it doesn't really make sense
>    to do so.  So enforce a sane configuration to prevent the user from
>    making a dumb mistake.
> 
> With these changes, when you disable CONFIG_FRAME_POINTER, "make
> oldconfig" will ask if you want to enable CONFIG_ORC_UNWINDER.

Yeah, so I think this is still suboptimal: the frame pointer and the Orc unwinders 
are configured in different places, and the user won't know about the various 
unwinder options unless stumbling across them by accidentally disabling frame 
pointers ...

Also, the Kconfig help text for frame pointers is now actively misleading:

  CONFIG_FRAME_POINTER:

  If you say Y here the resulting kernel image will be slightly
  larger and slower, but it gives very useful debugging information
  in case of kernel bugs. (precise oopses/stacktraces/warnings)

Please, as I suggested it before, make it a multiple choice option: frame, Orc, or 
the guess unwinder.

I'd only offer the 'guess' unwinder if EXPERT is selected, because it's a really 
sub-optimal selection all things considered.

Once things are tested and it's all rosy we can change the default x86 unwinder to 
Orc and organize the naming of the config variables to:

	CONFIG_UNWINDER_ORC
	CONFIG_UNWINDER_FRAME_POINTER
	CONFIG_UNWINDER_GUESS

Thanks,

	Ingo

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v4 0/2] ORC unwinder Josh Poimboeuf <jpoimboe@redhat.com> - 2017-07-25 01:40 +0200
  [PATCH v4 2/2] x86/kconfig: make it easier to switch to the new ORC unwinder Josh Poimboeuf <jpoimboe@redhat.com> - 2017-07-25 01:40 +0200
    Re: [PATCH v4 2/2] x86/kconfig: make it easier to switch to the new  ORC unwinder Ingo Molnar <mingo@kernel.org> - 2017-07-25 11:20 +0200
      Re: [PATCH v4 2/2] x86/kconfig: make it easier to switch to the new  ORC unwinder Josh Poimboeuf <jpoimboe@redhat.com> - 2017-07-25 16:00 +0200
        [tip:x86/asm] x86/kconfig: Consolidate unwinders into multiple  choice selection tip-bot for Josh Poimboeuf <tipbot@zytor.com> - 2017-07-26 14:20 +0200
    [tip:x86/asm] x86/kconfig: Make it easier to switch to the new ORC  unwinder tip-bot for Josh Poimboeuf <tipbot@zytor.com> - 2017-07-26 14:20 +0200
  Re: [PATCH v4 1/2] x86/unwind: add ORC unwinder "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> - 2017-07-28 18:50 +0200
    Re: [PATCH v4 1/2] x86/unwind: add ORC unwinder Josh Poimboeuf <jpoimboe@redhat.com> - 2017-07-28 20:00 +0200
      Re: [PATCH v4 1/2] x86/unwind: add ORC unwinder "Levin, Alexander (Sasha Levin)" <alexander.levin@verizon.com> - 2017-07-28 20:40 +0200
        Re: [PATCH v4 1/2] x86/unwind: add ORC unwinder Josh Poimboeuf <jpoimboe@redhat.com> - 2017-07-28 21:00 +0200
          Re: [PATCH v4 1/2] x86/unwind: add ORC unwinder Josh Poimboeuf <jpoimboe@redhat.com> - 2017-07-29 06:00 +0200

csiph-web