Re: [syzbot] [scsi?] WARNING in sg_remove_sfp_usercontext

From: Alexander Wetzel
Date: Thu Apr 04 2024 - 08:05:30 EST


On 04.04.24 12:57, Pengfei Xu wrote:
cc Alexander Wetzel,

I used syzkaller and found the similar problem.

And bisected and found the related commit:
27f58c04a8f4 scsi: sg: Avoid sg device teardown race
Reverted the commit on top of v6.9-rc2, this issue was gone.

There were now multiple reports about the issue. The first (main) discussion and fix is here:
https://lore.kernel.org/all/20240401191038.18359-1-Alexander@xxxxxxxxxxxxxx/

In a nutshell:
The new WARN_ON_ONCE() was a (very) stupid idea and must be undone.
It's ok to just drop the line with the WARN_ON_ONCE(). The rest of the proposed patch fixes a much less urgent issue.

Of course I can also break the fix into two patches, when this speeds up the merge...

Alexander