Re: [GIT PULL] first round of SCSI updates for the 5.14+ merge window

From: James Bottomley
Date: Thu Sep 02 2021 - 19:23:50 EST


On Thu, 2021-09-02 at 15:38 -0700, Linus Torvalds wrote:
> On Thu, Sep 2, 2021 at 9:50 AM James Bottomley
> <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote:
> > We also picked up a non trivial conflict with the already upstream
> > block tree in st.c
>
> Hmm. Resolving that conflict, I just reacted to how the st.c code
> passes in a NULL gendisk to scsi_ioctl() and then on to
> blk_execute_rq().
>
> Just checking that was fine, and I notice how *many* places do that.
>
> Should the blk_execute_rq() function even take that "struct gendisk
> *bd_disk" argument at all?
>
> Maybe the right thing to do would be for the people who care to just
> set rq->rq_disk before starting the request..
>
> But I guess it's traditional, and nobody cares.

It's certainly traditional, but Christoph has been caring a lot about
cleaning up our gendisks recently, so he might be interested in seeing
if he can fix it ...

James