Page MenuHomeFreeBSD

bhyve: Make gdb support optional
ClosedPublic

Authored by markj on Jul 12 2023, 1:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Aug 10, 1:51 PM
Unknown Object (File)
Wed, Aug 5, 3:55 AM
Unknown Object (File)
Sun, Aug 2, 4:17 AM
Unknown Object (File)
Sun, Aug 2, 1:33 AM
Unknown Object (File)
Sat, Aug 1, 7:33 AM
Unknown Object (File)
Wed, Jul 29, 6:26 AM
Unknown Object (File)
Tue, Jul 28, 12:36 PM
Unknown Object (File)
Tue, Jul 21, 9:34 AM
Subscribers

Details

Summary

Add a BHYVE_GDB_SUPPORT make variable that can be set by per-arch
makefiles. When set, BHYVE_GDB is defined and can be used as a
preprocessor predicate. Use it to guard gdb stub calls in MI code.

The arm64 bhyve port currently does not have a functional gdb stub, but
that's not critical to landing the port, so this mechanism slightly
reduces the friction of adding support for a new platform.

Diff Detail

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

Event Timeline

markj requested review of this revision.Jul 12 2023, 1:51 PM
jhb added inline comments.
usr.sbin/bhyve/bhyverun.c
149

Do you want to make '-G' here conditional on BHYVE_GDB? (Not sure how messy the usage() function ends up at the end of all this)

This revision is now accepted and ready to land.Jul 12 2023, 6:33 PM
This revision was automatically updated to reflect the committed changes.