qla2xxx: use t10_pi_tuple
Subsystems (2)
PATCH 1/1 qla2xxx: use t10_pi_tuple
Drop unused scsi_dif_tuple and use t10_pi_tuple instead of own sd_dif_tuple copy.
Signed-off-by Sebastian Herbszt <[email protected]>
drivers/scsi/qla2xxx/qla_def.h renamed from drivers/scsi/qla2xxx-orig/qla_def.h +1 -10
@@ -25,6 +25,7 @@#include <linux/firmware.h> #include <linux/aer.h> #include <linux/mutex.h> +#include <linux/t10-pi.h> #include <scsi/scsi.h> #include <scsi/scsi_host.h>
@@ -276,16 +277,6 @@struct req_que ;/* -* (sd.h is not exported, hence local inclusion) -* Data Integrity Field tuple. -*/ -struct sd_dif_tuple { -__be16 guard_tag;/* Checksum */ -__be16 app_tag;/* Opaque storage */ -__be32 ref_tag;/* Target LBA or indirect LBA */ -}; - -/* * SCSI Request Block */ struct srb_cmd {
drivers/scsi/qla2xxx/qla_isr.c renamed from drivers/scsi/qla2xxx-orig/qla_isr.c +5 -11
@@ -1711,12 +1711,6 @@ qla2x00_handle_sense(srb_t *sp, uint8_t } } -struct scsi_dif_tuple { -__be16 guard;/* Checksum */ -__be16 app_tag;/* APPL identifier */ -__be32 ref_tag;/* Target LBA or indirect LBA */ -}; -/* * Checks the guard or meta-data for the type of error * detected by the HBA. In case of errors, we set the
@@ -1779,7 +1773,7 @@ qla2x00_handle_dif_error(srb_t *sp, struif (scsi_prot_sg_count (cmd)) {uint32_t i, j =0 , k =0 , num_ent;struct scatterlist *sg; -struct sd_dif_tuple *spt; +struct t10_pi_tuple *pi;/* Patch the corresponding protection tags */ scsi_for_each_prot_sg(cmd, sg,
@@ -1801,12 +1795,12 @@ qla2x00_handle_dif_error(srb_t *sp, strureturn 1 ; } - spt =page_address (sg_page (sg)) + sg->offset; - spt += j; + pi =page_address (sg_page (sg)) + sg->offset; + pi += j; - spt->app_tag =0xffff ; + pi->app_tag =0xffff ;if (scsi_get_prot_type (cmd) ==SCSI_PROT_DIF_TYPE3 ) - spt->ref_tag =0xffffffff ; + pi->ref_tag =0xffffffff ; }return 0 ;
drivers/scsi/qla2xxx/qla_target.c renamed from drivers/scsi/qla2xxx-orig/qla_target.c +5 -5
@@ -2591,17 +2591,17 @@ qlt_handle_dif_error(struct scsi_qla_hos }#if 0 -struct sd_dif_tuple *spt; +struct t10_pi_tuple *pi;/* TODO: * This section came from initiator. Is it valid here? * should ulp be override with actual val??? */ - spt =page_address (sg_page (sg)) + sg->offset; - spt += j; + pi =page_address (sg_page (sg)) + sg->offset; + pi += j; - spt->app_tag =0xffff ; + pi->app_tag =0xffff ;if (cmd->se_cmd.prot_type ==SCSI_PROT_DIF_TYPE3 ) - spt->ref_tag =0xffffffff ; + pi->ref_tag =0xffffffff ;#endif }