> + > + if (dev->online_queues > 1) > + nvme_queue_scan(dev); > return; > Above this we do 'if (dev->online_queues < 2)'. Maybe we should change one or the other to make it consistent Otherwise looks fine Acked-by Jon Derrick: <jonathan.derrick at intel.com>