Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED Allocation code.

From	Scott Bauer <sbauer@eng.utah.edu>
Newsgroups	linux.kernel
Subject	Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED Allocation code.
Date	2016-12-20 17:40 +0100
Message-ID	<sQvCh-7WT-11@gated-at.bofh.it> (permalink)
References	(2 earlier) <sQe8q-4Wj-65@gated-at.bofh.it> <sQeL8-5pk-19@gated-at.bofh.it> <sQlWh-1OO-5@gated-at.bofh.it> <sQuGe-7mF-31@gated-at.bofh.it> <sQuPT-7q6-9@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 12/20/2016 08:46 AM, Christoph Hellwig wrote:
> On Tue, Dec 20, 2016 at 10:49:16AM -0500, Keith Busch wrote:
>> On Mon, Dec 19, 2016 at 10:17:44PM -0800, Christoph Hellwig wrote:
>>> As far as I can tell Security Send / Receive has always been intended to
>>> apply to the whole controller, even if that's something I would not
>>> personally think is a good idea.
>>
>> NVMe security commands required the namespace ID since the very
>> beginning. It's currently documented in figure 42 of section 5,
>> "Namespace Identifier Used" column.
> 
> Oh, for some reason I read a no there when looking it up.
> Good to know, although TCG spec still seem to ignore it.

Thanks Keith. Although TCG Spec currently ignores it in the future it may not.
In that case we should probably attempt to future proof it a bit. Since the
namespace ID is accessible via the block device structure I'll find a way to
include that in some opaque pointer that we can deliver through the core into
NVMe.

But this also brings up another question (and part of the reason I moved from
the block ioctl to fs ioctl): For drives with multiple namsepaces is it 
acceptable to allow a namespace, who has a segregated chunk of space, the ability
to perform actions outside of its range? Since the multiple namespaces portion
of the spec says there will be one Global LR a namespace that doesn't encompass
the entire LBA range can end up locking other LBAs via locking the global.

That's why I wanted to go to char dev because it seemed like a better fit for 
this scenario. Any thoughts on the above?

> 
> _______________________________________________
> Linux-nvme mailing list
> Linux-nvme@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-nvme
>

Thread

[PATCH v3 4/5] nvme: Implement resume_from_suspend and SED Allocation code. Scott Bauer <scott.bauer@intel.com> - 2016-12-19 20:50 +0100
  Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Keith Busch <keith.busch@intel.com> - 2016-12-19 23:00 +0100
    Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Scott Bauer <scott.bauer@intel.com> - 2016-12-19 23:40 +0100
      Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-20 07:20 +0100
        Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Keith Busch <keith.busch@intel.com> - 2016-12-20 16:40 +0100
          Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-20 16:50 +0100
            Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Scott Bauer <sbauer@eng.utah.edu> - 2016-12-20 17:40 +0100
              Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-21 10:10 +0100
            Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Scott Bauer <scott.bauer@intel.com> - 2016-12-20 19:10 +0100
              Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-21 10:40 +0100
  Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. kbuild test robot <lkp@intel.com> - 2016-12-20 05:20 +0100
  Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-20 07:30 +0100
  Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-20 07:50 +0100
  Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Jethro Beekman <kernel@jbeekman.nl> - 2016-12-25 15:50 +0100
    Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Scott Bauer <scott.bauer@intel.com> - 2016-12-27 23:30 +0100
      Re: [PATCH v3 4/5] nvme: Implement resume_from_suspend and SED  Allocation code. Christoph Hellwig <hch@infradead.org> - 2016-12-28 09:50 +0100

csiph-web