1. Computers
  2. Display Drivers
  3. Graphics Cards
  4. Memory
  5. Motherboards
  6. Processors
  7. Software
  8. Storage
  9. Operating Systems


Facebook RSS Twitter Twitter Google Plus


Phoronix Test Suite

OpenBenchmarking.org

Another Linus Rant About Linux DRM; Rejects Pull

Linux Kernel

Published on 06 December 2011 05:11 PM EST
Written by Michael Larabel in Linux Kernel
11 Comments

Linus Torvalds is known to make a few colorful remarks from time to time. Today he's become frustrated once again with the Linux DRM layer and has rejected a pull request for the Linux 3.2 kernel.

David Airlie had sent in a pull request today for the Linux 3.2 kernel that addresses an Intel VT-d issue with Ironlake graphics, a kexec fix where the GPU write-back engines need to be switched off, and then a trivial Radeon KMS fix. The kexec fix sent Linus over the edge.

In the past Linus Torvalds has been vocal about select Direct Rendering Manager (DRM) pull requests from David Airlie with calling the GEM memory management patches at first being untested crap, the lack of Nouveau DRM in the Linux kernel, questioning what's wrong with the whole DRM crowd, and showing why DRM has been problematic (late pull request), among other remarks.

Below is the response from Linus Torvalds for Airlie's pull request.
Quite frankly, I think it's too late for something like a kexec bugfix. Nobody cares. So kexec doesn't work - that's not something new. This doesn't smell like a regression to me. And the kcalloc things you mention *sound* like some kind of cleanup crap.

The DRM layer has been fairly good for a few releases, but I'm getting the feeling that I need to start pushing back, because I'm getting stuff that I don't think matters, and shouldn't be sent to me after -rc4.

So I'm not pulling this.

What the heck is up?

By now, I want fixes that either fix real regressions that people *care* about, or that help new unreleased hardware that people *will* care about and that cannot possibly mess up old users.

kexec? Who the f*ck cares? Really?

Linus

As it's not exactly a popular end-user Linux feature, for those that don't known, kexec allows a new kernel to be booted and switched to from an existing running Linux kernel without a full system reboot. Using kexec skips over the BIOS/UEFI hardware initialization -- this allows for a faster switch to a new kernel and other niche cases (e.g. there's a way to re-clock with the NVIDIA binary blob and then switching over to Nouveau, etc).

After Airlie replied to Linus, there was a soft response from Torvalds where he clarified his position.
So having looked at the patch itself, I don't dislike the notion of making sure certain fields are nicely initialized. So I don't hate the patch itself, but quite frankly, to me it doesn't smell even *remotely* like "regression fix". I don't think this is something that has ever worked.

And I do realize that some enterprise distros want to use kexec, but at the same time I say "that's *their* problem". We know kexec hasn't been horribly reliable, anybody who uses it should have be taking that into account.

I hope kexec gets more reliable, but I *also* really hope that our RC series will calm down, and on the whole, weighing the two concerns, when we're talking about something that has never worked before either, I think the thing is pretty clear.

That said, if there is some other real use-case ("this fixes problems with the BIOS from Xyz initializing things to random crap"), I'd have no real objections to the patch itself.

So my complaint is really a "I want to be more anal about the later -rc patches, I feel we're slipping", not a "I hate the patch per se".

Linus

Linus is basically wanting to reserve the post merge window now for "real" regression fixes and new hardware enablement, at least for the DRM subsystem.

The thread can be found on LKML.org for those wanting to read in full or check for updates.

About The Author
Michael Larabel is the principal author of Phoronix.com and founded the web-site in 2004 with a focus on enriching the Linux hardware experience and being the largest web-site devoted to Linux hardware reviews, particularly for products relevant to Linux gamers and enthusiasts but also commonly reviewing servers/workstations and embedded Linux devices. Michael has written more than 10,000 articles covering the state of Linux hardware support, Linux performance, graphics hardware drivers, and other topics. Michael is also the lead developer of the Phoronix Test Suite, Phoromatic, and OpenBenchmarking.org automated testing software. He can be followed via and or contacted via .
Latest Linux Hardware Reviews
  1. Intel Xeon E5-1680 v3 & E5-2687W v3 Compared To The Core i7 5960X On Linux
  2. Intel 120GB 530 Series SSD Linux Performance
  3. Btrfs/EXT4/XFS/F2FS RAID 0/1/5/6/10 Linux Benchmarks On Four SSDs
  4. AMD's Windows Catalyst Driver Remains Largely Faster Than Linux Drivers
Latest Linux Articles
  1. Mesa Git Yields Performance Improvements For Newer AMD GPUs
  2. Apple OS X 10.10 vs. Ubuntu 14.10 Performance
  3. Mesa 10.5-devel Brings Some Intel Haswell HD Graphics Changes Over Mesa 10.3
  4. NVIDIA vs. Nouveau Drivers With Linux 3.18 + Mesa 10.4-devel
Latest Linux News
  1. Git 2.2.0 Released With 550+ Changes
  2. GNOME 3.15.2 Released
  3. Quantum OS Aims For A Linux Desktop With QML, Wayland & Material Design
  4. New Open-Source, Linux Benchmarks To Feast On
  5. FreeBSD Plans For The Next Ten Years
  6. Qt 5.4 Planned For Release On 9 December
  7. Meizu's Ubuntu Phone Not Expected Until Early Next Year
  8. DragonFlyBSD 4.0 Drops i386 Support, Improves Graphics
  9. Expensive "Free/Libre Software Laptop" Uses A NVIDIA GPU
  10. QEMU 2.2-rc3 Released, Final Release Pushed Back By Couple Days
Latest Forum Discussions
  1. Updated and Optimized Ubuntu Free Graphics Drivers
  2. Hurrican SDL Port
  3. Roadmap to Catalyst 14.10 ?
  4. how to configure module phoromatic ?
  5. PulseAudio 6.0 Is Coming & Other Linux Audio Plans For The Future
  6. Debian Developer Resigns From The Systemd Maintainership Team
  7. Cant get working Kaveri APU - A10-7850k
  8. Script for Fan Speed Control