LINUX.ORG.RU
ФорумTalks

Линус теперь няшка

 , ,


1

1

https://www.phoronix.com/scan.php?page=news_item&px=Linus-Torvalds-New-Po...

Ответ к коммиту, который добавляет драйвер для контроллера BigBen в ядро и включает его по умолчанию:

We do *not* enable new random drivers by default. And we most *definitely* don't do it when they are odd-ball ones that most people have never heard of.

Yet the new «BigBen Interactive» driver that was added this merge window did exactly that.

Just don't do it.

Yes, yes, every developer always thinks that _their_ driver is so special and so magically important that it should be enabled by default. But no. When we have thousands of drivers, we don't randomly pick one new driver to be enabled by default just because some developer thinks it is special. It's not.

So the

default !EXPERT

was completely wrong in commit 256a90ed9e46 («HID: hid-bigbenff: driver for BigBen Interactive PS3OFMINIPAD gamepad»). Please don't do things like this.

Linus

Посмотрим насколько его хватит.

ox55ff ★★★★★
()

БелаяУточка.pcx

FiXer ★★☆☆☆
()

белки-истерички ИТТ.

Линус такой, какой и был.

legolegs ★★★★★
()
Ответ на: комментарий от das_tier

In comparison, here's his post from last November over a similar issue:

You add new drivers and then default them to «on».

THAT IS COMPLETELY UNACCEPTABLE.

I don't know why I have to say this every single merge window, but let's do it one more time:

As a developer, you think _your_ driver or feature is the most important thing ever, and you have the hardware.

AND ALMOST NOBODY ELSE CARES.

Read it and weep. Unless your hardware is completely ubiquitous, it damn well should not default to being defaulted everybody elses config. ... But something like CONFIG_DELL_SMBIOS sure as hell does not merit being default on. Not even if you have enabled WMI.

EVERY SINGLE «default» line that got added by this branch was wrong.

Stop doing this. It's a serious violation of peoples expectations. When I do «make oldconfig», I don't want some new random hardware support.

There was also another situation today testing Torvalds' patience when it came to a kernel oops, with the reformed Torvalds writing:

On my laptop I'm getting a kernel page fault with the current git tree, and I'm tentatively blaming commit

9ee3e06610fd («HID: i2c-hid: override HID descriptors for certain devices»)

but that's simply because it's the only thing that seems to touch this particular area in this merge window.

The oops looks like this: ... which is why I suspect that new i2c_hid_get_dmi_hid_report_desc_override() code.

I *think* the problem is that the i2c_hid_dmi_desc_override_table[] isn't terminated by a NULL entry, and I will test that next.

What makes me *very* unhappy about this is that if I'm right, I think it means that code was literally not tested at all by anybody who didn't have one of the entries in that list.

Vsevolod-linuxoid ★★★★★
()
Последнее исправление: Vsevolod-linuxoid (всего исправлений: 1)

Я помню MUZHIK-2 на пару неделек на дурочку отъезжал.
Может Линус тоже так расслаблялся?

Deleted
()
Ответ на: комментарий от das_tier

А раньше он бы матом обложил за этот коммит?

Так он и сейчас матом обложил. Просто матные слова не использовал. :) Ну примерно как в анекдоте про сварщиков - «Иван Петрович, пожалуйста перестаньте капать мне расплавленным металлом за шиворот.»

Stanson ★★★★★
()

Надолго ли его хватит?

Задать дебильный вопрос намного проще, чем дать на него умный ответ. Дебилов много, а Линуса один. Этим объяснишь, завтра новые придут.

Pacmu3ka
()

Так это не он пишет. Линус лежит связанный в углу и с шорохом компилирует новый RC. А пишет его дочь. Привыкайте.

Deleted
()
Ответ на: комментарий от Vsevolod-linuxoid

Главная разница с первым сообщением в том, что там реально все ясно, а в этих двух какой-то поток сознания

Deleted
()

хм, как ему удалось написать слова fuck и moron ни разу не использовав эту последовательность букв? Магия....

dikiy ★★☆☆☆
()
Ответ на: Надолго ли его хватит? от Pacmu3ka

Вообще, если один и тот же вопрос возникает регулярно, то дело скорее всего не в том что «все тупые», а в том что вопрос не простой и стоило бы его один раз подробно задокументировать и давать ссылочку, а не каждый раз соревноваться в изобретении новых креативных способов отфутболивания этого вопроса.

alpha ★★★★★
()
Ответ на: комментарий от ncrmnt

Никак иначе накодил индивидуальный цензор за время «отпуска».

ados ★★★★★
()
Ответ на: комментарий от alpha

Вот просто интересно, насколько «непростым» должно быть понимание, что ваше гребаное железо по умолчанию интересно только вам, а не всем остальным пользователям ядра?

Deleted
()
Последнее исправление: Deleted (всего исправлений: 1)
Ответ на: комментарий от Deleted

Это ванильное ядро, мастер-ветка.

Мне вот например с моего места наоборот интересно почему исходный код новых модулей включается в кодовую базу ядра, но по сути выключается из процесса тестирования.

Чтобы баги обнаруживались потом на рабочей машине пользователем, а ни в коем случае не тревожили мэтров на этапе когда их ещё можно заметить и исправить?

Полагаю этому есть разумные объяснения, но было бы неплохо их увидеть в каком-то более удобочитаемом виде, чем «this is NOT ACCEPTABLE!!!!».

И за двадцать лет уж можно было собраться и написать Kernel Driver Guide где задокументировать процесс добавления новых драйверов практически пошагово, в том числе расписать ответы на все часто задаваемые вопросы.

alpha ★★★★★
()
Ответ на: комментарий от Deleted

Линус, как обычно, сначала вызверился, потом пошел свою собственную матчасть изучать. Ну зато без мата теперь :)

alpha ★★★★★
()
Ответ на: комментарий от alpha

Какого тестирования, когда там драйвер для железки, которой ни у кого кроме закоммитившего нет?

nvidia
()
Ответ на: комментарий от alpha

Насколько я понимаю, суть в работе make oldconfig, т.е. в том, что пользователи запуская эту команду рассчитывают на то, что никаких новых драйверов не попадёт в сборку.

Т.е. сабжевый коммит ломает логику этой команды, а в приоритетах Линуса всегда была обратная совместимость для пользователя до последнего.

Deleted
()
Ответ на: комментарий от Deleted

Насколько я понимаю, суть в работе make oldconfig, т.е. в том, что пользователи запуская эту команду рассчитывают на то, что никаких новых драйверов не попадёт в сборку.

1) По твоему это самоочевидный факт не требующий пояснений?

2) Если бы это было так, как ты говоришь, там не было бы в принципе ни одного изменения, но изменения там бывают. Соответственно есть некий неявный процесс определения, что и как можно изменять и что нет.

До тех пор пока этот процесс существует только в голове Линуса, ему приходится страдать и объяснять его суть каждому встречному.

Для решения этой проблемы люди и придумали документацию.

alpha ★★★★★
()
Ответ на: комментарий от alpha

1) По твоему это самоочевидный факт не требующий пояснений?

Работа утилиты или то, что людям не нужны редкоземельные драйвера по дефолту? Первое - нет, второе - да.

Для решения этой проблемы люди и придумали документацию.

Ты так говоришь, будто мы тут обсуждаем непонятно откуда взявшийся код без доков и с комментариями на китайском. А тем временем Linux - одна из самых наиболее качественно документированных программных систем. Во всяком случае, опенсорсных.

Но всегда есть куда расти, да.

Deleted
()
Ответ на: комментарий от Deleted

Ты так говоришь, будто мы тут обсуждаем непонятно откуда взявшийся код без доков и с комментариями на китайском. А тем временем Linux - одна из самых наиболее качественно документированных программных систем. Во всяком случае, опенсорсных.

Не надо идеализировать Linux. Это крутая, не имеющая аналогов, великая и могучая система, но не идеальная в том числе и с точки зрения разработки.

И например, документация к коду и документация к процессам - разные вещи. И просто судя по тому что один и тот же вопрос всплывает в мейллистах и хранится там же вместо того чтобы быть сложенным в отдельный файл с названием «Should I enable my driver by default?», проблема с документацией наличествует.

И там не новички делают ошибки, а опытные майнтейнеры.

Стоило бы и тест написать на самом деле, не только доку.

alpha ★★★★★
()
Ответ на: комментарий от ncrmnt

Нет. Скрипт не пишут месяц. Там наверно ИИ, как у гугл транслейта.

SR_team ★★★★★
()
Ответ на: комментарий от Deleted

суть в работе make oldconfig, т.е. в том, что пользователи запуская эту команду рассчитывают на то, что никаких новых драйверов не попадёт в сборку.

Ха. Ха. Ха.

upcFrost ★★★★★
()

Злой ты, модератор!


Ну, чисто random brain farts:

- Линуса вызывали для обсуждения некоторых вопросов и стратегии.

- А джва - SJW засланцы устроили набег не просто так.

Т.е. чувак, под видом дурки, отплозал решать какие-то административные задачи.

Вернулся в аккурат перед объявлением сделки.

Доктор, я - псих?

Deleted
()

Гы, слова нормальные, а голос матерный.

ya-betmen ★★★★★
()

Больше не будет пальца Линуса?

Ушла эпоха...

cvs-255 ★★★★★
()
Ответ на: комментарий от alpha

Линус, как обычно, сначала вызверился, потом пошел свою собственную матчасть изучать. Ну зато без мата теперь :)

Это не значит, что он сам этих [неразборчиво] решил не [неразборчиво] на [неразборчиво]. Потому, что мат бывает ещё и завуалированным. Так что от Линуса Торвальдса это следует читать как «[неразборчиво]». Посему все [неразборчиво] CoCk'а опять пошли [неразборчиво] стройной шеренгой.

Quasar ★★★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.