Wedson Almeida Filho is a Microsoft engineer who has been prolific in his contributions to the Rust for the Linux kernel code over the past several years. Wedson has worked on many Rust Linux kernel features and even did a experimental EXT2 file-system driver port to Rust. But he’s had enough and is now stepping away from the Rust for Linux efforts.

From Wedon’s post on the kernel mailing list:

I am retiring from the project. After almost 4 years, I find myself lacking the energy and enthusiasm I once had to respond to some of the nontechnical nonsense, so it’s best to leave it up to those who still have it in them.

I truly believe the future of kernels is with memory-safe languages. I am no visionary but if Linux doesn’t internalize this, I’m afraid some other kernel will do to it what it did to Unix.

Lastly, I’ll leave a small, 3min 30s, sample for context here: https://youtu.be/WiPp9YEBV0Q?t=1529 – and to reiterate, no one is trying force anyone else to learn Rust nor prevent refactorings of C code."

  • @Flipper@feddit.org
    link
    fedilink
    303 months ago

    crash from undocumented function parameter invariants

    My favourite, as that was the exact point the dev was making in his talk, that the stuff is badly documented and that the function signature would document it perfectly.

    • @WarmApplePieShrek@lemmy.dbzer0.com
      link
      fedilink
      English
      -4
      edit-2
      3 months ago

      My favorite, as that is the exact point made by anti-rust people.

      What kind of type signature would prove the first block of any directory in an ext4 filesystem image isn’t a hole?

      • @pivot_root@lemmy.world
        link
        fedilink
        133 months ago

        The first directory block is a hole. But type == DIRENT, so no error is reported. After that, we get a directory block without ‘.’ and ‘…’ but with a valid dentry. This may cause some code that relies on dot or dotdot (such as make_indexed_dir()) to crash

        The problem isn’t that the block is a hole. It’s that the downstream function expects the directory block to contain . and .., and it gets given one without because of incorrect error handling.

        You can encode the invariant of “has dot and dot dot” using a refinement type and smart constructor. The refined type would be a directory block with a guarantee it meets that invariant, and an instance of it could only be created through a function that validates the invariant. If the invariant is met, you get the refined type. If it isn’t, you only get an error.

        This doesn’t work in C, but in languages with stricter type systems, refinement types are a huge advantage.

          • @pivot_root@lemmy.world
            link
            fedilink
            3
            edit-2
            2 months ago

            If it were poorly designed and used exceptions, yes. The correct way to design smart constructors is to not actually use a constructor directly but instead use a static method that forces the caller to handle both cases (or explicitly ignore the failure case). The static method would have a return type that either indicates “success and here’s the refined type” or “error and this is why.”

            In Rust terminology, that would be a Result<T, Error>.

            For Go, it would be (*RefinedType, error) (where dereferencing the first value without checking it would be at your own peril).

            C++ would look similar to Rust, but it doesn’t come as part of the standard library last I checked.

            C doesn’t have the language-level features to be able to do this. You can’t make a refined type that’s accessible as a type while also making it impossible to construct arbitrarily.

              • @pivot_root@lemmy.world
                link
                fedilink
                1
                edit-2
                2 months ago

                You’re going to need to cite that.

                I’m not familiar with C23 or many of the compiler-specific extensions, but in all the previous versions I worked with, there is no type visibility other than “fully exposed” or opaque and dangerous (void*).

                You could try wrapping your Foo in

                typedef struct {
                    Foo validated
                } ValidFoo;
                

                But nothing stops someone from being an idiot about it and constructing it by hand:

                ValidFoo trustMeBro;
                trustMeBro.validated = someFoo;
                otherFunction(trustMeBro);
                

                Or even just casting it.

                Foo* someFoo;
                otherFunction((ValidFoo*) someFoo);
                
                  • @pivot_root@lemmy.world
                    link
                    fedilink
                    22 months ago

                    That’s not the point, though. The point is to use a nominal type that asserts an invariant and make it impossible to create an instance of said type which violates the invariant.

                    Both validation functions and refinement types put the onus on the caller to ensure they’re not passing invalid data around, but only refinement types can guarantee it. Humans are fallible, and it’s easy to accidentally forget to put a check_if_valid() function somewhere or assume that some function earlier in the call stack did it for you.

                    With smart constructors and refinement types, the developer literally can’t pass an unvalidated type downstream by accident.

      • Preston Maness ☭
        link
        fedilink
        English
        33 months ago

        What kind of type signature would prove the first block of any directory in an ext4 filesystem image isn’t a hole?

        I don’t know if the type system proves it’s not a hole, but the type system certainly seems to force consumers to contend with the possibility by surfacing the outcomes at the type system level. That’s what the Either is doing in the example’s return type, is it not?

        fn get_or_create_inode(
            &self,
            ino: Ino
        ) -> Result<Either<ARef<Inode<T>>, inode::New<T>>>