linux-kernel - Re: [PATCH v1 7/7] nvmet: allow overriding the NVMe VS via configfs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <0af0f525-918d-d2b3-2f08-f4f19c83fab1@suse.de>
Date:   Thu, 1 Jun 2017 08:59:52 +0200
From:   Hannes Reinecke <hare@...e.de>
To:     Christoph Hellwig <hch@....de>,
        Johannes Thumshirn <jthumshirn@...e.de>
Cc:     Sagi Grimberg <sagi@...mberg.me>,
        Keith Busch <keith.busch@...el.com>,
        Max Gurtovoy <maxg@...lanox.com>,
        Linux NVMe Mailinglist <linux-nvme@...ts.infradead.org>,
        Linux Kernel Mailinglist <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v1 7/7] nvmet: allow overriding the NVMe VS via configfs

On 06/01/2017 08:52 AM, Christoph Hellwig wrote:
> On Wed, May 31, 2017 at 03:32:17PM +0200, Johannes Thumshirn wrote:
>> Allow overriding the announced NVMe Version of a via configfs.
>>
>> This is particularly helpful when debugging new features for the host
>> or target side without bumping the hard coded version (as the target
>> might not be fully compliant to the announced version yet).
> 
> It is.  Just bump to 1.3, please as Identify 3h is the only mandatory
> change: http://nvmexpress.org/changes/
> 
Actually, I still do like this.
With it it'll be very easy to test version compliance; and validate our
initiator behaviour against older versions.

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		   Teamlead Storage & Networking
hare@...e.de			               +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)