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
Unknown Object (File)
Thu, Jan 30, 8:25 PM
Unknown Object (File)
Tue, Jan 28, 2:50 AM
Unknown Object (File)
Mon, Jan 27, 9:02 AM
Unknown Object (File)
Fri, Jan 24, 5:58 PM
Unknown Object (File)
Thu, Jan 23, 6:39 PM
Unknown Object (File)
Mon, Jan 13, 7:31 PM
Unknown Object (File)
Thu, Jan 9, 10:40 AM
Unknown Object (File)
Dec 29 2024, 12:46 AM
View All 93 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 45102
Build 41990: 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–1461

Isn't the socket leaked?

usr.sbin/bhyvectl/bhyvectl.c
1700

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/
4 lines

Diff 104832

View Options

usr.sbin/bhyve/snapshot.c

Loading...
View Options

usr.sbin/bhyvectl/bhyvectl.c

Loading...