|
 |
9c84ec |
From 5a55ec38edd875fc6dc54c0483e1f96ad9cf8cf9 Mon Sep 17 00:00:00 2001
|
|
|
9c84ec |
Message-Id: <5a55ec38edd875fc6dc54c0483e1f96ad9cf8cf9.1488376602.git.dcaratti@redhat.com>
|
|
|
9c84ec |
From: Sabrina Dubroca <sd@queasysnail.net>
|
|
|
9c84ec |
Date: Sun, 27 Nov 2016 20:08:49 +0100
|
|
|
9c84ec |
Subject: [PATCH] drivers: Move driver_wired_get_ifstatus() to a common file
|
|
|
9c84ec |
|
|
|
9c84ec |
This continues refactoring of the common parts of wired drivers code
|
|
|
9c84ec |
into a shared file, so that they can be reused by other drivers.
|
|
|
9c84ec |
|
|
|
9c84ec |
Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
|
|
|
9c84ec |
---
|
|
|
9c84ec |
src/drivers/driver_macsec_qca.c | 31 +------------------------------
|
|
|
9c84ec |
src/drivers/driver_wired.c | 31 +------------------------------
|
|
|
9c84ec |
src/drivers/driver_wired_common.c | 29 +++++++++++++++++++++++++++++
|
|
|
9c84ec |
src/drivers/driver_wired_common.h | 1 +
|
|
|
9c84ec |
4 files changed, 32 insertions(+), 60 deletions(-)
|
|
|
9c84ec |
|
|
|
9c84ec |
diff --git a/src/drivers/driver_macsec_qca.c b/src/drivers/driver_macsec_qca.c
|
|
|
9c84ec |
index 31cb0dc..786e2e8 100644
|
|
|
9c84ec |
--- a/src/drivers/driver_macsec_qca.c
|
|
|
9c84ec |
+++ b/src/drivers/driver_macsec_qca.c
|
|
|
9c84ec |
@@ -99,35 +99,6 @@ static int macsec_qca_get_capa(void *priv, struct wpa_driver_capa *capa)
|
|
|
9c84ec |
}
|
|
|
9c84ec |
|
|
|
9c84ec |
|
|
|
9c84ec |
-#if defined(__FreeBSD__) || defined(__DragonFly__) || defined(__FreeBSD_kernel__)
|
|
|
9c84ec |
-static int macsec_qca_get_ifstatus(const char *ifname, int *status)
|
|
|
9c84ec |
-{
|
|
|
9c84ec |
- struct ifmediareq ifmr;
|
|
|
9c84ec |
- int s;
|
|
|
9c84ec |
-
|
|
|
9c84ec |
- s = socket(PF_INET, SOCK_DGRAM, 0);
|
|
|
9c84ec |
- if (s < 0) {
|
|
|
9c84ec |
- wpa_print(MSG_ERROR, "socket: %s", strerror(errno));
|
|
|
9c84ec |
- return -1;
|
|
|
9c84ec |
- }
|
|
|
9c84ec |
-
|
|
|
9c84ec |
- os_memset(&ifmr, 0, sizeof(ifmr));
|
|
|
9c84ec |
- os_strlcpy(ifmr.ifm_name, ifname, IFNAMSIZ);
|
|
|
9c84ec |
- if (ioctl(s, SIOCGIFMEDIA, (caddr_t) &ifmr) < 0) {
|
|
|
9c84ec |
- wpa_printf(MSG_ERROR, "ioctl[SIOCGIFMEDIA]: %s",
|
|
|
9c84ec |
- strerror(errno));
|
|
|
9c84ec |
- close(s);
|
|
|
9c84ec |
- return -1;
|
|
|
9c84ec |
- }
|
|
|
9c84ec |
- close(s);
|
|
|
9c84ec |
- *status = (ifmr.ifm_status & (IFM_ACTIVE | IFM_AVALID)) ==
|
|
|
9c84ec |
- (IFM_ACTIVE | IFM_AVALID);
|
|
|
9c84ec |
-
|
|
|
9c84ec |
- return 0;
|
|
|
9c84ec |
-}
|
|
|
9c84ec |
-#endif /* defined(__FreeBSD__) || defined(__DragonFly__) || defined(FreeBSD_kernel__) */
|
|
|
9c84ec |
-
|
|
|
9c84ec |
-
|
|
|
9c84ec |
static void __macsec_drv_init(struct macsec_qca_data *drv)
|
|
|
9c84ec |
{
|
|
|
9c84ec |
int ret = 0;
|
|
|
9c84ec |
@@ -243,7 +214,7 @@ static void * macsec_qca_init(void *ctx, const char *ifname)
|
|
|
9c84ec |
int status;
|
|
|
9c84ec |
wpa_printf(MSG_DEBUG, "%s: waiting for link to become active",
|
|
|
9c84ec |
__func__);
|
|
|
9c84ec |
- while (macsec_qca_get_ifstatus(ifname, &status) == 0 &&
|
|
|
9c84ec |
+ while (driver_wired_get_ifstatus(ifname, &status) == 0 &&
|
|
|
9c84ec |
status == 0)
|
|
|
9c84ec |
sleep(1);
|
|
|
9c84ec |
}
|
|
|
9c84ec |
diff --git a/src/drivers/driver_wired.c b/src/drivers/driver_wired.c
|
|
|
9c84ec |
index 953fa3d..db83683 100644
|
|
|
9c84ec |
--- a/src/drivers/driver_wired.c
|
|
|
9c84ec |
+++ b/src/drivers/driver_wired.c
|
|
|
9c84ec |
@@ -390,35 +390,6 @@ static int wpa_driver_wired_get_capa(void *priv, struct wpa_driver_capa *capa)
|
|
|
9c84ec |
}
|
|
|
9c84ec |
|
|
|
9c84ec |
|
|
|
9c84ec |
-#if defined(__FreeBSD__) || defined(__DragonFly__) || defined(__FreeBSD_kernel__)
|
|
|
9c84ec |
-static int wpa_driver_wired_get_ifstatus(const char *ifname, int *status)
|
|
|
9c84ec |
-{
|
|
|
9c84ec |
- struct ifmediareq ifmr;
|
|
|
9c84ec |
- int s;
|
|
|
9c84ec |
-
|
|
|
9c84ec |
- s = socket(PF_INET, SOCK_DGRAM, 0);
|
|
|
9c84ec |
- if (s < 0) {
|
|
|
9c84ec |
- wpa_printf(MSG_ERROR, "socket: %s", strerror(errno));
|
|
|
9c84ec |
- return -1;
|
|
|
9c84ec |
- }
|
|
|
9c84ec |
-
|
|
|
9c84ec |
- os_memset(&ifmr, 0, sizeof(ifmr));
|
|
|
9c84ec |
- os_strlcpy(ifmr.ifm_name, ifname, IFNAMSIZ);
|
|
|
9c84ec |
- if (ioctl(s, SIOCGIFMEDIA, (caddr_t) &ifmr) < 0) {
|
|
|
9c84ec |
- wpa_printf(MSG_ERROR, "ioctl[SIOCGIFMEDIA]: %s",
|
|
|
9c84ec |
- strerror(errno));
|
|
|
9c84ec |
- close(s);
|
|
|
9c84ec |
- return -1;
|
|
|
9c84ec |
- }
|
|
|
9c84ec |
- close(s);
|
|
|
9c84ec |
- *status = (ifmr.ifm_status & (IFM_ACTIVE | IFM_AVALID)) ==
|
|
|
9c84ec |
- (IFM_ACTIVE | IFM_AVALID);
|
|
|
9c84ec |
-
|
|
|
9c84ec |
- return 0;
|
|
|
9c84ec |
-}
|
|
|
9c84ec |
-#endif /* defined(__FreeBSD__) || defined(__DragonFly__) || defined(FreeBSD_kernel__) */
|
|
|
9c84ec |
-
|
|
|
9c84ec |
-
|
|
|
9c84ec |
static void * wpa_driver_wired_init(void *ctx, const char *ifname)
|
|
|
9c84ec |
{
|
|
|
9c84ec |
struct wpa_driver_wired_data *drv;
|
|
|
9c84ec |
@@ -477,7 +448,7 @@ static void * wpa_driver_wired_init(void *ctx, const char *ifname)
|
|
|
9c84ec |
int status;
|
|
|
9c84ec |
wpa_printf(MSG_DEBUG, "%s: waiting for link to become active",
|
|
|
9c84ec |
__func__);
|
|
|
9c84ec |
- while (wpa_driver_wired_get_ifstatus(ifname, &status) == 0 &&
|
|
|
9c84ec |
+ while (driver_wired_get_ifstatus(ifname, &status) == 0 &&
|
|
|
9c84ec |
status == 0)
|
|
|
9c84ec |
sleep(1);
|
|
|
9c84ec |
}
|
|
|
9c84ec |
diff --git a/src/drivers/driver_wired_common.c b/src/drivers/driver_wired_common.c
|
|
|
9c84ec |
index 52f22de..e55e2c7 100644
|
|
|
9c84ec |
--- a/src/drivers/driver_wired_common.c
|
|
|
9c84ec |
+++ b/src/drivers/driver_wired_common.c
|
|
|
9c84ec |
@@ -162,3 +162,32 @@ int wired_multicast_membership(int sock, int ifindex, const u8 *addr, int add)
|
|
|
9c84ec |
return -1;
|
|
|
9c84ec |
#endif /* __linux__ */
|
|
|
9c84ec |
}
|
|
|
9c84ec |
+
|
|
|
9c84ec |
+
|
|
|
9c84ec |
+#if defined(__FreeBSD__) || defined(__DragonFly__) || defined(__FreeBSD_kernel__)
|
|
|
9c84ec |
+int driver_wired_get_ifstatus(const char *ifname, int *status)
|
|
|
9c84ec |
+{
|
|
|
9c84ec |
+ struct ifmediareq ifmr;
|
|
|
9c84ec |
+ int s;
|
|
|
9c84ec |
+
|
|
|
9c84ec |
+ s = socket(PF_INET, SOCK_DGRAM, 0);
|
|
|
9c84ec |
+ if (s < 0) {
|
|
|
9c84ec |
+ wpa_printf(MSG_ERROR, "socket: %s", strerror(errno));
|
|
|
9c84ec |
+ return -1;
|
|
|
9c84ec |
+ }
|
|
|
9c84ec |
+
|
|
|
9c84ec |
+ os_memset(&ifmr, 0, sizeof(ifmr));
|
|
|
9c84ec |
+ os_strlcpy(ifmr.ifm_name, ifname, IFNAMSIZ);
|
|
|
9c84ec |
+ if (ioctl(s, SIOCGIFMEDIA, (caddr_t) &ifmr) < 0) {
|
|
|
9c84ec |
+ wpa_printf(MSG_ERROR, "ioctl[SIOCGIFMEDIA]: %s",
|
|
|
9c84ec |
+ strerror(errno));
|
|
|
9c84ec |
+ close(s);
|
|
|
9c84ec |
+ return -1;
|
|
|
9c84ec |
+ }
|
|
|
9c84ec |
+ close(s);
|
|
|
9c84ec |
+ *status = (ifmr.ifm_status & (IFM_ACTIVE | IFM_AVALID)) ==
|
|
|
9c84ec |
+ (IFM_ACTIVE | IFM_AVALID);
|
|
|
9c84ec |
+
|
|
|
9c84ec |
+ return 0;
|
|
|
9c84ec |
+}
|
|
|
9c84ec |
+#endif /* defined(__FreeBSD__) || defined(__DragonFly__) || defined(FreeBSD_kernel__) */
|
|
|
9c84ec |
diff --git a/src/drivers/driver_wired_common.h b/src/drivers/driver_wired_common.h
|
|
|
9c84ec |
index e2d8bbe..c8e347a 100644
|
|
|
9c84ec |
--- a/src/drivers/driver_wired_common.h
|
|
|
9c84ec |
+++ b/src/drivers/driver_wired_common.h
|
|
|
9c84ec |
@@ -26,5 +26,6 @@ int driver_wired_get_ifflags(const char *ifname, int *flags);
|
|
|
9c84ec |
int driver_wired_set_ifflags(const char *ifname, int flags);
|
|
|
9c84ec |
int driver_wired_multi(const char *ifname, const u8 *addr, int add);
|
|
|
9c84ec |
int wired_multicast_membership(int sock, int ifindex, const u8 *addr, int add);
|
|
|
9c84ec |
+int driver_wired_get_ifstatus(const char *ifname, int *status);
|
|
|
9c84ec |
|
|
|
9c84ec |
#endif /* DRIVER_WIRED_COMMON_H */
|
|
|
9c84ec |
--
|
|
|
9c84ec |
2.7.4
|
|
|
9c84ec |
|