From mboxrd@z Thu Jan  1 00:00:00 1970
From: hch@lst.de (Christoph Hellwig)
Date: Thu, 24 Dec 2015 16:45:12 +0100
Subject: [PATCH 3/3] nvme: make SG_IO support optional
In-Reply-To: <20151224152945.GA7262@localhost.localdomain>
References: <1450952155-8693-1-git-send-email-hch@lst.de>
 <1450952155-8693-4-git-send-email-hch@lst.de>
 <567BCB6C.6000502@dev.mellanox.co.il> <20151224111248.GB15268@lst.de>
 <567BD794.2020607@dev.mellanox.co.il> <20151224114040.GA17820@lst.de>
 <20151224152945.GA7262@localhost.localdomain>
Message-ID: <20151224154512.GA22789@lst.de>

On Thu, Dec 24, 2015@03:29:46PM +0000, Keith Busch wrote:
> On Thu, Dec 24, 2015@12:40:40PM +0100, Christoph Hellwig wrote:
> > On Thu, Dec 24, 2015@01:31:32PM +0200, Sagi Grimberg wrote:
> > > OK, I see what you mean. We can look at having multipathd ioctl'ing
> > > identify_ns at some point...
> > 
> > block/for-next has the nguid available as a sysfs attribute, that's
> > all we should need.
> 
> Off the top of your head, do you happen to know how to point multipathd
> to the new attribute? Is this a code change or just a configuration rule?

With the most recent mutlipath-tools there should be no need to patch
mutipath-tools.  Just make sure udev sets the ID_SERIAL attribute
correctly, which I think some folks were already looking into.  An
explicit getuid callout that just reads the file should also work
with any versions, although the most recent ones call it deprectated.