From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: [PATCH] MdeModulePkg/NvmExpressDxe: fix check for Cap.Css To: devel@edk2.groups.io From: littlefox@lf-net.org X-Originating-Location: Zwickau, Saxony, DE (92.117.127.159) X-Originating-Platform: Linux Chrome 98 User-Agent: GROUPS.IO Web Poster MIME-Version: 1.0 Date: Wed, 16 Mar 2022 12:56:25 -0700 Message-ID: Content-Type: multipart/alternative; boundary="icBeQgAfFVu0gFMQS5QJ" --icBeQgAfFVu0gFMQS5QJ Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Fix the check for NVMe command set being supported by the controller. Was problematic with qemu (6.2.0, Debian 1:6.2+dfsg-3), which sets 0xC1 in = that register, making the OVMF think the NVMe controller does not support N= VMe. Uncovered by commit 9dd14fc ( https://github.com/tianocore/edk2/commit/9dd1= 4fc91c174eae87fd122c7ac70073a363527f ) , which changed the number of bits i= ncluded in the Css register from 4 to 8. Signed-off-by: Mara Sophie Grosch littlefox@lf-net.org Patch is on GitHub: https://github.com/tianocore/edk2/pull/2637 $ git diff --stat master MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressHci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --icBeQgAfFVu0gFMQS5QJ Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable

Fix the check for NVMe command se= t being supported by the controller.

Was problematic with qemu (6.2.0,= Debian 1:6.2+dfsg-3), which sets 0xC1 in that register, making the OVMF th= ink the NVMe controller does not support NVMe.

Uncovered by commit 9dd14fc, which changed the number of bits included in the Css= register from 4 to 8.

Signed-off-by: Mara Sop= hie Grosch littlefox@lf-net.org

Patch is on GitHu= b: https://github.com/tianocore/edk2/pull/2637

 

$ git diff --= stat master

 MdeModu= lePkg/Bus/Pci/NvmExpressDxe/NvmExpressHci.c | 2 +-

 1 file = changed, 1 insertion(+), 1 deletion(-)

--icBeQgAfFVu0gFMQS5QJ--