linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Sterba <dsterba@suse.cz>
To: Brendan Hide <brendan@swiftspirit.co.za>
Cc: Gene Czarcinski <gene@czarc.net>,
	linux-btrfs@vger.kernel.org, Josef Bacik <jbacik@fusionio.com>
Subject: Re: [PATCH 6/6] Btrfs-progs: detect if the disk we are formatting is a ssd
Date: Mon, 21 Jan 2013 14:01:22 +0100	[thread overview]
Message-ID: <20130121130122.GH19967@twin.jikos.cz> (raw)
In-Reply-To: <50FB0CA1.4070204@swiftspirit.co.za>

On Sat, Jan 19, 2013 at 11:14:09PM +0200, Brendan Hide wrote:
> On 2013/01/19 08:06 PM, Gene Czarcinski wrote:
> >Signed-off-by: Josef Bacik <jbacik@fusionio.com>
> >Signed-off-by: Gene Czarcinski <gene@czarc.net>
> >---
> >-values are raid0, raid1, raid10 or single.
> >+values are raid0, raid1, raid10, single or dup.  Single device will have dup
> >+set by default except in the case of SSDs which will default to single.  This is
> >+because SSDs can remap blocks internally so duplicate blocks could end up in the
> >+same erase block which negates the benefits of doing metadata duplication.
> Can't help but suggest that a "NO_DEDUP" command could be added to the SATA
> Transport Protocol/SCSI Command set. Not sure where to submit that idea ...
> :-/

You might get your answer from lkml, fsdevel or linux-scsi.

david

  reply	other threads:[~2013-01-21 13:01 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-19 18:06 [PATCH 0/6] patches from Fedora 18 (resubmitted) Gene Czarcinski
2013-01-19 18:06 ` [PATCH 1/6] Fedora 18 - btrfs-init-dev-list.patch Gene Czarcinski
2013-01-22 21:56   ` Gene Czarcinski
2013-01-19 18:06 ` [PATCH 2/6] Fedora 18 - build-fixes.patch Gene Czarcinski
2013-01-22 21:25   ` Gene Czarcinski
2013-01-23 18:12     ` David Sterba
2013-01-19 18:06 ` [PATCH 3/6] Fedora 18 - fix-labels.patch Gene Czarcinski
2013-01-22 21:45   ` Gene Czarcinski
2013-01-19 18:06 ` [PATCH 4/6] Fedora 18 - modified valgrind.patch Gene Czarcinski
2013-01-22 21:53   ` Gene Czarcinski
2013-01-23 19:40   ` Eric Sandeen
2013-01-19 18:06 ` [PATCH 5/6] Btrfs-progs: add btrfs device ready command Gene Czarcinski
2013-01-19 18:06 ` [PATCH 6/6] Btrfs-progs: detect if the disk we are formatting is a ssd Gene Czarcinski
2013-01-19 21:14   ` Brendan Hide
2013-01-21 13:01     ` David Sterba [this message]
2013-01-23  2:35     ` Chris Samuel

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=20130121130122.GH19967@twin.jikos.cz \
    --to=dsterba@suse.cz \
    --cc=brendan@swiftspirit.co.za \
    --cc=gene@czarc.net \
    --cc=jbacik@fusionio.com \
    --cc=linux-btrfs@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).