svn commit: r267551 - head/sys/cam/ctl

[Edward Tomasz Napierala]

Author: trasz
Date: Mon Jun 16 19:05:51 2014
New Revision: 267551
URL: http://svnweb.freebsd.org/changeset/base/267551

Log:
  Add comment explaining a potential problem with just added LUN ID.
  
  Reminded by:	mav@
  Sponsored by:	The FreeBSD Foundation

Modified:
  head/sys/cam/ctl/ctl_frontend_iscsi.c

Modified: head/sys/cam/ctl/ctl_frontend_iscsi.c
==============================================================================
--- head/sys/cam/ctl/ctl_frontend_iscsi.c	Mon Jun 16 18:33:32 2014	(r267550)
+++ head/sys/cam/ctl/ctl_frontend_iscsi.c	Mon Jun 16 19:05:51 2014	(r267551)
@@ -2143,6 +2143,10 @@ cfiscsi_devid(struct ctl_scsiio *ctsio, 
 
 	/*
 	 * desc1 is for the unique LUN name.
+	 *
+	 * XXX: According to SPC-3, LUN must report the same ID through
+	 *      all the ports.  The code below, however, reports the
+	 *      ID only via iSCSI.
 	 */
 	desc1->proto_codeset = (SCSI_PROTO_ISCSI << 4) | SVPD_ID_CODESET_UTF8;
 	desc1->id_type = SVPD_ID_PIV | SVPD_ID_ASSOC_LUN |