> +static inline int nvme_next_ring_index(struct nvme_queue *nvmeq, u16 index)
> +{
> + if (++index == nvmeq->q_depth)
> + return 0;
> +
> + return index;Can you please just drop this helper? It makes the code not only less readable but also longer. Otherwise the change looks fine to me.
