[lvc-project] [PATCH] drm/vmwgfx: Handle surface check failure correctly

Fedor Pchelkin pchelkin at ispras.ru
Fri Oct 11 15:21:56 MSK 2024


On Fri, 11. Oct 14:06, Nikolay Kuratov wrote:
> Добрый день!
> > Также не выдержана длина строк описания. Все они сильно разной длины.
> Это как раз из-за того что я пытаюсь уместиться в 80 строк :)

Эмм.. нет? Первая строка обрезана на ~60 символах, тогда как в неё ещё
много слов может поместиться, которые перенесены на следующую строку.
Посмотрите на итоговое отправленное письмо, может в текстовом редакторе у
Вас не так отображается (зигзагообразный первый абзац).
https://lore.kernel.org/dri-devel/20241002122429.1981822-1-kniv@yandex-team.ru/

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

> Про 75 впервые слышу, принял к сведению.

80 тоже нормально. 75 - это скорее ориентир.

https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-format
  The canonical patch message body contains the following:
        ...
        The body of the explanation, line wrapped at 75 columns, which
	will be copied to the permanent changelog to describe this patch.

> Проверился с checkpatch, (помнится, проверяет длину строк) - никаких замечаний к патчу не было.

На слишком короткие строки он замечаний делать не будет.

>
> Будем надеяться, что с Заком дело пойдет.

Патч уже в tip-репозитории
https://cgit.freedesktop.org/drm/drm-tip/commit/?id=26498b8d54373d31a621d7dec95c4bd842563b3b

> 
> По поводу последнего нашего долга по бекпорту
> UNCHECKED_FUNC_RES.STAT: drivers/md/md-cluster.c:520
> отписал в Svacer почему его не закрыл вместе с остальными. Корректность там действительно мы вносим, у меня есть некоторые сомнения, что мы (и патч чинящий нас) в сумме слишком интрузивны.

Является ли исправляемая Вашим патчем проблема Major с точки зрения
безопасности? Тегов Fixes, Cc:stable, каких-либо упоминаний возможных
последствий отсутствия обработки таймаута в патче не нахожу. Может он и не
требуется для stable?



More information about the lvc-project mailing list