rust: compile libcore with edition 2024 for 1.87+
commit f4daa80d6be7d3c55ca72a8e560afc4e21f886aa upstream. Rust 1.87 (released on 2025-05-15) compiles core library with edition 2024 instead of 2021 [1]. Ensure that the edition matches libcore's expectation to avoid potential breakage. [ J3m3 reported in Zulip [2] that the `rust-analyzer` target was broken after this patch -- indeed, we need to avoid `core-cfgs` since those are passed to the `rust-analyzer` target. So, instead, I tweaked the patch to create a new `core-edition` variable and explicitly mention the `--edition` flag instead of reusing `core-cfg`s. In addition, pass a new argument using this new variable to `generate_rust_analyzer.py` so that we set the right edition there. By the way, for future reference: the `filter-out` change is needed for Rust < 1.87, since otherwise we would skip the `--edition=2021` we just added, ending up with no edition flag, and thus the compiler would default to the 2015 one. [2] https://rust-for-linux.zulipchat.com/#narrow/channel/291565/topic/x/near/520206547 - Miguel ] Cc: stable@vger.kernel.org # Needed in 6.12.y and later (Rust is pinned in older LTSs). Link: https://github.com/rust-lang/rust/pull/138162 [1] Reported-by: est31 <est31@protonmail.com> Closes: https://github.com/Rust-for-Linux/linux/issues/1163 Signed-off-by: Gary Guo <gary@garyguo.net> Link: https://lore.kernel.org/r/20250517085600.2857460-1-gary@garyguo.net Signed-off-by: Miguel Ojeda <ojeda@kernel.org> [ Solved conflicts for 6.12.y backport. - Miguel ] Signed-off-by: Miguel Ojeda <ojeda@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
1814e71a4e
commit
60d8db49ef
+8
-6
@@ -53,6 +53,8 @@ endif
|
|||||||
core-cfgs = \
|
core-cfgs = \
|
||||||
--cfg no_fp_fmt_parse
|
--cfg no_fp_fmt_parse
|
||||||
|
|
||||||
|
core-edition := $(if $(call rustc-min-version,108700),2024,2021)
|
||||||
|
|
||||||
quiet_cmd_rustdoc = RUSTDOC $(if $(rustdoc_host),H, ) $<
|
quiet_cmd_rustdoc = RUSTDOC $(if $(rustdoc_host),H, ) $<
|
||||||
cmd_rustdoc = \
|
cmd_rustdoc = \
|
||||||
OBJTREE=$(abspath $(objtree)) \
|
OBJTREE=$(abspath $(objtree)) \
|
||||||
@@ -95,8 +97,8 @@ rustdoc-macros: $(src)/macros/lib.rs FORCE
|
|||||||
|
|
||||||
# Starting with Rust 1.82.0, skipping `-Wrustdoc::unescaped_backticks` should
|
# Starting with Rust 1.82.0, skipping `-Wrustdoc::unescaped_backticks` should
|
||||||
# not be needed -- see https://github.com/rust-lang/rust/pull/128307.
|
# not be needed -- see https://github.com/rust-lang/rust/pull/128307.
|
||||||
rustdoc-core: private skip_flags = -Wrustdoc::unescaped_backticks
|
rustdoc-core: private skip_flags = --edition=2021 -Wrustdoc::unescaped_backticks
|
||||||
rustdoc-core: private rustc_target_flags = $(core-cfgs)
|
rustdoc-core: private rustc_target_flags = --edition=$(core-edition) $(core-cfgs)
|
||||||
rustdoc-core: $(RUST_LIB_SRC)/core/src/lib.rs FORCE
|
rustdoc-core: $(RUST_LIB_SRC)/core/src/lib.rs FORCE
|
||||||
+$(call if_changed,rustdoc)
|
+$(call if_changed,rustdoc)
|
||||||
|
|
||||||
@@ -372,7 +374,7 @@ quiet_cmd_rustc_library = $(if $(skip_clippy),RUSTC,$(RUSTC_OR_CLIPPY_QUIET)) L
|
|||||||
cmd_rustc_library = \
|
cmd_rustc_library = \
|
||||||
OBJTREE=$(abspath $(objtree)) \
|
OBJTREE=$(abspath $(objtree)) \
|
||||||
$(if $(skip_clippy),$(RUSTC),$(RUSTC_OR_CLIPPY)) \
|
$(if $(skip_clippy),$(RUSTC),$(RUSTC_OR_CLIPPY)) \
|
||||||
$(filter-out $(skip_flags),$(rust_flags) $(rustc_target_flags)) \
|
$(filter-out $(skip_flags),$(rust_flags)) $(rustc_target_flags) \
|
||||||
--emit=dep-info=$(depfile) --emit=obj=$@ \
|
--emit=dep-info=$(depfile) --emit=obj=$@ \
|
||||||
--emit=metadata=$(dir $@)$(patsubst %.o,lib%.rmeta,$(notdir $@)) \
|
--emit=metadata=$(dir $@)$(patsubst %.o,lib%.rmeta,$(notdir $@)) \
|
||||||
--crate-type rlib -L$(objtree)/$(obj) \
|
--crate-type rlib -L$(objtree)/$(obj) \
|
||||||
@@ -383,7 +385,7 @@ quiet_cmd_rustc_library = $(if $(skip_clippy),RUSTC,$(RUSTC_OR_CLIPPY_QUIET)) L
|
|||||||
|
|
||||||
rust-analyzer:
|
rust-analyzer:
|
||||||
$(Q)$(srctree)/scripts/generate_rust_analyzer.py \
|
$(Q)$(srctree)/scripts/generate_rust_analyzer.py \
|
||||||
--cfgs='core=$(core-cfgs)' \
|
--cfgs='core=$(core-cfgs)' $(core-edition) \
|
||||||
$(realpath $(srctree)) $(realpath $(objtree)) \
|
$(realpath $(srctree)) $(realpath $(objtree)) \
|
||||||
$(rustc_sysroot) $(RUST_LIB_SRC) $(KBUILD_EXTMOD) > \
|
$(rustc_sysroot) $(RUST_LIB_SRC) $(KBUILD_EXTMOD) > \
|
||||||
$(if $(KBUILD_EXTMOD),$(extmod_prefix),$(objtree))/rust-project.json
|
$(if $(KBUILD_EXTMOD),$(extmod_prefix),$(objtree))/rust-project.json
|
||||||
@@ -407,9 +409,9 @@ define rule_rustc_library
|
|||||||
endef
|
endef
|
||||||
|
|
||||||
$(obj)/core.o: private skip_clippy = 1
|
$(obj)/core.o: private skip_clippy = 1
|
||||||
$(obj)/core.o: private skip_flags = -Wunreachable_pub
|
$(obj)/core.o: private skip_flags = --edition=2021 -Wunreachable_pub
|
||||||
$(obj)/core.o: private rustc_objcopy = $(foreach sym,$(redirect-intrinsics),--redefine-sym $(sym)=__rust$(sym))
|
$(obj)/core.o: private rustc_objcopy = $(foreach sym,$(redirect-intrinsics),--redefine-sym $(sym)=__rust$(sym))
|
||||||
$(obj)/core.o: private rustc_target_flags = $(core-cfgs)
|
$(obj)/core.o: private rustc_target_flags = --edition=$(core-edition) $(core-cfgs)
|
||||||
$(obj)/core.o: $(RUST_LIB_SRC)/core/src/lib.rs \
|
$(obj)/core.o: $(RUST_LIB_SRC)/core/src/lib.rs \
|
||||||
$(wildcard $(objtree)/include/config/RUSTC_VERSION_TEXT) FORCE
|
$(wildcard $(objtree)/include/config/RUSTC_VERSION_TEXT) FORCE
|
||||||
+$(call if_changed_rule,rustc_library)
|
+$(call if_changed_rule,rustc_library)
|
||||||
|
|||||||
@@ -18,7 +18,7 @@ def args_crates_cfgs(cfgs):
|
|||||||
|
|
||||||
return crates_cfgs
|
return crates_cfgs
|
||||||
|
|
||||||
def generate_crates(srctree, objtree, sysroot_src, external_src, cfgs):
|
def generate_crates(srctree, objtree, sysroot_src, external_src, cfgs, core_edition):
|
||||||
# Generate the configuration list.
|
# Generate the configuration list.
|
||||||
cfg = []
|
cfg = []
|
||||||
with open(objtree / "include" / "generated" / "rustc_cfg") as fd:
|
with open(objtree / "include" / "generated" / "rustc_cfg") as fd:
|
||||||
@@ -34,7 +34,7 @@ def generate_crates(srctree, objtree, sysroot_src, external_src, cfgs):
|
|||||||
crates_indexes = {}
|
crates_indexes = {}
|
||||||
crates_cfgs = args_crates_cfgs(cfgs)
|
crates_cfgs = args_crates_cfgs(cfgs)
|
||||||
|
|
||||||
def append_crate(display_name, root_module, deps, cfg=[], is_workspace_member=True, is_proc_macro=False):
|
def append_crate(display_name, root_module, deps, cfg=[], is_workspace_member=True, is_proc_macro=False, edition="2021"):
|
||||||
crates_indexes[display_name] = len(crates)
|
crates_indexes[display_name] = len(crates)
|
||||||
crates.append({
|
crates.append({
|
||||||
"display_name": display_name,
|
"display_name": display_name,
|
||||||
@@ -43,7 +43,7 @@ def generate_crates(srctree, objtree, sysroot_src, external_src, cfgs):
|
|||||||
"is_proc_macro": is_proc_macro,
|
"is_proc_macro": is_proc_macro,
|
||||||
"deps": [{"crate": crates_indexes[dep], "name": dep} for dep in deps],
|
"deps": [{"crate": crates_indexes[dep], "name": dep} for dep in deps],
|
||||||
"cfg": cfg,
|
"cfg": cfg,
|
||||||
"edition": "2021",
|
"edition": edition,
|
||||||
"env": {
|
"env": {
|
||||||
"RUST_MODFILE": "This is only for rust-analyzer"
|
"RUST_MODFILE": "This is only for rust-analyzer"
|
||||||
}
|
}
|
||||||
@@ -53,6 +53,7 @@ def generate_crates(srctree, objtree, sysroot_src, external_src, cfgs):
|
|||||||
display_name,
|
display_name,
|
||||||
deps,
|
deps,
|
||||||
cfg=[],
|
cfg=[],
|
||||||
|
edition="2021",
|
||||||
):
|
):
|
||||||
append_crate(
|
append_crate(
|
||||||
display_name,
|
display_name,
|
||||||
@@ -60,12 +61,13 @@ def generate_crates(srctree, objtree, sysroot_src, external_src, cfgs):
|
|||||||
deps,
|
deps,
|
||||||
cfg,
|
cfg,
|
||||||
is_workspace_member=False,
|
is_workspace_member=False,
|
||||||
|
edition=edition,
|
||||||
)
|
)
|
||||||
|
|
||||||
# NB: sysroot crates reexport items from one another so setting up our transitive dependencies
|
# NB: sysroot crates reexport items from one another so setting up our transitive dependencies
|
||||||
# here is important for ensuring that rust-analyzer can resolve symbols. The sources of truth
|
# here is important for ensuring that rust-analyzer can resolve symbols. The sources of truth
|
||||||
# for this dependency graph are `(sysroot_src / crate / "Cargo.toml" for crate in crates)`.
|
# for this dependency graph are `(sysroot_src / crate / "Cargo.toml" for crate in crates)`.
|
||||||
append_sysroot_crate("core", [], cfg=crates_cfgs.get("core", []))
|
append_sysroot_crate("core", [], cfg=crates_cfgs.get("core", []), edition=core_edition)
|
||||||
append_sysroot_crate("alloc", ["core"])
|
append_sysroot_crate("alloc", ["core"])
|
||||||
append_sysroot_crate("std", ["alloc", "core"])
|
append_sysroot_crate("std", ["alloc", "core"])
|
||||||
append_sysroot_crate("proc_macro", ["core", "std"])
|
append_sysroot_crate("proc_macro", ["core", "std"])
|
||||||
@@ -155,6 +157,7 @@ def main():
|
|||||||
parser = argparse.ArgumentParser()
|
parser = argparse.ArgumentParser()
|
||||||
parser.add_argument('--verbose', '-v', action='store_true')
|
parser.add_argument('--verbose', '-v', action='store_true')
|
||||||
parser.add_argument('--cfgs', action='append', default=[])
|
parser.add_argument('--cfgs', action='append', default=[])
|
||||||
|
parser.add_argument("core_edition")
|
||||||
parser.add_argument("srctree", type=pathlib.Path)
|
parser.add_argument("srctree", type=pathlib.Path)
|
||||||
parser.add_argument("objtree", type=pathlib.Path)
|
parser.add_argument("objtree", type=pathlib.Path)
|
||||||
parser.add_argument("sysroot", type=pathlib.Path)
|
parser.add_argument("sysroot", type=pathlib.Path)
|
||||||
@@ -171,7 +174,7 @@ def main():
|
|||||||
assert args.sysroot in args.sysroot_src.parents
|
assert args.sysroot in args.sysroot_src.parents
|
||||||
|
|
||||||
rust_project = {
|
rust_project = {
|
||||||
"crates": generate_crates(args.srctree, args.objtree, args.sysroot_src, args.exttree, args.cfgs),
|
"crates": generate_crates(args.srctree, args.objtree, args.sysroot_src, args.exttree, args.cfgs, args.core_edition),
|
||||||
"sysroot": str(args.sysroot),
|
"sysroot": str(args.sysroot),
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user