[CC += Branden]
Hi, Jeremy (and Branden)!
On 11/11/21 02:53, Jeremy Kerr wrote:
This change adds a brief description for the new Management Component
Transport Protocol (MCTP) support added to Linux as of bc49d8169.
This is a fairly regular sockets-API implementation, so we're just
describing the semantics of socket, bind, sendto and recvfrom for the
new protocol.
Signed-off-by: Jeremy Kerr <jk@xxxxxxxxxxxxxxxxxxxx>
I have a few nitpicks below :)
Thanks!
Alex
---
v2:
- Fix synopsis variable formatting
- Semantic newlines
- remove unnecessary escape
- make custom constants more obvious
- Add URI breakpoints
- fix sockaddr_mctp misuse
- update to reflect upstream sockaddr_mctp types
---
man7/address_families.7 | 7 ++
man7/mctp.7 | 187 ++++++++++++++++++++++++++++++++++++++++
2 files changed, 194 insertions(+)
create mode 100644 man7/mctp.7
diff --git a/man7/address_families.7 b/man7/address_families.7
index 3e535e66d..3c8299e69 100644
--- a/man7/address_families.7
+++ b/man7/address_families.7
@@ -405,6 +405,13 @@ XDP (express data path) interface (since Linux 4.18).
See
.I Documentation/networking/af_xdp.rst
in the Linux kernel source tree for details.
+.TP
+.B AF_MCTP
+.\" commit: bc49d8169aa72295104f1558830c568efb946315
+MCTP (Management Component Transport Protocol) interface (since Linux 5.15),
+as defined by the DMTF specification DSP0236.
+For further information, see
+.BR mctp (7).
.SH SEE ALSO
.BR socket (2),
.BR socket (7)
diff --git a/man7/mctp.7 b/man7/mctp.7
new file mode 100644
index 000000000..506826ed6
--- /dev/null
+++ b/man7/mctp.7
@@ -0,0 +1,187 @@
+.\" Copyright (c) 2021 Jeremy Kerr <jk@xxxxxxxxxxxxxxxxxxxx>
+.\"
+.\" %%%LICENSE_START(GPLv2+_DOC_FULL)
+.\" This is free documentation; you can redistribute it and/or
+.\" modify it under the terms of the GNU General Public License as
+.\" published by the Free Software Foundation; either version 2 of
+.\" the License, or (at your option) any later version.
+.\"
+.\" The GNU General Public License's references to "object code"
+.\" and "executables" are to be interpreted as the output of any
+.\" document formatting or typesetting system, including
+.\" intermediate and printed output.
+.\"
+.\" This manual is distributed in the hope that it will be useful,
+.\" but WITHOUT ANY WARRANTY; without even the implied warranty of
+.\" MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+.\" GNU General Public License for more details.
+.\"
+.\" You should have received a copy of the GNU General Public
+.\" License along with this manual; if not, see
+.\" <http://www.gnu.org/licenses/>.
+.\" %%%LICENSE_END
+.\" commit: bc49d8169aa72295104f1558830c568efb946315
+.TH MCTP 7 2021-10-14 "Linux" "Linux Programmer's Manual"
+.SH NAME
+mctp \- Management Component Transport Protocol
+.SH SYNOPSIS
+.nf
+.B #include <sys/socket.h>
+.B #include <linux/mctp.h>
I prefer alphabetic sorting of includes. And except for the one that
provides the prototype of the function, I also want them to have a
comment saying why they are needed.
See for example membarrier(2):
[
SYNOPSIS
#include <linux/membarrier.h> /* Definition of MEMBARRIER_*
constants */
#include <sys/syscall.h> /* Definition of SYS_* constants */
#include <unistd.h>
int syscall(SYS_membarrier, int cmd, unsigned int flags, int
cpu_id);
Note: glibc provides no wrapper for membarrier(), neces-
sitating the use of syscall(2).
]
Since syscall() is provided by <unistd.h>, it doesn't have a comment.
Any other include has its comment. Also, please have a look at that
page for the formatting of those comments.
+.PP
+.IB mctp_socket " = socket(AF_MCTP, SOCK_DGRAM, 0);"
+.fi
+.SH DESCRIPTION
+Linux implements the Management Component Transport Protocol (MCTP),
+specified by DMTF spec DSP0236, currently at version 1.
+This is a connectionless protocol, typically used between devices within a
+server system.
+Message reliability and ordering are not guaranteed, but message boundaries are
+preserved.
+.PP
+The API for MCTP messaging uses a standard sockets interface, using the
+.BR sendto (2)
+and
+.BR recvfrom (2)
+classes of system calls to transfer messages.
+Messages may be fragmented into packets before transmission, and reassembled at
+the remote endpoint.
Break at the comma.
+This fragmentation and reassembly is transparent to userspace.
+.PP
+.SS Address format
+MCTP addresses (also referred to as EIDs, or Endpoint Identifiers) are
+single-byte values, typed as
+.BR mctp_eid_t .
Types should be in italics.
Branden, I thought this was specified somewhere, but I can't find it.
Do you know where it is? Or maybe if your more up to date
groff_man[_style](7) pages mention that?
groff_man(7) (groff 1.22.4):
[
.I [text]
Set text in italics. If the macro is given no ar-
guments, the text of the next input line is set in
italics.
Use italics for file and path names, for environ-
ment variables, for enumeration or preprocessor
constants in C, for variable (user-determined)
portions of syntax synopses, for the first occur-
rence only of a technical concept being intro-
duced, for names of works of software (including
commands and functions, but excluding names of op-
erating systems or their kernels), and anywhere a
parameter requiring replacement by the user is en-
countered. An exception involves variable text in
a context that is already marked up in italics,
such as file or path names with variable compo-
nents; in such cases, follow the convention of
mathematical typography: set the file or path name
in italics as usual (see .IR below), but use roman
for the variable part, and italics again in run-
ning roman text when referring to the variable ma-
terial.
]
Anyway, for you Jeremy, I have other pages to follow for consistency:
For example, gettimeofday(2).
+Packets between a local and remote endpoint are identified by the source
Break after "by" (or perhaps just before it).
+and destination EIDs, plus a three-bit tag value.
+.PP
+Addressing data is passed in socket system calls through
+.B struct sockaddr_mctp
Type in italics too.
+defined as:
+.PP
+.in +4n
+.EX
+typedef uint8_t mctp_eid_t;
+
+struct mctp_addr {
+ mctp_eid_t s_addr;
+};
+
+struct sockaddr_mctp {
+ unsigned short smctp_family; /* = AF_MCTP */
+ uint16_t __smctp_pad0; /* pad, must be zero */
+ int smctp_network; /* local network identifier */
+ struct mctp_addr smctp_addr; /* EID */
+ uint8_t smctp_type; /* message type byte */
+ uint8_t smctp_tag; /* tag value, including TO flag */
+ uint8_t __smctp_pad1; /* pad, must be zero */
Do we want to tie the implementation to this pad?
Future implementations of sockaddr_mctp are not going to use that byte
for anything else?
If that definition is not to be set in stone, we probably want to cover
it with a layer of mistery. system_data_types(7) for example mentions:
[
The structures described in this manual page shall con-
tain, at least, the members shown in their definition, in
no particular order.
]
Something similar might be good for this page. Maybe "trailing fields
may be added in the future to this structure. The structure should be
zeroed before use, so that future fields are zeroed" or something like
that (I'm not very inspired for the wording, sorry :), and then remove
the pad field.
+};
+.EE
+.in
+.SS Sending messages
+Messages can be transmitted using the
+.BR sendto (2)
+and
+.BR sendmsg (2)
+system calls, by providing a
+.B struct sockaddr_mctp
+describing the addressing parameters.
+.PP
+.in +4n
+.EX
+struct sockaddr_mctp addr;
+ssize_t len;
+char *buf;
+
+/* unused fields must be zero */
+memset(&addr, 0, sizeof(addr));
+
+/* set message destination */
+addr.smctp_family = AF_MCTP;
+addr.smctp_network = 0;
+addr.smctp_addr.s_addr = 8; /* remote EID */
+addr.smctp_tag = MCTP_TAG_OWNER;
+addr.smctp_type = MYPROGRAM_MCTP_TYPE_ECHO; /* example type */
+
+buf = "hello, world!"
+
+len = sendto(sd, buf, 13, 0,
+ (struct sockaddr *)&addr, sizeof(addr));
+.EE
+.in
+.PP
+Here, the sender owns the message tag; so
+.B MCTP_TAG_OWNER
+is used as the tag data.
+The kernel will allocate a specific tag value for this message.
+If no tag is available,
+.BR sendto (2)
+will return an error, with errno set to
+.BR EBUSY .
This is correct. Please ignore the following comment.
Only for Branden: I just noticed a difference between man-pages(7) and
groff_man(7) advise: groff_man(7) advises to use italics for
preprocessor constants, while man-pages(7) recommends bold:
[
Special macros, which are usually in uppercase, are in
bold (e.g., MAXINT). Exception: don't boldface NULL.
]
I find it better with bold, since that differentiates variables from
constants.
+This allocated tag remains associated with the socket, so that any replies to
+the sent message will be received by the same socket.
+.PP
+Sending a MCTP message requires the
+.B CAP_NET_RAW
+capability.
+.SS Receiving messages
+Messages can be received using the
+.BR recvfrom (2)
+and
+.BR recvmsg (2)
+system calls.
+.PP
+.in +4n
+.EX
+struct sockaddr_mctp addr;
+socklen_t addrlen;
+char buf[13];
+
+addrlen = sizeof(addr);
+
+len = recvfrom(sd, buf, sizeof(buf), 0,
+ (struct sockaddr *)&addr, &addrlen);
+.EE
+.in
+.PP
+In order to receive an incoming message, the receiver will need to either have
+previously sent a message to the same endpoint, or performed a
+.BR bind (2)
+to receive all messages of a certain type:
+.PP
+.in +4n
+.EX
+struct sockaddr_mctp addr;
+
+addr.smctp_family = AF_MCTP;
+addr.smctp_network = MCTP_NET_ANY;
+addr.smctp_addr.s_addr = MCTP_ADDR_ANY;
+addr.smctp_type = MYPROGRAM_MCTP_TYPE_ECHO; /* our 'echo' type */
+
+int rc = bind(sd, (struct sockaddr *)&addr, sizeof(addr));
+.EE
+.in
+.PP
+This call requires the
+.B CAP_NET_BIND_SERVICE
+capability, and will result in the socket receiving all messages sent to
+locally-assigned EIDs, for the specified message type.
+.PP
+After a
+.BR recvfrom (2)
+or
+.BR recvmsg (2)
+returns a success condition, the provided address argument will be populated
+with the sender's network and EID, as well as the tag value used for the
+message.
+Any reply to this message should pass the same address and tag value (with the
+TO bit cleared) to indicate that is is directed to the same remote endpoint.
+.SH SEE ALSO
+.BR socket (7)
+.PP
+The kernel source file
+.IR Documentation/networking/mctp.rst .
+.PP
+The DSP0236 specification, at
+.UR https://www.dmtf.org\:/standards\:/pmci
Please, use \: after /, not before. So '/\:'. I had a discussion with
Branden, and he showed me the merits of that style:
<https://lists.gnu.org/archive/html/groff/2021-10/msg00046.html>
+.UE .
--
Alejandro Colomar
Linux man-pages comaintainer; http://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/
