Bhyve: SMBIOS bcdrev calculated from major/minor values
ClosedPublic
Actions

Authored by rgrimes on Apr 5 2020, 10:25 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 16, 11:49 AM

	Unknown Object (File)
	Sun, Sep 21, 1:27 AM

	Unknown Object (File)
	Wed, Sep 17, 4:42 PM

	Unknown Object (File)
	Aug 9 2025, 6:42 AM

	Unknown Object (File)
	Aug 5 2025, 10:56 AM

	Unknown Object (File)
	Jun 26 2025, 3:13 AM

	Unknown Object (File)
	Jun 24 2025, 2:15 AM

	Unknown Object (File)
	Jun 24 2025, 1:43 AM

View All 73 Files

Subscribers

None

Details

Reviewers

bcran
grehan
jhb

Group Reviewers

bhyve

Summary

In the past changes have been made to smbios->minor without updating the smbios->bcdrev value. Correct that by calculating bcdrev from the major/minor values.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

rgrimes created this revision.Apr 5 2020, 10:25 AM

bcran accepted this revision.Apr 5 2020, 5:51 PM

This revision is now accepted and ready to land.Apr 5 2020, 5:51 PM

jhb accepted this revision.Apr 6 2020, 4:39 PM

jhb added inline comments.

usr.sbin/bhyve/smbiostbl.c
771	If you want the comment, please style(9) it by adding a blank line before and a period. I think you could probably also get away with not needing the comment since the code is a few lines below and fairly obvious.

Remove comment

This revision now requires review to proceed.Apr 7 2020, 11:15 PM

rgrimes marked an inline comment as done.Apr 7 2020, 11:15 PM

bcran accepted this revision.Apr 8 2020, 3:24 AM

This revision is now accepted and ready to land.Apr 8 2020, 3:24 AM

This was fixed in rS359719, so I think this review should be closed.

jhb closed this revision.Dec 30 2020, 9:37 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

	smbiostbl.c
	smbiostbl.c.orig

2 lines

Diff 70324

View Options

usr.sbin/bhyve/smbiostbl.c

Bhyve: SMBIOS bcdrev calculated from major/minor valuesClosedPublicActions