Re: [GIT PULL] SCSI fixes for 4.15-rc3
From: Linus Torvalds
Date: Tue Dec 12 2017 - 12:16:22 EST
On Tue, Dec 12, 2017 at 8:21 AM, James Bottomley
<James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> The most important one is the bfa fix because it's easy to oops the
> kernel with this driver, a regression in the new timespec conversion in
> aacraid and a regression in the Fibre Channel ELS handling patch. The
> other three are a theoretical problem with termination in the
> vendor/host matching code and a use after free in lpfc.
No, this is just complete garbage.
> Johannes Thumshirn (1):
> scsi: bfa: fix access to bfad_im_port_s
This is utter shite, and doesn't even compile cleanly.
Sure, it's "just" a warning, and the code works. But no, I'm not
pulling crap like this. If you save a pointer in an integer
"hostdata[0]" field, then you damn well do the proper casts or helper
functions or something, you don't just ignore the compiler when it
very reasonably warns about it.
What the hell is going on? Nobody compiled this stuff at all? Or
nobody cares about new build warnings?
That is not acceptable.
Linus