Re: [PATCH RESEND] Kbuild: Add an option to enable GCC VTA

[Michal Marek]

Dne 25.11.2014 v 00:46 Josh Stone napsal(a):
> On 11/24/2014 01:46 PM, Michal Marek wrote:
>> Dne 21.11.2014 v 19:40 Josh Stone napsal(a):
>>> Due to recent codegen issues, gcc -fvar-tracking-assignments was
>>> unconditionally disabled in commit 2062afb4f804a ("Fix gcc-4.9.0
>>> miscompilation of load_balance() in scheduler").  However, this reduces
>> [...]
>>> With gcc-4.9.2-1.fc22, I can now build v3.18-rc5 with Fedora's i686 and
>>> x86_64 configs, and this is completely clean with GCC_COMPARE_DEBUG.
>>
>> According to gcc's bug#61801, this is really fixed in 4.9.2 (commit
>> 556537c4 in the git mirror). So how about checking for this minimal
>> version instead of a new Kconfig option?
> 
> That's possible, if the new Kconfig option is really undesirable.  But
> given that there's a similar DWARF4 option, which this pairs well with,
> I thought a VTA option was a good choice.
> 
> Besides 4.9.2 though, this particular fix has also been backported to
> 4.8.4, and who knows what various distro maintainers may do.

My idea was that if we switch -fvar-tracking-assignments on with gcc
4.9.2+, then yes, some users with patched compilers will not benefit
from it, but it will eventually work out of the box for "everybody."
With the kconfig option, an uninformed user has no idea whether or not
it is a good idea to switch it on. It should at least mention that it is
safe to enable with 4.9.2+. And BTW

    make KCLFAGS=-fvar-tracking-assignments

works as well, as the user-supplied flags are applied last.

Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/