Slashdot is powered by your submissions, so send in your scoop

 



Forgot your password?
typodupeerror
Linux

Linus Torvalds Gently Criticizes Build-Slowing Testing Code Left in Linux 6.15-rc1 (phoronix.com) 25

"The big set of open-source graphics driver updates for Linux 6.15 have been merged," writes Phoronix, "but Linux creator Linus Torvalds isn't particularly happy with the pull request." The new "hdrtest" code is for the Intel Xe kernel driver and is around trying to help ensure the Direct Rendering Manager header files are self-contained and pass kernel-doc tests — basic maintenance checks on the included DRM header files to ensure they are all in good shape.
But Torvalds accused the code of not only slowing down the full-kernel builds, but also leaving behind "random" files for dependencies "that then make the source tree nasty," reports Tom's Hardware: While Torvalds was disturbed by the code that was impacting the latest Linux kernel, beginning his post with a "Grr," he remained precise in his objections to it. "I did the pull, resolved the (trivial) conflicts, but I notice that this ended up containing the disgusting 'hdrtest' crap that (a) slows down the build because it's done for a regular allmodconfig build rather than be some simple thing that you guys can run as needed (b) also leaves random 'hdrtest' turds around in the include directories," he wrote.

Torvalds went on to state that he had previously complained about this issue, and inquired why the hdr testing is being done as a regular part of the build. Moreover, he highlighted that the resulting 'turds' were breaking filename completion. Torvalds underlined this point — and his disgust — by stating, "this thing needs to *die*." In a shot of advice to fellow Linux developers, Torvalds said, "If you want to do that hdrtest thing, do it as part of your *own* checks. Don't make everybody else see that disgusting thing...."

He then noted that he had decided to mark hdrtest as broken for now, to prevent its inclusion in regular builds.

As of Saturday, all of the DRM-Next code had made it into Linux 6.15 Git, notes Phoronix. "But Linus Torvalds is expecting all this 'hdrtest' mess to be cleaned up."

Linus Torvalds Gently Criticizes Build-Slowing Testing Code Left in Linux 6.15-rc1

Comments Filter:
  • Regression in 15 (Score:4, Interesting)

    by gavron ( 1300111 ) on Monday March 31, 2025 @01:51AM (#65270939)

    First, I like Phoronix and support Michael Larabel's many year effort.
    Second, I like Linux and he speaks to truth which as a coder I appreciate.

    I always run the latest release kernel, and sometimes the RC-5/6 branch.
    This time I waited til 15. Sadly I also added a second monitor.

    DRM is just a bear. It took several "set as primary monitor" "change resolution"
    and "hard reboot" (soft would hang) to get things to were I could down downres the
    max hardres the primary (laptop Dell 7xxx) and keep the backup (HP 24" HDMI)
    without "windows" crossing "displays". Sorry for the quotes, but it's getting so the
    terminology is not keeping up with what shows up on the monitors. I'd even say
    "displays on the monitors" but "displays" means something too.

    NOT a complaint. NOT a whine. I know what's involved and the devs are doing great.
    And yes, Linus is not wrong. This needs a little bit of a whoa-horsey pullback and fixing
    things. Some of those things are not part of a test suite because there are so many variables
    and so much subjective interaction.

    Again, I support Phorotnix and Michael and will continue to do so.
    I agree with Linus.
    I back the devs as I can.

    The product WILL get better. I can't wait.

    Ehud
    Tucson Arizona USA (that country currently fucking itself in the world, ugh)

    • Is having two displays with different DPI numbers still a hassle? My secondary screen is 4k tv for watching media.

    • I have a similar issue with my laptop. While my Linux installation has been extremely stable, the thing that is most likely to break is multi-monitor support, particularly with EVDI or my docking station driver. Btw, thanks Dell for copying Apple's braindead move of removing the HDMI port from the laptop. That required me to purchase a docking station whose drivers sometimes break upon OS updates, often consume lots of CPU for seemingly no reason, and cause the screen to black out many times during norma
  • Good (Score:5, Insightful)

    by devslash0 ( 4203435 ) on Monday March 31, 2025 @03:46AM (#65271023)

    Looks like he's a responsible maintainer who questions and controls tech debt before it becomes a serious issue. Many people could learn from that.

  • by zawarski ( 1381571 ) on Monday March 31, 2025 @03:51AM (#65271027)
    Is the news?
    • I presume he got kidnapped and is trying to discretely attract attention.

      I see no other rational explanation for "Linus Torvalds" and "gently" occurring in the same sentence, perkele!

    • by vbdasc ( 146051 )

      Yes, only uttering "crap" and "turd". From Linus, this is indeed gentle.

  • But to make your code stand out above the rest -- or should I say, under everybody's shoe -- won't be worth bragging points at the bar

  • Why is this news? Something writes a pull request and their test routine isn't deemed to be clean code, so the pull request gets not accepted yet. This happens in thousands of projects every day. It's not like Linus fired the developer or something.

    • Which is part of the point. Software developers are lazy. Rather than write neat, clean-as-can-be code, they throw the pot of spaghetti, sauce and all, at the wall and call it a day. When you have terabyte drives and gigs of RAM, what does it matter if your code is bloated and makes the system run through extraneous hoops?

      Whenever we do video driver updates on our laptops before sending them out we ask the question: why does an onboard video card need a 600 MB, or more, driver update when all it's doing

    • by Dan667 ( 564390 )
      Maybe the news is Linux is getting soft. Back in my day I would have learned new colorful insults from his code review.
  • by Sneftel ( 15416 ) on Monday March 31, 2025 @06:12AM (#65271119)

    Linus Torvalds Gently Criticizes

    That'll be the day.

  • by SuperDre ( 982372 ) on Monday March 31, 2025 @06:21AM (#65271129) Homepage
    So what you're saying is that the Linux build doesn't have any tests during its build process, leaving the possibility of added bugs in there....
    • Not sure if you're serious or trolling but testing is a separate step in the build process, you'll have a build target in the Makefile and a test target (typically broken into multiple sub steps). With something as big as the linux kernel each component/module is going to have it's own build and test steps that have dependencies on each other and all fall under the top level build and test. Let's say there are 100 components that have a dependency on hdrtest. If you want to test your component(s), their
    • If the area that was being tested was truly important to even 1% of Linux installs, perhaps this would be good. Having unit tests specific to a hardware device that most systems don't use, is of absolutely no utility for finding bugs. Oh noes this module that less than 0.1% of users will ever know exists has a subtle issue. Oh noes oh noes.
  • . . . is bogus. It slows down the iterative develop-build process FAR too much - even for most realistically helpful “unit tests”. It’s MUCH better to simply make such testing optional, and require it before merge into a broadly shared branch (especially the main code base).

    And I fully agree with Linus that testing shouldn’t pollute the code tree with random artifacts. The output should go into a separate, easily deleted, directory (preferably under /tmp on UNIX in order to harvest t

"It is better to have tried and failed than to have failed to try, but the result's the same." - Mike Dennison

Working...