On 04/07/2016 03:29 PM, Nicholas A. Bellinger wrote:
The whole point of the alloc_session callback is so that extra pre-allocation doesn't happen until after se_node_acl lookup has finished, and is what ib_srpt needs to be using given SRP's completely existent spec level security model.
Sorry but I consider this as hair splitting. Who cares about whether or not command buffer allocation + freeing happens for login attempts that will fail because no ACL has been set up yet? That scenario is not performance critical.
Bart. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html