Move the code for the snapshotting IPC thread into a separate file
and define macros for adding new IPC commands.
No functional change intended.
Details
Details
- Reviewers
- None
- Group Reviewers
bhyve
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 69799 Build 66682: arc lint + arc unit
Event Timeline
Comment Actions
only suggestions, not a request
| usr.sbin/bhyve/ipc.c | ||
|---|---|---|
| 60–74 | what do you think about each ipc command constructing their own nvlist as a reply? and then let handle_message() destroy the passed in nvl list? there are two different class errors that can occur here, one is that the requested command is not supported and the second class is the handler encountered an error...might be worth being able to differentiate between those two different types of errors...only mentioning, your decision | |
| 98–110 | looking at the suggested edit...then handle_message() would be responsible for destroying nvl and constructing the reply message. | |