From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: Bernd Schubert <bernd.schubert@fastmail.fm>
Cc: dgilbert@interlog.com,
"Martin K. Petersen" <martin.petersen@oracle.com>,
Mike Snitzer <snitzer@redhat.com>, Hannes Reinecke <hare@suse.de>,
emilne@redhat.com,
device-mapper development <dm-devel@redhat.com>,
linux-scsi@vger.kernel.org, Sumit.Saxena@lsi.com
Subject: Re: SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails]
Date: Thu, 26 Sep 2013 10:42:00 -0400 [thread overview]
Message-ID: <yq1r4cbablz.fsf@sermon.lab.mkp.net> (raw)
In-Reply-To: <52443992.4040806@fastmail.fm> (Bernd Schubert's message of "Thu, 26 Sep 2013 15:41:38 +0200")
>>>>> "Bernd" == Bernd Schubert <bernd.schubert@fastmail.fm> writes:
Bernd,
Bernd> Both types of systems we have in-house neither block limits vpd
Bernd> nor READ_CAP16 return anything that would indicate discard is
Bernd> supported. But UNMAP and WRITE SAME unmap(*) just work fine.
I have a collection of different SSDs in a tray connected to an LSI
SAS2008 ASIC. The 510 is the only drive that does not have LBPME=1.
Chances are it's because DRAT and RZAT are not set but it could also be
that the 510 is blacklisted.
Bernd> I certainly don't want to cause any more write-same trouble, but
Bernd> as all layers initially have to assume write same is supported
Bernd> anyway and need to dynamically disable it if it fails, can't we
Bernd> also enable discard by default with WRITE SAME16 unmap?
No thanks :)
But you can force discards on by doing a:
# echo -n unmap > /sys/class/scsi_disk/x:y:z:n/provisioning_mode
or
# echo -n writesame_16 > /sys/class/scsi_disk/x:y:z:n/provisioning_mode
Create a udev rule if you like.
In any case I wouldn't recommend using TRIM on that drive...
Bernd> PS: LSI SATL with FWv17 seems to have an unmap bug - I cannot
Bernd> unmap the last sector:
Yes, it appears there's an off-by-one bug in the UNMAP translation.
Sumit, is this something you guys can look into?
--
Martin K. Petersen Oracle Linux Engineering
next prev parent reply other threads:[~2013-09-26 14:42 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-19 16:13 [PATCH] dm mpath: disable WRITE SAME if it fails Mike Snitzer
2013-09-20 21:21 ` SCSI's heuristics for enabling WRITE SAME still need work [was: dm mpath: disable WRITE SAME if it fails] Mike Snitzer
2013-09-20 22:03 ` Martin K. Petersen
2013-09-21 15:28 ` Douglas Gilbert
2013-09-23 18:18 ` Ewan Milne
2013-09-24 5:39 ` [dm-devel] " Hannes Reinecke
2013-09-24 12:34 ` Mike Snitzer
2013-09-24 13:49 ` Martin K. Petersen
2013-09-24 15:15 ` Mike Snitzer
2013-09-25 20:52 ` Bernd Schubert
2013-09-25 22:12 ` Douglas Gilbert
2013-09-26 0:44 ` Martin K. Petersen
2013-09-26 5:39 ` Douglas Gilbert
2013-09-26 13:41 ` Bernd Schubert
2013-09-26 14:42 ` Martin K. Petersen [this message]
2013-09-26 15:34 ` Bernd Schubert
2013-09-26 15:47 ` Douglas Gilbert
2013-09-26 18:42 ` Saxena, Sumit
2013-09-24 19:12 ` [dm-devel] " Jeremy Linton
2013-09-24 19:37 ` Douglas Gilbert
2013-09-24 9:37 ` Paolo Bonzini
2013-09-24 13:25 ` James Bottomley
2013-09-24 18:39 ` [dm-devel] " Mikulas Patocka
2013-09-24 20:44 ` Martin K. Petersen
2013-09-24 22:02 ` Mike Snitzer
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=yq1r4cbablz.fsf@sermon.lab.mkp.net \
--to=martin.petersen@oracle.com \
--cc=Sumit.Saxena@lsi.com \
--cc=bernd.schubert@fastmail.fm \
--cc=dgilbert@interlog.com \
--cc=dm-devel@redhat.com \
--cc=emilne@redhat.com \
--cc=hare@suse.de \
--cc=linux-scsi@vger.kernel.org \
--cc=snitzer@redhat.com \
/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.