Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 946835 - www-client/firefox-133.0.3[pgo] fails to build: undefined reference to `SyscallAsm'
Summary: www-client/firefox-133.0.3[pgo] fails to build: undefined reference to `Sysca...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal
Assignee: Mozilla Gentoo Team
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2024-12-22 16:51 UTC by John Turner
Modified: 2024-12-23 06:44 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
emerge --info (emerge-into.txt,6.62 KB, text/plain)
2024-12-22 16:51 UTC, John Turner
Details
xaa (first half of compressed log) (xaa,805.82 KB, application/x-xz)
2024-12-22 17:06 UTC, John Turner
Details
xab (second half of compressed log) (xab,805.82 KB, application/octet-stream)
2024-12-22 17:07 UTC, John Turner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description John Turner 2024-12-22 16:51:46 UTC
Created attachment 914736 [details]
emerge --info

Firefox 133.0.3 fails to build for me. (build log posted below)
Comment 1 John Turner 2024-12-22 16:53:55 UTC
https://0x0.st/8rrI.log.zst

Not sure how to upload this build log, it seems to be too large, and eventually this link will expire.
Comment 2 John Turner 2024-12-22 17:06:21 UTC
Created attachment 914737 [details]
xaa (first half of compressed log)
Comment 3 John Turner 2024-12-22 17:07:20 UTC
Created attachment 914738 [details]
xab (second half of compressed log)

Use "cat xaa xab > build.log.xz" to cat the files into one, and then decompress like normal.
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-12-22 20:14:02 UTC
[20:12] <sam_> I think this is exposed by juippis' fixes to PGO for GCC
[20:13] <sam_> then sandbox::Syscall::Call gets cloned using the profile data
[20:13] <sam_> and it ends up in a different LTO partition
[20:13] <sam_> so, in a way, this is great
[20:13] <sam_> it shows PGO is working :D
[20:13] <sam_> because it deduced that it's cold
Comment 5 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-12-22 20:16:37 UTC
I'm pretty sure https://src.fedoraproject.org/rpms/firefox/blob/rawhide/f/mozilla-1516803.patch will fix it.
Comment 6 Joonas Niilola gentoo-dev 2024-12-23 06:25:13 UTC
I gather from IRC that the rebased patch worked? I guess when I returned the "old approach" in https://gitweb.gentoo.org/repo/gentoo.git/commit/www-client/firefox?id=47d3ff466c8238445180dacff7bc2bba4977285c I also took the older version of that mozilla-1516803.patch along with it, and that needs an update. Weird that I tested firefox[pgo] just fine with gcc and lto, though.
Comment 7 Joonas Niilola gentoo-dev 2024-12-23 06:29:34 UTC
Wait, https://src.fedoraproject.org/rpms/firefox/raw/rawhide/f/mozilla-1516803.patch this bit is identical to what I have in firefox-133-patches-05.tar.xz -> 0015-gcc-lto-pgo-gentoo.patch

Could you copypaste the contents of your 0015-gcc-lto-pgo-gentoo.patch or attach the patch here?
Comment 8 Joonas Niilola gentoo-dev 2024-12-23 06:33:12 UTC
Oh, right, this again. Yeah the patch seems malformed due to some copy-paste error and I guess it doesn't get applied. Will test and fix.
Comment 9 Larry the Git Cow gentoo-dev 2024-12-23 06:43:30 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5eb5df285e2324f1b16d12cd4ed6ddbc8eb30a9d

commit 5eb5df285e2324f1b16d12cd4ed6ddbc8eb30a9d
Author:     Joonas Niilola <juippis@gentoo.org>
AuthorDate: 2024-12-23 06:40:34 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2024-12-23 06:43:27 +0000

    www-client/firefox: fix a "typo" in 133.0.3's patchset
    
     - one part of 0015-gcc-lto-pgo-gentoo.patch didn't apply because it was read
       as comments due to copy-pasting missing a crucial position line.
    
    Closes: https://bugs.gentoo.org/946835
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 www-client/firefox/Manifest               | 2 +-
 www-client/firefox/firefox-133.0.3.ebuild | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
Comment 10 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-12-23 06:44:11 UTC
Thanks all!