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
Unknown Object (File)
Thu, May 21, 1:32 AM
Unknown Object (File)
Tue, May 19, 9:01 PM
Unknown Object (File)
Mon, May 18, 11:53 PM
Unknown Object (File)
Sun, May 17, 12:47 AM
Unknown Object (File)
Sun, May 17, 12:47 AM
Unknown Object (File)
Tue, May 12, 4:15 AM
Unknown Object (File)
Mon, Apr 27, 5:59 PM
Unknown Object (File)
Apr 22 2026, 9:45 PM
View All 63 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...