On Fri, 26 Mar 2021, Peter Zijlstra wrote: > Hi, another week, another update :-) > > Respin of the !RETPOLINE optimization patches. > > Boris, the first 3 should probably go into tip/x86/core, it's an ungodly > tangle > since it relies on the insn decoder patches in tip/x86/core, the NOP patches > in > tip/x86/cpu and the alternative patches in tip/x86/alternatives. > > Just to make life easy I'd suggest merging everything in x86/core and > forgetting about the other topic branches (that's what I ended up doing > locally). > > The remaining 13 patches depend on the first 3 as well as on the work in > tip/objtool/core, just to make life more interesting still ;-) > > All except the last 4 patches should be fairly uncontroversial (I hope...). > > There's a fair number of new patches and another few have been completely > rewritten, but it all seems to work nicely.
Reviewed-by: Miroslav Benes <mbe...@suse.cz> for the objtool changes. All looks much better in this version. I have only one minor thing. There are only two call sites of elf_add_string(). The one in elf_create_section() passes shstrtab, the other one in elf_create_undef_symbol() NULL. elf_add_string() then retrieves it itself. I think it would be nicer to just call find_section_by_name() in elf_create_undef_symbol(), pass it down and make it consistent. Might be a matter of taste. Miroslav