Page MenuHomeFreeBSD
Differential D51535

bhyve/tpm_ppi_qemu: Cast address of the TPP3 field to uint32_t
ClosedPublic
Actions

Authored by bnovkov on Jul 25 2025, 6:10 PM.
Tags
None
Referenced Files
F163110153: D51535.id159241.diff
Mon, Jul 20, 4:02 AM
Unknown Object (File)
Tue, Jul 14, 5:25 PM
Unknown Object (File)
Mon, Jul 13, 6:42 PM
Unknown Object (File)
Mon, Jul 13, 12:13 AM
Unknown Object (File)
Wed, Jul 1, 3:13 PM
Unknown Object (File)
Fri, Jun 26, 8:13 AM
Unknown Object (File)
Fri, Jun 26, 7:51 AM
Unknown Object (File)
May 21 2026, 1:32 AM
View All 70 Files
Subscribers
bcran
imp
markj
rgrimes

Details

Reviewers
corvink
markj
Group Reviewers
bhyve
Commits
rGc7a1776752ec: bhyve/tpm_ppi_qemu: Cast address of the TPP3 field to uint32_t
Summary

Fix a format specifier issue when building 'tpm_ppi_qemu.c' with
'dsdt_line' marked as 'printflike'.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bnovkov created this revision.Jul 25 2025, 6:10 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptJul 25 2025, 6:10 PM
bnovkov requested review of this revision.Jul 25 2025, 6:10 PM
Harbormaster completed remote builds in B65724: Diff 159125.Jul 25 2025, 6:10 PM
bnovkov added a parent revision: D51534: bhyve/acpi: Mark dsdt_line as 'printflike'.Jul 25 2025, 6:10 PM
imp added inline comments.Jul 25 2025, 6:54 PM
usr.sbin/bhyve/tpm_ppi_qemu.c
210

(unsigned int) is really the only way to fix the identifier. *OR* casting it to the maximum int which is overkill here imho.
*or* switching to DES' implementation of the C24 stuff that lets us match uint32_t to a %blah...

markj accepted this revision as: markj.Jul 26 2025, 9:27 PM
markj added a subscriber: markj.
markj added inline comments.
usr.sbin/bhyve/tpm_ppi_qemu.c
210

Casting the sizeof rather than whole expression would perhaps be clearer.

I don't quite understand why casting to unsigned int would be better.

This revision is now accepted and ready to land.Jul 26 2025, 9:27 PM
Closed by commit rGc7a1776752ec: bhyve/tpm_ppi_qemu: Cast address of the TPP3 field to uint32_t (authored by bnovkov). · Explain WhyJul 27 2025, 4:34 PM
This revision was automatically updated to reflect the committed changes.
bnovkov added a commit: rGc7a1776752ec: bhyve/tpm_ppi_qemu: Cast address of the TPP3 field to uint32_t.

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
2 lines

Diff 159241

View Options

usr.sbin/bhyve/tpm_ppi_qemu.c

Loading...