File: tcp_loopback_PR3718.patch

package info (click to toggle)
pmix 6.0.0%2Breally5.0.9-2
  • links: PTS, VCS
  • area: main
  • in suites: sid
  • size: 11,756 kB
  • sloc: ansic: 125,921; sh: 4,265; python: 2,530; makefile: 2,109; xml: 1,611; perl: 1,151; lex: 138
file content (156 lines) | stat: -rw-r--r-- 6,800 bytes parent folder | download
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
From c8c0c7e922fde83640118bf19fe0c78cb8ce2f79 Mon Sep 17 00:00:00 2001
From: Ralph Castain <rhc@pmix.org>
Date: Sun, 9 Nov 2025 11:02:53 -0700
Subject: [PATCH] Enable use of loopback interface

We used to have support for both Unix sockets and TCP interfaces, so we explicitly
ignored all loopback devices (leaving local operations to the Unix sockets).

However, at some point years back we removed the Unix socket support, transitioning
solely over to TCP interfaces. Nobody remembered the TCP support ignoring loopbacks,
and we always picked up public interfaces - so nobody realized the limitation.

Now that we have a directive to not allow remote connections, translating into wanting
a loopback device, we cannot continue to reject loopbacks. So we need to fix that bug
to enable selection of the loopback.

Signed-off-by: Ralph Castain <rhc@pmix.org>
---
 src/mca/pif/base/pif_base_components.c | 9 +--------
 src/mca/pif/bsdx_ipv4/pif_bsdx.c       | 7 +------
 src/mca/pif/bsdx_ipv6/pif_bsdx_ipv6.c  | 9 +--------
 src/mca/pif/pif.h                      | 3 +--
 src/mca/pif/posix_ipv4/pif_posix.c     | 5 -----
 src/util/pmix_if.h                     | 3 +--
 6 files changed, 5 insertions(+), 31 deletions(-)

Index: pmix/src/mca/pif/base/pif_base_components.c
===================================================================
--- pmix.orig/src/mca/pif/base/pif_base_components.c	2025-11-09 21:45:04.557091406 +0100
+++ pmix/src/mca/pif/base/pif_base_components.c	2025-11-09 21:45:04.550659240 +0100
@@ -3,7 +3,7 @@
  * Copyright (c) 2015-2020 Intel, Inc.  All rights reserved.
  * Copyright (c) 2015-2016 Research Organization for Information Science
  *                         and Technology (RIST). All rights reserved.
- * Copyright (c) 2021-2022 Nanook Consulting.  All rights reserved.
+ * Copyright (c) 2021-2025 Nanook Consulting  All rights reserved.
  * $COPYRIGHT$
  *
  * Additional copyrights may follow
@@ -24,7 +24,6 @@
 /* instantiate the global list of interfaces */
 pmix_list_t pmix_if_list = PMIX_LIST_STATIC_INIT;
 bool pmix_if_do_not_resolve = false;
-bool pmix_if_retain_loopback = false;
 
 static int pmix_pif_base_register(pmix_mca_base_register_flag_t flags);
 static int pmix_pif_base_open(pmix_mca_base_open_flag_t flags);
@@ -49,12 +48,6 @@
                                                 PMIX_MCA_BASE_VAR_TYPE_BOOL,
                                                 &pmix_if_do_not_resolve);
 
-    pmix_if_retain_loopback = false;
-    (void) pmix_mca_base_framework_var_register(&pmix_pif_base_framework, "retain_loopback",
-                                                "If nonzero, retain loopback interfaces",
-                                                PMIX_MCA_BASE_VAR_TYPE_BOOL,
-                                                &pmix_if_retain_loopback);
-
     return PMIX_SUCCESS;
 }
 
Index: pmix/src/mca/pif/bsdx_ipv4/pif_bsdx.c
===================================================================
--- pmix.orig/src/mca/pif/bsdx_ipv4/pif_bsdx.c	2025-11-09 21:45:04.557091406 +0100
+++ pmix/src/mca/pif/bsdx_ipv4/pif_bsdx.c	2025-11-09 21:45:04.550971748 +0100
@@ -125,12 +125,7 @@
             continue;
         }
 
-        /* skip interface if it is a loopback device (IFF_LOOPBACK set) */
-        if (!pmix_if_retain_loopback && 0 != (cur_ifaddrs->ifa_flags & IFF_LOOPBACK)) {
-            continue;
-        }
-
-        /* or if it is a point-to-point interface */
+        /* skip if it is a point-to-point interface */
         /* TODO: do we really skip p2p? */
         if (0 != (cur_ifaddrs->ifa_flags & IFF_POINTOPOINT)) {
             continue;
Index: pmix/src/mca/pif/bsdx_ipv6/pif_bsdx_ipv6.c
===================================================================
--- pmix.orig/src/mca/pif/bsdx_ipv6/pif_bsdx_ipv6.c	2025-11-09 21:45:04.557091406 +0100
+++ pmix/src/mca/pif/bsdx_ipv6/pif_bsdx_ipv6.c	2025-11-09 21:45:04.551238508 +0100
@@ -119,14 +119,7 @@
             continue;
         }
 
-        /* skip interface if it is a loopback device (IFF_LOOPBACK set) */
-        if (!pmix_if_retain_loopback && 0 != (cur_ifaddrs->ifa_flags & IFF_LOOPBACK)) {
-            pmix_output_verbose(1, pmix_pif_base_framework.framework_output,
-                                "skipping loopback interface %s.\n", cur_ifaddrs->ifa_name);
-            continue;
-        }
-
-        /* or if it is a point-to-point interface */
+        /* skip if it is a point-to-point interface */
         /* TODO: do we really skip p2p? */
         if (0 != (cur_ifaddrs->ifa_flags & IFF_POINTOPOINT)) {
             pmix_output_verbose(1, pmix_pif_base_framework.framework_output,
Index: pmix/src/mca/pif/pif.h
===================================================================
--- pmix.orig/src/mca/pif/pif.h	2025-11-09 21:45:04.557091406 +0100
+++ pmix/src/mca/pif/pif.h	2025-11-09 21:45:04.551463280 +0100
@@ -4,7 +4,7 @@
  * Copyright (c) 2015      Los Alamos National Security, LLC. All rights
  *                         reserved.
  * Copyright (c) 2016-2019 Intel, Inc.  All rights reserved.
- * Copyright (c) 2021-2022 Nanook Consulting.  All rights reserved.
+ * Copyright (c) 2021-2025 Nanook Consulting  All rights reserved.
  * $COPYRIGHT$
  *
  * Additional copyrights may follow
@@ -76,7 +76,6 @@
 
 /* global flags */
 extern bool pmix_if_do_not_resolve;
-extern bool pmix_if_retain_loopback;
 
 /**
  * Structure for if components.
Index: pmix/src/mca/pif/posix_ipv4/pif_posix.c
===================================================================
--- pmix.orig/src/mca/pif/posix_ipv4/pif_posix.c	2025-11-09 21:45:04.557091406 +0100
+++ pmix/src/mca/pif/posix_ipv4/pif_posix.c	2025-11-09 21:45:04.551707097 +0100
@@ -226,11 +226,6 @@
             continue;
         }
 #endif
-#if 0
-        if (!pmix_if_retain_loopback && (ifr->ifr_flags & IFF_LOOPBACK) != 0) {
-            continue;
-        }
-#endif
 
         intf = PMIX_NEW(pmix_pif_t);
         if (NULL == intf) {
Index: pmix/src/util/pmix_if.h
===================================================================
--- pmix.orig/src/util/pmix_if.h	2025-11-09 21:45:04.557091406 +0100
+++ pmix/src/util/pmix_if.h	2025-11-09 21:45:04.552020382 +0100
@@ -14,7 +14,7 @@
  * Copyright (c) 2008      Sun Microsystems, Inc.  All rights reserved.
  * Copyright (c) 2013      Cisco Systems, Inc.  All rights reserved.
  * Copyright (c) 2016-2020 Intel, Inc.  All rights reserved.
- * Copyright (c) 2021-2023 Nanook Consulting.  All rights reserved.
+ * Copyright (c) 2021-2025 Nanook Consulting  All rights reserved.
  * $COPYRIGHT$
  *
  * Additional copyrights may follow
@@ -77,7 +77,6 @@
 
 /* global flags */
 PMIX_EXPORT extern bool pmix_if_do_not_resolve;
-PMIX_EXPORT extern bool pmix_if_retain_loopback;
 
 /**
  *  Lookup an interface by address and return its name.