[PATCH] nvme-multipath: avoid crash on invalid subsystem cntlid enumeration

[hch@lst.de (Christoph Hellwig)]

On Tue, Apr 02, 2019@02:55:40AM -0600, Keith Busch wrote:
> On Mon, Apr 01, 2019@04:50:13AM -0700, Hannes Reinecke wrote:
> > On 4/1/19 7:17 AM, Keith Busch wrote:
> > > We might want to use this patch anyway. It is possible some process
> > > reference count is preventing the final release when we want to remove
> > > a controller, and a subsequent discover, even if the cntlid is unqiue,
> > > would create a duplicate disk name.
> > > 
> > I don't mind which patch is taken.
> > But crashing on misbehaving targets is never a good idea.
> 
> I 100% agree. :)
>  
> > Do I need to send the other patch?
> 
> Can we do both? I like the warning when an invalid subsystem is detected,
> but your original patch fixes the unique gendisk name, so I'd vote
> to keep that good code, but justify it with a slightly different
> changelog. Something along the following:

Yes, let's do both.  Hannes, please resend a series with this patch
with the updated changelog and the duplicate cntlid check one.