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

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

Tags

None

Referenced Files

	F153962962: D24300.id70226.diff
	Sat, Apr 25, 3:25 AM

	Unknown Object (File)
	Tue, Apr 21, 1:55 PM

	Unknown Object (File)
	Sun, Apr 19, 9:34 PM

	Unknown Object (File)
	Fri, Apr 17, 12:25 AM

	Unknown Object (File)
	Thu, Apr 16, 7:45 PM

	Unknown Object (File)
	Wed, Apr 15, 6:28 PM

	Unknown Object (File)
	Mon, Apr 6, 3:48 PM

	Unknown Object (File)
	Mon, Apr 6, 3:19 PM

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