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)
	Feb 2 2024, 5:20 AM

	Unknown Object (File)
	Dec 20 2023, 2:07 AM

	Unknown Object (File)
	Nov 26 2023, 7:52 AM

	Unknown Object (File)
	Nov 22 2023, 12:40 PM

	Unknown Object (File)
	Nov 12 2023, 9:18 AM

	Unknown Object (File)
	Nov 11 2023, 2:39 AM

	Unknown Object (File)
	Nov 10 2023, 8:01 AM

	Unknown Object (File)
	Nov 9 2023, 8:06 AM

View All 27 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