On Thu, Nov 22, 2018 at 03:16:23PM +0100, David Disseldorp wrote: > All fabrics return a const string. In all cases *except* iSCSI the > get_fabric_name() string matches fabric_ops.name. > > Both fabric_ops.get_fabric_name() and fabric_ops.name are user facing, > with the former being used for PR/ALUA state and the latter for configFS > (config/target/$name), so we unfortunately need to keep both strings > around for now. Would it make sense to just use .name unless .fabric_name is set to mostly avoid the duplication?