On Wed, Feb 25, 2009 at 6:40 PM, FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote: > I'm talking about 'iqn.' stuff in your patch. OK >> Yes, it works fine, thanks! To make sure we're on the same page, your patch >> does four changes to vpd page 0x83 >> 1. doesn't report protocol id anymore (PIV set to zero) > Yes. I checked some SAS, FC, iSCSI drives and none sets PIV bit. So I > guess that it's fine to drop PIV bit and the protocol bit. by "drives" you mean targets? >> 2. reports LUN association (zero) instead of target port association >> 3. reports T10 identifier type instead of vendor specific >> 4. uses the T10 IET assigned name in the identifier >> I am fine with the patch and it allows me to achieve the interop >> with the vmware stack, to make it clearer I would add the LU association > I see, thanks for confirmation. I'll merge it. thanks, lets just document changes 1-4 and the ASS LU in the change-log, for the future... Or. -- 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