]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
vspawn: prefer "VSOCK" spelling over "vsock" and "VSock"
authorLennart Poettering <lennart@poettering.net>
Wed, 21 Feb 2024 15:18:24 +0000 (16:18 +0100)
committerLennart Poettering <lennart@poettering.net>
Wed, 21 Feb 2024 15:18:24 +0000 (16:18 +0100)
The vsock(7) man page appears to prefer "VSOCK" spelling for the
concept, let's also use it as much as we can.

man/systemd-vmspawn.xml
src/vmspawn/vmspawn.c

index b618999b8ea679000fba81cb3dcde7443c252e56..c11809bc5a64d3b087c0ac67cc0317e5f0acda43 100644 (file)
@@ -42,7 +42,8 @@
     file descriptors must be passed with the names <literal>kvm</literal> and <literal>vhost-vsock</literal>
     respectively.</para>
 
-    <para>Note: on Ubuntu/Debian derivatives systemd-vmspawn requires the user to be in the <literal>kvm</literal> group to use the VSock options.</para>
+    <para>Note: on Ubuntu/Debian derivatives systemd-vmspawn requires the user to be in the
+    <literal>kvm</literal> group to use the VSOCK options.</para>
   </refsect1>
 
   <refsect1>
           <term><option>--qemu-vsock=</option><replaceable>BOOL</replaceable></term>
 
           <listitem>
-            <para>Configure whether to use VSock networking.</para>
+            <para>Configure whether to use VSOCK networking.</para>
 
-            <para>If the option is not specified VSock support will be detected automatically.
-            If yes is specified VSocks are always used, and vice versa if no is set VSocks are never used.</para>
+            <para>If the option is not specified VSOCK support will be detected automatically.  If yes is
+            specified VSOCK is always used, and vice versa if no is set VSOCK are never used.</para>
             <xi:include href="version-info.xml" xpointer="v255"/>
           </listitem>
         </varlistentry>
index dd72f240024cd9d0a5b3e5df3c0d825978ed0a70..925b56abfe8545d2a9940c79f1c7dfb61af58636 100644 (file)
@@ -467,11 +467,11 @@ static int open_vsock(void) {
 
         r = bind(vsock_fd, &bind_addr.sa, sizeof(bind_addr.vm));
         if (r < 0)
-                return log_error_errno(errno, "Failed to bind to vsock to address %u:%u: %m", bind_addr.vm.svm_cid, bind_addr.vm.svm_port);
+                return log_error_errno(errno, "Failed to bind to VSOCK address %u:%u: %m", bind_addr.vm.svm_cid, bind_addr.vm.svm_port);
 
         r = listen(vsock_fd, SOMAXCONN_DELUXE);
         if (r < 0)
-                return log_error_errno(errno, "Failed to listen on vsock: %m");
+                return log_error_errno(errno, "Failed to listen on VSOCK: %m");
 
         return TAKE_FD(vsock_fd);
 }
@@ -545,13 +545,13 @@ static int vmspawn_dispatch_vsock_connections(sd_event_source *source, int fd, u
         assert(userdata);
 
         if (revents != EPOLLIN) {
-                log_warning("Got unexpected poll event for vsock fd.");
+                log_warning("Got unexpected poll event for VSOCK fd.");
                 return 0;
         }
 
         conn_fd = accept4(fd, NULL, NULL, SOCK_CLOEXEC|SOCK_NONBLOCK);
         if (conn_fd < 0) {
-                log_warning_errno(errno, "Failed to accept connection from vsock fd (%m), ignoring...");
+                log_warning_errno(errno, "Failed to accept connection from VSOCK fd (%m), ignoring...");
                 return 0;
         }
 
@@ -1134,7 +1134,7 @@ static int run_virtual_machine(int kvm_device_fd, int vhost_device_fd) {
         if (arg_qemu_vsock < 0) {
                 r = qemu_check_vsock_support();
                 if (r < 0)
-                        return log_error_errno(r, "Failed to check for VSock support: %m");
+                        return log_error_errno(r, "Failed to check for VSOCK support: %m");
 
                 use_vsock = r;
         }
@@ -1185,7 +1185,7 @@ static int run_virtual_machine(int kvm_device_fd, int vhost_device_fd) {
 
                 r = vsock_fix_child_cid(device_fd, &child_cid, arg_machine);
                 if (r < 0)
-                        return log_error_errno(r, "Failed to fix CID for the guest vsock socket: %m");
+                        return log_error_errno(r, "Failed to fix CID for the guest VSOCK socket: %m");
 
                 r = strv_extend(&cmdline, "-device");
                 if (r < 0)
@@ -1549,13 +1549,13 @@ static int run_virtual_machine(int kvm_device_fd, int vhost_device_fd) {
         if (use_vsock) {
                 notify_sock_fd = open_vsock();
                 if (notify_sock_fd < 0)
-                        return log_error_errno(notify_sock_fd, "Failed to open vsock: %m");
+                        return log_error_errno(notify_sock_fd, "Failed to open VSOCK: %m");
 
                 r = cmdline_add_vsock(&cmdline, notify_sock_fd);
                 if (r == -ENOMEM)
                         return log_oom();
                 if (r < 0)
-                        return log_error_errno(r, "Failed to call getsockname on vsock: %m");
+                        return log_error_errno(r, "Failed to call getsockname on VSOCK: %m");
         }
 
         if (DEBUG_LOGGING) {
@@ -1606,7 +1606,7 @@ static int run_virtual_machine(int kvm_device_fd, int vhost_device_fd) {
         if (use_vsock) {
                 r = setup_notify_parent(event, notify_sock_fd, &exit_status, &notify_event_source);
                 if (r < 0)
-                        return log_error_errno(r, "Failed to setup event loop to handle vsock notify events: %m");
+                        return log_error_errno(r, "Failed to setup event loop to handle VSOCK notify events: %m");
         }
 
         /* shutdown qemu when we are shutdown */
@@ -1624,7 +1624,7 @@ static int run_virtual_machine(int kvm_device_fd, int vhost_device_fd) {
 
         if (use_vsock) {
                 if (exit_status == INT_MAX) {
-                        log_debug("Couldn't retrieve inner EXIT_STATUS from vsock");
+                        log_debug("Couldn't retrieve inner EXIT_STATUS from VSOCK");
                         return EXIT_SUCCESS;
                 }
                 if (exit_status != 0)