• Miguel Ojeda's avatar
    rust: upgrade to Rust 1.77.1 · b481dd85
    Miguel Ojeda authored
    This is the next upgrade to the Rust toolchain, from 1.76.0 to 1.77.1
    (i.e. the latest) [1].
    
    See the upgrade policy [2] and the comments on the first upgrade in
    commit 3ed03f4d ("rust: upgrade to Rust 1.68.2").
    
    # Unstable features
    
    The `offset_of` feature (single-field `offset_of!`) that we were using
    got stabilized in Rust 1.77.0 [3].
    
    Therefore, now the only unstable features allowed to be used outside the
    `kernel` crate is `new_uninit`, though other code to be upstreamed may
    increase the list.
    
    Please see [4] for details.
    
    # Required changes
    
    Rust 1.77.0 merged the `unused_tuple_struct_fields` lint into `dead_code`,
    thus upgrading it from `allow` to `warn` [5]. In turn, this made `rustc`
    complain about the `ThisModule`'s pointer field being never read, but
    the previous patch adds the `as_ptr` method to it, needed by Binder [6],
    so that we do not need to locally `allow` it.
    
    # Other changes
    
    Rust 1.77.0 introduces the `--check-cfg` feature [7], for which there
    is a Call for Testing going on [8]. We were requested to test it and
    we found it useful [9] -- we will likely enable it in the future.
    
    # `alloc` upgrade and reviewing
    
    The vast majority of changes are due to our `alloc` fork being upgraded
    at once.
    
    There are two kinds of changes to be aware of: the ones coming from
    upstream, which we should follow as closely as possible, and the updates
    needed in our added fallible APIs to keep them matching the newer
    infallible APIs coming from upstream.
    
    Instead of taking a look at the diff of this patch, an alternative
    approach is reviewing a diff of the changes between upstream `alloc` and
    the kernel's. This allows to easily inspect the kernel additions only,
    especially to check if the fallible methods we already have still match
    the infallible ones in the new version coming from upstream.
    
    Another approach is reviewing the changes introduced in the additions in
    the kernel fork between the two versions. This is useful to spot
    potentially unintended changes to our additions.
    
    To apply these approaches, one may follow steps similar to the following
    to generate a pair of patches that show the differences between upstream
    Rust and the kernel (for the subset of `alloc` we use) before and after
    applying this patch:
    
        # Get the difference with respect to the old version.
        git -C rust checkout $(linux/scripts/min-tool-version.sh rustc)
        git -C linux ls-tree -r --name-only HEAD -- rust/alloc |
            cut -d/ -f3- |
            grep -Fv README.md |
            xargs -IPATH cp rust/library/alloc/src/PATH linux/rust/alloc/PATH
        git -C linux diff --patch-with-stat --summary -R > old.patch
        git -C linux restore rust/alloc
    
        # Apply this patch.
        git -C linux am rust-upgrade.patch
    
        # Get the difference with respect to the new version.
        git -C rust checkout $(linux/scripts/min-tool-version.sh rustc)
        git -C linux ls-tree -r --name-only HEAD -- rust/alloc |
            cut -d/ -f3- |
            grep -Fv README.md |
            xargs -IPATH cp rust/library/alloc/src/PATH linux/rust/alloc/PATH
        git -C linux diff --patch-with-stat --summary -R > new.patch
        git -C linux restore rust/alloc
    
    Now one may check the `new.patch` to take a look at the additions (first
    approach) or at the difference between those two patches (second
    approach). For the latter, a side-by-side tool is recommended.
    
    Link: https://github.com/rust-lang/rust/blob/stable/RELEASES.md#version-1770-2024-03-21 [1]
    Link: https://rust-for-linux.com/rust-version-policy [2]
    Link: https://github.com/rust-lang/rust/pull/118799 [3]
    Link: https://github.com/Rust-for-Linux/linux/issues/2 [4]
    Link: https://github.com/rust-lang/rust/pull/118297 [5]
    Link: https://lore.kernel.org/rust-for-linux/20231101-rust-binder-v1-2-08ba9197f637@google.com/#Z31rust:kernel:lib.rs [6]
    Link: https://doc.rust-lang.org/nightly/unstable-book/compiler-flags/check-cfg.html [7]
    Link: https://github.com/rust-lang/rfcs/pull/3013#issuecomment-1936648479 [8]
    Link: https://github.com/rust-lang/rust/issues/82450#issuecomment-1947462977 [9]
    Reviewed-by: default avatarAlice Ryhl <aliceryhl@google.com>
    Tested-by: default avatarBoqun Feng <boqun.feng@gmail.com>
    Link: https://lore.kernel.org/r/20240217002717.57507-1-ojeda@kernel.org
    [ Upgraded to 1.77.1. Removed `allow(dead_code)` thanks to the previous
      patch. Reworded accordingly. No changes to `alloc` during the beta. ]
    Signed-off-by: default avatarMiguel Ojeda <ojeda@kernel.org>
    b481dd85
lib.rs 9.42 KB