>> + pr_info("creating controller %d for subsystem %s for NQN %s.\n",
>> + ctrl->cntlid, ctrl->subsys->subsysnqn, ctrl->hostnqn);
>
> That's going to be a crazy long line in the dmesg log..
Have a better idea how to provide this information?
>
> Otherwise looks fine:
>
>
> Reviewed-by: Christoph Hellwig <hch at lst.de>
>