Page MenuHomeFreeBSD

bhyve: Initial some NVMe controller health log data.

Authored by on Mar 27 2020, 12:49 PM.
Referenced Files
Unknown Object (File)
Thu, May 4, 4:12 PM
Unknown Object (File)
Mar 16 2023, 11:04 AM
Unknown Object (File)
Mar 1 2023, 6:12 PM
Unknown Object (File)
Feb 12 2023, 4:56 PM
Unknown Object (File)
Feb 11 2023, 1:28 AM
Unknown Object (File)
Dec 25 2022, 5:27 PM
Unknown Object (File)
Dec 15 2022, 6:18 PM
Unknown Object (File)
Dec 12 2022, 12:26 PM



Currently, bhyve NVMe controller does not set logpage health data. For some OSs, they think NVMe device is falling without reasonable values.

These fields are:

Available spare
Available spare threshold

This patch will set some reasonable values.

Test Plan

From guest, check these values.

Diff Detail

Lint Passed
No Test Coverage
Build Status
Buildable 30132
Build 27939: arc lint + arc unit

Event Timeline

Should have a better way to initial health log.

This looks good to me. Please rebase against the latest and I'd be happy to commit this.


Per style(9) this white space should remain.


Perhaps, reword as

/* Set nominal Health values checked by implementations */

Thank you chuck.
I am bring back and update this patch as your comments.

Looks good. Thank you for contributing this!

This revision is now accepted and ready to land.Aug 2 2020, 9:53 PM

I am sorry, I am new to this review system.
@chuck Do I need to do any action now or just wait?

From this page FreeBSD Wiki: Phabricator, I should commit this by myself

Once the reviewers are happy with your change, they will accept your patch (which will be denoted by a green tick on the web site). At this point, you are free to submit the change to svn. Just use svn commit as usual.

But I don't think I have permission for this.

I'm happy with this change but need to wait for maintainer approval before I commit it

I'm happy with this change but need to wait for maintainer approval before I commit it

I'm good with it if you're good with it :)

This revision was automatically updated to reflect the committed changes.