Re: [PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via configfs

From	Christoph Hellwig <hch@lst.de>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via configfs
Date	2017-05-31 11:50 +0200
Message-ID	<tN8ql-31c-7@gated-at.bofh.it> (permalink)
References	<tMKo1-4Kd-3@gated-at.bofh.it> <tMKo1-4Kd-1@gated-at.bofh.it> <tMLDt-5ux-47@gated-at.bofh.it> <tMLWO-5BU-33@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue, May 30, 2017 at 11:48:09AM +0200, Johannes Thumshirn wrote:
> On 05/30/2017 11:24 AM, Christoph Hellwig wrote:
> > This should be stored as a uuid_t (or rather uuid_be in the current
> > kernel, but I'm about to rename it), and use uuid_be_to_bin / uuid_to_bin
> > for parsing.
> 
> OK, thought you ask me to do that. Which one do you prefer?

I think you'll have to use the old names, unless we get the uuid
changes stabilized soon and can pull the branch for it into the
nvme / block tree.  So go with those for now, and I'll do any
after the fact cleanup later.

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via configfs Johannes Thumshirn <jthumshirn@suse.de> - 2017-05-30 10:10 +0200
  Re: [PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via  configfs Hannes Reinecke <hare@suse.de> - 2017-05-30 10:30 +0200
  Re: [PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via         configfs Christoph Hellwig <hch@lst.de> - 2017-05-30 11:30 +0200
    Re: [PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via  configfs Johannes Thumshirn <jthumshirn@suse.de> - 2017-05-30 11:50 +0200
      Re: [PATCH 2/7] nvmet: add uuid field to nvme_ns and populate via         configfs Christoph Hellwig <hch@lst.de> - 2017-05-31 11:50 +0200

csiph-web