They got to him

  • Buffalox@lemmy.world
    link
    fedilink
    English
    arrow-up
    132
    arrow-down
    13
    ·
    11 days ago

    On Tuesday, he submitted a 10-patch series that delivered a 23-fold throughput improvement in a synthetic benchmark

    Impressive, probably kind of like I can answer math questions very very fast.

    Reviewers have since identified a race condition and raised concerns about memory reclamation.

    That what I thought, I can answer math questions fast, but I don’t claim to be able to answer them correctly.

      • Buffalox@lemmy.world
        link
        fedilink
        English
        arrow-up
        43
        arrow-down
        4
        ·
        11 days ago

        Maybe it did, but apparently the code was not properly reviewed initially, so there could be other hidden problems.
        Of course that’s the same for human made code, but there’s a problem in only judging code from how it performs, and not actually understanding the code. And that seems to be the situation with a lot of AI code.

        • db2@lemmy.world
          link
          fedilink
          English
          arrow-up
          24
          arrow-down
          2
          ·
          11 days ago

          And particularly disturbing when the human understanding failure is from the original and lead Linux developer.

          • Buffalox@lemmy.world
            link
            fedilink
            English
            arrow-up
            10
            arrow-down
            3
            ·
            edit-2
            10 days ago

            100% that is the worst place to have the problem originate.
            I have rarely disagreed with Linus Torvalds on things regarding the kernel, and maybe I’m wrong, but I strongly disagree with him on this issue. Contributing code nobody understands cannot be a good thing.

            • amorpheus@lemmy.world
              link
              fedilink
              English
              arrow-up
              10
              arrow-down
              4
              ·
              10 days ago

              Contributing code nobody understands cannot be a good thing.

              You people act like Linux adopted an all-AI pipeline.

            • auzy1@lemmy.world
              link
              fedilink
              English
              arrow-up
              1
              arrow-down
              2
              ·
              edit-2
              10 days ago

              The code still undergoes the exact same code review as before, because humans were the original slop coders.

              The standards needed for code to be accepted haven’t changed at all. Even the code Linus personally is reviewed…

              • Buffalox@lemmy.world
                link
                fedilink
                English
                arrow-up
                5
                arrow-down
                1
                ·
                10 days ago

                The code still undergoes the exact same code review as before.

                That’s not good enough, because it is well documented that AI often make mistakes and absolutely needs to undergo way stricter review.
                It is also well documented that although you can achieve some results quickly, AI code also quickly becomes impossible to maintain.

                If nobody understands the code, as it seems was the case here, and it is introduced anyway, the kernel is doomed to deteriorate.

                My wife has already noticed deteriorating quality in Debian, with system hangs after they loosened the requirements for AI code. The quality of Debian now is lower than it has been in a decade!

                Maybe you need to watch something like this:

                https://www.youtube.com/watch?v=l9ARF2KcDgc

                  • Buffalox@lemmy.world
                    link
                    fedilink
                    English
                    arrow-up
                    2
                    arrow-down
                    1
                    ·
                    edit-2
                    10 days ago

                    Lol. That guy banned me from his community

                    Maybe because you are making stupid ignorant comments.

        • Diplomjodler@lemmy.world
          link
          fedilink
          English
          arrow-up
          6
          arrow-down
          7
          ·
          10 days ago

          So, let me get this right. You’re disapproving on how things were handled based on your speculation of how it went?

          • Buffalox@lemmy.world
            link
            fedilink
            English
            arrow-up
            8
            arrow-down
            8
            ·
            10 days ago

            Not based on speculation, based on numerous analysis of the problems with using AI code you don’t understand.
            Clearly Linus failed to understand the code, otherwise he wouldn’t have suggested it for submission.

            You seem to have a problem with criticism of Linus, is it a religious thing?

            • auzy1@lemmy.world
              link
              fedilink
              English
              arrow-up
              1
              arrow-down
              2
              ·
              edit-2
              10 days ago

              You understand it less it sounds like.

              1. Race Conditions are common. This isn’t new to AI. Nobody writes stories about them though unless they lead to security issues (like with XFS, which had one for 10 years and only fixed recently).
              2. There is a lot happening in the kernel, its not always a matter of looking in one tiny location. A race condition is pretty broad and can lead to many things.
              3. Patches get rejected all the time, for reasons including race conditions which are identified.
              4. If you check the post, other developers had tried and failed to fix the same issue. He’s not the only one. It was just yet another failed attempt
              5. Optimisation is HARD, and always has been.
              6. Finally, you say “suggested it for submission”. That code would go to a staging repo first for testing. He wasn’t suggesting pushing it immediately to the main tree…
              7. Standard development tools (at least for my development work) wouldn’t catch race conditions (maybe commercial ones / high end ones would)… AI is actually far better at detecting these things

              This isn’t a unique or interesting story in any capacity… It’s boring and normal

            • Diplomjodler@lemmy.world
              link
              fedilink
              English
              arrow-up
              2
              arrow-down
              7
              ·
              10 days ago

              Of course of course. How dare you blaspheme against our Lord and Saviour, filthy infidel!

              But anyway, I’d like to be as enlightened as you. Could you name any of those numerous analyses that detail exactly what Torvalds did and what he failed to understand?

              • Buffalox@lemmy.world
                link
                fedilink
                English
                arrow-up
                3
                arrow-down
                2
                ·
                edit-2
                10 days ago

                No I can’t, because those are so general and commonly known, that if you somehow are unaware, you must have been living under a rock for the past year.
                The second option is that you are just trolling or for some reason is in denial.
                Either way I’m not wasting my time on any of those options.

      • devfuuu@lemmy.world
        link
        fedilink
        English
        arrow-up
        20
        arrow-down
        6
        ·
        11 days ago

        Only while there are still reviewers who care to review the slop. Which is a rare breed and will quickly soon vanish.

        • corsicanguppy@lemmy.ca
          link
          fedilink
          English
          arrow-up
          2
          arrow-down
          1
          ·
          10 days ago

          I bet some projects will get more tolerance and love than others, based on value and utility. The kernel will get more, and Systemd will get less.