nvme: Be less verbose when cancelling I/O or admin commands

[imp]

Summary

When we're resetting, and there's outstanding I/O that we're cancelling,
only report we're cancelling the I/O once rather than once per
I/O. Likewise when we reschedule the I/O. We don't need to say for each
one that we're cancelling/rescheduling something, and then report the
I/O that we're doing. Likewise with cancelling admin commands (we never
retry them here, so a similar change isn't needed).

Sponsored by: Netflix

[mav]

I'd just use !TAILQ_EMPTY() instead of TAILQ_FIRST() != NULL. Also while there I would remove extra STAILQ_EMPTY() before STAILQ_FIRST().

[imp]

Great suggestion...

[jhb]

sys/dev/nvme/nvme_qpair.c

1255–1256

I would also maybe go ahead and spell out report.

[imp]

sys/dev/nvme/nvme_qpair.c

1255–1256

OK. I'd already made the first change... And I spelled it out here and elsewhere.

[chuck]

sys/dev/nvme/nvme_qpair.c

1294

Would it make sense to have the logging similar for Admin and I/O? I.e., either change this to done aborting outstanding i/o or change the Admin case above to done aborting?

[imp]

sys/dev/nvme/nvme_qpair.c

1294

Yes. I think it would.
Thanks!