x86/boot: Fix intermediate file names to generate 32 bit code

The "base" and "offset" definition were inverted, "base" file should be the
files without offsets applied while "offset" should have the offsets applied.

Also update an old usage of "final" to "apply offset" to make more clear and
consistent (in former commit messages the "final" term was used instead of
"offset").

Fixes: aa9045e771 ('x86/boot: Rework how 32bit C is linked/included for early boot')
Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
This commit is contained in:
Frediano Ziglio authored and Andrew Cooper committed 2024-11-06 13:47:23 +00:00
1 parent c6b84ccf8d
commit e75b327c45
2 files changed
+4 -4

No files matched your search

+3 -3
View File
@@ -44,7 +44,7 @@ text_gap := 0x010200
text_diff := 0x408020
$(obj)/build32.base.lds: AFLAGS-y += -DGAP=$(text_gap) -DTEXT_DIFF=$(text_diff)
$(obj)/build32.offset.lds: AFLAGS-y += -DGAP=$(text_gap) -DTEXT_DIFF=$(text_diff) -DFINAL
$(obj)/build32.offset.lds: AFLAGS-y += -DGAP=$(text_gap) -DTEXT_DIFF=$(text_diff) -DAPPLY_OFFSET
$(obj)/build32.base.lds $(obj)/build32.offset.lds: $(src)/build32.lds.S FORCE
$(call if_changed_dep,cpp_lds_S)
@@ -75,10 +75,10 @@ cmd_combine = \
$(PYTHON) $(srctree)/tools/combine_two_binaries.py \
--gap $(text_gap) \
--text-diff $(text_diff) \
--script $(obj)/build32.offset.lds \
--script $(obj)/build32.base.lds \
--bin1 $(obj)/built-in-32.base.bin \
--bin2 $(obj)/built-in-32.offset.bin \
--map $(obj)/built-in-32.offset.map \
--map $(obj)/built-in-32.base.map \
--exports cmdline_parse_early,reloc,reloc_trampoline32 \
--output $@
+1 -1
View File
@@ -15,7 +15,7 @@
* with this program. If not, see <http://www.gnu.org/licenses/>.
*/
#ifdef FINAL
#ifndef APPLY_OFFSET
# undef GAP
# define GAP 0
# define MULT 0
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.