>>>>> "nab" == Nicholas A Bellinger <nab@xxxxxxxxxxxxxxx> writes: nab> The following incremental patch saves the current sess_prot_type nab> into se_node_acl, and will always reset sess_prot_type if a nab> previous saved value exists. So the PI setting for the fabric's nab> session with backend devices not supporting PI is persistent across nab> session restart. nab> I also noticed the internal DIF emulation was not honoring se_cmd-> nab> prot_checks for the WRPROTECT/RDPROTECT == 0x3 case, so nab> sbc_dif_v1_verify() has been updated to follow which checks have nab> been calculated based on WRPROTECT/RDPROTECT in nab> sbc_set_prot_op_checks(). nab> Finally in sbc_check_prot(), if PROTECT is non-zero for a backend nab> device with DIF disabled, and sess_prot_type is not set go ahead nab> and return INVALID_CDB_FIELD. Looks good to me. Reviewed-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx> -- 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html