All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: "Pasi Kärkkäinen" <pasik@iki.fi>
Cc: "Martin K. Petersen" <martin.petersen@oracle.com>,
	Ankit Jain <jankit@suse.com>,
	linux-scsi@vger.kernel.org,
	James.Bottomley@hansenpartnership.com, Eric.Moore@lsi.com
Subject: Re: [SCSI] mpt2sas T10 DIF fixes
Date: Sun, 19 Aug 2012 12:38:43 -0400	[thread overview]
Message-ID: <yq1628eet0s.fsf@sermon.lab.mkp.net> (raw)
In-Reply-To: <20120819153004.GU19851@reaktio.net> ("Pasi Kärkkäinen"'s message of "Sun, 19 Aug 2012 18:30:04 +0300")

>>>>> "Pasi" == Pasi Kärkkäinen <pasik@iki.fi> writes:

Pasi> So physically the disk is using 520 bytes/sector (and needs to
Pasi> support that), but logically the HBA presents 512 bytes/sector +
Pasi> PI.. is that correct?

Yep. The beauty of PI is that we can store the extra stuff without
having to deal with weird block sizes.


Pasi> I guess what I'm asking is: Can I use sg_format to enable
Pasi> checksums/protection on any disk that physically supports 520
Pasi> bytes/sector, or does the disk need to have special T10 data
Pasi> protection support aswell?

The disk needs to support T10 PI (PROTECT=1 in the Standard Inquiry VPD
page). We don't support driving the HBA in PI mode with a non-PI target.

-- 
Martin K. Petersen	Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2012-08-19 16:40 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-21  1:36 [SCSI] mpt2sas T10 DIF fixes Martin K. Petersen
2012-02-21  1:36 ` [PATCH 1/3] mpt2sas: Return the correct sense key for DIF errors Martin K. Petersen
2012-02-21  1:36 ` [PATCH 2/3] mpt2sas: Add a module parameter that permits overriding protection capabilities Martin K. Petersen
2012-02-21  1:36 ` [PATCH 3/3] mpt2sas: Do not check DIF for unwritten blocks Martin K. Petersen
2012-08-09  8:14 ` [SCSI] mpt2sas T10 DIF fixes Ankit Jain
2012-08-17 16:00   ` Martin K. Petersen
2012-08-19 14:49     ` Pasi Kärkkäinen
2012-08-19 15:18       ` Martin K. Petersen
2012-08-19 15:30         ` Pasi Kärkkäinen
2012-08-19 16:38           ` Martin K. Petersen [this message]
2012-08-19 20:19             ` Pasi Kärkkäinen
2012-08-20 10:48             ` Pasi Kärkkäinen
2012-08-20 11:06               ` Pasi Kärkkäinen
2012-08-21  0:43                 ` Martin K. Petersen
2012-08-21  6:13                   ` Pasi Kärkkäinen
2012-08-21  0:40               ` Martin K. Petersen
2012-08-21  6:12                 ` Pasi Kärkkäinen
2012-08-21 15:50                 ` Jiang, Dave
2012-08-22 18:36                   ` Martin K. Petersen
2012-10-01  9:17     ` Pasi Kärkkäinen
2012-10-02 22:24       ` Martin K. Petersen

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=yq1628eet0s.fsf@sermon.lab.mkp.net \
    --to=martin.petersen@oracle.com \
    --cc=Eric.Moore@lsi.com \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=jankit@suse.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=pasik@iki.fi \
    /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.