Page MenuHomeFreeBSD
Differential D30805

vmm: Fix ivrs_drv device_printf usage
ClosedPublic
Actions

Authored by khng on Jun 17 2021, 6:11 PM.
Tags
None
Referenced Files
F159670414: D30805.id91026.diff
Tue, Jun 16, 9:47 PM
Unknown Object (File)
Sat, Jun 13, 12:50 AM
Unknown Object (File)
Mon, Jun 8, 11:51 AM
Unknown Object (File)
Mon, Jun 8, 2:46 AM
Unknown Object (File)
Tue, May 19, 2:06 PM
Unknown Object (File)
Tue, May 19, 6:55 AM
Unknown Object (File)
May 17 2026, 9:38 PM
Unknown Object (File)
May 15 2026, 5:50 PM
View All Files
Subscribers
bcran
imp
jhb
rgrimes

Details

Reviewers
grehan
imp
jhb
Group Reviewers
bhyve
Commits
rGda4e35def95a: vmm: Fix ivrs_drv device_printf usage
rG67d02e13011f: vmm: Fix ivrs_drv device_printf usage
rG210e6aec4f83: vmm: Fix ivrs_drv device_printf usage
Summary

The original %b description string is wrong.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 39958
Build 36847: arc lint + arc unit

Event Timeline

khng created this revision.Jun 17 2021, 6:11 PM
Herald added a reviewer: bhyve. · View Herald TranscriptJun 17 2021, 6:11 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript
khng requested review of this revision.Jun 17 2021, 6:11 PM
Harbormaster completed remote builds in B39958: Diff 91026.Jun 17 2021, 6:11 PM
imp accepted this revision.Jun 17 2021, 9:31 PM
This revision is now accepted and ready to land.Jun 17 2021, 9:31 PM
jhb added a subscriber: jhb.Jun 17 2021, 11:13 PM

Wait, how did this compile exactly? Does it stop at the first invalid character so that '\008' becomes '\0' and '8'?

jhb accepted this revision.Jun 17 2021, 11:13 PM
imp added a comment.Jun 17 2021, 11:24 PM
In D30805#692800, @jhb wrote:

Wait, how did this compile exactly? Does it stop at the first invalid character so that '\008' becomes '\0' and '8'?

\0 and a bit name that starts with a number is legal, but not the expected results

Closed by commit rG210e6aec4f83: vmm: Fix ivrs_drv device_printf usage (authored by khng). · Explain WhyJun 19 2021, 6:09 AM
This revision was automatically updated to reflect the committed changes.
khng added a commit: rG210e6aec4f83: vmm: Fix ivrs_drv device_printf usage.
khng added a commit: rG67d02e13011f: vmm: Fix ivrs_drv device_printf usage.Jul 13 2021, 5:59 PM
khng added a commit: rGda4e35def95a: vmm: Fix ivrs_drv device_printf usage.Sep 3 2021, 7:08 PM

Revision Contents
Changeset List

PathSize
sys/
amd64/
vmm/
amd/
30 lines

Diff 91026

View Options

sys/amd64/vmm/amd/ivrs_drv.c

Loading...