From: Douglas Gilbert <dgilbert@interlog.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: "Nicholas A. Bellinger" <nab@daterainc.com>,
target-devel <target-devel@vger.kernel.org>,
lkml <linux-kernel@vger.kernel.org>,
linux-scsi <linux-scsi@vger.kernel.org>,
Christoph Hellwig <hch@lst.de>, Hannes Reinecke <hare@suse.de>,
Chris Mason <chris.mason@fusionio.com>,
Roland Dreier <roland@purestorage.com>,
Zach Brown <zab@redhat.com>, Kent Overstreet <kmo@daterainc.com>,
Theodore Tso <tytso@mit.edu>,
James Bottomley <JBottomley@Parallels.com>,
Nicholas Bellinger <nab@linux-iscsi.org>
Subject: Re: [PATCH 0/9] target: Add support for EXTENDED_COPY (VAAI) offload emulation
Date: Fri, 23 Aug 2013 15:13:10 +0200 [thread overview]
Message-ID: <52175FE6.4040309@interlog.com> (raw)
In-Reply-To: <yq17gfc60yd.fsf@sermon.lab.mkp.net>
On 13-08-23 02:33 PM, Martin K. Petersen wrote:
>>>>>> "Doug" == Douglas Gilbert <dgilbert@interlog.com> writes:
>
> Doug> The SCSI opcodes associated with it (0x83 and 0x84) have been
> Doug> renamed THIRD PARTY COPY OUT and IN, and
>
> Where did you see that? My SPC still has EXTENDED COPY.
SCSI _opcodes_ == SCSI operation codes. In other words the
name associated with all service actions (commands) that have
as their first byte 0x83 and 0x84 .
spc4r36h.pdf Annex F.3.1 and changed in spc4r34. Yes, well
hidden but IMO useful. So now we can use the term "Third party
copy" to cover:
- EXTENDED COPY(LID1) and associated commands
also found in SPC-2 and SPC-3 (with the "LID1" suffix)
- EXTENDED COPY(LID4) and associated commands
- the XCOPY LITE commands: POPULATE TOKEN and WRITE USING TOKEN
> Doug> Confused? I certainly was.
>
> Yeah, this is UNMAP all over again, just 100 times worse :(
Well what EXTENDED COPY (offload copy) is trying to do
ain't simple but obviously there could be a substantial
performance pay-off.
There is a "Third-party copy implementation and usage" Annex
(D) in spc4r36h.pdf . It could do with some more explanatory
text.
> Anyway. Excited to see nab posting the patches! My copy offload code
> from the spring has been getting stale both in the T10 and the kernel
> sense. But at least we know what I'll be working on next week :)
BTW I have ported the sg_xcopy "LID1" xcopy logic into
my ddpt utility. That gives two advantages:
- can cope with ibs!=obs
- runs on OSes other than Linux
Doug Gilbert
prev parent reply other threads:[~2013-08-23 13:13 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-23 2:26 [PATCH 0/9] target: Add support for EXTENDED_COPY (VAAI) offload emulation Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 1/9] target: Make target_core_subsystem defined as non static Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 2/9] target: Make spc_parse_naa_6h_vendor_specific " Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 3/9] target: Make helpers non static for EXTENDED_COPY command setup Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 4/9] target: Add global device list for EXTENDED_COPY Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 5/9] target: Avoid non-existent tg_pt_gp_mem in target_alua_state_check Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 6/9] target: Add support for EXTENDED_COPY copy offload emulation Nicholas A. Bellinger
2013-08-23 2:26 ` [PATCH 7/9] target: Enable EXTENDED_COPY setup in spc_parse_cdb Nicholas A. Bellinger
2013-08-23 2:27 ` [PATCH 8/9] target: Add Third Party Copy (3PC) bit in INQUIRY response Nicholas A. Bellinger
2013-08-23 2:27 ` [PATCH 9/9] target: Enable global EXTENDED_COPY setup/release Nicholas A. Bellinger
2013-08-23 11:04 ` [PATCH 0/9] target: Add support for EXTENDED_COPY (VAAI) offload emulation Douglas Gilbert
2013-08-23 12:33 ` Martin K. Petersen
2013-08-23 13:13 ` Douglas Gilbert [this message]
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=52175FE6.4040309@interlog.com \
--to=dgilbert@interlog.com \
--cc=JBottomley@Parallels.com \
--cc=chris.mason@fusionio.com \
--cc=hare@suse.de \
--cc=hch@lst.de \
--cc=kmo@daterainc.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=nab@daterainc.com \
--cc=nab@linux-iscsi.org \
--cc=roland@purestorage.com \
--cc=target-devel@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=zab@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.