[lvc-project] [PATCH 6.1 v2 2/3] scsi: aic79xx: check for non-NULL scb in ahd_handle_pkt_busfree

[Boris Belyavtsev]

On Mon Apr 21, 2025 at 5:01 PM +07, Fedor Pchelkin wrote:
> В названии всех патчей перед наименованием функции присутствует таб или
> это какой-то другой артефакт..

Простите, не понимаю где. Патчи гитом формировались вроде как должно
быть нормально?

> Будет в логе запись:
> scb pointer is NULL
>
> Сомневаюсь, что это как-то позволит пользователю (и разработчику ядра,
> которому он передаст логи) понять, что произошло.
>
> Лучше с WARN_ON_ONCE. Это неожидаемое поведение, лучше о таком сообщать
> громко, т.к. драйвер древний и к корректной обработке подобного вида
> ситуаций не приспособлен.
>
> > +                     return (1);
> > +             }
>
> Вот здесь добавляется return посередине обработки какого-либо события -
> быть уверенными, что драйвер или firmware в этом случае не войдут в
> какое-нибудь неопределённое состояние, мы не можем. WARNING с return в
> целом неплохо подходит.
>
> >               ahd_abort_scbs(ahd, SCB_GET_TARGET(ahd, scb), 'A',
> >                              SCB_GET_LUN(scb), SCB_GET_TAG(scb),
> >                              ROLE_INITIATOR, CAM_UNEXP_BUSFREE);
> > --
> > 2.43.0

Можно поменять на
	if (WARN_ON_ONCE(scb == NULL))
		return (1);