Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-17 Thread kernel test robot
Hi Masahiro, I love your patch! Perhaps something to improve: [auto build test WARNING on powerpc/next] [also build test WARNING on linus/master v5.12-rc3] [cannot apply to kbuild/for-next asm-generic/master next-20210317] [If your patch is applied to the wrong git tree, kindly drop us a note. An

Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-09 Thread Nicolas Pitre
On Tue, 9 Mar 2021, Rasmus Villemoes wrote: > On 09/03/2021 20.54, Nicolas Pitre wrote: > > On Wed, 10 Mar 2021, Masahiro Yamada wrote: > > > > >>> I'm not sure I do understand every detail here, especially since it is > >>> so far away from the version that I originally contributed. But the > >

Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-09 Thread Rasmus Villemoes
On 09/03/2021 20.54, Nicolas Pitre wrote: > On Wed, 10 Mar 2021, Masahiro Yamada wrote: > >>> I'm not sure I do understand every detail here, especially since it is >>> so far away from the version that I originally contributed. But the >>> concept looks good. >>> >>> I still think that there is

Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-09 Thread Nicolas Pitre
On Wed, 10 Mar 2021, Masahiro Yamada wrote: > On Wed, Mar 10, 2021 at 2:36 AM Nicolas Pitre wrote: > > > > On Wed, 10 Mar 2021, Masahiro Yamada wrote: > > > > > Commit a555bdd0c58c ("Kbuild: enable TRIM_UNUSED_KSYMS again, with some > > > guarding") re-enabled this feature, but Linus is still unh

Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-09 Thread Masahiro Yamada
On Wed, Mar 10, 2021 at 2:36 AM Nicolas Pitre wrote: > > On Wed, 10 Mar 2021, Masahiro Yamada wrote: > > > Commit a555bdd0c58c ("Kbuild: enable TRIM_UNUSED_KSYMS again, with some > > guarding") re-enabled this feature, but Linus is still unhappy about > > the build time. > > > > The reason of the

Re: [PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-09 Thread Nicolas Pitre
On Wed, 10 Mar 2021, Masahiro Yamada wrote: > Commit a555bdd0c58c ("Kbuild: enable TRIM_UNUSED_KSYMS again, with some > guarding") re-enabled this feature, but Linus is still unhappy about > the build time. > > The reason of the slowness is the recursion - this basically works in > two loops. >

[PATCH v2 3/4] kbuild: re-implement CONFIG_TRIM_UNUSED_KSYMS to make it work in one-pass

2021-03-09 Thread Masahiro Yamada
Commit a555bdd0c58c ("Kbuild: enable TRIM_UNUSED_KSYMS again, with some guarding") re-enabled this feature, but Linus is still unhappy about the build time. The reason of the slowness is the recursion - this basically works in two loops. In the first loop, Kbuild builds the entire tree based on t