The OpenNET Project / Index page

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



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

Оглавление

Релиз ядра Linux 5.18, opennews (??), 23-Май-22, (0) [смотреть все]

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


29. "Релиз ядра Linux 5.18"  +7 +/
Сообщение от Аноним (35), 23-Май-22, 14:08 
Мой патч не принимают уже 2-й год. Хотя там всё по делу. У кого есть подобный опыт из опеннетчиков?
Ответить | Правка | Наверх | Cообщить модератору

34. "Релиз ядра Linux 5.18"  +1 +/
Сообщение от Аноним (5), 23-Май-22, 14:16 
Скинь сюда. Добавлю твой патч себе, чтоб тебе не так обидно было.
Ответить | Правка | Наверх | Cообщить модератору

58. "Релиз ядра Linux 5.18"  +1 +/
Сообщение от Аноним (1), 23-Май-22, 15:48 
Проводи через Шигорина. Или просто сделай патч только для Альта.  
Ответить | Правка | К родителю #29 | Наверх | Cообщить модератору

142. "Релиз ядра Linux 5.18"  +1 +/
Сообщение от Аноним (-), 24-Май-22, 04:43 
Издеваешься? Альты за всю свою историю отправили считанные единицы патчей в кернел, в разы меньше даже убунт.

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

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

174. "Релиз ядра Linux 5.18"  +3 +/
Сообщение от Семен (??), 24-Май-22, 10:56 
Пинговать никого не надо, надо слать сразу патч по почте всем активным мейнтейнерам затрагиваемой подсистемы. Для этого есть скрипт ./scripts/get_maintainer.pl которому передается имя файла патча.

Если в кратце, то клонируем девелоп репозиторий ядра. Вносим изменения.

Далее вводим:

git add --all
git commit -S

Первой строкой вводим заголовок патча в формате
[подсистема] Заголовок в повелительном наклонении.

Например:
[drm/amdgpu] Fix null pointer dereference
[selinux] Remove unnecessary code

После заголовка два перевода строки(одна пустая строка), и пишем описание.
Далее два перевода строки(одна пустая строка) и пишем:
Signed-off: Ваше имя <емаил@почта.ру>

Сохраняем и фиксируем коммит.

Дальше вводим:
git format-patch -1

Если надо взять несколько последних коммитов и оформить, как серию патчей
то вводим git format-patch -количество коммитов.

Далее:
./scripts/checkpatch.pl имя-файла-патча.patch

Проверяем патч на ошибки. Если нет ошибок переходим к следующему пункту.

Нужно отправлять патч в plain формате, но лучше отправлять через git, чтобы исключить возможные ошибки.
Вводим или вносим функцию в ваш .bashrc


function kpsend() {
  local patch=${1:?}
  shift
  git send-email \
    --cc-cmd="./scripts/get_maintainer.pl --norolestats ${patch}" \
    $@ "${patch}"
}

дальше вводим:
kpsend имя-файла-патча.patch

Дальше жмем ентер-ентер много раз, если надо добавляем емайлы получателей. Если все успешно скрипт выдаст OK. Значит ваш патч успешно был отправлен всем мейнтейнерам чьи подсистемы затрагивает патч. Через пару часов проверяете https://lore.kernel.org/ и смотрите дошел ли ваш патч. Иногда на лоре бывают задержки до суток.


Большие изменения нельзя отправлять одним патчем. Делим на несколько более мелких патчей.
Если патч зависит от какого коммита, то надо это указать.

Тоже самое, но другим языком:
http://nickdesaulniers.github.io/blog/2017/05/16/submitting-.../

Полезно прочесть и закрепить знания:
https://www.kernel.org/doc/html/latest/process/2.Process.htm...
https://www.kernel.org/doc/html/latest/process/submitting-pa...


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

175. "Релиз ядра Linux 5.18"  +/
Сообщение от Семен (??), 24-Май-22, 11:01 
опечатка должно быть
Signed-off-by вместо Signed-off
и опечатка
Вместо
[drm/amdgpu] Fix null pointer dereference
[selinux] Remove unnecessary code

drm/amdgpu: Fix null pointer dereference
selinux: Remove unnecessary code

Пример оформления патча:
https://lore.kernel.org/lkml/YoyIn5L8cIwxHxR0@hovoldcon...

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

212. "Релиз ядра Linux 5.18"  +/
Сообщение от Аноним (-), 24-Май-22, 20:22 
> Пинговать никого не надо, надо слать сразу патч по почте всем активным
> мейнтейнерам затрагиваемой подсистемы.

Ну как, отправляя патч нехило бы знать кому он потенциально интересен и особый бонус - если с ними это заранее обсудили. Так с лопаты unsolicited вгрузить - если это тривиальщина, типа исправления опечатки, может прокатить сразу. А если нет - неплохо бы понимать насколько этому рады, в именно том виде, все причастные. В неочевидных случаях шлют сначала как RFC (request for comment) и собирают коменты о том что там не так.

> Для этого есть скрипт ./scripts/get_maintainer.pl
> которому передается имя файла патча.

Лучше все же понимать кому патч может быть интересен. Кроме номинального майнтайнера еще бывает целая клика вокруг и если они вынут NAKи из кармана, майнтайнер против них не попрет. Даже одного NAK'а может хватить чтобы зарубить инициативу.

> Если в кратце, то клонируем девелоп репозиторий ядра. Вносим изменения.

Кто-то делает патчи к ядру без этой части? =)

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

223. "Релиз ядра Linux 5.18"  +1 +/
Сообщение от Семен (??), 24-Май-22, 21:20 
Все верно, поэтому дал выше пояснительные ссылки для дополнительного изучения. Чтобы не было вопросов почему лыжи не едут по асфальту...

> Кто-то делает патчи к ядру без этой части? =)

Возможно и такое. Бывает, что патч отправляют на старую версию ядра, но вернуть патч назад по течению более проблемно. Патч на момент отправки должен быть на апстрим гит, чтобы у людей не было проблем с применением и не сидеть резолвить конфликты. Или еще хуже кто-то пофиксил этот баг в апстриме. Плюс к примеру надо брать ремоте репозиторий drm-next для отправки патчей в подсистему drm, или репозиторий xfs для отправки патча в xfs. Потому что все патчи сначала аккумулируются в  репозитории более низкого уровня, а только потом обычной со следующей версией ядра(или со следующим циклом исправлений, зависит от мейнтейнера) перетекают в основной репозиторий ядра. И да размытые формулировки многие не любят, любят все тесты и реальные цифры.

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

102. "Релиз ядра Linux 5.18"  +3 +/
Сообщение от Семен (??), 23-Май-22, 19:44 
Вы не правильно видимо отправляете. У меня принимают патчи без проблем. Обычно если с патчем что-то не то, то вам должны ответить в чем причина. Выбывают иногда какие-то религиозные убеждения у мейнтейнеров, и они ошибку за ошибку не считают, притом иногда доходит до маразма, когда мейнтейнер придумывает проблемы которых нет и оторван от реальности. Увы но это человеческий фактор, сколько людей столько мнений. Вот таакой Если нужна помощь с отправкой напишите мне, помогу как правильно оформить и отправить патч.
Ответить | Правка | К родителю #29 | Наверх | Cообщить модератору

158. "Релиз ядра Linux 5.18"  +/
Сообщение от edo (ok), 24-Май-22, 06:24 
ситуация не вроде этой?
https://habr.com/ru/post/657607/
Ответить | Правка | К родителю #29 | Наверх | Cообщить модератору

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

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




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

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