Make netlink optional

src/CMakeLists.txt

@@ -179,15 +179,15 @@ if (${WITH_FIO})
 endif (${WITH_FIO})
 
 # vitastor-nbd
-pkg_check_modules(NL3 REQUIRED libnl-3.0 libnl-genl-3.0)
+pkg_check_modules(NL3 libnl-3.0 libnl-genl-3.0)
-add_executable(vitastor-nbd   
+add_executable(vitastor-nbd
-	nbd_proxy.cpp 
+	nbd_proxy.cpp
 )
 target_include_directories(vitastor-nbd PUBLIC ${NL3_INCLUDE_DIRS})
 target_link_libraries(vitastor-nbd vitastor_client ${NL3_LIBRARIES})
-if(HAVE_NBD_NETLINK_H)
+if (HAVE_NBD_NETLINK_H AND "${NL3_LIBRARIES}")
 	target_compile_definitions(vitastor-nbd PUBLIC HAVE_NBD_NETLINK_H)
-endif()
+endif (HAVE_NBD_NETLINK_H AND "${NL3_LIBRARIES}")
 
 # vitastor-nfs
 add_executable(vitastor-nfs

src/nbd_proxy.cpp

@@ -23,6 +23,7 @@
 #include "cluster_client.h"
 #include "epoll_manager.h"
 
+#ifdef HAVE_NBD_NETLINK_H
 #include <netlink/attr.h>
 #include <netlink/genl/ctrl.h>
 #include <netlink/genl/genl.h>
@@ -31,9 +32,6 @@
 #include <netlink/netlink.h>
 #include <netlink/socket.h>
 #include <netlink/errno.h>
-
-#ifdef HAVE_NBD_NETLINK_H
-
 #include <linux/nbd-netlink.h>
 
 #define fail(...) { fprintf(stderr, __VA_ARGS__); exit(1); }
@@ -226,22 +224,6 @@ nla_put_failure:
 
 #undef fail
 
-#else
-
-static int netlink_configure(const int *sockfd, int sock_size, int dev_num, uint64_t size,
-    uint64_t blocksize, uint64_t flags, uint64_t cflags, uint64_t timeout, uint64_t conn_timeout)
-{
-    fprintf(stderr, "netlink is not supported\n");
-    exit(1);
-    return 0;
-}
-
-static void netlink_disconnect(uint32_t dev_num)
-{
-    fprintf(stderr, "netlink is not supported\n");
-    exit(1);
-}
-
 #endif
 
 #ifndef MSG_ZEROCOPY
@@ -339,14 +321,18 @@ public:
                 fprintf(stderr, "device name or number is missing\n");
                 exit(1);
             }
-
             if (cfg["netlink"].is_null())
             {
                 unmap(cfg["dev_num"].uint64_value());
             }
             else
             {
+#ifdef HAVE_NBD_NETLINK_H
                 netlink_disconnect(cfg["dev_num"].uint64_value());
+#else
+                fprintf(stderr, "netlink support is disabled in this build\n");
+                exit(1);
+#endif
             }
         }
         else if (cfg["command"] == "ls" || cfg["command"] == "list" || cfg["command"] == "list-mapped")
@@ -518,48 +504,38 @@ public:
 
         if (!cfg["netlink"].is_null())
         {
+#ifdef HAVE_NBD_NETLINK_H
             int devnum = -1;
             if (!cfg["dev_num"].is_null())
             {
                 devnum = (int)cfg["dev_num"].uint64_value();
             }
-
             uint64_t flags = NBD_FLAG_SEND_FLUSH;
             uint64_t cflags = 0;
-
+#ifdef NBD_FLAG_READ_ONLY
-            #ifdef NBD_FLAG_READ_ONLY
             if (!cfg["nbd_ro"].is_null())
-            {
                 flags |= NBD_FLAG_READ_ONLY;
-            }
+#endif
-            #endif
+#ifdef NBD_CFLAG_DESTROY_ON_DISCONNECT
-
-            #ifdef NBD_CFLAG_DESTROY_ON_DISCONNECT
             if (!cfg["nbd_destroy_on_disconnect"].is_null())
-            {
                 cflags |= NBD_CFLAG_DESTROY_ON_DISCONNECT;
-            }
+#endif
-            #endif
+#ifdef NBD_CFLAG_DISCONNECT_ON_CLOSE
-
-            #ifdef NBD_CFLAG_DISCONNECT_ON_CLOSE
             if (!cfg["nbd_disconnect_on_close"].is_null())
-            {
                 cflags |= NBD_CFLAG_DISCONNECT_ON_CLOSE;
-            }
+#endif
-            #endif
-
             int err = netlink_configure(sockfd + 1, 1, devnum, device_size, 4096, flags, cflags, nbd_timeout, nbd_lease);
             if (err < 0)
             {
-                if (err == -NLE_BUSY)
+                errno = (err == -NLE_BUSY ? EBUSY : EIO);
-                    errno = EBUSY;
+                fprintf(stderr, "netlink_configure failed: %s (code %d)\n", nl_geterror(err), err);
-                else
-                    errno = EIO;
-                perror("netlink_configure");
                 exit(1);
             }
-
             printf("/dev/nbd%d\n", err);
+#else
+            fprintf(stderr, "netlink support is disabled in this build\n");
+            exit(1);
+#endif
         }
         else
         {