On Sun, 22 Feb 2009 13:46:46 +0200 Or Gerlitz <ogerlitz@xxxxxxxxxxxx> wrote: > All in all, it seems something has to change (both regarding the association and the vendor identification). Indeed, fixing things to comply with T10 is possible, however we believe the SNS approach allows the user to set its own target name, configurable through the standard stgt management mechanisms, which may improve/ease target identification and network manageability. Your patch does the right thing for only iSCSI? We need to do the right things for other protocols. A simple patch like this doesn't work for you? I guess that we still need to add a new user interface to change 0x83 VPD page freely though. diff --git a/usr/spc.c b/usr/spc.c index ac5c3de..2fc8e8e 100644 --- a/usr/spc.c +++ b/usr/spc.c @@ -133,8 +133,8 @@ static void update_vpd_83(struct scsi_lu *lu, void *id) struct vpd *vpd_pg = lu->attrs.lu_vpd[3]; uint8_t *data = vpd_pg->data; - data[0] = (PIV_ISCSI << 4) | INQ_CODE_ASCII; - data[1] = PIV_VALID | ASS_TGT_PORT | DESG_VENDOR; + data[0] = INQ_CODE_ASCII; + data[1] = DESG_T10; data[3] = SCSI_ID_LEN; strncpy((char *)data + 4, id, SCSI_ID_LEN); @@ -1029,7 +1029,7 @@ int spc_lu_init(struct scsi_lu *lu) snprintf(lu->attrs.product_rev, sizeof(lu->attrs.product_rev), "%s", "0001"); snprintf(lu->attrs.scsi_id, sizeof(lu->attrs.scsi_id), - "deadbeaf%d:%" PRIu64, tgt->tid, lu->lun); + "IET %04x%04" PRIx64, tgt->tid, lu->lun); snprintf(lu->attrs.scsi_sn, sizeof(lu->attrs.scsi_sn), "beaf%d%" PRIu64, tgt->tid, lu->lun); -- To unsubscribe from this list: send the line "unsubscribe stgt" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
