Bug 211621 - [FlatPak SDK] Some packages in the SDK have missing debug symbols
Summary: [FlatPak SDK] Some packages in the SDK have missing debug symbols
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Philippe Normand
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-08 05:10 PDT by Carlos Alberto Lopez Perez
Modified: 2020-05-12 10:05 PDT (History)
2 users (show)

See Also:


Attachments
crash log (171.92 KB, text/plain)
2020-05-08 05:33 PDT, Philippe Normand
no flags Details
Patch (2.65 KB, patch)
2020-05-12 05:26 PDT, Philippe Normand
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2020-05-08 05:10:48 PDT
On the backtrace from https://build.webkit.org/results/GTK%20Linux%2064-bit%20Debug%20(Tests)/r261362%20(6280)/compositing/video/video-border-radius-crash-log.txt there are missing symbols for some libraries of the SDK.

I observe that on:

/usr/lib/x86_64-linux-gnu/GL/lib/dri/swrast_dri.so
/usr/lib/x86_64-linux-gnu/ffmpeg/./libavcodec.so.58
Comment 1 Philippe Normand 2020-05-08 05:33:32 PDT
Created attachment 398854 [details]
crash log

build bot links are not future-proof.
Comment 2 Philippe Normand 2020-05-12 05:26:22 PDT
Created attachment 399121 [details]
Patch
Comment 3 Philippe Normand 2020-05-12 05:27:31 PDT
(In reply to Carlos Alberto Lopez Perez from comment #0)
> On the backtrace from
> https://build.webkit.org/results/GTK%20Linux%2064-bit%20Debug%20(Tests)/
> r261362%20(6280)/compositing/video/video-border-radius-crash-log.txt there
> are missing symbols for some libraries of the SDK.
> 
> I observe that on:
> 
> /usr/lib/x86_64-linux-gnu/GL/lib/dri/swrast_dri.so

This should be better now with Sdk.Debug.

> /usr/lib/x86_64-linux-gnu/ffmpeg/./libavcodec.so.58

Unfortunately it seems the FDO ffmpeg flatpak extension is explicitly compiled without debug support.
Comment 4 Carlos Alberto Lopez Perez 2020-05-12 06:41:09 PDT
(In reply to Philippe Normand from comment #3)
> (In reply to Carlos Alberto Lopez Perez from comment #0)
> > On the backtrace from
> > https://build.webkit.org/results/GTK%20Linux%2064-bit%20Debug%20(Tests)/
> > r261362%20(6280)/compositing/video/video-border-radius-crash-log.txt there
> > are missing symbols for some libraries of the SDK.
> > 
> > I observe that on:
> > 
> > /usr/lib/x86_64-linux-gnu/GL/lib/dri/swrast_dri.so
> 
> This should be better now with Sdk.Debug.
> 

I like the idea of installing Sdk.Debug also for release, but not sure how that will help with this specific bug, because the backtrace I posted above its already from the debug bot. And I understand Sdk.Debug was already installed there. However, the symbols for swrast_dri.so were missing.
Comment 5 Philippe Normand 2020-05-12 07:22:43 PDT
webkit-flatpak --command=bash --debug
[📦🌐🐱 org.webkit.Webkit WebKit]$ file /usr/lib/x86_64-linux-gnu/GL/lib/dri/swrast_dri.so
/usr/lib/x86_64-linux-gnu/GL/lib/dri/swrast_dri.so: symbolic link to /usr/lib/x86_64-linux-gnu/GL/default/lib/dri/swrast_dri.so
[📦🌐🐱 org.webkit.Webkit WebKit]$ file /usr/lib/x86_64-linux-gnu/GL/default/lib/dri/swrast_dri.so
/usr/lib/x86_64-linux-gnu/GL/default/lib/dri/swrast_dri.so: symbolic link to libgallium_dri.so
[📦🌐🐱 org.webkit.Webkit WebKit]$ file /usr/lib/x86_64-linux-gnu/GL/default/lib/dri/libgallium_dri.so 
/usr/lib/x86_64-linux-gnu/GL/default/lib/dri/libgallium_dri.so: ELF 64-bit LSB shared object, x86-64, version 1 (GNU/Linux), dynamically linked, BuildID[sha1]=5ec01c8e318371839e3923926af535b2c6787298, stripped
[📦🌐🐱 org.webkit.Webkit WebKit]$
Comment 6 Philippe Normand 2020-05-12 07:24:08 PDT
Perhaps we should build our own versions of the Mesa and ffmpeg extensions. Those shipped in Flathub are stripped and/or without debug support.
Comment 7 Carlos Alberto Lopez Perez 2020-05-12 09:53:44 PDT
(In reply to Philippe Normand from comment #6)
> Perhaps we should build our own versions of the Mesa and ffmpeg extensions.
> Those shipped in Flathub are stripped and/or without debug support.

I see.

Maybe we can start by opening an issue on their bugtracker requesting to ship the debug symbols on the Debug flatpaks. ?


In any case I like the patch from this bug, I agree with the idea of installing also the flatpak debug layers also on release builds.
Comment 8 Philippe Normand 2020-05-12 10:01:55 PDT
(In reply to Carlos Alberto Lopez Perez from comment #7)
> (In reply to Philippe Normand from comment #6)
> > Perhaps we should build our own versions of the Mesa and ffmpeg extensions.
> > Those shipped in Flathub are stripped and/or without debug support.
> 
> I see.
> 
> Maybe we can start by opening an issue on their bugtracker requesting to
> ship the debug symbols on the Debug flatpaks. ?
> 

Yes. I mentioned it on IRC and got no reply. Let's try a bugreport...

> 
> In any case I like the patch from this bug, I agree with the idea of
> installing also the flatpak debug layers also on release builds.

Thanks
Comment 9 EWS 2020-05-12 10:05:20 PDT
Committed r261559: <https://trac.webkit.org/changeset/261559>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 399121 [details].