git: c42692360611 - main - bhyve: Use NVMEV to read the ASQS field of AQA

From: John Baldwin <jhb_at_FreeBSD.org>
Date: Mon, 29 Jan 2024 19:03:05 UTC
The branch main has been updated by jhb:

URL: https://cgit.FreeBSD.org/src/commit/?id=c42692360611c8afce6ee7bb937a858d64a133f4

commit c42692360611c8afce6ee7bb937a858d64a133f4
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2024-01-29 18:51:07 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2024-01-29 18:51:07 +0000

    bhyve: Use NVMEV to read the ASQS field of AQA
    
    This is not a functional change, but just being consistent instead of
    omitting a shift by 0.
    
    Reviewed by:    corvink, chuck, imp
    Sponsored by:   Chelsio Communications
    Differential Revision:  https://reviews.freebsd.org/D43600
---
 usr.sbin/bhyve/pci_nvme.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/usr.sbin/bhyve/pci_nvme.c b/usr.sbin/bhyve/pci_nvme.c
index 4084f9b007ba..dd6c9a41d17f 100644
--- a/usr.sbin/bhyve/pci_nvme.c
+++ b/usr.sbin/bhyve/pci_nvme.c
@@ -1065,7 +1065,7 @@ pci_nvme_init_controller(struct pci_nvme_softc *sc)
 	 * cleared to 0h produces undefined results" for both ACQS and
 	 * ASQS. If zero, set CFS and do not become ready.
 	 */
-	asqs = ONE_BASED(sc->regs.aqa & NVME_AQA_REG_ASQS_MASK);
+	asqs = ONE_BASED(NVMEV(NVME_AQA_REG_ASQS, sc->regs.aqa));
 	if (asqs < 2) {
 		EPRINTLN("%s: illegal ASQS value %#x (aqa=%#x)", __func__,
 		    asqs - 1, sc->regs.aqa);