> +static void nvme_rdma_destroy_qp(struct nvme_rdma_queue *queue) > +{ > + rdma_destroy_qp(queue->cm_id); > + queue->qp = NULL; Why is setting the qp to NULL needed here? I'd rather not do this if we don't have a good reason reason for it.