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

2022-11-29 Thread Maxwell G via devel
On Tue Nov 29, 2022 at 12:57 CST, Jeremy Linton wrote: > Hi, > > > On 6/16/22 15:53, Ben Cotton wrote: > > https://fedoraproject.org/wiki/Changes/fno-omit-frame-pointer > > > > This document represents a proposed Change. As part of the Changes > > process, proposals are publicly announced in

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

2022-11-29 Thread Vitaly Zaitsev via devel
On 29/11/2022 19:57, Jeremy Linton wrote: Why not turn this on just for rawhide and leave it off for the main distro releases? Mass rebuild is a huge pain for maintainers due to FTBFS issues, and doing it multiple times is unacceptable. -- Sincerely, Vitaly Zaitsev (vit...@easycoding.org)

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

2022-11-29 Thread Jeremy Linton
Hi, On 6/16/22 15:53, Ben Cotton wrote: https://fedoraproject.org/wiki/Changes/fno-omit-frame-pointer This document represents a proposed Change. As part of the Changes process, proposals are publicly announced in order to receive community feedback. This proposal will only be implemented if

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

2022-11-22 Thread Demi Marie Obenour
On 11/22/22 10:19, Florian Weimer wrote: > * Neal Gompa: > >> On Tue, Nov 22, 2022 at 6:54 AM Florian Weimer wrote: >>> >>> Why isn't this something that is up to the toolchain team to decide? >>> >> >> The toolchain team doesn't work with the full corpus of packages, >> doesn't really interface

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

2022-11-22 Thread Florian Weimer
* Neal Gompa: > On Tue, Nov 22, 2022 at 6:54 AM Florian Weimer wrote: >> >> Why isn't this something that is up to the toolchain team to decide? >> > > The toolchain team doesn't work with the full corpus of packages, > doesn't really interface with most of the packagers, and doesn't work > with

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

2022-11-22 Thread Neal Gompa
On Tue, Nov 22, 2022 at 6:54 AM Florian Weimer wrote: > > Why isn't this something that is up to the toolchain team to decide? > The toolchain team doesn't work with the full corpus of packages, doesn't really interface with most of the packagers, and doesn't work with most of the upstreams that

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

2022-11-22 Thread Florian Weimer
Why isn't this something that is up to the toolchain team to decide? What makes FESCo more qualified than the toolchain maintainers to determine the best approach for Fedora? Thanks, Florian ___ devel mailing list -- devel@lists.fedoraproject.org To

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

2022-11-18 Thread Naheem Zaffar
> > what is the impact of the proposed change on non-x86 platforms? I > > assume the proposal focuses on x86, but the distro wide flags are shared > > across all platforms in Fedora, it means aarch64, ppc64le and s390x. > > With RISC-V waiting behind the door ... > > None, because this proposal is

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

2022-11-10 Thread Demi Marie Obenour
On 11/10/22 08:59, Michael Catanzaro wrote: > On Wed, Nov 9 2022 at 09:39:04 AM +0100, Vitaly Zaitsev via devel > 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,

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

2022-11-10 Thread Kevin Fenzi
On Wed, Nov 09, 2022 at 04:26:23AM +, Naheem Zaffar wrote: > On Tue, 8 Nov 2022, 19:22 Vitaly Zaitsev via devel, < > devel@lists.fedoraproject.org> wrote: > > > On 08/11/2022 19:53, Naheem Zaffar wrote: > > > Has there been any consideration to turn on frame pointers for atleast > > > dev

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

2022-11-10 Thread Vitaly Zaitsev via devel
On 10/11/2022 14:59, Michael Catanzaro wrote: Of course, without the frame pointers, profiling software is impossible and performance engineers are unable to investigate performance problems using Fedora. 99.999% shouldn't suffer because of 0.001%. -- Sincerely, Vitaly Zaitsev

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

2022-11-10 Thread Neal Gompa
On Thu, Nov 10, 2022 at 9:00 AM Michael Catanzaro wrote: > > On Wed, Nov 9 2022 at 09:39:04 AM +0100, Vitaly Zaitsev via devel > 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

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

2022-11-10 Thread Michael Catanzaro
On Wed, Nov 9 2022 at 09:39:04 AM +0100, Vitaly Zaitsev via devel 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

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

2022-11-09 Thread Sérgio Basto
On Wed, 2022-11-09 at 19:27 -0500, Demi Marie Obenour wrote: > On 11/9/22 05:21, Dan Horák wrote: > > On Tue, 01 Nov 2022 11:26:13 - > > Daan De Meyer via devel wrote: > > > > > I've added a new section to the proposal with the benchmark > > > results of some benchmarks we performed against

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

2022-11-09 Thread Demi Marie Obenour
On 11/9/22 05:21, Dan Horák wrote: > On Tue, 01 Nov 2022 11:26:13 - > Daan De Meyer via devel wrote: > >> I've added a new section to the proposal with the benchmark results of some >> benchmarks we performed against a Fedora 37 system built with frame pointers >> and a regular Fedora 37

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

2022-11-09 Thread Florian Weimer
* Demi Marie Obenour: > On 11/8/22 18:46, Frank Ch. Eigler wrote: >> Demi Marie Obenour writes: >> >>> Three other options I can think of: >>> [...] >> >> Another one: >> >> 4. Speed up out-of-context backtracer(s), possibly consuming >> kernel-perf-ringbuffer stack dumps, or possibly

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

2022-11-09 Thread Dan Horák
On Tue, 01 Nov 2022 11:26:13 - Daan De Meyer via devel wrote: > I've added a new section to the proposal with the benchmark results of some > benchmarks we performed against a Fedora 37 system built with frame pointers > and a regular Fedora 37 system. The impact on most benchmarks seems

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

2022-11-09 Thread Daniel P . Berrangé
On Wed, Nov 09, 2022 at 04:26:23AM +, Naheem Zaffar wrote: > On Tue, 8 Nov 2022, 19:22 Vitaly Zaitsev via devel, < > devel@lists.fedoraproject.org> wrote: > > > On 08/11/2022 19:53, Naheem Zaffar wrote: > > > Has there been any consideration to turn on frame pointers for atleast > > > dev

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

2022-11-09 Thread Vitaly Zaitsev via devel
On 09/11/2022 05:26, Naheem Zaffar wrote: Not all builds in rawhide are release builds. You will get for instance the whole gnome stack beta releases and rc releases during development. To achieve the goal, you must rebuild every single package in a dependency tree, including such important

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

2022-11-08 Thread Naheem Zaffar
On Tue, 8 Nov 2022, 19:22 Vitaly Zaitsev via devel, < devel@lists.fedoraproject.org> wrote: > On 08/11/2022 19:53, Naheem Zaffar wrote: > > Has there been any consideration to turn on frame pointers for atleast > > dev releases? > > > Fedora has no dev releases. Mass rebuild is a huge pain for

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

2022-11-08 Thread Demi Marie Obenour
On 11/8/22 18:46, Frank Ch. Eigler wrote: > Demi Marie Obenour writes: > >> Three other options I can think of: >> [...] > > Another one: > > 4. Speed up out-of-context backtracer(s), possibly consuming > kernel-perf-ringbuffer stack dumps, or possibly using another > event source

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

2022-11-08 Thread Frank Ch. Eigler
Demi Marie Obenour writes: > Three other options I can think of: > [...] Another one: 4. Speed up out-of-context backtracer(s), possibly consuming kernel-perf-ringbuffer stack dumps, or possibly using another event source to trigger and work via ptrace and /proc/$pid/mem - FChE

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

2022-11-08 Thread Vitaly Zaitsev via devel
On 08/11/2022 19:53, Naheem Zaffar wrote: Has there been any consideration to turn on frame pointers for atleast dev releases? Fedora has no dev releases. Mass rebuild is a huge pain for maintainers due to FTBFS issues, and doing it multiple times is unacceptable. If someone needs extended

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

2022-11-08 Thread Daniel P . Berrangé
On Tue, Nov 08, 2022 at 06:53:46PM +, Naheem Zaffar wrote: > Has there been any consideration to turn on frame pointers for atleast dev > releases? > > AFAIK the kernel already has a separate configuration during the > development phase of the distro-cycle This practice already causes

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

2022-11-08 Thread Naheem Zaffar
Has there been any consideration to turn on frame pointers for atleast dev releases? AFAIK the kernel already has a separate configuration during the development phase of the distro-cycle Can this be extended for rebuilds of alpha and beta releases in rawhide to turn on frame pointers as an

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

2022-11-07 Thread Demi Marie Obenour
On 11/7/22 09:59, Daniel Alley wrote: > The references of the proposal has some discussion on this subject. > Basically, this was already considered and rejected by the kernel > developers including Torvalds directly on the basis that DWARF > unwinding support is simply too complex and

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

2022-11-07 Thread Daniel Alley
> On 11/1/22 07:38, Dominique Martinet wrote: > > Has adding kernel support for DWARF unwinding been considered instead? The references of the proposal has some discussion on this subject. Basically, this was already considered and rejected by the kernel developers including Torvalds directly

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

2022-11-01 Thread Demi Marie Obenour
On 11/1/22 07:38, Dominique Martinet wrote: > Daan De Meyer via devel wrote on Tue, Nov 01, 2022 at 11:26:13AM -: >> I've added a new section to the proposal with the benchmark results of >> some benchmarks we performed against a Fedora 37 system built with >> frame pointers and a regular

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

2022-11-01 Thread Dominique Martinet
Daan De Meyer via devel wrote on Tue, Nov 01, 2022 at 11:26:13AM -: > I've added a new section to the proposal with the benchmark results of > some benchmarks we performed against a Fedora 37 system built with > frame pointers and a regular Fedora 37 system. The impact on most > benchmarks

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

2022-11-01 Thread Daan De Meyer via devel
I've added a new section to the proposal with the benchmark results of some benchmarks we performed against a Fedora 37 system built with frame pointers and a regular Fedora 37 system. The impact on most benchmarks seems limited aside from the CPython benchmark suite (pyperformance). See the

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

2022-07-13 Thread Christian Hergert
It's not even a question worth asking because it's both impractical and unlikely to actually fix the situation we have. ___ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora

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

2022-07-12 Thread Sven Kieske
On So, 2022-07-10 at 10:36 -0700, Gordon Messmer wrote: > On 7/10/22 04:38, Vitaly Zaitsev via devel wrote: > > Have you rebuilt all system packages with -fno-omit-frame-pointer or > > just tested packages? > > > No.  Early in the thread, Tomasz Torcz posted a link to a Phoronix > article as

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

2022-07-11 Thread Daniel Alley
> Unfortunately, no, there's no in-kernel DWARF unwinder due to the complexity > involved. > Instead, the kernel uses ORC and has an unwinder for that. Adding ORC support > to all of > Linux userspace so that we can unwind it in the kernel isn't likely to > happen, since > all tooling would

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

2022-07-11 Thread Christian Hergert
> My comment was specifically about sysprof. I've been told that the > GNOME developers will not even consider anything else. This means that > we need to fix sysprof. If we do that, it will be possible to use GNOME > OS for profiling on older CPUs, and hardware-assisted backtraces on > newer

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

2022-07-11 Thread Daan De Meyer via devel
> You still might have LBR buffers deep enough for your purposes, I think > that's worth checking. They have been around for much longer (on > Intel). We've been using LBR opportunistically for a while if available to augment frame pointer based stacks. It turns out to be quite helpful at the

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

2022-07-11 Thread Florian Weimer
* Daan De Meyer via devel: >> If we can get SHSTK to work, the value of the DWARF integration and >> performance work will diminish fairly quickly because most developers >> will soon have CPUs with fairly deep (32 entry) LBR buffers, SHSTK >> support, or both. > > This seems like a fairly bold

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

2022-07-11 Thread Daan De Meyer via devel
ware support. Cheers, Daan From: Florian Weimer Sent: 11 July 2022 07:12 To: Matthias Clasen Cc: Development discussions related to Fedora Subject: Re: F37 proposal: Add -fno-omit-frame-pointer to default compilation flags (System-Wide Change proposal) * Mat

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

2022-07-11 Thread Florian Weimer
* Matthias Clasen: > On Wed, Jul 6, 2022 at 3:06 PM Florian Weimer wrote: > >> If the GNOME's sysprof does not work with Fedora, fix it or use >> something else. Do not change how Fedora is built. > > The result of that attitude is that performance work in the desktop > space is happening on

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

2022-07-10 Thread Florian Weimer
* Demi Marie Obenour: > That is the problem right here: .eh_frame-based unwinding is too slow, > so it has to be done offline in userspace. What about instead adding > ORC information to userspace? That would be much faster to use. I'm not sure ORC covers all the registers that could be used

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

2022-07-10 Thread Florian Weimer
* Fabio Valentini: > On Fri, Jul 8, 2022 at 9:20 PM Christian Hergert wrote: >> >> > So it looks like what you folks are doing is actually very similar to what >> > Facebook is doing. That is interesting, and explains why some GNOME >> > developers are jumping on the bandwagon of this Change

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

2022-07-10 Thread Jeff Law
On 7/10/2022 11:36 AM, Gordon Messmer wrote: On 7/10/22 04:38, Vitaly Zaitsev via devel wrote: Have you rebuilt all system packages with -fno-omit-frame-pointer or just tested packages? No.  Early in the thread, Tomasz Torcz posted a link to a Phoronix article as evidence that Fedora's

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

2022-07-10 Thread Gordon Messmer
On 7/10/22 10:55, Vitaly Zaitsev via devel wrote: On 10/07/2022 19:36, Gordon Messmer wrote: No.  Early in the thread, Tomasz Torcz posted a link to a Phoronix article as evidence that Fedora's performance was behind other distributions. All packages in the dependency tree must be rebuilt

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

2022-07-10 Thread Vitaly Zaitsev via devel
On 10/07/2022 19:36, Gordon Messmer wrote: No.  Early in the thread, Tomasz Torcz posted a link to a Phoronix article as evidence that Fedora's performance was behind other distributions. All packages in the dependency tree must be rebuilt with -fno-omit-frame-pointer flag, otherwise the

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

2022-07-10 Thread Gordon Messmer
On 7/10/22 04:38, Vitaly Zaitsev via devel wrote: Have you rebuilt all system packages with -fno-omit-frame-pointer or just tested packages? No.  Early in the thread, Tomasz Torcz posted a link to a Phoronix article as evidence that Fedora's performance was behind other distributions.  At

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

2022-07-10 Thread Demi Marie Obenour
On 7/9/22 12:05, Christian Hergert wrote: >> Why does the unwinding need to happen in the kernel? The kernel can >> already asynchronously invoke userspace code in the form of signal >> handlers. Is the problem that it is necessary to collect profiling >> information in the middle of a system

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

2022-07-10 Thread drago01
On Sunday, July 10, 2022, Kevin Kofler via devel < devel@lists.fedoraproject.org> wrote: > drago01 wrote: > > > On Sunday, July 10, 2022, Kevin Kofler via devel < > > devel@lists.fedoraproject.org> wrote: > > > >> drago01 wrote: > >> > One thing which seems to be completely missing in the

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

2022-07-10 Thread Kevin Kofler via devel
drago01 wrote: > On Sunday, July 10, 2022, Kevin Kofler via devel < > devel@lists.fedoraproject.org> wrote: > >> drago01 wrote: >> > One thing which seems to be completely missing in the conversation is >> > how this affects battery life. Having lower performance, means the CPU >> > has to do

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

2022-07-10 Thread drago01
On Sunday, July 10, 2022, Kevin Kofler via devel < devel@lists.fedoraproject.org> wrote: > drago01 wrote: > > One thing which seems to be completely missing in the conversation is how > > this affects battery life. Having lower performance, means the CPU has to > > do more work / be in a higher

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

2022-07-10 Thread Vitaly Zaitsev via devel
On 09/07/2022 22:32, Gordon Messmer wrote: I cannot replicate any of the differences that I would expect to be able to.  More than anything else, their results look like evidence of a bug in the Xeon Platinum 8380. Have you rebuilt all system packages with -fno-omit-frame-pointer or just

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

2022-07-10 Thread Kevin Kofler via devel
drago01 wrote: > One thing which seems to be completely missing in the conversation is how > this affects battery life. Having lower performance, means the CPU has to > do more work / be in a higher power state for a longer time frame. Who in their right mind does CPU profiling on battery?

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

2022-07-10 Thread drago01
One thing which seems to be completely missing in the conversation is how this affects battery life. Having lower performance, means the CPU has to do more work / be in a higher power state for a longer time frame. ___ devel mailing list --

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

2022-07-10 Thread Daan De Meyer via devel
Demi Marie Obenour Sent: 09 July 2022 04:02 To: devel@lists.fedoraproject.org Subject: Re: F37 proposal: Add -fno-omit-frame-pointer to default compilation flags (System-Wide Change proposal) On 7/8/22 20:18, Christian Hergert wrote: >> That is the problem right here: .eh_frame-based un

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

2022-07-09 Thread Gordon Messmer
On 6/21/22 21:40, Gordon Messmer wrote: On 6/21/22 13:10, Matthew Miller wrote: Phoronix credits this to those distros shipping with P-state Performance by default. Yes, but I doubt that for several reasons: First, it's a claim without evidence.  That setting isn't the only difference

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

2022-07-09 Thread Christian Hergert
> Why does the unwinding need to happen in the kernel? The kernel can > already asynchronously invoke userspace code in the form of signal > handlers. Is the problem that it is necessary to collect profiling > information in the middle of a system call, where another syscall > would see

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

2022-07-08 Thread Demi Marie Obenour
On 7/8/22 20:18, Christian Hergert wrote: >> That is the problem right here: .eh_frame-based unwinding is too slow, so it >> has to be >> done offline in userspace. What about instead adding ORC information to >> userspace? That >> would be much faster to use. > > I'm not familiar with ORC,

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

2022-07-08 Thread Christian Hergert
> That is the problem right here: .eh_frame-based unwinding is too slow, so it > has to be > done offline in userspace. What about instead adding ORC information to > userspace? That > would be much faster to use. I'm not familiar with ORC, but there are a few things that initially come to

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

2022-07-08 Thread Demi Marie Obenour
On 7/8/22 15:29, Christian Hergert wrote: >> Frank Ch. Eigler mentions that elfutils has a more modern unwinding library. >> Could that perhaps solve your performance issues with libunwind? > > I don't think so. The problem is two-fold. > > First, we have to capture enough of the stack to do

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

2022-07-08 Thread Fabio Valentini
On Fri, Jul 8, 2022 at 9:20 PM Christian Hergert wrote: > > > So it looks like what you folks are doing is actually very similar to what > > Facebook is doing. That is interesting, and explains why some GNOME > > developers are jumping on the bandwagon of this Change proposal. > > To be fair,

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

2022-07-08 Thread Christian Hergert
> Frank Ch. Eigler mentions that elfutils has a more modern unwinding library. > Could that perhaps solve your performance issues with libunwind? I don't think so. The problem is two-fold. First, we have to capture enough of the stack to do offline unwinding. I think the default many people do

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

2022-07-08 Thread Christian Hergert
> So it looks like what you folks are doing is actually very similar to what > Facebook is doing. That is interesting, and explains why some GNOME > developers are jumping on the bandwagon of this Change proposal. To be fair, we've been complaining about it internally in GNOME for probably

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

2022-07-08 Thread Ben Beasley
I don’t think we should be gatekeeping who can propose or discuss a Change. I do think that the opinions of the upstream and downstream GCC maintainers should be weighed quite heavily when considering a change to the default compiler flags. As a FESCo member, I’m waiting to see if the Change

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

2022-07-08 Thread Demi Marie Obenour
On 7/7/22 16:13, Christian Hergert wrote: > Sysprof has modular data collection backends, and not everything requires > linking against libunwind. > > For those not familiar with Sysprof, or profiling the desktop at large, > generally a single program is not the problem. The performance

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

2022-07-08 Thread Kevin Kofler via devel
Theodore Papadopoulo wrote: > gdb and similar tools are confused, notably because local variables are > "optimsed out" (which I suspect is related to frame pointer). It is not. This is related to compiling with any optimization at all, and you definitely do not want production binaries compiled

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

2022-07-08 Thread Daniel P . Berrangé
On Thu, Jul 07, 2022 at 08:13:52PM -, Christian Hergert wrote: > Sysprof has modular data collection backends, and not everything > requires linking against libunwind. > > For those not familiar with Sysprof, or profiling the desktop at > large, generally a single program is not the problem.

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

2022-07-08 Thread Kevin Kofler via devel
PS: One more question: Christian Hergert wrote: > We have an in-tree parser for ELF that allows us to avoid a lot of > extraneous code when extracting symbols. Partially because libunwind is > incredibly slow (by profiler requirements), and partially because > historically we never had to stash

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

2022-07-08 Thread Kevin Kofler via devel
Christian Hergert wrote: > For those not familiar with Sysprof, or profiling the desktop at large, > generally a single program is not the problem. The performance problems > often exist across a number of processes. […] > The most used collector, however, is the perf collector which is just >

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

2022-07-08 Thread Theodore Papadopoulo
On 7/6/22 08:30, Vitaly Zaitsev via devel wrote: On 05/07/2022 21:15, Matthias Clasen wrote: And I doubt that you'd be able to notice a 'smaller than 1% slowdown' on your system. 4% slowdown is unacceptable. At least for Fedora Workstation, being a useful system for developers with working

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

2022-07-07 Thread Demi Marie Obenour
On 7/7/22 10:46, Frank Ch. Eigler wrote: > > ngompa13 wrote: > >> [...] >> I agree, this is a completely unacceptable statement to make. The >> problem isn't sysprof, the problem is that profiling is garbage on >> Linux by default. > > That's an overstatement. > >> And while maybe most

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

2022-07-07 Thread Michael Catanzaro
On Tue, Jul 5 2022 at 01:42:05 PM +0200, Fabio Valentini wrote: No - I think the problem is that adding those flags to the default build configuration will affect the whole system - all executables and shared libraries, not only "leaf" binaries. And that makes your benchmarks (and those run by

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

2022-07-07 Thread Christian Hergert
Sysprof has modular data collection backends, and not everything requires linking against libunwind. For those not familiar with Sysprof, or profiling the desktop at large, generally a single program is not the problem. The performance problems often exist across a number of processes. That

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

2022-07-07 Thread Frank Ch. Eigler
ngompa13 wrote: > [...] > I agree, this is a completely unacceptable statement to make. The > problem isn't sysprof, the problem is that profiling is garbage on > Linux by default. That's an overstatement. > And while maybe most developers may not bother to do profiling right > now, we don't

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

2022-07-07 Thread Neal Gompa
On Thu, Jul 7, 2022 at 7:43 AM Matthias Clasen wrote: > > > > On Wed, Jul 6, 2022 at 3:06 PM Florian Weimer wrote: > >> >> If the GNOME's sysprof does not >> work with Fedora, fix it or use something else. Do not change how >> Fedora is built. > > > The result of that attitude is that

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

2022-07-07 Thread Matthias Clasen
On Wed, Jul 6, 2022 at 3:06 PM Florian Weimer wrote: > If the GNOME's sysprof does not > work with Fedora, fix it or use something else. Do not change how > Fedora is built. > The result of that attitude is that performance work in the desktop space is happening on GNOME OS images, or in

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

2022-07-06 Thread Frank Ch. Eigler
Michael Catanzaro writes: > I can point you to documentation for sysprof: > https://gitlab.gnome.org/GNOME/sysprof#debugging-symbols > which says that every library should be built with > -fno-omit-frame-pointer. Given that sysprof is a userspace program, it's not in a giant rush, so it should

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

2022-07-06 Thread Jeff Law
On 7/6/2022 1:05 PM, Florian Weimer wrote: * Michael Catanzaro: On Wed, Jul 6 2022 at 08:06:45 AM -0600, Jeff Law wrote: If I'm understanding things correctly, the original proposal is trying to make a very special case of profiling work better -- a case that 99.9% of Fedora users do not

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

2022-07-06 Thread Kevin Kofler via devel
Michael Catanzaro wrote: > I can point you to documentation for sysprof: > > https://gitlab.gnome.org/GNOME/sysprof#debugging-symbols > > which says that every library should be built with > -fno-omit-frame-pointer. And why is that? Do they not use libunwind, or GDB, or any other sane

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

2022-07-06 Thread Kevin Kofler via devel
Michael Catanzaro wrote: > Problem is that in order to get good profiling results today, you need > to rebuild all dependencies with frame pointers enabled. And that is > not realistic. Nobody does that. Actually, the Facebook developers, the ones who are proposing this very Change, claim that

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

2022-07-06 Thread Kevin Kofler via devel
Marek Polacek wrote: > On Tue, Jul 05, 2022 at 03:47:26PM -0400, Matthias Clasen wrote: >> (Un)acceptable for whom? > > GCC maintainers in Fedora, at least. What I do not understand is why a Change that wants to change the default GCC flags is even under discussion at all without the buy-in

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

2022-07-06 Thread Kevin Kofler via devel
Dan Čermák wrote: > Please never run ASAN in production workloads: > https://www.openwall.com/lists/oss-security/2016/02/17/9 > > tl;dr; you'll create a local root exploit. Oh, the joys of automagically added insecure environment variable handlers… Good to know! Kevin Kofler

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

2022-07-06 Thread Daan De Meyer via devel
I've just updated the proposal with an extended description describing the use cases enabled by frame pointers in more details. More specifically, on top of describing the profiling use case in much more detail, I've also added a section on BPF debugging tooling, such as bcc and bpftrace, which

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

2022-07-06 Thread Florian Weimer
* Michael Catanzaro: > On Wed, Jul 6 2022 at 08:06:45 AM -0600, Jeff Law > wrote: >> If I'm understanding things correctly, the original proposal is trying >> to make a very special case of profiling work better -- a case that >> 99.9% of Fedora users do not need or care about.That seems

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

2022-07-06 Thread Florian Weimer
* Jeff Law: > If I'm understanding things correctly, the original proposal is trying > to make a very special case of profiling work better -- a case that > 99.9% of Fedora users do not need or care about.    That seems like a > particularly bad cost/benefit for this proposal. It became clear

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

2022-07-06 Thread Michael Catanzaro
On Wed, Jul 6 2022 at 10:05:17 AM -0700, Tom Stellard wrote: With the current profiling methods, are you able to at least narrow down which libraries applications spend the most time in? Or do you really need detailed profile information for every single library in order to determine where

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

2022-07-06 Thread Tom Stellard
On 7/6/22 08:42, Michael Catanzaro wrote: On Wed, Jul 6 2022 at 04:20:50 PM +0100, Jonathan Wakely wrote: You build locally and profile using your locally built packages. Problem is that in order to get good profiling results today, you need to rebuild all dependencies with frame pointers

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

2022-07-06 Thread Tom Stellard
On 7/6/22 08:20, Jonathan Wakely wrote: On Wed, 6 Jul 2022 at 15:57, Jeff Law wrote: On 7/6/2022 8:20 AM, Michael Catanzaro wrote: On Wed, Jul 6 2022 at 08:06:45 AM -0600, Jeff Law wrote: If I'm understanding things correctly, the original proposal is trying to make a very special case

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

2022-07-06 Thread Michael Catanzaro
On Wed, Jul 6 2022 at 04:20:50 PM +0100, Jonathan Wakely wrote: You build locally and profile using your locally built packages. Problem is that in order to get good profiling results today, you need to rebuild all dependencies with frame pointers enabled. And that is not realistic. Nobody

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

2022-07-06 Thread Jonathan Wakely
On Wed, 6 Jul 2022 at 15:57, Jeff Law wrote: > > > > On 7/6/2022 8:20 AM, Michael Catanzaro wrote: > > On Wed, Jul 6 2022 at 08:06:45 AM -0600, Jeff Law > > wrote: > >> If I'm understanding things correctly, the original proposal is trying > >> to make a very special case of profiling work

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

2022-07-06 Thread Vitaly Zaitsev via devel
On 06/07/2022 16:20, Michael Catanzaro wrote: But all Fedora users benefit from performance improvements implemented as a result of profiling. I don't think so. Only the proposal owners will get benefit. -- Sincerely, Vitaly Zaitsev (vit...@easycoding.org)

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

2022-07-06 Thread Jeff Law
On 7/6/2022 8:20 AM, Michael Catanzaro wrote: On Wed, Jul 6 2022 at 08:06:45 AM -0600, Jeff Law wrote: If I'm understanding things correctly, the original proposal is trying to make a very special case of profiling work better -- a case that 99.9% of Fedora users do not need or care about.   

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

2022-07-06 Thread Michael Catanzaro
On Wed, Jul 6 2022 at 08:06:45 AM -0600, Jeff Law wrote: If I'm understanding things correctly, the original proposal is trying to make a very special case of profiling work better -- a case that 99.9% of Fedora users do not need or care about.That seems like a particularly bad cost/benefit

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

2022-07-06 Thread Michael Catanzaro
On Wed, Jul 6 2022 at 09:26:44 AM -0400, Marek Polacek wrote: I think you may be underestimating how much even 1% matters. For Fedora Workstation, the primary concern should be to make sure sysprof works nicely. That's our profiling tool, and it currently doesn't work well at all with

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

2022-07-06 Thread Jeff Law
On 7/6/2022 7:26 AM, Marek Polacek wrote: On Tue, Jul 05, 2022 at 03:47:26PM -0400, Matthias Clasen wrote: On Tue, Jul 5, 2022 at 3:40 PM Marek Polacek wrote: Maybe not, but even ~1% is still an unacceptable slowdown. It would take about a year for the compiler to catch up.

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

2022-07-06 Thread Marek Polacek
On Tue, Jul 05, 2022 at 03:47:26PM -0400, Matthias Clasen wrote: > On Tue, Jul 5, 2022 at 3:40 PM Marek Polacek wrote: > > > > > Maybe not, but even ~1% is still an unacceptable slowdown. It would take > > about a year for the compiler to catch up. > > > > > (Un)acceptable for whom? GCC

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

2022-07-06 Thread Dan Čermák
Kevin Kofler via devel writes: > Fabio Valentini wrote: >> And if we say this argument is valid, then should we also build all our >> packages with ASAN / TSAN / etc. instrumentation, as well? > > And ASAN would actually have tangible benefits for end users, namely > preventing some memory bug

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

2022-07-06 Thread Vitaly Zaitsev via devel
On 05/07/2022 21:15, Matthias Clasen wrote: And I doubt that you'd be able to notice a 'smaller than 1% slowdown' on your system. 4% slowdown is unacceptable. At least for Fedora Workstation, being a useful system for developers with working debugging and profiling tools should have some

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

2022-07-05 Thread Kevin Kofler via devel
Fabio Valentini wrote: > And if we say this argument is valid, then should we also build all our > packages with ASAN / TSAN / etc. instrumentation, as well? And ASAN would actually have tangible benefits for end users, namely preventing some memory bug exploits, whereas frame pointers only slow

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

2022-07-05 Thread Florian Weimer
* Matthias Clasen: > not to mention hardware continuously getting faster too... The proposal is about enabling this feature for older hardware. Recent x86-64 CPUs can maintain an array of return addresses in hardware (so basically the backtrace is available directly). Kernel patches to enable

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

2022-07-05 Thread Fabio Valentini
On Tue, Jul 5, 2022 at 9:16 PM Matthias Clasen wrote: > > On Thu, Jun 30, 2022 at 4:55 AM Kevin Kofler via devel > wrote: >> >> Daan De Meyer via devel wrote: >> > Which shows a smaller than 1% slowdown between the binary built with frame >> > pointers and the binary built without frame

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

2022-07-05 Thread Matthias Clasen
On Tue, Jul 5, 2022 at 3:40 PM Marek Polacek wrote: > > Maybe not, but even ~1% is still an unacceptable slowdown. It would take > about a year for the compiler to catch up. > > (Un)acceptable for whom? And why would it be unacceptable? You just said compilers will make up for it quickly, not

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

2022-07-05 Thread Marek Polacek
On Tue, Jul 05, 2022 at 03:15:02PM -0400, Matthias Clasen wrote: > On Thu, Jun 30, 2022 at 4:55 AM Kevin Kofler via devel < > devel@lists.fedoraproject.org> wrote: > > > Daan De Meyer via devel wrote: > > > Which shows a smaller than 1% slowdown between the binary built with > > frame > > >

  1   2   >