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
F147606787: D34863.id104832.diff
Thu, Mar 12, 7:02 AM
Unknown Object (File)
Thu, Mar 5, 11:34 AM
Unknown Object (File)
Fri, Feb 13, 7:15 PM
Unknown Object (File)
Fri, Feb 13, 4:30 PM
Unknown Object (File)
Wed, Feb 11, 1:24 AM
Unknown Object (File)
Feb 8 2026, 9:35 AM
Unknown Object (File)
Feb 8 2026, 3:15 AM
Unknown Object (File)
Feb 8 2026, 2:15 AM
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...