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

Tux3 Gets Harshly Criticized Over Code Quality

Linux Kernel

Published on 19 May 2014 11:15 AM EDT
Written by Michael Larabel in Linux Kernel
19 Comments

On Friday the Tux3 file-system was called for review and offered to the mainline Linux kernel. Tux3 has been under development for more than six years but it seems that even after all this time and improvements, the code quality still isn't the best and the work is being scrutinized.

XFS developer Dave Chinner came out Sunday afternoon criticizing the Tux3 code. Dave began, "I had a quick look at the code. This is not a code review - it's a message to tell everyone else not to waste their time looking at the code right now..."
The code is a dog's breakfast of #ifdef hackery, stuff that doesn't work (lots of code surrounded by "#if 0"), there's "#if __KERNEL__ ... #else .... #endif" all through the code, etc. The "declarations within code" stuff is just horrible - it's not even used consistently so it just looks like laziness to me. [In other words], the code is an ugly mess and needs a serious amount of cleanup work.

There's also several kernel hacks within the code. One of the long talked about features was also the Tux3 built-in snapshotting capability that's supposed to be superior to ZFS and Btrfs, but Chinner couldn't find any of that code, only some code comments about adding snapshots.

Tux3 Gets Harshly Criticized Over Code Quality


Dave concluded:
Tux3 is just a prototype of a standard journaling filesystem. The tux3 code is still missing large parts of it's intended core functionality and there is nothing to tell us when that might appear. It really appears to me that tux3 is where btrfs was 5-6 years ago - the core of an idea, but a long, long way from being feature complete or production ready. btrfs still doesn't handle ENOSPC well and given that tux3's is following the same development path (BUG on ENOSPC) it doesn't fill me with any confidence that tux3 is going to turn out any better than btrfs in 5 years time.

Really, I don't see how you plan to bring tux3 to be feature complete and production ready in less than 2-3 years. The current code is barely functional at this point and there's still questions that haven't been answered about whether core tux3 functionality can even be made to work properly, let alone integrated effectively.

IMO, it's a waste of time right now asking anyone to review this code for inclusion until it has been cleaned up, the core infrastructure problems have been solved and the core filesystem code is much closer to feature complete.....

Lead Tux3 developer Daniel Phillips has yet to respond to these critical comments. For what it's worth, it was also Dave Chinner in 2012 who was taking shots at EXT4 and Btrfs.

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 Launches The Core i7 5960X, Mighty Powerful Haswell-E CPUs
  2. AMD Radeon R9 290: Gallium3D vs. Catalyst Drivers
  3. AMD Radeon R9 290 Open-Source Driver Works, But Has A Ways To Go
  4. Trying The Configurable 45 Watt TDP With AMD's A10-7800 / A6-7400K
Latest Linux Articles
  1. Testing For The Latest Linux Kernel Power Regression
  2. The Most Energy Efficient Radeon GPU For AMD Linux Gaming
  3. 20-Way Radeon Comparison With Open-Source Graphics For Steam On Linux Gaming
  4. Preview: OS X 10.10 Yosemite vs. Ubuntu Linux GPU Performance
Latest Linux News
  1. Enlightenment E19 RC3 Shows Off The New Wayland Compositor
  2. Metro Redux Is Going To Require OpenGL 4.x On Linux
  3. Jailhouse v0.1 Released As A Basic Hypervisor For Linux
  4. Google's Chromebook "Samus" Now Supported By Coreboot
  5. Chrome 38 Now In Beta With Exciting Advancements
  6. Ubuntu's Utopic Unicorn 14.10 Beta 1 Released
  7. Genode OS 14.08 Has New GUI Architecture, Pluggable VFS
  8. Another Intel Linux Power Regression Is Being Investigated
  9. DNF Makes It A Step Closer To Replacing Yum On Fedora
  10. OS Battle: Linux Takes 1.7% Desktop Marketshare
Latest Forum Discussions
  1. Canonical Joined The Khronos Group To Help Mir/Wayland Drivers
  2. Users defect to Linux as OpenBSD removes Lynx from base system
  3. Radeon HD5670 and Ubuntu 14.04
  4. Updated and Optimized Ubuntu Free Graphics Drivers
  5. AMD Releases UVD Video Decode Support For R600 GPUs
  6. Best Radeon for a Power Mac G5?
  7. OC capability - Intel Core i5 4690K & Biostar Hi-Fi Z97WE
  8. Announcing radeontop, a tool for viewing the GPU usage