Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
Anish == Anish Bhatt an...@chelsio.com writes: Anish, Just a heads up that we've got some changes due to land shortly that'll provide interfaces for you to get the correct transfer lengths, size of the protection interval, etc. Your patch should be rebased on top of those changes. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
Thanks for the heads up, will wait for those. Please cc me/Manoj on the patches -Anish From: Martin K. Petersen [martin.peter...@oracle.com] Sent: Thursday, August 21, 2014 2:47 PM To: Anish Bhatt Cc: linux-scsi@vger.kernel.org; h...@infradead.org; jbottom...@parallels.com; micha...@cs.wisc.edu; Karen Xie; Manoj Malviya Subject: Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support Anish == Anish Bhatt an...@chelsio.com writes: Anish, Just a heads up that we've got some changes due to land shortly that'll provide interfaces for you to get the correct transfer lengths, size of the protection interval, etc. Your patch should be rebased on top of those changes. -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
On 8/7/2014 1:37 AM, Anish Bhatt wrote: Signed-off-by: Anish Bhatt an...@chelsio.com Signed-off by: Manoj Malvia manojmalv...@chelsio.com Signed-off by: Karen Xie k...@chelsio.com --- drivers/scsi/libiscsi.c | 61 +++-- drivers/scsi/libiscsi_tcp.c | 296 include/scsi/libiscsi.h | 6 + include/scsi/libiscsi_tcp.h | 22 4 files changed, 349 insertions(+), 36 deletions(-) Hey Anish, Against what target did you guys test this stuff? Sagi. -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
We tested against a Sanblaze target. -Anish From: Sagi Grimberg [sa...@dev.mellanox.co.il] Sent: Thursday, August 14, 2014 2:25 AM To: Anish Bhatt Cc: linux-scsi@vger.kernel.org; h...@infradead.org; jbottom...@parallels.com; micha...@cs.wisc.edu; Karen Xie; Manoj Malviya Subject: Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support On 8/7/2014 1:37 AM, Anish Bhatt wrote: Signed-off-by: Anish Bhatt an...@chelsio.com Signed-off by: Manoj Malvia manojmalv...@chelsio.com Signed-off by: Karen Xie k...@chelsio.com --- drivers/scsi/libiscsi.c | 61 +++-- drivers/scsi/libiscsi_tcp.c | 296 include/scsi/libiscsi.h | 6 + include/scsi/libiscsi_tcp.h | 22 4 files changed, 349 insertions(+), 36 deletions(-) Hey Anish, Against what target did you guys test this stuff? Sagi. -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
Mike, I presume you meant the
8846bab
scsi_cmnd: Introduce scsi_transfer_length helper
d77e653
libiscsi, iser: Adjust data_length to include protection information
Martin's fix after that
5616b0
use the scsi data buffer length to extract transfer size
I see the commits in the branches, but not in master. Do the patches need to be
rebased onto drivers-for-3.17 then ?
-Anish
From: Anish Bhatt
Sent: Wednesday, August 06, 2014 7:35 PM
To: Mike Christie
Cc: linux-scsi@vger.kernel.org; h...@infradead.org; jbottom...@parallels.com;
Karen Xie; Manoj Malviya
Subject: RE: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
This patch was made over scsi-devel/master, I'll rebase it to Christoph's tree
-Anish
From: Mike Christie [micha...@cs.wisc.edu]
Sent: Wednesday, August 06, 2014 7:25 PM
To: Anish Bhatt
Cc: linux-scsi@vger.kernel.org; h...@infradead.org; jbottom...@parallels.com;
Karen Xie; Manoj Malviya
Subject: Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
On 08/06/2014 05:37 PM, Anish Bhatt wrote
@@ -436,12 +473,12 @@ static int iscsi_prep_scsi_cmd_pdu(struct iscsi_task
*task)
/* No unsolicit Data-Out's */
hdr-flags |= ISCSI_FLAG_CMD_FINAL;
} else {
+ unsigned in_len = iscsi_scsi_in_total_length(sc);
hdr-flags |= ISCSI_FLAG_CMD_FINAL;
zero_data(hdr-dlength);
- hdr-data_length = cpu_to_be32(scsi_in(sc)-length);
-
if (sc-sc_data_direction == DMA_FROM_DEVICE)
hdr-flags |= ISCSI_FLAG_CMD_READ;
+ hdr-data_length = cpu_to_be32(in_len);
}
What tree and branch did you make these over? Have you seen the patches
from Sagi for iser and libiscsi support?
Could you rebuild this over Christoph's tree which has his patches:
git://git.infradead.org/users/hch/scsi-queue.git
--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
On 08/06/2014 05:37 PM, Anish Bhatt wrote
@@ -436,12 +473,12 @@ static int iscsi_prep_scsi_cmd_pdu(struct iscsi_task
*task)
/* No unsolicit Data-Out's */
hdr-flags |= ISCSI_FLAG_CMD_FINAL;
} else {
+ unsigned in_len = iscsi_scsi_in_total_length(sc);
hdr-flags |= ISCSI_FLAG_CMD_FINAL;
zero_data(hdr-dlength);
- hdr-data_length = cpu_to_be32(scsi_in(sc)-length);
-
if (sc-sc_data_direction == DMA_FROM_DEVICE)
hdr-flags |= ISCSI_FLAG_CMD_READ;
+ hdr-data_length = cpu_to_be32(in_len);
}
What tree and branch did you make these over? Have you seen the patches
from Sagi for iser and libiscsi support?
Could you rebuild this over Christoph's tree which has his patches:
git://git.infradead.org/users/hch/scsi-queue.git
--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
This patch was made over scsi-devel/master, I'll rebase it to Christoph's tree
-Anish
From: Mike Christie [micha...@cs.wisc.edu]
Sent: Wednesday, August 06, 2014 7:25 PM
To: Anish Bhatt
Cc: linux-scsi@vger.kernel.org; h...@infradead.org; jbottom...@parallels.com;
Karen Xie; Manoj Malviya
Subject: Re: [PATCH scsi 1/1] libiscsi : Add T10 Data Integrity Feature support
On 08/06/2014 05:37 PM, Anish Bhatt wrote
@@ -436,12 +473,12 @@ static int iscsi_prep_scsi_cmd_pdu(struct iscsi_task
*task)
/* No unsolicit Data-Out's */
hdr-flags |= ISCSI_FLAG_CMD_FINAL;
} else {
+ unsigned in_len = iscsi_scsi_in_total_length(sc);
hdr-flags |= ISCSI_FLAG_CMD_FINAL;
zero_data(hdr-dlength);
- hdr-data_length = cpu_to_be32(scsi_in(sc)-length);
-
if (sc-sc_data_direction == DMA_FROM_DEVICE)
hdr-flags |= ISCSI_FLAG_CMD_READ;
+ hdr-data_length = cpu_to_be32(in_len);
}
What tree and branch did you make these over? Have you seen the patches
from Sagi for iser and libiscsi support?
Could you rebuild this over Christoph's tree which has his patches:
git://git.infradead.org/users/hch/scsi-queue.git
--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
