From: "Pedro André" <PEDA@bang-olufsen.dk>
To: "iwd@lists.linux.dev" <iwd@lists.linux.dev>
Cc: "Pedro André" <PEDA@bang-olufsen.dk>
Subject: [PATCH] add AllowRoaming station property
Date: Wed, 18 Oct 2023 12:27:26 +0000 [thread overview]
Message-ID: <20231018122534.33455-1-peda@bang-olufsen.dk> (raw)
From: Pedro Andre <peda@bang-olufsen.dk>
This adds an AllowRoaming property to the station struct that makes it
possible to indicate to the station/interface that it should not go
into roaming state (when set to false). This property defaults to true
(i.e. it defaults to the normal iwd behaviour that allows roaming).
---
src/station.c | 40 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/src/station.c b/src/station.c
index 065687d..00254b7 100644
--- a/src/station.c
+++ b/src/station.c
@@ -130,6 +130,7 @@ struct station {
bool autoconnect : 1;
bool autoconnect_can_start : 1;
bool netconfig_after_roam : 1;
+ bool allow_roaming : 1;
};
struct anqp_entry {
@@ -2754,6 +2755,9 @@ static bool station_cannot_roam(struct station *station)
const struct l_settings *config = iwd_get_config();
bool disabled;
+ if (!station->allow_roaming)
+ return true;
+
/*
* Disable roaming with hardware that can roam automatically. Note this
* is now required for recent kernels which have CQM event support on
@@ -4155,6 +4159,38 @@ static bool station_property_get_state(struct l_dbus *dbus,
return true;
}
+static bool station_property_get_allow_roaming(struct l_dbus *dbus,
+ struct l_dbus_message *message,
+ struct l_dbus_message_builder *builder,
+ void *user_data)
+{
+ struct station *station = user_data;
+ bool roaming = station->allow_roaming;
+
+ l_dbus_message_builder_append_basic(builder, 'b', &roaming);
+ return true;
+}
+
+static struct l_dbus_message *station_property_set_allow_roaming(
+ struct l_dbus *dbus,
+ struct l_dbus_message *message,
+ struct l_dbus_message_iter *new_value,
+ l_dbus_property_complete_cb_t complete,
+ void *user_data)
+{
+ struct station *station = user_data;
+ bool roaming;
+
+ if (!l_dbus_message_iter_get_variant(new_value, "b", &roaming))
+ return dbus_error_invalid_args(message);
+
+ l_debug("Setting allow_roaming %s", roaming ? "true" : "false");
+
+ station->allow_roaming = roaming;
+
+ return l_dbus_message_new_method_return(message);
+}
+
void station_foreach(station_foreach_func_t func, void *user_data)
{
const struct l_queue_entry *entry;
@@ -4358,6 +4394,8 @@ static struct station *station_create(struct netdev *netdev)
station->roam_bss_list = l_queue_new();
+ station->allow_roaming = true;
+
return station;
}
@@ -4484,6 +4522,8 @@ static void station_setup_interface(struct l_dbus_interface *interface)
station_property_get_scanning, NULL);
l_dbus_interface_property(interface, "State", 0, "s",
station_property_get_state, NULL);
+ l_dbus_interface_property(interface, "AllowRoaming", 0, "b",
+ station_property_get_allow_roaming, station_property_set_allow_roaming);
}
static void station_destroy_interface(void *user_data)
--
2.39.2
next reply other threads:[~2023-10-18 12:27 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-18 12:27 Pedro André [this message]
2023-10-18 12:51 ` [PATCH] add AllowRoaming station property James Prestwood
2023-10-18 14:32 ` Denis Kenzior
2023-10-18 14:56 ` James Prestwood
2023-10-18 14:20 ` Denis Kenzior
2023-10-19 11:55 ` Pedro André
2023-10-20 14:57 ` Denis Kenzior
2023-10-20 16:29 ` James Prestwood
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20231018122534.33455-1-peda@bang-olufsen.dk \
--to=peda@bang-olufsen.dk \
--cc=iwd@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).