Page MenuHomeFreeBSD
Differential D34863

bhyve/snapshot: ..back to SOCK_STREAM
ClosedPublic
Actions

Authored by rew on Apr 10 2022, 5:15 AM.
Tags
None
Referenced Files
F142607394: D34863.id105368.diff
Wed, Jan 21, 11:34 AM
F142549061: D34863.id.diff
Tue, Jan 20, 9:17 PM
Unknown Object (File)
Tue, Jan 20, 3:35 PM
Unknown Object (File)
Tue, Jan 20, 2:46 PM
Unknown Object (File)
Tue, Jan 20, 1:49 AM
Unknown Object (File)
Mon, Jan 19, 8:13 PM
Unknown Object (File)
Fri, Jan 16, 5:18 AM
Unknown Object (File)
Dec 19 2025, 3:05 PM
View All Files
Subscribers
bcran
evgueni.gavrilov_itglobal.com
imp
jhb
markj
rgrimes

Details

Reviewers
markj
Group Reviewers
bhyve
Commits
rG10e30f7d8fcb: bhyve/snapshot: ..back to SOCK_STREAM
rG690b7ea08179: bhyve/snapshot: ..back to SOCK_STREAM
Summary

Now that nvlist_send()/nvlist_recv() are being used, ditch the datagram
socket.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45341
Build 42229: arc lint + arc unit

Event Timeline

rew created this revision.Apr 10 2022, 5:15 AM
Herald added a reviewer: bhyve. · View Herald TranscriptApr 10 2022, 5:15 AM
Herald added subscribers: bcran, rgrimes, imp. · View Herald Transcript
rew requested review of this revision.Apr 10 2022, 5:15 AM
Harbormaster completed remote builds in B45102: Diff 104832.Apr 10 2022, 5:16 AM
jhb added a subscriber: jhb.Apr 12 2022, 10:16 PM
jhb added inline comments.
usr.sbin/bhyvectl/bhyvectl.c
1702

I think you still want SUN_LEN here? (And do we need to set addr.sun_len as well?)

evgueni.gavrilov_itglobal.com added a subscriber: evgueni.gavrilov_itglobal.com.Apr 21 2022, 12:23 PM
rew updated this revision to Diff 105368.Apr 24 2022, 4:18 PM

feedback from jhb..use SUN_LEN()

Harbormaster completed remote builds in B45341: Diff 105368.Apr 24 2022, 4:18 PM
markj added a subscriber: markj.Apr 25 2022, 3:27 PM
markj added inline comments.
usr.sbin/bhyve/snapshot.c
1460

Isn't the socket leaked?

usr.sbin/bhyvectl/bhyvectl.c
1698

Wrap this line since you're modifying it anyway?

rew updated this revision to Diff 105401.Apr 25 2022, 4:02 PM

address feedback from markj:

  • wrap long line
  • close file descriptor being leaked
Harbormaster completed remote builds in B45359: Diff 105401.Apr 25 2022, 4:02 PM
markj accepted this revision as: markj.Apr 25 2022, 5:38 PM
This revision is now accepted and ready to land.Apr 25 2022, 5:38 PM
Closed by commit rG690b7ea08179: bhyve/snapshot: ..back to SOCK_STREAM (authored by rew). · Explain WhyApr 28 2022, 3:45 PM
This revision was automatically updated to reflect the committed changes.
rew added a commit: rG690b7ea08179: bhyve/snapshot: ..back to SOCK_STREAM.
jhb added a commit: rG10e30f7d8fcb: bhyve/snapshot: ..back to SOCK_STREAM.Jan 26 2023, 6:56 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
13 lines
bhyvectl/
8 lines

Diff 105368

View Options

usr.sbin/bhyve/snapshot.c

Loading...
View Options

usr.sbin/bhyvectl/bhyvectl.c

Loading...