Page MenuHomeFreeBSD

bhyve: Clear lid to 0 for internal device errors for NVMe AENs.
ClosedPublic

Authored by jhb on Nov 23 2022, 11:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 23, 1:58 PM
Unknown Object (File)
Sun, Mar 9, 7:11 PM
Unknown Object (File)
Wed, Mar 5, 1:52 PM
Unknown Object (File)
Wed, Mar 5, 1:50 AM
Unknown Object (File)
Tue, Mar 4, 10:10 PM
Unknown Object (File)
Mon, Feb 24, 2:46 PM
Unknown Object (File)
Feb 9 2025, 11:48 PM
Unknown Object (File)
Feb 4 2025, 7:38 PM

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Nov 23 2022, 11:16 PM
This revision is now accepted and ready to land.Nov 24 2022, 6:32 AM

These changes are good but not really necessary from a functionality point of view. In both cases, the emulation is returning an error, making the LID undefined from the guest's perspective. But I'm all for good hygiene.

+1 on chuck's results. This can be anything.... but unassigned variables are allowed to have any behavior when accessing them now (well, always, but usually it was just some random value)... So I'm good with the change. It might be simpler to assign lid=0 before the case statement, though.

I think the real issue here is avoiding leaking host hypervisor stack memory to a guest, not if NVMe clients care about the value of lid for these specific cases.