> +static int nvme_check_ctrl_fabric_info(struct nvme_ctrl *ctrl, struct nvme_id_ctrl *id) Overly long line. I'd also rename this to something like nvmf_validate_identify_ctrl. Otherwise this looks good.