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

X.Org: "A Wasteland of Unreviewedness"

X.Org

Published on 17 May 2012 04:06 AM EDT
Written by Michael Larabel in X.Org
9 Comments

After David Airlie brought up the new DDX driver API for the X.Org Server, a new discussion was born concerning the lack of patch review taking place for the X.Org Server.

David Airlie commented on the developers' mailing list about the lack of patch review for the new API patches, he wonders how he's "going to get the next 50 patches in at this rate some time this year." Alan Coopersmith then responded with how there seems to be a harder time overall in getting patch reviews done. Coopersmith says, "I've got no ideas how to fix this quickly, but we need to get it fixed."

The Oracle X.Org developer started a new mailing list thread about the lack of reviews. David Airlie went on to comment:
I've pretty much no idea how to deal with it sanely. We've moved to larger scale development model without a larger set of developers. The kernel isn't even as stringent wrt to reviews as xorg-devel is.

I'm guessing we'll probably have to have review swap parties or meetings or something insane like that to clear the backlog on occasions, it would be nice if patchwork could be kept up to date, but it would involve anyone handling patches to jump on and clean up the ones they've merged already.

I spent a major amount of my time either reviewing kernel patches, or persuading others to review other peoples patches so I don't have to, I'm not sure if we need some more tracking from Keith or others on what unreviewed stuff is outstanding and who best to direct it to, but again it involves a time commitment from someone and I've no idea who could afford it.

Like the input guys have a bit of crossover work, the build system stuff seems to be covered, but the rest of the server is a wasteland of unreviewedness.

Keith Packard's response to this wasteland of unreviewedness was, "Small changes seem to get reviewed pretty quickly; it's big changes that take actual thought that just aren't getting looked at quickly (if at all)."

Peter Hutterer's response is in favor of loosening the review requirements for xorg-server code. Alan Coopersmith also re-sent some patches that have been sent to the list before, but went un-reviewed, so if no one steps up to the plate, he's just going to be sending patches un-reviewed for merging.

But, on the plus side, the rants got Aaron Plattner of NVIDIA to review the first four patches of David Airlie's for the driver API changes, per this message.

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. Acer B286HK: A 28-inch UHD LED 4K Monitor For As Low As $350
  2. Intel Xeon E5-1680 v3 & E5-2687W v3 Compared To The Core i7 5960X On Linux
  3. Intel 120GB 530 Series SSD Linux Performance
  4. Btrfs/EXT4/XFS/F2FS RAID 0/1/5/6/10 Linux Benchmarks On Four SSDs
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. Jolla Tablet Could Have MicroSDHC, Split Screen, 3.5G Upgrade
  2. Intel Has Last Round Of DRM Changes For Linux 3.19, Starts Dropping DRI1/UMS
  3. Fedora 21 Release Candidate 1 Awaits Your Testing
  4. GCC 5 Adds Support For ARM's Cortex-A17
  5. KWayland Server Component Coming For KDE Plasma 5.2
  6. NVIDIA Posts Tegra Gallium3D Patch For K1+ Support
  7. Ubuntu 14.10 MacBook Air Tests With Linux 3.18, Mesa 10.5
  8. AMD Richland APU Support Added To Coreboot
  9. 2014 Holiday Shopping Reminder, Happy Thanksgiving
  10. Python 3 Support Added To The GNOME Shell
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