* Re: [PATCH 4/4] Revert "mm/page_alloc: make should_fail_alloc_page() static" [not found] ` <fb642720-b651-e93f-4656-7042493efba8@nvidia.com> @ 2021-07-15 8:35 ` Jesper Dangaard Brouer 2021-07-16 0:04 ` John Hubbard 0 siblings, 1 reply; 3+ messages in thread From: Jesper Dangaard Brouer @ 2021-07-15 8:35 UTC (permalink / raw) To: John Hubbard, Mel Gorman, Andrew Morton, acme, Jiri Olsa Cc: brouer, Desmond Cheong Zhi Xi, Zhang Qiang, Yanfei Xu, Chuck Lever, Matteo Croce, Linux-MM, LKML, bpf Cc. Jiri Olsa + Arnaldo On 14/07/2021 09.06, John Hubbard wrote: > On 7/13/21 8:21 AM, Mel Gorman wrote: >> From: Matteo Croce <mcroce@microsoft.com> >> >> This reverts commit f7173090033c70886d925995e9dfdfb76dbb2441. >> >> Fix an unresolved symbol error when CONFIG_DEBUG_INFO_BTF=y: >> >> LD vmlinux >> BTFIDS vmlinux >> FAILED unresolved symbol should_fail_alloc_page >> make: *** [Makefile:1199: vmlinux] Error 255 >> make: *** Deleting file 'vmlinux' > > Yes! I ran into this yesterday. Your patch fixes this build failure > for me, so feel free to add: > > Tested-by: John Hubbard <jhubbard@nvidia.com> > > > However, I should add that I'm still seeing another build failure, after > fixing the above: > > LD vmlinux > BTFIDS vmlinux > FAILED elf_update(WRITE): no error This elf_update(WRITE) error is new to me. > make: *** [Makefile:1176: vmlinux] Error 255 > make: *** Deleting file 'vmlinux' It is annoying that vmlinux is deleted in this case, because I usually give Jiri the output from 'resolve_btfids -v' on vmlinux. $ ./tools/bpf/resolve_btfids/resolve_btfids -v vmlinux.failed You can do: $ git diff diff --git a/scripts/link-vmlinux.sh b/scripts/link-vmlinux.sh index 3b261b0f74f0..02dec10a7d75 100755 --- a/scripts/link-vmlinux.sh +++ b/scripts/link-vmlinux.sh @@ -302,7 +302,8 @@ cleanup() rm -f .tmp_symversions.lds rm -f .tmp_vmlinux* rm -f System.map - rm -f vmlinux + # rm -f vmlinux + mv vmlinux vmlinux.failed rm -f vmlinux.o } > > > ...and un-setting CONFIG_DEBUG_INFO_BTF makes that disappear. Maybe someone > who is understands the BTFIDS build step can shed some light on that; I'm > not there yet. :) I'm just a user/consume of output from the BTFIDS build step, I think Jiri Olsa own the tool resolve_btfids, and ACME pahole. I've hit a number of issues in the past that Jiri and ACME help resolve quickly. The most efficient solution I've found was to upgrade pahole to a newer version. What version of pahole does your build system have? What is your GCC version? --Jesper ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 4/4] Revert "mm/page_alloc: make should_fail_alloc_page() static" 2021-07-15 8:35 ` [PATCH 4/4] Revert "mm/page_alloc: make should_fail_alloc_page() static" Jesper Dangaard Brouer @ 2021-07-16 0:04 ` John Hubbard 2021-07-16 6:04 ` John Hubbard 0 siblings, 1 reply; 3+ messages in thread From: John Hubbard @ 2021-07-16 0:04 UTC (permalink / raw) To: Jesper Dangaard Brouer, Mel Gorman, Andrew Morton, acme, Jiri Olsa Cc: brouer, Desmond Cheong Zhi Xi, Zhang Qiang, Yanfei Xu, Chuck Lever, Matteo Croce, Linux-MM, LKML, bpf ... >> LD vmlinux >> BTFIDS vmlinux >> FAILED elf_update(WRITE): no error > > This elf_update(WRITE) error is new to me. > >> make: *** [Makefile:1176: vmlinux] Error 255 >> make: *** Deleting file 'vmlinux' > > It is annoying that vmlinux is deleted in this case, because I usually give Jiri the output from > 'resolve_btfids -v' on vmlinux. > > $ ./tools/bpf/resolve_btfids/resolve_btfids -v vmlinux.failed > > You can do: > $ git diff > diff --git a/scripts/link-vmlinux.sh b/scripts/link-vmlinux.sh > index 3b261b0f74f0..02dec10a7d75 100755 > --- a/scripts/link-vmlinux.sh > +++ b/scripts/link-vmlinux.sh > @@ -302,7 +302,8 @@ cleanup() > rm -f .tmp_symversions.lds > rm -f .tmp_vmlinux* > rm -f System.map > - rm -f vmlinux > + # rm -f vmlinux > + mv vmlinux vmlinux.failed > rm -f vmlinux.o > } > > >> >> >> ...and un-setting CONFIG_DEBUG_INFO_BTF makes that disappear. Maybe someone >> who is understands the BTFIDS build step can shed some light on that; I'm >> not there yet. :) > > I'm just a user/consume of output from the BTFIDS build step, I think Jiri Olsa own the tool > resolve_btfids, and ACME pahole. I've hit a number of issues in the past that Jiri and ACME help > resolve quickly. > The most efficient solution I've found was to upgrade pahole to a newer version. > > What version of pahole does your build system have? > > What is your GCC version? > Just a quick answer first on the versions: this is an up to date Arch Linux system: gcc: 11.1.0 pahole: 1.21 I'll try to get the other step done later this evening. thanks, -- John Hubbard NVIDIA ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 4/4] Revert "mm/page_alloc: make should_fail_alloc_page() static" 2021-07-16 0:04 ` John Hubbard @ 2021-07-16 6:04 ` John Hubbard 0 siblings, 0 replies; 3+ messages in thread From: John Hubbard @ 2021-07-16 6:04 UTC (permalink / raw) To: Jesper Dangaard Brouer, Mel Gorman, Andrew Morton, acme, Jiri Olsa Cc: brouer, Desmond Cheong Zhi Xi, Zhang Qiang, Yanfei Xu, Chuck Lever, Matteo Croce, Linux-MM, LKML, bpf On 7/15/21 5:04 PM, John Hubbard wrote: ... >>> ...and un-setting CONFIG_DEBUG_INFO_BTF makes that disappear. Maybe someone >>> who is understands the BTFIDS build step can shed some light on that; I'm >>> not there yet. :) >> >> I'm just a user/consume of output from the BTFIDS build step, I think Jiri Olsa own the tool >> resolve_btfids, and ACME pahole. I've hit a number of issues in the past that Jiri and ACME help >> resolve quickly. >> The most efficient solution I've found was to upgrade pahole to a newer version. >> >> What version of pahole does your build system have? >> >> What is your GCC version? >> > > Just a quick answer first on the versions: this is an up to date Arch Linux system: > > gcc: 11.1.0 > pahole: 1.21 > > I'll try to get the other step done later this evening. ...and...I've lost the repro completely. The only thing I changed was that I attempted to update pahole. This caused Arch Linux reinstall pahole, claiming that 1.21 is already the current version. It acts as if there was something wrong with the pahole installation. This seems unlikely, given that the system is merely on a routine update schedule. However, that's the data I have. If it ever comes up again I'll be able to run resolve_btfids, using your steps here, so thanks for posting those! thanks, -- John Hubbard NVIDIA ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-16 6:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210713152100.10381-1-mgorman@techsingularity.net>
[not found] ` <20210713152100.10381-5-mgorman@techsingularity.net>
[not found] ` <fb642720-b651-e93f-4656-7042493efba8@nvidia.com>
2021-07-15 8:35 ` [PATCH 4/4] Revert "mm/page_alloc: make should_fail_alloc_page() static" Jesper Dangaard Brouer
2021-07-16 0:04 ` John Hubbard
2021-07-16 6:04 ` John Hubbard
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).