[lvc-project] [PATCH] hwmon: Use 64-bit arithmetic instead of 32-bit

[Fedor Pchelkin]

Здравствуйте! Ниже упомяну несколько важных вещей. Прочитайте,
пожалуйста.

On Thu, 30. Jan 06:46, Guenter Roeck wrote:
> First, this does not affect the entire hardware monitoring subsystem,
> just a single driver. The subject is misleading.
> 
> Second, the result of the calculation is used as parameter of a function
> which takes unsigned int as parameter, and calculates jiffies from it.
> If the number of jiffies to be calculated ever exceeds 4,294,967,295
> microseconds, the code has severe other problems to be concerned about.
> 
> In other words, your patch does not fix anything, and neither does the
> referenced commit. It just helps hiding a (hopefully non-existing) different
> problem.
> 
> Yes, commit b52f45110502 "fixes" a similar "problem", and also completely
> misses the point, since the 64-bit result of the calculation is also used
> as parameter to usecs_to_jiffies(). The same applies to commit 8338b74a750c5
> and maybe others.
> 
> I am not going to apply this patch.

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

К сожалению, наше с Вами взаимодействие сейчас ведётся не в лучшем виде,
т.к. вести обсуждение сразу по единовременно высылаемой Вами куче патчей
с одинаковыми ошибками, неоднократно просить переделывать одни и те же
моменты в разных патчах, а в итоге получать по почте заново ту же кучу
патчей зачастую с похожими ошибками, не зная, были ли замечания по
улучшению пропущены, оспорены на основе своей точки зрения или просто
проигнорированы - не есть правильный и плодотворный подход к обсуждению
исправлений. Он в том числе ведёт к тому, что патчи во время
предварительного ревью просматриваются мной поверхностно (иногда просто
устаёшь твердить одно и то же), и смысл всего происходящего теряется.

Исходя из этого предлагаю следующий на мой взгляд разумный подход:

* единовременно высылать и вести обсуждение только по одному (максимум
  двум) патчам; когда патч на наш общий взгляд становится быть готовым к
  отправке в международное сообщество, он туда отправляется, дальше на
  предварительное обсуждение в lvc-patches высылается следущий патч;

* когда высылается новая версия патча на основе предложений по улучшению
  прошлой, в названии патча указывается номер версии (v1, v2, v3, ...).
  Например, "[PATCH v2] fix something". Также желателен краткий changelog
  того, что поменялось по сравнению с предыдущей версией. Этот пункт
  отражён в инструкции
https://portal.linuxtesting.ru/How-to-send-patches-to-kernel.html#Подготовка-второй-версии-патча

* более высоким приоритетом ставить исправления для предупреждений,
  оцененных в Svacer как Major/Critical. Minor - ну, это Minor, для них
  патчи вообще обычно готовятся, когда других более приоритетных и
  занятных задач по разметке или исправлению Major/Critical нет. К тому
  же, для них зачастую требуется более тщательно продумывать описание =>
  отсюда следующий пункт;

* общение в списках рассылки и подготовка патчей в ядро Linux ведутся на
  английском языке. Для нас это не родной язык, так что для каждого это
  индивидуальный вопрос, но важной составляющей патча является его
  описание, формулируемое на английском, которое позволяет донести до
  других людей, а зачем патч вообще нужен и какую проблему он решает.
  Важно грамотно составлять это самое описание. Если в нём написана
  какая-то непонятица с несоблюдением грамматических правил английского
  языка, то у читателей патча возникают разные вопросы. По идее,
  помогает чтение текста других патчей и обсуждений в списках рассылок
  (в основном там используется нормальный технический английский),
  накапливается багаж нужных фраз и выражений и т.п. Но если с этим
  есть сложности, то на крайний случай лучше обращаться к
  онлайн-переводчику и Интернету.

Сейчас можно попробовать довести этот патч до ума, исправив нужное
место - видимо, требуется поменять тип подозрительной переменной. И
думается, правильным будет на этом патче и отработать предложенный выше
подход. А дальше постепенно патч за патчем.