Re: [PATCH v2 6/6] block: skip queue if NULL on blk_cleanup_queue()

Luis Chamberlain
2021-07-20T00:24:41Z

On Mon, Jul 19, 2021 at 10:50:13AM +0100, Christoph Hellwig wrote:
> On Thu, Jul 15, 2021 at 12:07:26PM -0700, Luis Chamberlain wrote:
> > > For all drivers using blk_alloc_disk/blk_mq_alloc_disk there should
> > > always be a queue. The others ones aren't ready to handle errors
> > > from add_disk yet in any way I think (and I plan to fix this up
> > > ASAP).
> >
> > Have an example in mind?
>
> The only ones left are nvme, dasd and scsi. NVMe is trivial (attached),
> dasd needs a little more work, I need to send up a WIP to the
> maintainers. scsi is the real problem and will require a fair amount
> of work.

Alright, I'll wait until these changes are merged before sending those
driver's conversions. Without this patch though, there's no way to test
the error injection in case the driver was buggy and removed the queue
by mistake. Recall my first patch moved that check to the beginning too.

So we either skip that error injection test ... or not sure what else to do.

Luis