[PATCH] x86: Alias memset to __builtin_memset.
From: Clement Courbet
Date: Tue Mar 24 2020 - 10:06:47 EST
Thanks for the comments. Answers below.
> This ifdef is unnecessary
> This needs to be within #ifndef CONFIG_FORTIFY_SOURCE
Thanks, fixed.
> shouldn't this just be done universally ?
It looks like every architecture does its own magic with memory
functions. I'm not very familiar with how the linux kernel is
organized, do you have a pointer for where this would go if enabled
globally ?
> Who's adding it for 64b?
> Any idea where it's coming from in your
> build? Maybe a local modification to be removed?
Actually this is from our local build configuration. Apparently this
is needed to build on some architectures that redefine common
symbols, but the authors seemed to feel strongly that this should be
on for all architectures. I've reached out to the authors for an
extended rationale.
On the other hand I think that there is no reason to prevent people
from building with freestanding if we can easily allow them to.