The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

В ядре Linux найдена забытая заплата, влияющая на производительность CPU AMD, opennews (ok), 27-Сен-22, (0) [смотреть все] +2

Сообщения [Сортировка по времени | RSS]


50. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +2 +/
Сообщение от An2 (?), 28-Сен-22, 02:03 
И интересно, что человек из АМД, проделавший всю работу [1], и пославший исправление, в итоге, не является автором патча [2]. Человек из Интел оформил его исправление по-другому - и вуаля - автор!

[1] https://lore.kernel.org/lkml/20220921063638.2489-1-kprateek..../
[2] https://lore.kernel.org/lkml/67154681-0e0a-69d7-d4ee-21067c9.../

Ответить | Правка | К родителю #11 | Наверх | Cообщить модератору

62. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +3 +/
Сообщение от edo (ok), 28-Сен-22, 05:52 
и что? вы думаете, что эта разработчики трудятся исключительно чтобы потешить тщеславие?

как вижу ситуацию я:
1. сотрудник amd обнаружил в коде задержку, которая не требуется современным системам amd и опубликовал соответствующий патч, в которой отключил эту задержку на zen;
2. сотрудник intel произвёл мини-расследование и обнаружил, что эта задержка была добавлена для актуальных 20 лет назад чипсетов intel и предложил свою версию патча, которая отключает эту задержку на всех не-штеуд системах (предполагая, что проблема затронула только intel);
3. другие разработчики предположили, что эта задежка, возможно, была необходима и для других старых систем:
https://lore.kernel.org/lkml/Yyy6l94G0O2B7Yh1@rhlx01.hs.../

идёт вполне нормальная коллективная работа; и, вполне вероятно, скоро в ядро вольют третью версию патча.

Ответить | Правка | Наверх | Cообщить модератору

139. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +/
Сообщение от An2 (?), 29-Сен-22, 02:19 
> и предложил свою версию патча

Т.е. на код-ревью выкидываешь то, что кто-то написал, - и подаёшь своё "новое". И несмотря на почти один и тот же эффект не ставишь оригинального автора хотя бы в со-авторы. Хм.

Ответить | Правка | Наверх | Cообщить модератору

72. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +/
Сообщение от Аноним (72), 28-Сен-22, 07:11 
Но первое исправление совсем неполное, там только для zen отключение.
Завтра amd выпустит новую архитектуру izen и её также будет тормозить.
Не говоря уже об всех предыдущих процессорах amd и всех остальных производителей.
Ответить | Правка | К родителю #50 | Наверх | Cообщить модератору

107. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +2 +/
Сообщение от n00by (ok), 28-Сен-22, 11:26 
> И интересно, что человек из АМД, проделавший всю работу [1], и пославший
> исправление, в итоге, не является автором патча [2]. Человек из Интел
> оформил его исправление по-другому - и вуаля - автор!
> [1] https://lore.kernel.org/lkml/20220921063638.2489-1-kprateek..../
> [2] https://lore.kernel.org/lkml/67154681-0e0a-69d7-d4ee-21067c9.../

И интересно, что в оригинале оно выглядит не совсем так, как интерпретировано выше:

Reported-by: K Prateek Nayak <***@amd.com>
Suggested-by: Rafael J. Wysocki <***@intel.com>
Signed-off-by: Dave Hansen <***@linux.intel.com>
Reviewed-by: Mario Limonciello <***@amd.com>
Tested-by: K Prateek Nayak <***@amd.com>
Link: https://lore.kernel.org/all/20220921063638.2489-1-kprateek.n.../
Link: https://lkml.kernel.org/r/20220922184745.3252932-1-dave.hans...

Ответить | Правка | К родителю #50 | Наверх | Cообщить модератору

141. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +/
Сообщение от An2 (?), 29-Сен-22, 02:26 
> Reported-by:

Если кто-то обнаруживает проблему и описывает, как на неё наткнуться, т.е. создаёт хороший баг-репорт, то его записывают в эту графу.

АМДшник же не просто нашёл условия, при которых тормозит, он нашёл код, который приводит к тормозам. И исправил. Да, оказалось, можно чуть лучше оформить исправление. В любом случае не надо было его деградировать до "просто" сообщившего об ошибке, а хотя бы добавить в "Co-Authored-By:".

Ответить | Правка | Наверх | Cообщить модератору

144. "В ядре Linux найдена забытая заплата, влияющая на производит..."  +1 +/
Сообщение от n00by (ok), 29-Сен-22, 11:28 
Да не исправил он, а решил для частного случая: boot_cpu_has(X86_FEATURE_ZEN).
Если бы это приняли, то для других процессоров вероятно вообще бы никогда не исправили.

Интелы по сути признали, что только их изделиям свойственна проблема: boot_cpu_data.x86_vendor != X86_VENDOR_INTEL

АМД произвели ревью, то есть проконтролировали, что всё сделано правильно. Обнаружившего и так два раза упомянули, плюс ссылка на первую версию.

Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру