Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sctp: update manpages #965

Closed
wants to merge 1 commit into from
Closed

sctp: update manpages #965

wants to merge 1 commit into from

Conversation

llfw
Copy link
Contributor

@llfw llfw commented Dec 29, 2023

Add missing documentation for sctp_sendv(3) and sctp_recvv(3).

Add a note that sctp_send(3), sctp_sendx(3) and sctp_recvmsg(3) are deprecated by RFC 6458.

Add a STANDARDS section to all functions specified in RFC 6458 to indicate their standards conformance.


this should probably be reviewed by someone who knows more about SCTP than i do.

@llfw llfw force-pushed the sctp-docs branch 2 times, most recently from d0e64d4 to 68195ea Compare December 29, 2023 18:03
@igalic
Copy link
Contributor

igalic commented Dec 29, 2023

cc @kprovost @tuexen

@llfw llfw force-pushed the sctp-docs branch 3 times, most recently from 00818b7 to f92ec8b Compare December 30, 2023 12:36
@bsdimp bsdimp self-assigned this Jan 29, 2024
.Sh DESCRIPTION
The
.Fn sctp_send
system call
is used to transmit a message to another SCTP endpoint.
The
.Fn sctp_sendx
function is used to transmit a message to multiple endpoint addresses.
Copy link
Contributor

@tuexen tuexen Jan 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe something like

When issuing an implicit association setup, the
.Fn sctp_sendx
function allows to specify multiple addresses of the peer, which can then be used for the association setup.
This is equivalent to using
.Xr sctp_connectx 3 .

would be more accurate.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i replaced the text with this, which i think is still accurate, but is slightly shorter to fit the style of the paragraph:

The
.Fn sctp_sendx
function is used to specify multiple peer addresses when creating an implicit
association, as in
.Xr sctp_connectx 3 .

@tuexen
Copy link
Contributor

tuexen commented Jan 30, 2024

Looking at, for example IP6(4), it might be useful to add references to RFC 6458 in the SEE ALSO section to the man pages where you added a STANDARDS section.

lib/libc/net/sctp_recvmsg.3 Outdated Show resolved Hide resolved
lib/libc/net/sctp_recvmsg.3 Outdated Show resolved Hide resolved
@bsdbcr
Copy link
Contributor

bsdbcr commented Jan 31, 2024

The man page changes need a line break after a sentence stop. Running "mandoc -Tlint" and the textproc/igor port find these sort of issues.

Add missing documentation for sctp_sendv(3) and sctp_recvv(3).

Add a note that sctp_send(3), sctp_sendx(3) and sctp_recvmsg(3) are
deprecated by RFC 6458.

Add a STANDARDS section to all functions specified in RFC 6458 to
indicate their standards conformance.

MFC after:	2 weeks
@llfw
Copy link
Contributor Author

llfw commented Jan 31, 2024

  • fixed lint errors
  • added .Rs for RFC 6458
  • changed the description of sctp_sendx
  • rebased on current main (although there were no conflicts here)

@bsdbcr
Copy link
Contributor

bsdbcr commented Feb 1, 2024

The man page changes look good now, thank you! 👍

freebsd-git pushed a commit that referenced this pull request Feb 3, 2024
Add missing documentation for sctp_sendv(3) and sctp_recvv(3).

Add a note that sctp_send(3), sctp_sendx(3) and sctp_recvmsg(3) are
deprecated by RFC 6458.

Add a STANDARDS section to all functions specified in RFC 6458 to
indicate their standards conformance.

MFC after:	2 weeks
Reviewed by: imp, bcr, kp, tuexen
Pull Request: #965
@bsdimp bsdimp closed this Feb 3, 2024
@llfw llfw deleted the sctp-docs branch February 3, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants