Paths

Table of Contentst

Differential D38872

bhyve: Exit with EX_OSERR if init checkpoint or restore time failed
ClosedPublic
Actions

Authored by gusev.vitaliy_gmail.com on Mar 2 2023, 6:47 PM.

Tags

Referenced Files

	F131883586: D38872.id118249.diff
	Sat, Oct 11, 11:04 PM

	F131867959: D38872.id118384.diff
	Sat, Oct 11, 8:07 PM

	F131867949: D38872.id118249.diff
	Sat, Oct 11, 8:06 PM

	F131867946: D38872.id118201.diff
	Sat, Oct 11, 8:06 PM

	F131821497: D38872.diff
	Sat, Oct 11, 11:09 AM

	F131768047: D38872.diff
	Sat, Oct 11, 12:30 AM

	Unknown Object (File)
	Wed, Oct 8, 5:58 PM

	Unknown Object (File)
	Fri, Oct 3, 5:22 AM

Subscribers

Details

Reviewers

markj
jhb
corvink

Group Reviewers

Commits

rG6efb053bea5f: bhyve: exit with EX_OSERR if init checkpoint or restore time failed
rGd213429e42d4: bhyve: exit with EX_OSERR if init checkpoint or restore time failed

Summary

Part of Capsicum integration for snapshots.

Sponsored by: vStack

Test Plan

Run VM, Suspend, Resume.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

gusev.vitaliy_gmail.com created this revision.Mar 2 2023, 6:47 PM

Herald added a reviewer: bhyve. · View Herald TranscriptMar 2 2023, 6:47 PM

Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript

gusev.vitaliy_gmail.com requested review of this revision.Mar 2 2023, 6:47 PM

gusev.vitaliy_gmail.com mentioned this in D35454: bhyve: snapshot capsicum support.Mar 2 2023, 6:49 PM

gusev.vitaliy_gmail.com mentioned this in D38857: bhyve: Init checkpoint before caph_enter().

gusev.vitaliy_gmail.com added a child revision: D38857: bhyve: Init checkpoint before caph_enter().Mar 2 2023, 8:30 PM

gusev.vitaliy_gmail.com added a parent revision: D38856: bhyve: [snapshot] Add cap limits for ipc socket.

markj accepted this revision as: markj.Mar 2 2023, 10:03 PM

markj added inline comments.

usr.sbin/bhyve/bhyverun.c
1582	pthread_create() returns a positive error number in the failure case, and this check won't catch it. I suspect it should just check `init_checkpoint_thread(ctx) != 0`.

This revision is now accepted and ready to land.Mar 2 2023, 10:03 PM

corvink accepted this revision.Mar 3 2023, 7:04 AM

Fix note, use init_checkpoint_thread(ctx) != 0

This revision now requires review to proceed.Mar 3 2023, 11:47 AM

gusev.vitaliy_gmail.com marked an inline comment as done.Mar 3 2023, 11:48 AM

gusev.vitaliy_gmail.com added inline comments.

usr.sbin/bhyve/bhyverun.c
1582	Done.

@markj @corvink Please re-accept.

markj accepted this revision as: markj.Mar 3 2023, 2:27 PM

This revision is now accepted and ready to land.Mar 3 2023, 2:27 PM

Closed by commit rGd213429e42d4: bhyve: exit with EX_OSERR if init checkpoint or restore time failed (authored by gusev.vitaliy_gmail.com, committed by corvink). · Explain WhyMar 6 2023, 1:06 PM

This revision was automatically updated to reflect the committed changes.

corvink added a commit: rGd213429e42d4: bhyve: exit with EX_OSERR if init checkpoint or restore time failed.

corvink added a commit: rG6efb053bea5f: bhyve: exit with EX_OSERR if init checkpoint or restore time failed.Mar 17 2023, 10:31 AM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

7 lines

Diff 118249

usr.sbin/bhyve/bhyverun.c

Loading...