„Zamknij mordę, ty *$%#!”, czyli Linus Torvalds krytykuje programistę

Linus Torvalds bardzo dosadnie skrytykował niejakiego Mauro Carvalho Chehaba, programistę z Red Hata za zgłoszenie niewłaściwego sposobu naprawienia usterki w Pulseaudio i innych aplikacjach zewnętrznych.

Chehab stał się obiektem, delikatnie rzecz ujmując, bardzo dosadnej krytyki ze strony Torvaldsa. Programista, pracujący nad kernelem Linuxa oraz nad Red Hatem, próbował załatać usterkę w systemie. Zaproponowane przez Chehaba rozwiązanie ponoć sprawiało jeszcze większe problemy Było, jak to Torvalds określił, „absolutnym ścierwem”. Twórca Linuksa się wściekł. – Mauro, zamknij się, do k**** nędzy!. Jestem wściekły, ponieważ twój cały email jest po prostu zły, a latka która psuła inne rzeczy była w oczywisty sposób ścierwem. Cała łatka to zepsute g**** - czytamy w mailu od Torvaldsa, który został opublikowany.

Twórca Linuxa jest najwyraźniej człowiekiem dosadnym. Z jednej strony to dobrze, przynajmniej nie ukrywa niczego przed ludźmi. Z drugiej jednak… kto by chciał takiego szefa?

Całość listu w języku angielskim publikuję poniżej:

From Linus Torvalds
Subject Re: [Regression w/ patch] Media commit causes user space to misbahave (was: Re: Linux 3.8-rc1)
-------
Mauro Carvalho Chehab wrote:

Are you saying that pulseaudio is entering on some weird loop if the returned value is not -EINVAL? That seems a bug at pulseaudio.
----
Mauro, SHUT THE F**K UP!

It's a bug alright - in the kernel. How long have you been a maintainer? And you *still* haven't learnt the first rule of kernel maintenance?

If a change results in user programs breaking, it's a bug in the kernel. We never EVER blame the user programs. How hard can this be to understand?

To make matters worse, commit f0ed2ce840b3 is clearly total and utter CRAP even if it didn't break applications. ENOENT is not a valid error return from an ioctl. Never has been, never will be. ENOENT means "No such file and directory", and is for path operations. ioctl's are done on files that have already been opened, there's no way in hell that ENOENT would ever be valid.

So, on a first glance, this doesn't sound like a regression, but, instead, it looks tha pulseaudio/tumbleweed has some serious bugs and/or regressions.

Shut up, Mauro. And I don't _ever_ want to hear that kind of obvious garbage and idiocy from a kernel maintainer again. Seriously.

I'd wait for Rafael's patch to go through you, but I have another error report in my mailbox of all KDE media applications being broken by v3.8-rc1, and I bet it's the same kernel bug. And you've shown yourself to not be competent in this issue, so I'll apply it directly and immediately myself.

WE DO NOT BREAK USERSPACE!

Seriously. How hard is this rule to understand? We particularly don't break user space with TOTAL CRAP. I'm angry, because your whole email was so _horribly_ wrong, and the patch that broke things was so obviously crap. The whole patch is incredibly broken shit. It adds an insane error code (ENOENT), and then because it's so insane, it adds a few places to fix it up ("ret == -ENOENT ? -EINVAL : ret").

The fact that you then try to make *excuses* for breaking user space, and blaming some external program that *used* to work, is just shameful. It's not how we work.

Fix your f*cking "compliance tool", because it is obviously broken. And fix your approach to kernel programming.

-Linus

Za: chip.pl