On 01/10/2016 12:28 PM, Nicholas A. Bellinger wrote:
This patch does a simple conversion of tcm_fc code to use
proper modern core_tpg_get_initiator_node_acl() lookup using
se_node_acl->acl_kref, and drops the legacy list walk from
ft_acl_get().
Note the original lookup also took node_name into account,
but since ft_init_nodeacl() only ever sets port_name for
se_node_acl->acl_group within configfs, this is purely
a mechanical change.
Hi Nic,
This patch modifies ft_prli_locked() such that the ACL list lookup only
happens if FC_SPP_EST_IMG_PAIR has been set by the initiator. That is a
behavior change and this has not been mentioned in the patch
description. I think the patch description should mention that behavior
change and also that it should explain why that behavior change is
acceptable - if it is acceptable.
Bart.
--
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