All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: Jens Axboe <axboe@kernel.dk>
Cc: linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org,
	adilger@dilger.ca
Subject: Re: [PATCHSET v2] Add support for write life time hints
Date: Wed, 14 Jun 2017 11:45:56 -0400	[thread overview]
Message-ID: <yq160fyvah7.fsf@oracle.com> (raw)
In-Reply-To: <1497412919-19400-1-git-send-email-axboe@kernel.dk> (Jens Axboe's message of "Tue, 13 Jun 2017 22:01:49 -0600")


Jens,

> A new iteration of this patchset, previously known as write streams.
> As before, this patchset aims at enabling applications split up
> writes into separate streams, based on the perceived life time
> of the data written. This is useful for a variety of reasons:
>
> - With NVMe 1.3 compliant devices, the device can expose multiple
>   streams. Separating data written into streams based on life time
>   can drastically reduce the write amplification. This helps device
>   endurance, and increases performance. Testing just performed
>   internally at Facebook with these patches showed up to a 25%
>   reduction in NAND writes in a RocksDB setup.
>
> - Software caching solutions can make more intelligent decisions
>   on how and where to place data.
>
> Contrary to previous patches, we're not exposing numeric stream values
> anymore.  I've previously advocated for just doing a set of hints that
> makes sense instead. See the coverage from the LSFMM summit this year:

I am all for having these write hints. But one request I would like to
make is that we just make them flags and abolish all notions of the term
"streams" from block for this particular use case (since it is more
hinty than streamy).

There are devices coming where a proper stream ID is prerequisite to
separate data streams for other reasons than bucketing based on data
lifetime. So my preference would be that we make the lifetime hints be
flags in block. That does not preclude using Streams Directives to
implement them in the NVMe NAND flash case. But it does not cause
conflicts with the use cases that need "proper" stream IDs for QoS or
colocation avoidance purposes in SCSI.

-- 
Martin K. Petersen	Oracle Linux Engineering

  parent reply	other threads:[~2017-06-14 15:45 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-14  4:01 [PATCHSET v2] Add support for write life time hints Jens Axboe
2017-06-14  4:01 ` [PATCH 01/10] block: add support for carrying stream information in a bio Jens Axboe
2017-06-14  4:01 ` [PATCH 02/10] blk-mq: expose stream write stats through debugfs Jens Axboe
2017-06-14  4:07   ` Andreas Dilger
2017-06-14  4:01 ` [PATCH 03/10] fs: add support for an inode to carry stream related data Jens Axboe
2017-06-14  4:01 ` [PATCH 04/10] fs: add support for allowing applications to pass in write life time hints Jens Axboe
2017-06-14  4:06   ` Andreas Dilger
2017-06-14  4:01 ` [PATCH 05/10] fs: add O_DIRECT support for sending down bio stream information Jens Axboe
2017-06-14  4:01 ` [PATCH 06/10] fs: add support for buffered writeback to pass down " Jens Axboe
2017-06-14  4:01 ` [PATCH 07/10] ext4: add support for passing in stream information for buffered writes Jens Axboe
2017-06-14  4:01 ` [PATCH 08/10] xfs: " Jens Axboe
2017-06-14  4:01 ` [PATCH 09/10] btrfs: " Jens Axboe
2017-06-14  4:01 ` [PATCH 10/10] nvme: add support for streams and directives Jens Axboe
2017-06-14  4:10   ` Andreas Dilger
2017-06-14 15:45 ` Martin K. Petersen [this message]
2017-06-14 15:53   ` [PATCHSET v2] Add support for write life time hints Jens Axboe
2017-06-14 16:00     ` Martin K. Petersen
2017-06-14 16:03       ` Jens Axboe
2017-06-14 16:01     ` Christoph Hellwig
2017-06-14 16:02       ` Jens Axboe
2017-06-14 16:04       ` Martin K. Petersen
2017-06-14 16:26         ` Jens Axboe
2017-06-14 17:22         ` Jens Axboe
2017-06-14 23:39         ` Andreas Dilger
2017-06-15  3:26           ` Jens Axboe
2017-06-15  3:53             ` Andreas Dilger
2017-06-15  3:57               ` Jens Axboe
2017-06-15  4:38                 ` Jens Axboe
2017-06-16 13:58             ` Christoph Hellwig
2017-06-16 14:40               ` Jens Axboe
2017-06-16 15:52                 ` Christoph Hellwig
2017-06-16 15:55                   ` Jens Axboe

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=yq160fyvah7.fsf@oracle.com \
    --to=martin.petersen@oracle.com \
    --cc=adilger@dilger.ca \
    --cc=axboe@kernel.dk \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.