Re: F37 proposal: Add -fno-omit-frame-pointer to default compilation flags (System-Wide Change proposal)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/10/22 08:59, Michael Catanzaro wrote:
> On Wed, Nov 9 2022 at 09:39:04 AM +0100, Vitaly Zaitsev via devel 
> <devel@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
>> According to tests, this will slow down your system to 2.5%+, which is
>> unacceptable for a general purpose distribution.
> 
> Of course, without the frame pointers, profiling software is impossible 
> and performance engineers are unable to investigate performance 
> problems using Fedora. Seems kind of silly to block performance work 
> just for a 2.5% benefit....
> 
> Michael

Have you considered submitting kernel patches for DWARF unwinding?  The
actual unwinding can be done in the vdso, so bugs do not crash the kernel.
That eliminates the main reason for Linus’s objection IIUC.  It does mean
that user stack unwinding needs to move from NMI context to just before
system call return, but that is a good thing anyway as it allows faulting
in stack pages if needed.
-- 
Sincerely,
Demi Marie Obenour (she/her/hers)
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux