tools/bootconfig: allow overriding CFLAGS and LDFLAGS

[PATCH 0/2] tools/bootconfig: allow overriding CFLAGS and LDFLAGS

Posted by Brian Masney 10 months, 2 weeks ago

Allow the user to override the CFLAGS and LDFLAGS similar to what's
currently done on other kernel tools. These changes allows bootconfig to
be compiled with the hardened compiler and linker flags.

Brian Masney (2):
  tools/bootconfig: allow overriding CFLAGS assignment
  tools/bootconfig: specify LDFLAGS as an argument to CC

 tools/bootconfig/Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.47.0

Re: [PATCH 0/2] tools/bootconfig: allow overriding CFLAGS and LDFLAGS

Posted by Masami Hiramatsu (Google) 10 months, 1 week ago

On Fri, 28 Mar 2025 14:38:56 -0400
Brian Masney <bmasney@redhat.com> wrote:

> Allow the user to override the CFLAGS and LDFLAGS similar to what's
> currently done on other kernel tools. These changes allows bootconfig to
> be compiled with the hardened compiler and linker flags.
> 
> Brian Masney (2):
>   tools/bootconfig: allow overriding CFLAGS assignment
>   tools/bootconfig: specify LDFLAGS as an argument to CC
> 

Both looks good to me. Let me pick those up.

Thanks!

>  tools/bootconfig/Makefile | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> -- 
> 2.47.0
> 


-- 
Masami Hiramatsu (Google) <mhiramat@kernel.org>

Re: [PATCH 0/2] tools/bootconfig: allow overriding CFLAGS and LDFLAGS

Posted by Masami Hiramatsu (Google) 8 months, 3 weeks ago

On Mon, 31 Mar 2025 21:23:37 +0900
Masami Hiramatsu (Google) <mhiramat@kernel.org> wrote:

> On Fri, 28 Mar 2025 14:38:56 -0400
> Brian Masney <bmasney@redhat.com> wrote:
> 
> > Allow the user to override the CFLAGS and LDFLAGS similar to what's
> > currently done on other kernel tools. These changes allows bootconfig to
> > be compiled with the hardened compiler and linker flags.
> > 
> > Brian Masney (2):
> >   tools/bootconfig: allow overriding CFLAGS assignment
> >   tools/bootconfig: specify LDFLAGS as an argument to CC
> > 
> 
> Both looks good to me. Let me pick those up.
> 

Sorry, somehow I missed these patches. I confirmed

tools/bootconfig$ make LDFLAGS=-static

passed the tests and it is statically linked! :)

tools/bootconfig$ file bootconfig 
bootconfig: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), statically linked, BuildID[sha1]=88323308c8d2cc45d1985ef94751bfd60199b32b, for GNU/Linux 3.2.0, with debug_info, not stripped


Thanks!

> Thanks!
> 
> >  tools/bootconfig/Makefile | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > -- 
> > 2.47.0
> > 
> 
> 
> -- 
> Masami Hiramatsu (Google) <mhiramat@kernel.org>


-- 
Masami Hiramatsu (Google) <mhiramat@kernel.org>

Re: [PATCH 0/2] tools/bootconfig: allow overriding CFLAGS and LDFLAGS

Posted by Brian Masney 10 months ago

Hi Masami,

On Mon, Mar 31, 2025 at 8:23 AM Masami Hiramatsu <mhiramat@kernel.org> wrote:
> On Fri, 28 Mar 2025 14:38:56 -0400
> Brian Masney <bmasney@redhat.com> wrote:
>
> > Allow the user to override the CFLAGS and LDFLAGS similar to what's
> > currently done on other kernel tools. These changes allows bootconfig to
> > be compiled with the hardened compiler and linker flags.
> >
> > Brian Masney (2):
> >   tools/bootconfig: allow overriding CFLAGS assignment
> >   tools/bootconfig: specify LDFLAGS as an argument to CC
> >
>
> Both looks good to me. Let me pick those up.

Should these show up in linux-next by now? I see them in your tree at:

https://web.git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace.git/log/?h=bootconfig/tools

But not in the for-next branch.

Thanks,
Brian