2012-02-15 17:31:37 -06:00
|
|
|
/* -*- Mode: C; tab-width: 4; indent-tabs-mode: t; c-basic-offset: 4 -*- */
|
|
|
|
|
/* NetworkManager -- Network link manager
|
|
|
|
|
*
|
|
|
|
|
* This program is free software; you can redistribute it and/or modify
|
|
|
|
|
* it under the terms of the GNU General Public License as published by
|
|
|
|
|
* the Free Software Foundation; either version 2 of the License, or
|
|
|
|
|
* (at your option) any later version.
|
|
|
|
|
*
|
|
|
|
|
* This program is distributed in the hope that it will be useful,
|
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
|
* GNU General Public License for more details.
|
|
|
|
|
*
|
|
|
|
|
* You should have received a copy of the GNU General Public License along
|
|
|
|
|
* with this program; if not, write to the Free Software Foundation, Inc.,
|
|
|
|
|
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
|
|
|
|
|
*
|
2016-07-05 22:24:47 +02:00
|
|
|
* Copyright 2011 - 2016 Red Hat, Inc.
|
2012-02-15 17:31:37 -06:00
|
|
|
*/
|
|
|
|
|
|
2016-02-19 14:57:48 +01:00
|
|
|
#include "nm-default.h"
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2016-09-29 13:49:01 +02:00
|
|
|
#include "nm-device-bond.h"
|
|
|
|
|
|
2013-10-01 18:27:25 +02:00
|
|
|
#include <errno.h>
|
|
|
|
|
#include <stdlib.h>
|
2012-02-15 17:31:37 -06:00
|
|
|
|
|
|
|
|
#include "NetworkManagerUtils.h"
|
|
|
|
|
#include "nm-device-private.h"
|
2016-11-21 00:43:52 +01:00
|
|
|
#include "platform/nm-platform.h"
|
2014-09-08 10:15:02 -05:00
|
|
|
#include "nm-device-factory.h"
|
2014-10-21 22:09:52 -04:00
|
|
|
#include "nm-core-internal.h"
|
2015-01-06 19:09:46 +01:00
|
|
|
#include "nm-ip4-config.h"
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2014-08-02 15:14:26 +02:00
|
|
|
#include "nm-device-logging.h"
|
|
|
|
|
_LOG_DECLARE_SELF(NMDeviceBond);
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2016-09-29 13:49:01 +02:00
|
|
|
/*****************************************************************************/
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2016-09-29 13:49:01 +02:00
|
|
|
struct _NMDeviceBond {
|
|
|
|
|
NMDevice parent;
|
|
|
|
|
};
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2016-09-29 13:49:01 +02:00
|
|
|
struct _NMDeviceBondClass {
|
|
|
|
|
NMDeviceClass parent;
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
G_DEFINE_TYPE (NMDeviceBond, nm_device_bond, NM_TYPE_DEVICE)
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2016-10-02 18:22:50 +02:00
|
|
|
/*****************************************************************************/
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2015-04-17 15:15:38 +02:00
|
|
|
static NMDeviceCapabilities
|
2012-09-27 12:12:15 -04:00
|
|
|
get_generic_capabilities (NMDevice *dev)
|
2012-02-15 17:31:37 -06:00
|
|
|
{
|
2014-09-24 17:46:15 -05:00
|
|
|
return NM_DEVICE_CAP_CARRIER_DETECT | NM_DEVICE_CAP_IS_SOFTWARE;
|
2012-02-15 17:31:37 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static gboolean
|
2014-05-30 13:44:53 -05:00
|
|
|
check_connection_compatible (NMDevice *device, NMConnection *connection)
|
2012-02-15 17:31:37 -06:00
|
|
|
{
|
|
|
|
|
NMSettingBond *s_bond;
|
|
|
|
|
|
2014-05-30 13:44:53 -05:00
|
|
|
if (!NM_DEVICE_CLASS (nm_device_bond_parent_class)->check_connection_compatible (device, connection))
|
2013-03-07 07:44:36 -05:00
|
|
|
return FALSE;
|
|
|
|
|
|
2012-02-15 17:31:37 -06:00
|
|
|
s_bond = nm_connection_get_setting_bond (connection);
|
2014-05-30 13:44:53 -05:00
|
|
|
if (!s_bond || !nm_connection_is_type (connection, NM_SETTING_BOND_SETTING_NAME))
|
2012-02-15 17:31:37 -06:00
|
|
|
return FALSE;
|
|
|
|
|
|
|
|
|
|
/* FIXME: match bond properties like mode, etc? */
|
|
|
|
|
|
|
|
|
|
return TRUE;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static gboolean
|
2012-09-27 12:12:15 -04:00
|
|
|
complete_connection (NMDevice *device,
|
|
|
|
|
NMConnection *connection,
|
|
|
|
|
const char *specific_object,
|
|
|
|
|
const GSList *existing_connections,
|
|
|
|
|
GError **error)
|
2012-02-15 17:31:37 -06:00
|
|
|
{
|
2014-08-05 14:13:42 -04:00
|
|
|
NMSettingBond *s_bond;
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2017-04-18 12:09:02 +02:00
|
|
|
nm_utils_complete_generic (nm_device_get_platform (device),
|
2016-03-08 13:57:20 +01:00
|
|
|
connection,
|
2012-02-15 17:31:37 -06:00
|
|
|
NM_SETTING_BOND_SETTING_NAME,
|
|
|
|
|
existing_connections,
|
|
|
|
|
NULL,
|
2014-08-25 16:21:59 +02:00
|
|
|
_("Bond connection"),
|
2014-08-05 17:11:57 -04:00
|
|
|
"bond",
|
2012-02-15 17:31:37 -06:00
|
|
|
TRUE);
|
|
|
|
|
|
|
|
|
|
s_bond = nm_connection_get_setting_bond (connection);
|
|
|
|
|
if (!s_bond) {
|
|
|
|
|
s_bond = (NMSettingBond *) nm_setting_bond_new ();
|
|
|
|
|
nm_connection_add_setting (connection, NM_SETTING (s_bond));
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
return TRUE;
|
|
|
|
|
}
|
|
|
|
|
|
2016-10-02 18:22:50 +02:00
|
|
|
/*****************************************************************************/
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2013-08-28 10:38:46 +02:00
|
|
|
static gboolean
|
2016-07-04 16:25:39 +02:00
|
|
|
set_bond_attr (NMDevice *device, NMBondMode mode, const char *attr, const char *value)
|
2013-08-28 10:38:46 +02:00
|
|
|
{
|
2014-08-02 15:14:26 +02:00
|
|
|
NMDeviceBond *self = NM_DEVICE_BOND (device);
|
2013-10-01 18:27:25 +02:00
|
|
|
gboolean ret;
|
2013-10-01 20:15:03 +02:00
|
|
|
int ifindex = nm_device_get_ifindex (device);
|
2013-10-01 18:27:25 +02:00
|
|
|
|
2016-07-04 16:25:39 +02:00
|
|
|
if (!_nm_setting_bond_option_supported (attr, mode))
|
|
|
|
|
return FALSE;
|
|
|
|
|
|
2017-04-18 12:09:02 +02:00
|
|
|
ret = nm_platform_sysctl_master_set_option (nm_device_get_platform (device), ifindex, attr, value);
|
2014-08-02 15:14:26 +02:00
|
|
|
if (!ret)
|
2016-10-06 21:28:40 +02:00
|
|
|
_LOGW (LOGD_PLATFORM, "failed to set bonding attribute '%s' to '%s'", attr, value);
|
2013-10-01 18:27:25 +02:00
|
|
|
return ret;
|
2013-08-28 10:38:46 +02:00
|
|
|
}
|
|
|
|
|
|
2013-08-28 11:47:44 +02:00
|
|
|
static gboolean
|
2017-06-21 11:19:03 +02:00
|
|
|
ignore_option (NMSettingBond *s_bond, const char *option, const char *value)
|
2013-08-28 11:47:44 +02:00
|
|
|
{
|
2017-06-21 11:19:03 +02:00
|
|
|
const char *defvalue;
|
|
|
|
|
|
|
|
|
|
if (nm_streq0 (option, NM_SETTING_BOND_OPTION_MIIMON)) {
|
|
|
|
|
/* The default value for miimon, when missing in the setting, is
|
|
|
|
|
* 0 if arp_interval is != 0, and 100 otherwise. So, let's ignore
|
|
|
|
|
* miimon=0 (which means that miimon is disabled) and accept any
|
|
|
|
|
* other value. Adding miimon=100 does not cause any harm.
|
|
|
|
|
*/
|
|
|
|
|
defvalue = "0";
|
|
|
|
|
} else
|
|
|
|
|
defvalue = nm_setting_bond_get_option_default (s_bond, option);
|
|
|
|
|
|
|
|
|
|
return nm_streq0 (value, defvalue);
|
2013-08-28 11:47:44 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static void
|
|
|
|
|
update_connection (NMDevice *device, NMConnection *connection)
|
|
|
|
|
{
|
|
|
|
|
NMSettingBond *s_bond = nm_connection_get_setting_bond (connection);
|
|
|
|
|
int ifindex = nm_device_get_ifindex (device);
|
2017-06-05 14:48:08 +02:00
|
|
|
NMBondMode mode = NM_BOND_MODE_UNKNOWN;
|
2013-08-28 11:47:44 +02:00
|
|
|
const char **options;
|
|
|
|
|
|
|
|
|
|
if (!s_bond) {
|
|
|
|
|
s_bond = (NMSettingBond *) nm_setting_bond_new ();
|
|
|
|
|
nm_connection_add_setting (connection, (NMSetting *) s_bond);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/* Read bond options from sysfs and update the Bond setting to match */
|
|
|
|
|
options = nm_setting_bond_get_valid_options (s_bond);
|
2017-06-05 14:48:08 +02:00
|
|
|
for (; *options; options++) {
|
2017-04-18 12:09:02 +02:00
|
|
|
gs_free char *value = nm_platform_sysctl_master_get_option (nm_device_get_platform (device), ifindex, *options);
|
2016-03-16 11:22:07 +01:00
|
|
|
char *p;
|
2013-08-28 11:47:44 +02:00
|
|
|
|
2017-02-07 11:05:01 +01:00
|
|
|
if ( value
|
|
|
|
|
&& _nm_setting_bond_get_option_type (s_bond, *options) == NM_BOND_OPTION_TYPE_BOTH) {
|
2016-03-16 11:22:07 +01:00
|
|
|
p = strchr (value, ' ');
|
|
|
|
|
if (p)
|
|
|
|
|
*p = '\0';
|
|
|
|
|
}
|
|
|
|
|
|
2017-06-07 18:51:41 +02:00
|
|
|
if (value && nm_streq (*options, NM_SETTING_BOND_OPTION_MODE))
|
2017-06-05 14:48:08 +02:00
|
|
|
mode = _nm_setting_bond_mode_from_string (value);
|
|
|
|
|
|
|
|
|
|
if (!_nm_setting_bond_option_supported (*options, mode))
|
|
|
|
|
continue;
|
|
|
|
|
|
2016-03-16 11:22:07 +01:00
|
|
|
if ( value
|
|
|
|
|
&& value[0]
|
2017-06-21 11:19:03 +02:00
|
|
|
&& !ignore_option (s_bond, *options, value)) {
|
2013-08-28 11:47:44 +02:00
|
|
|
/* Replace " " with "," for arp_ip_targets from the kernel */
|
2016-07-05 22:24:47 +02:00
|
|
|
if (strcmp (*options, NM_SETTING_BOND_OPTION_ARP_IP_TARGET) == 0) {
|
2016-03-16 11:22:07 +01:00
|
|
|
for (p = value; *p; p++) {
|
2013-08-28 11:47:44 +02:00
|
|
|
if (*p == ' ')
|
|
|
|
|
*p = ',';
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
nm_setting_bond_add_option (s_bond, *options, value);
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2014-06-18 20:17:57 +02:00
|
|
|
static gboolean
|
|
|
|
|
master_update_slave_connection (NMDevice *self,
|
|
|
|
|
NMDevice *slave,
|
|
|
|
|
NMConnection *connection,
|
|
|
|
|
GError **error)
|
|
|
|
|
{
|
|
|
|
|
g_object_set (nm_connection_get_setting_connection (connection),
|
|
|
|
|
NM_SETTING_CONNECTION_MASTER, nm_device_get_iface (self),
|
|
|
|
|
NM_SETTING_CONNECTION_SLAVE_TYPE, NM_SETTING_BOND_SETTING_NAME,
|
|
|
|
|
NULL);
|
|
|
|
|
return TRUE;
|
|
|
|
|
}
|
|
|
|
|
|
2013-07-05 16:41:51 +02:00
|
|
|
static void
|
2013-10-01 20:15:03 +02:00
|
|
|
set_arp_targets (NMDevice *device,
|
2016-07-04 16:25:39 +02:00
|
|
|
NMBondMode mode,
|
2013-10-01 18:27:25 +02:00
|
|
|
const char *value,
|
|
|
|
|
const char *delim,
|
|
|
|
|
const char *prefix)
|
2013-07-05 16:41:51 +02:00
|
|
|
{
|
2013-10-01 18:27:25 +02:00
|
|
|
char **items, **iter, *tmp;
|
|
|
|
|
|
2013-10-01 20:15:03 +02:00
|
|
|
if (!value || !*value)
|
|
|
|
|
return;
|
|
|
|
|
|
2013-10-01 18:27:25 +02:00
|
|
|
items = g_strsplit_set (value, delim, 0);
|
|
|
|
|
for (iter = items; iter && *iter; iter++) {
|
|
|
|
|
if (*iter[0]) {
|
|
|
|
|
tmp = g_strdup_printf ("%s%s", prefix, *iter);
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_ARP_IP_TARGET, tmp);
|
2013-10-01 18:27:25 +02:00
|
|
|
g_free (tmp);
|
|
|
|
|
}
|
2013-07-05 16:41:51 +02:00
|
|
|
}
|
2013-10-01 18:27:25 +02:00
|
|
|
g_strfreev (items);
|
2013-07-05 16:41:51 +02:00
|
|
|
}
|
|
|
|
|
|
2013-10-01 18:27:25 +02:00
|
|
|
static void
|
2013-10-01 20:15:03 +02:00
|
|
|
set_simple_option (NMDevice *device,
|
2016-07-04 16:25:39 +02:00
|
|
|
NMBondMode mode,
|
2013-10-01 18:27:25 +02:00
|
|
|
NMSettingBond *s_bond,
|
|
|
|
|
const char *opt)
|
2013-07-05 16:41:51 +02:00
|
|
|
{
|
2013-10-01 20:15:03 +02:00
|
|
|
const char *value;
|
2013-10-01 18:27:25 +02:00
|
|
|
|
|
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, opt);
|
2013-10-01 20:15:03 +02:00
|
|
|
if (!value)
|
|
|
|
|
value = nm_setting_bond_get_option_default (s_bond, opt);
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, opt, value);
|
2013-10-01 18:27:25 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static NMActStageReturn
|
|
|
|
|
apply_bonding_config (NMDevice *device)
|
|
|
|
|
{
|
2016-07-04 16:25:39 +02:00
|
|
|
NMDeviceBond *self = NM_DEVICE_BOND (device);
|
2013-10-01 18:27:25 +02:00
|
|
|
NMConnection *connection;
|
|
|
|
|
NMSettingBond *s_bond;
|
2013-10-01 20:15:03 +02:00
|
|
|
int ifindex = nm_device_get_ifindex (device);
|
2016-07-04 16:25:39 +02:00
|
|
|
const char *mode_str, *value;
|
2013-10-01 20:15:03 +02:00
|
|
|
char *contents;
|
2013-10-01 18:27:25 +02:00
|
|
|
gboolean set_arp_interval = TRUE;
|
2016-07-04 16:25:39 +02:00
|
|
|
NMBondMode mode;
|
2013-10-01 18:27:25 +02:00
|
|
|
|
|
|
|
|
/* Option restrictions:
|
|
|
|
|
*
|
|
|
|
|
* arp_interval conflicts miimon > 0
|
|
|
|
|
* arp_interval conflicts [ alb, tlb ]
|
|
|
|
|
* arp_validate needs [ active-backup ]
|
|
|
|
|
* downdelay needs miimon
|
|
|
|
|
* updelay needs miimon
|
|
|
|
|
* primary needs [ active-backup, tlb, alb ]
|
|
|
|
|
*
|
|
|
|
|
* clearing miimon requires that arp_interval be 0, but clearing
|
|
|
|
|
* arp_interval doesn't require miimon to be 0
|
|
|
|
|
*/
|
|
|
|
|
|
2015-07-14 16:53:24 +02:00
|
|
|
connection = nm_device_get_applied_connection (device);
|
2013-10-01 18:27:25 +02:00
|
|
|
g_assert (connection);
|
|
|
|
|
s_bond = nm_connection_get_setting_bond (connection);
|
|
|
|
|
g_assert (s_bond);
|
|
|
|
|
|
2016-07-04 16:25:39 +02:00
|
|
|
mode_str = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_MODE);
|
|
|
|
|
if (!mode_str)
|
|
|
|
|
mode_str = "balance-rr";
|
|
|
|
|
|
|
|
|
|
mode = _nm_setting_bond_mode_from_string (mode_str);
|
|
|
|
|
if (mode == NM_BOND_MODE_UNKNOWN) {
|
|
|
|
|
_LOGW (LOGD_BOND, "unknown bond mode '%s'", mode_str);
|
|
|
|
|
return NM_ACT_STAGE_RETURN_FAILURE;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/* Set mode first, as some other options (e.g. arp_interval) are valid
|
|
|
|
|
* only for certain modes.
|
|
|
|
|
*/
|
2016-07-05 22:24:47 +02:00
|
|
|
|
|
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_MODE, mode_str);
|
2013-10-01 18:27:25 +02:00
|
|
|
|
|
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_MIIMON);
|
|
|
|
|
if (value && atoi (value)) {
|
|
|
|
|
/* clear arp interval */
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_ARP_INTERVAL, "0");
|
2013-10-01 18:27:25 +02:00
|
|
|
set_arp_interval = FALSE;
|
|
|
|
|
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_MIIMON, value);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_UPDELAY);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_DOWNDELAY);
|
2013-10-01 18:27:25 +02:00
|
|
|
} else if (!value) {
|
2016-07-04 16:25:39 +02:00
|
|
|
/* If not given, and arp_interval is not given or disabled, default to 100 */
|
2013-10-01 18:27:25 +02:00
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_ARP_INTERVAL);
|
2016-07-04 16:25:39 +02:00
|
|
|
if (_nm_utils_ascii_str_to_int64 (value, 10, 0, G_MAXUINT32, 0) == 0)
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_MIIMON, "100");
|
2013-07-05 16:41:51 +02:00
|
|
|
}
|
|
|
|
|
|
2013-10-01 18:27:25 +02:00
|
|
|
if (set_arp_interval) {
|
2016-07-05 22:24:47 +02:00
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_ARP_INTERVAL);
|
2013-10-01 18:27:25 +02:00
|
|
|
/* Just let miimon get cleared automatically; even setting miimon to
|
|
|
|
|
* 0 (disabled) clears arp_interval.
|
|
|
|
|
*/
|
2013-07-05 16:41:51 +02:00
|
|
|
}
|
|
|
|
|
|
2016-07-04 16:25:39 +02:00
|
|
|
/* ARP validate: value > 0 only valid in active-backup mode */
|
2013-10-01 18:27:25 +02:00
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_ARP_VALIDATE);
|
|
|
|
|
if ( value
|
2016-03-15 17:37:06 +01:00
|
|
|
&& !nm_streq (value, "0")
|
|
|
|
|
&& !nm_streq (value, "none")
|
2016-07-04 16:25:39 +02:00
|
|
|
&& mode == NM_BOND_MODE_ACTIVEBACKUP)
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_ARP_VALIDATE, value);
|
2013-10-01 18:27:25 +02:00
|
|
|
else
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_ARP_VALIDATE, "0");
|
2013-10-01 18:27:25 +02:00
|
|
|
|
2016-07-04 16:25:39 +02:00
|
|
|
/* Primary */
|
|
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_PRIMARY);
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_PRIMARY, value ? value : "");
|
2013-10-01 18:27:25 +02:00
|
|
|
|
2016-07-04 16:25:39 +02:00
|
|
|
/* ARP targets: clear and initialize the list */
|
2017-04-18 12:09:02 +02:00
|
|
|
contents = nm_platform_sysctl_master_get_option (nm_device_get_platform (device), ifindex,
|
2016-07-05 22:24:47 +02:00
|
|
|
NM_SETTING_BOND_OPTION_ARP_IP_TARGET);
|
2016-07-04 16:25:39 +02:00
|
|
|
set_arp_targets (device, mode, contents, " \n", "-");
|
2013-10-01 18:27:25 +02:00
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_ARP_IP_TARGET);
|
2016-07-04 16:25:39 +02:00
|
|
|
set_arp_targets (device, mode, value, ",", "+");
|
|
|
|
|
g_free (contents);
|
2016-03-15 17:37:06 +01:00
|
|
|
|
2016-07-04 16:39:28 +02:00
|
|
|
/* AD actor system: don't set if empty */
|
|
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_AD_ACTOR_SYSTEM);
|
|
|
|
|
if (value)
|
2016-07-05 22:24:47 +02:00
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_AD_ACTOR_SYSTEM, value);
|
|
|
|
|
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_ACTIVE_SLAVE);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_AD_ACTOR_SYS_PRIO);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_AD_SELECT);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_AD_USER_PORT_KEY);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_ALL_SLAVES_ACTIVE);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_ARP_ALL_TARGETS);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_FAIL_OVER_MAC);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_LACP_RATE);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_LP_INTERVAL);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_NUM_GRAT_ARP);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_NUM_UNSOL_NA);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_MIN_LINKS);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_PACKETS_PER_SLAVE);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_PRIMARY_RESELECT);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_RESEND_IGMP);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_TLB_DYNAMIC_LB);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_USE_CARRIER);
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_XMIT_HASH_POLICY);
|
2013-10-01 18:27:25 +02:00
|
|
|
|
|
|
|
|
return NM_ACT_STAGE_RETURN_SUCCESS;
|
2013-07-05 16:41:51 +02:00
|
|
|
}
|
|
|
|
|
|
2012-02-29 12:39:57 -06:00
|
|
|
static NMActStageReturn
|
2017-02-22 17:04:00 +01:00
|
|
|
act_stage1_prepare (NMDevice *dev, NMDeviceStateReason *out_failure_reason)
|
2012-02-29 12:39:57 -06:00
|
|
|
{
|
|
|
|
|
NMActStageReturn ret = NM_ACT_STAGE_RETURN_SUCCESS;
|
|
|
|
|
gboolean no_firmware = FALSE;
|
|
|
|
|
|
2017-02-22 17:04:00 +01:00
|
|
|
ret = NM_DEVICE_CLASS (nm_device_bond_parent_class)->act_stage1_prepare (dev, out_failure_reason);
|
2013-10-01 18:27:25 +02:00
|
|
|
if (ret != NM_ACT_STAGE_RETURN_SUCCESS)
|
|
|
|
|
return ret;
|
2012-02-29 12:39:57 -06:00
|
|
|
|
2013-10-01 18:27:25 +02:00
|
|
|
/* Interface must be down to set bond options */
|
|
|
|
|
nm_device_take_down (dev, TRUE);
|
|
|
|
|
ret = apply_bonding_config (dev);
|
2017-02-22 17:04:00 +01:00
|
|
|
if (ret != NM_ACT_STAGE_RETURN_FAILURE)
|
2016-10-21 14:37:05 +02:00
|
|
|
ret = nm_device_hw_addr_set_cloned (dev, nm_device_get_applied_connection (dev), FALSE);
|
2013-10-01 18:27:25 +02:00
|
|
|
nm_device_bring_up (dev, TRUE, &no_firmware);
|
2012-02-29 12:39:57 -06:00
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
|
}
|
|
|
|
|
|
2012-02-28 18:35:30 -06:00
|
|
|
static gboolean
|
2013-11-07 01:08:02 -06:00
|
|
|
enslave_slave (NMDevice *device,
|
|
|
|
|
NMDevice *slave,
|
|
|
|
|
NMConnection *connection,
|
|
|
|
|
gboolean configure)
|
2012-02-28 18:35:30 -06:00
|
|
|
{
|
2014-08-02 15:14:26 +02:00
|
|
|
NMDeviceBond *self = NM_DEVICE_BOND (device);
|
2013-11-07 01:08:02 -06:00
|
|
|
gboolean success = TRUE, no_firmware = FALSE;
|
2012-11-14 14:05:30 -06:00
|
|
|
const char *slave_iface = nm_device_get_ip_iface (slave);
|
2017-03-01 14:55:51 +01:00
|
|
|
NMConnection *master_con;
|
2012-02-28 18:35:30 -06:00
|
|
|
|
2013-10-11 14:59:26 -04:00
|
|
|
nm_device_master_check_slave_physical_port (device, slave, LOGD_BOND);
|
|
|
|
|
|
2013-11-07 01:08:02 -06:00
|
|
|
if (configure) {
|
|
|
|
|
nm_device_take_down (slave, TRUE);
|
2017-04-18 12:09:02 +02:00
|
|
|
success = nm_platform_link_enslave (nm_device_get_platform (device),
|
platform: add self argument to platform functions
Most nm_platform_*() functions operate on the platform
singleton nm_platform_get(). That made sense because the
NMPlatform instance was mainly to hook fake platform for
testing.
While the implicit argument saved some typing, I think explicit is
better. Especially, because NMPlatform could become a more usable
object then just a hook for testing.
With this change, NMPlatform instances can be used individually, not
only as a singleton instance.
Before this change, the constructor of NMLinuxPlatform could not
call any nm_platform_*() functions because the singleton was not
yet initialized. We could only instantiate an incomplete instance,
register it via nm_platform_setup(), and then complete initialization
via singleton->setup().
With this change, we can create and fully initialize NMPlatform instances
before/without setting them up them as singleton.
Also, currently there is no clear distinction between functions
that operate on the NMPlatform instance, and functions that can
be used stand-alone (e.g. nm_platform_ip4_address_to_string()).
The latter can not be mocked for testing. With this change, the
distinction becomes obvious. That is also useful because it becomes
clearer which functions make use of the platform cache and which not.
Inside nm-linux-platform.c, continue the pattern that the
self instance is named @platform. That makes sense because
its type is NMPlatform, and not NMLinuxPlatform what we
would expect from a paramter named @self.
This is a major diff that causes some pain when rebasing. Try
to rebase to the parent commit of this commit as a first step.
Then rebase on top of this commit using merge-strategy "ours".
2015-04-18 12:36:09 +02:00
|
|
|
nm_device_get_ip_ifindex (device),
|
2013-11-07 01:08:02 -06:00
|
|
|
nm_device_get_ip_ifindex (slave));
|
|
|
|
|
nm_device_bring_up (slave, TRUE, &no_firmware);
|
2012-11-14 14:05:30 -06:00
|
|
|
|
2014-02-25 16:44:01 -05:00
|
|
|
if (!success)
|
|
|
|
|
return FALSE;
|
|
|
|
|
|
2014-08-02 15:14:26 +02:00
|
|
|
_LOGI (LOGD_BOND, "enslaved bond slave %s", slave_iface);
|
2017-03-01 14:55:51 +01:00
|
|
|
|
|
|
|
|
/* The active_slave option can be set only after the interface is enslaved */
|
|
|
|
|
master_con = nm_device_get_applied_connection (device);
|
|
|
|
|
if (master_con) {
|
|
|
|
|
NMSettingBond *s_bond = nm_connection_get_setting_bond (master_con);
|
|
|
|
|
const char *active;
|
|
|
|
|
|
|
|
|
|
if (s_bond) {
|
|
|
|
|
active = nm_setting_bond_get_option_by_name (s_bond, "active_slave");
|
|
|
|
|
if (active && nm_streq0 (active, nm_device_get_iface (slave))) {
|
2017-04-18 12:09:02 +02:00
|
|
|
nm_platform_sysctl_master_set_option (nm_device_get_platform (device),
|
2017-03-01 14:55:51 +01:00
|
|
|
nm_device_get_ifindex (device),
|
|
|
|
|
"active_slave",
|
|
|
|
|
active);
|
|
|
|
|
_LOGD (LOGD_BOND, "setting slave %s as active one for master %s",
|
|
|
|
|
active, nm_device_get_iface (device));
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
2014-02-25 16:44:01 -05:00
|
|
|
} else
|
2014-08-02 15:14:26 +02:00
|
|
|
_LOGI (LOGD_BOND, "bond slave %s was enslaved", slave_iface);
|
2012-02-28 18:35:30 -06:00
|
|
|
|
2014-02-25 16:44:01 -05:00
|
|
|
return TRUE;
|
2012-02-28 18:35:30 -06:00
|
|
|
}
|
|
|
|
|
|
2015-12-02 09:56:17 +01:00
|
|
|
static void
|
2014-02-25 16:44:01 -05:00
|
|
|
release_slave (NMDevice *device,
|
|
|
|
|
NMDevice *slave,
|
|
|
|
|
gboolean configure)
|
2012-02-28 18:35:30 -06:00
|
|
|
{
|
2014-08-02 15:14:26 +02:00
|
|
|
NMDeviceBond *self = NM_DEVICE_BOND (device);
|
2015-12-02 09:56:17 +01:00
|
|
|
gboolean success, no_firmware = FALSE;
|
2016-10-21 14:37:05 +02:00
|
|
|
gs_free char *address = NULL;
|
2012-02-28 18:35:30 -06:00
|
|
|
|
2014-02-25 16:44:01 -05:00
|
|
|
if (configure) {
|
2016-10-21 14:37:05 +02:00
|
|
|
/* When the last slave is released the bond MAC will be set to a random
|
|
|
|
|
* value by kernel; remember the current one and restore it afterwards.
|
|
|
|
|
*/
|
|
|
|
|
address = g_strdup (nm_device_get_hw_address (device));
|
|
|
|
|
|
2017-04-18 12:09:02 +02:00
|
|
|
success = nm_platform_link_release (nm_device_get_platform (device),
|
platform: add self argument to platform functions
Most nm_platform_*() functions operate on the platform
singleton nm_platform_get(). That made sense because the
NMPlatform instance was mainly to hook fake platform for
testing.
While the implicit argument saved some typing, I think explicit is
better. Especially, because NMPlatform could become a more usable
object then just a hook for testing.
With this change, NMPlatform instances can be used individually, not
only as a singleton instance.
Before this change, the constructor of NMLinuxPlatform could not
call any nm_platform_*() functions because the singleton was not
yet initialized. We could only instantiate an incomplete instance,
register it via nm_platform_setup(), and then complete initialization
via singleton->setup().
With this change, we can create and fully initialize NMPlatform instances
before/without setting them up them as singleton.
Also, currently there is no clear distinction between functions
that operate on the NMPlatform instance, and functions that can
be used stand-alone (e.g. nm_platform_ip4_address_to_string()).
The latter can not be mocked for testing. With this change, the
distinction becomes obvious. That is also useful because it becomes
clearer which functions make use of the platform cache and which not.
Inside nm-linux-platform.c, continue the pattern that the
self instance is named @platform. That makes sense because
its type is NMPlatform, and not NMLinuxPlatform what we
would expect from a paramter named @self.
This is a major diff that causes some pain when rebasing. Try
to rebase to the parent commit of this commit as a first step.
Then rebase on top of this commit using merge-strategy "ours".
2015-04-18 12:36:09 +02:00
|
|
|
nm_device_get_ip_ifindex (device),
|
2014-02-25 16:44:01 -05:00
|
|
|
nm_device_get_ip_ifindex (slave));
|
2013-01-21 15:12:24 +01:00
|
|
|
|
2014-02-25 16:44:01 -05:00
|
|
|
if (success) {
|
2014-08-02 15:14:26 +02:00
|
|
|
_LOGI (LOGD_BOND, "released bond slave %s",
|
|
|
|
|
nm_device_get_ip_iface (slave));
|
2014-02-25 16:44:01 -05:00
|
|
|
} else {
|
2014-08-02 15:14:26 +02:00
|
|
|
_LOGW (LOGD_BOND, "failed to release bond slave %s",
|
|
|
|
|
nm_device_get_ip_iface (slave));
|
2014-02-25 16:44:01 -05:00
|
|
|
}
|
|
|
|
|
|
2017-04-18 12:09:02 +02:00
|
|
|
nm_platform_process_events (nm_device_get_platform (device));
|
2016-10-21 14:37:05 +02:00
|
|
|
if (nm_device_update_hw_address (device))
|
|
|
|
|
nm_device_hw_addr_set (device, address, "restore", FALSE);
|
|
|
|
|
|
2014-02-25 16:44:01 -05:00
|
|
|
/* Kernel bonding code "closes" the slave when releasing it, (which clears
|
|
|
|
|
* IFF_UP), so we must bring it back up here to ensure carrier changes and
|
|
|
|
|
* other state is noticed by the now-released slave.
|
|
|
|
|
*/
|
2014-08-02 15:14:26 +02:00
|
|
|
if (!nm_device_bring_up (slave, TRUE, &no_firmware))
|
|
|
|
|
_LOGW (LOGD_BOND, "released bond slave could not be brought up.");
|
2015-12-02 10:53:16 +01:00
|
|
|
} else {
|
|
|
|
|
_LOGI (LOGD_BOND, "bond slave %s was released",
|
|
|
|
|
nm_device_get_ip_iface (slave));
|
2013-01-21 14:33:40 -06:00
|
|
|
}
|
2012-02-28 18:35:30 -06:00
|
|
|
}
|
|
|
|
|
|
2014-09-05 08:50:02 -05:00
|
|
|
static gboolean
|
|
|
|
|
create_and_realize (NMDevice *device,
|
|
|
|
|
NMConnection *connection,
|
|
|
|
|
NMDevice *parent,
|
2015-12-09 15:13:57 +01:00
|
|
|
const NMPlatformLink **out_plink,
|
2014-09-05 08:50:02 -05:00
|
|
|
GError **error)
|
|
|
|
|
{
|
|
|
|
|
const char *iface = nm_device_get_iface (device);
|
|
|
|
|
NMPlatformError plerr;
|
|
|
|
|
|
|
|
|
|
g_assert (iface);
|
|
|
|
|
|
2017-04-18 12:09:02 +02:00
|
|
|
plerr = nm_platform_link_bond_add (nm_device_get_platform (device), iface, out_plink);
|
2016-01-25 13:03:51 +01:00
|
|
|
if (plerr != NM_PLATFORM_ERROR_SUCCESS) {
|
2014-09-05 08:50:02 -05:00
|
|
|
g_set_error (error, NM_DEVICE_ERROR, NM_DEVICE_ERROR_CREATION_FAILED,
|
|
|
|
|
"Failed to create bond interface '%s' for '%s': %s",
|
|
|
|
|
iface,
|
|
|
|
|
nm_connection_get_id (connection),
|
2017-08-21 19:18:45 +02:00
|
|
|
nm_platform_error_to_string_a (plerr));
|
2014-09-05 08:50:02 -05:00
|
|
|
return FALSE;
|
|
|
|
|
}
|
|
|
|
|
return TRUE;
|
|
|
|
|
}
|
|
|
|
|
|
2017-03-03 12:08:39 +01:00
|
|
|
static gboolean
|
|
|
|
|
check_changed_options (NMSettingBond *s_a, NMSettingBond *s_b, GError **error)
|
|
|
|
|
{
|
|
|
|
|
guint i, num;
|
2017-04-07 11:56:53 -05:00
|
|
|
const char *name = NULL, *value_a = NULL, *value_b = NULL;
|
2017-03-03 12:08:39 +01:00
|
|
|
|
|
|
|
|
/* Check that options in @s_a have compatible changes in @s_b */
|
|
|
|
|
|
|
|
|
|
num = nm_setting_bond_get_num_options (s_a);
|
|
|
|
|
for (i = 0; i < num; i++) {
|
|
|
|
|
nm_setting_bond_get_option (s_a, i, &name, &value_a);
|
|
|
|
|
|
|
|
|
|
/* We support changes to these */
|
|
|
|
|
if (NM_IN_STRSET (name,
|
|
|
|
|
NM_SETTING_BOND_OPTION_ACTIVE_SLAVE,
|
|
|
|
|
NM_SETTING_BOND_OPTION_PRIMARY)) {
|
|
|
|
|
continue;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/* Missing in @s_b, but has a default value in @s_a */
|
|
|
|
|
value_b = nm_setting_bond_get_option_by_name (s_b, name);
|
|
|
|
|
if ( !value_b
|
|
|
|
|
&& nm_streq0 (value_a, nm_setting_bond_get_option_default (s_a, name))) {
|
|
|
|
|
continue;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/* Reject any other changes */
|
|
|
|
|
if (!nm_streq0 (value_a, value_b)) {
|
|
|
|
|
g_set_error (error,
|
|
|
|
|
NM_DEVICE_ERROR,
|
|
|
|
|
NM_DEVICE_ERROR_INCOMPATIBLE_CONNECTION,
|
|
|
|
|
"Can't reapply '%s' bond option",
|
|
|
|
|
name);
|
|
|
|
|
return FALSE;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
return TRUE;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static gboolean
|
|
|
|
|
can_reapply_change (NMDevice *device,
|
|
|
|
|
const char *setting_name,
|
|
|
|
|
NMSetting *s_old,
|
|
|
|
|
NMSetting *s_new,
|
|
|
|
|
GHashTable *diffs,
|
|
|
|
|
GError **error)
|
|
|
|
|
{
|
|
|
|
|
NMDeviceClass *device_class;
|
|
|
|
|
NMSettingBond *s_bond_old, *s_bond_new;
|
|
|
|
|
|
|
|
|
|
/* Only handle bond setting here, delegate other settings to parent class */
|
|
|
|
|
if (nm_streq (setting_name, NM_SETTING_BOND_SETTING_NAME)) {
|
|
|
|
|
if (!nm_device_hash_check_invalid_keys (diffs,
|
|
|
|
|
NM_SETTING_BOND_SETTING_NAME,
|
|
|
|
|
error,
|
|
|
|
|
NM_SETTING_BOND_OPTIONS))
|
|
|
|
|
return FALSE;
|
|
|
|
|
|
|
|
|
|
s_bond_old = NM_SETTING_BOND (s_old);
|
|
|
|
|
s_bond_new = NM_SETTING_BOND (s_new);
|
|
|
|
|
|
|
|
|
|
if ( !check_changed_options (s_bond_old, s_bond_new, error)
|
|
|
|
|
|| !check_changed_options (s_bond_new, s_bond_old, error)) {
|
|
|
|
|
return FALSE;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
return TRUE;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
device_class = NM_DEVICE_CLASS (nm_device_bond_parent_class);
|
|
|
|
|
return device_class->can_reapply_change (device,
|
|
|
|
|
setting_name,
|
|
|
|
|
s_old,
|
|
|
|
|
s_new,
|
|
|
|
|
diffs,
|
|
|
|
|
error);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static void
|
|
|
|
|
reapply_connection (NMDevice *device, NMConnection *con_old, NMConnection *con_new)
|
|
|
|
|
{
|
|
|
|
|
NMDeviceBond *self = NM_DEVICE_BOND (device);
|
|
|
|
|
const char *value;
|
|
|
|
|
NMSettingBond *s_bond;
|
|
|
|
|
NMBondMode mode;
|
|
|
|
|
|
|
|
|
|
NM_DEVICE_CLASS (nm_device_bond_parent_class)->reapply_connection (device,
|
|
|
|
|
con_old,
|
|
|
|
|
con_new);
|
|
|
|
|
|
|
|
|
|
_LOGD (LOGD_BOND, "reapplying bond settings");
|
|
|
|
|
s_bond = nm_connection_get_setting_bond (con_new);
|
|
|
|
|
g_return_if_fail (s_bond);
|
|
|
|
|
|
|
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_MODE);
|
|
|
|
|
if (!value)
|
|
|
|
|
value = "balance-rr";
|
|
|
|
|
|
|
|
|
|
mode = _nm_setting_bond_mode_from_string (value);
|
|
|
|
|
g_return_if_fail (mode != NM_BOND_MODE_UNKNOWN);
|
|
|
|
|
|
|
|
|
|
/* Primary */
|
|
|
|
|
value = nm_setting_bond_get_option_by_name (s_bond, NM_SETTING_BOND_OPTION_PRIMARY);
|
|
|
|
|
set_bond_attr (device, mode, NM_SETTING_BOND_OPTION_PRIMARY, value ? value : "");
|
|
|
|
|
|
|
|
|
|
/* Active slave */
|
|
|
|
|
set_simple_option (device, mode, s_bond, NM_SETTING_BOND_OPTION_ACTIVE_SLAVE);
|
|
|
|
|
}
|
|
|
|
|
|
2016-10-02 18:22:50 +02:00
|
|
|
/*****************************************************************************/
|
2012-02-28 18:35:30 -06:00
|
|
|
|
2012-02-15 17:31:37 -06:00
|
|
|
static void
|
|
|
|
|
nm_device_bond_init (NMDeviceBond * self)
|
|
|
|
|
{
|
2017-06-01 23:08:30 +02:00
|
|
|
nm_assert (nm_device_is_master (NM_DEVICE (self)));
|
2012-02-15 17:31:37 -06:00
|
|
|
}
|
|
|
|
|
|
core/dbus: rework D-Bus implementation to use lower layer GDBusConnection API
Previously, we used the generated GDBusInterfaceSkeleton types and glued
them via the NMExportedObject base class to our NM types. We also used
GDBusObjectManagerServer.
Don't do that anymore. The resulting code was more complicated despite (or
because?) using generated classes. It was hard to understand, complex, had
ordering-issues, and had a runtime and memory overhead.
This patch refactors this entirely and uses the lower layer API GDBusConnection
directly. It replaces the generated code, GDBusInterfaceSkeleton, and
GDBusObjectManagerServer. All this is now done by NMDbusObject and NMDBusManager
and static descriptor instances of type GDBusInterfaceInfo.
This adds a net plus of more then 1300 lines of hand written code. I claim
that this implementation is easier to understand. Note that previously we
also required extensive and complex glue code to bind our objects to the
generated skeleton objects. Instead, now glue our objects directly to
GDBusConnection. The result is more immediate and gets rid of layers of
code in between.
Now that the D-Bus glue us more under our control, we can address issus and
bottlenecks better, instead of adding code to bend the generated skeletons
to our needs.
Note that the current implementation now only supports one D-Bus connection.
That was effectively the case already, although there were places (and still are)
where the code pretends it could also support connections from a private socket.
We dropped private socket support mainly because it was unused, untested and
buggy, but also because GDBusObjectManagerServer could not export the same
objects on multiple connections. Now, it would be rather straight forward to
fix that and re-introduce ObjectManager on each private connection. But this
commit doesn't do that yet, and the new code intentionally supports only one
D-Bus connection.
Also, the D-Bus startup was simplified. There is no retry, either nm_dbus_manager_start()
succeeds, or it detects the initrd case. In the initrd case, bus manager never tries to
connect to D-Bus. Since the initrd scenario is not yet used/tested, this is good enough
for the moment. It could be easily extended later, for example with polling whether the
system bus appears (like was done previously). Also, restart of D-Bus daemon isn't
supported either -- just like before.
Note how NMDBusManager now implements the ObjectManager D-Bus interface
directly.
Also, this fixes race issues in the server, by no longer delaying
PropertiesChanged signals. NMExportedObject would collect changed
properties and send the signal out in idle_emit_properties_changed()
on idle. This messes up the ordering of change events w.r.t. other
signals and events on the bus. Note that not only NMExportedObject
messed up the ordering. Also the generated code would hook into
notify() and process change events in and idle handle, exhibiting the
same ordering issue too.
No longer do that. PropertiesChanged signals will be sent right away
by hooking into dispatch_properties_changed(). This means, changing
a property in quick succession will no longer be combined and is
guaranteed to emit signals for each individual state. Quite possibly
we emit now more PropertiesChanged signals then before.
However, we are now able to group a set of changes by using standard
g_object_freeze_notify()/g_object_thaw_notify(). We probably should
make more use of that.
Also, now that our signals are all handled in the right order, we
might find places where we still emit them in the wrong order. But that
is then due to the order in which our GObjects emit signals, not due
to an ill behavior of the D-Bus glue. Possibly we need to identify
such ordering issues and fix them.
Numbers (for contrib/rpm --without debug on x86_64):
- the patch changes the code size of NetworkManager by
- 2809360 bytes
+ 2537528 bytes (-9.7%)
- Runtime measurements are harder because there is a large variance
during testing. In other words, the numbers are not reproducible.
Currently, the implementation performs no caching of GVariants at all,
but it would be rather simple to add it, if that turns out to be
useful.
Anyway, without strong claim, it seems that the new form tends to
perform slightly better. That would be no surprise.
$ time (for i in {1..1000}; do nmcli >/dev/null || break; echo -n .; done)
- real 1m39.355s
+ real 1m37.432s
$ time (for i in {1..2000}; do busctl call org.freedesktop.NetworkManager /org/freedesktop org.freedesktop.DBus.ObjectManager GetManagedObjects > /dev/null || break; echo -n .; done)
- real 0m26.843s
+ real 0m25.281s
- Regarding RSS size, just looking at the processes in similar
conditions, doesn't give a large difference. On my system they
consume about 19MB RSS. It seems that the new version has a
slightly smaller RSS size.
- 19356 RSS
+ 18660 RSS
2018-02-26 13:51:52 +01:00
|
|
|
static const NMDBusInterfaceInfoExtended interface_info_device_bond = {
|
|
|
|
|
.parent = NM_DEFINE_GDBUS_INTERFACE_INFO_INIT (
|
|
|
|
|
NM_DBUS_INTERFACE_DEVICE_BOND,
|
|
|
|
|
.signals = NM_DEFINE_GDBUS_SIGNAL_INFOS (
|
|
|
|
|
&nm_signal_info_property_changed_legacy,
|
|
|
|
|
),
|
|
|
|
|
.properties = NM_DEFINE_GDBUS_PROPERTY_INFOS (
|
|
|
|
|
NM_DEFINE_DBUS_PROPERTY_INFO_EXTENDED_READABLE_L ("HwAddress", "s", NM_DEVICE_HW_ADDRESS),
|
|
|
|
|
NM_DEFINE_DBUS_PROPERTY_INFO_EXTENDED_READABLE_L ("Carrier", "b", NM_DEVICE_CARRIER),
|
|
|
|
|
NM_DEFINE_DBUS_PROPERTY_INFO_EXTENDED_READABLE_L ("Slaves", "ao", NM_DEVICE_SLAVES),
|
|
|
|
|
),
|
|
|
|
|
),
|
|
|
|
|
.legacy_property_changed = TRUE,
|
|
|
|
|
};
|
|
|
|
|
|
2012-02-15 17:31:37 -06:00
|
|
|
static void
|
|
|
|
|
nm_device_bond_class_init (NMDeviceBondClass *klass)
|
|
|
|
|
{
|
core/dbus: rework D-Bus implementation to use lower layer GDBusConnection API
Previously, we used the generated GDBusInterfaceSkeleton types and glued
them via the NMExportedObject base class to our NM types. We also used
GDBusObjectManagerServer.
Don't do that anymore. The resulting code was more complicated despite (or
because?) using generated classes. It was hard to understand, complex, had
ordering-issues, and had a runtime and memory overhead.
This patch refactors this entirely and uses the lower layer API GDBusConnection
directly. It replaces the generated code, GDBusInterfaceSkeleton, and
GDBusObjectManagerServer. All this is now done by NMDbusObject and NMDBusManager
and static descriptor instances of type GDBusInterfaceInfo.
This adds a net plus of more then 1300 lines of hand written code. I claim
that this implementation is easier to understand. Note that previously we
also required extensive and complex glue code to bind our objects to the
generated skeleton objects. Instead, now glue our objects directly to
GDBusConnection. The result is more immediate and gets rid of layers of
code in between.
Now that the D-Bus glue us more under our control, we can address issus and
bottlenecks better, instead of adding code to bend the generated skeletons
to our needs.
Note that the current implementation now only supports one D-Bus connection.
That was effectively the case already, although there were places (and still are)
where the code pretends it could also support connections from a private socket.
We dropped private socket support mainly because it was unused, untested and
buggy, but also because GDBusObjectManagerServer could not export the same
objects on multiple connections. Now, it would be rather straight forward to
fix that and re-introduce ObjectManager on each private connection. But this
commit doesn't do that yet, and the new code intentionally supports only one
D-Bus connection.
Also, the D-Bus startup was simplified. There is no retry, either nm_dbus_manager_start()
succeeds, or it detects the initrd case. In the initrd case, bus manager never tries to
connect to D-Bus. Since the initrd scenario is not yet used/tested, this is good enough
for the moment. It could be easily extended later, for example with polling whether the
system bus appears (like was done previously). Also, restart of D-Bus daemon isn't
supported either -- just like before.
Note how NMDBusManager now implements the ObjectManager D-Bus interface
directly.
Also, this fixes race issues in the server, by no longer delaying
PropertiesChanged signals. NMExportedObject would collect changed
properties and send the signal out in idle_emit_properties_changed()
on idle. This messes up the ordering of change events w.r.t. other
signals and events on the bus. Note that not only NMExportedObject
messed up the ordering. Also the generated code would hook into
notify() and process change events in and idle handle, exhibiting the
same ordering issue too.
No longer do that. PropertiesChanged signals will be sent right away
by hooking into dispatch_properties_changed(). This means, changing
a property in quick succession will no longer be combined and is
guaranteed to emit signals for each individual state. Quite possibly
we emit now more PropertiesChanged signals then before.
However, we are now able to group a set of changes by using standard
g_object_freeze_notify()/g_object_thaw_notify(). We probably should
make more use of that.
Also, now that our signals are all handled in the right order, we
might find places where we still emit them in the wrong order. But that
is then due to the order in which our GObjects emit signals, not due
to an ill behavior of the D-Bus glue. Possibly we need to identify
such ordering issues and fix them.
Numbers (for contrib/rpm --without debug on x86_64):
- the patch changes the code size of NetworkManager by
- 2809360 bytes
+ 2537528 bytes (-9.7%)
- Runtime measurements are harder because there is a large variance
during testing. In other words, the numbers are not reproducible.
Currently, the implementation performs no caching of GVariants at all,
but it would be rather simple to add it, if that turns out to be
useful.
Anyway, without strong claim, it seems that the new form tends to
perform slightly better. That would be no surprise.
$ time (for i in {1..1000}; do nmcli >/dev/null || break; echo -n .; done)
- real 1m39.355s
+ real 1m37.432s
$ time (for i in {1..2000}; do busctl call org.freedesktop.NetworkManager /org/freedesktop org.freedesktop.DBus.ObjectManager GetManagedObjects > /dev/null || break; echo -n .; done)
- real 0m26.843s
+ real 0m25.281s
- Regarding RSS size, just looking at the processes in similar
conditions, doesn't give a large difference. On my system they
consume about 19MB RSS. It seems that the new version has a
slightly smaller RSS size.
- 19356 RSS
+ 18660 RSS
2018-02-26 13:51:52 +01:00
|
|
|
NMDBusObjectClass *dbus_object_class = NM_DBUS_OBJECT_CLASS (klass);
|
2012-02-15 17:31:37 -06:00
|
|
|
NMDeviceClass *parent_class = NM_DEVICE_CLASS (klass);
|
|
|
|
|
|
2014-10-09 12:42:29 -05:00
|
|
|
NM_DEVICE_CLASS_DECLARE_TYPES (klass, NM_SETTING_BOND_SETTING_NAME, NM_LINK_TYPE_BOND)
|
2013-08-28 11:47:44 +02:00
|
|
|
|
core/dbus: rework D-Bus implementation to use lower layer GDBusConnection API
Previously, we used the generated GDBusInterfaceSkeleton types and glued
them via the NMExportedObject base class to our NM types. We also used
GDBusObjectManagerServer.
Don't do that anymore. The resulting code was more complicated despite (or
because?) using generated classes. It was hard to understand, complex, had
ordering-issues, and had a runtime and memory overhead.
This patch refactors this entirely and uses the lower layer API GDBusConnection
directly. It replaces the generated code, GDBusInterfaceSkeleton, and
GDBusObjectManagerServer. All this is now done by NMDbusObject and NMDBusManager
and static descriptor instances of type GDBusInterfaceInfo.
This adds a net plus of more then 1300 lines of hand written code. I claim
that this implementation is easier to understand. Note that previously we
also required extensive and complex glue code to bind our objects to the
generated skeleton objects. Instead, now glue our objects directly to
GDBusConnection. The result is more immediate and gets rid of layers of
code in between.
Now that the D-Bus glue us more under our control, we can address issus and
bottlenecks better, instead of adding code to bend the generated skeletons
to our needs.
Note that the current implementation now only supports one D-Bus connection.
That was effectively the case already, although there were places (and still are)
where the code pretends it could also support connections from a private socket.
We dropped private socket support mainly because it was unused, untested and
buggy, but also because GDBusObjectManagerServer could not export the same
objects on multiple connections. Now, it would be rather straight forward to
fix that and re-introduce ObjectManager on each private connection. But this
commit doesn't do that yet, and the new code intentionally supports only one
D-Bus connection.
Also, the D-Bus startup was simplified. There is no retry, either nm_dbus_manager_start()
succeeds, or it detects the initrd case. In the initrd case, bus manager never tries to
connect to D-Bus. Since the initrd scenario is not yet used/tested, this is good enough
for the moment. It could be easily extended later, for example with polling whether the
system bus appears (like was done previously). Also, restart of D-Bus daemon isn't
supported either -- just like before.
Note how NMDBusManager now implements the ObjectManager D-Bus interface
directly.
Also, this fixes race issues in the server, by no longer delaying
PropertiesChanged signals. NMExportedObject would collect changed
properties and send the signal out in idle_emit_properties_changed()
on idle. This messes up the ordering of change events w.r.t. other
signals and events on the bus. Note that not only NMExportedObject
messed up the ordering. Also the generated code would hook into
notify() and process change events in and idle handle, exhibiting the
same ordering issue too.
No longer do that. PropertiesChanged signals will be sent right away
by hooking into dispatch_properties_changed(). This means, changing
a property in quick succession will no longer be combined and is
guaranteed to emit signals for each individual state. Quite possibly
we emit now more PropertiesChanged signals then before.
However, we are now able to group a set of changes by using standard
g_object_freeze_notify()/g_object_thaw_notify(). We probably should
make more use of that.
Also, now that our signals are all handled in the right order, we
might find places where we still emit them in the wrong order. But that
is then due to the order in which our GObjects emit signals, not due
to an ill behavior of the D-Bus glue. Possibly we need to identify
such ordering issues and fix them.
Numbers (for contrib/rpm --without debug on x86_64):
- the patch changes the code size of NetworkManager by
- 2809360 bytes
+ 2537528 bytes (-9.7%)
- Runtime measurements are harder because there is a large variance
during testing. In other words, the numbers are not reproducible.
Currently, the implementation performs no caching of GVariants at all,
but it would be rather simple to add it, if that turns out to be
useful.
Anyway, without strong claim, it seems that the new form tends to
perform slightly better. That would be no surprise.
$ time (for i in {1..1000}; do nmcli >/dev/null || break; echo -n .; done)
- real 1m39.355s
+ real 1m37.432s
$ time (for i in {1..2000}; do busctl call org.freedesktop.NetworkManager /org/freedesktop org.freedesktop.DBus.ObjectManager GetManagedObjects > /dev/null || break; echo -n .; done)
- real 0m26.843s
+ real 0m25.281s
- Regarding RSS size, just looking at the processes in similar
conditions, doesn't give a large difference. On my system they
consume about 19MB RSS. It seems that the new version has a
slightly smaller RSS size.
- 19356 RSS
+ 18660 RSS
2018-02-26 13:51:52 +01:00
|
|
|
dbus_object_class->interface_infos = NM_DBUS_INTERFACE_INFOS (&interface_info_device_bond);
|
|
|
|
|
|
2017-06-01 23:08:30 +02:00
|
|
|
parent_class->is_master = TRUE;
|
2012-09-27 12:12:15 -04:00
|
|
|
parent_class->get_generic_capabilities = get_generic_capabilities;
|
|
|
|
|
parent_class->check_connection_compatible = check_connection_compatible;
|
|
|
|
|
parent_class->complete_connection = complete_connection;
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2013-08-28 11:47:44 +02:00
|
|
|
parent_class->update_connection = update_connection;
|
2014-06-18 20:17:57 +02:00
|
|
|
parent_class->master_update_slave_connection = master_update_slave_connection;
|
2012-02-15 17:31:37 -06:00
|
|
|
|
2014-09-05 08:50:02 -05:00
|
|
|
parent_class->create_and_realize = create_and_realize;
|
2012-09-27 12:12:15 -04:00
|
|
|
parent_class->act_stage1_prepare = act_stage1_prepare;
|
2017-01-14 17:04:17 +01:00
|
|
|
parent_class->get_configured_mtu = nm_device_get_configured_mtu_for_wired;
|
2012-02-28 18:35:30 -06:00
|
|
|
parent_class->enslave_slave = enslave_slave;
|
|
|
|
|
parent_class->release_slave = release_slave;
|
2017-03-03 12:08:39 +01:00
|
|
|
parent_class->can_reapply_change = can_reapply_change;
|
|
|
|
|
parent_class->reapply_connection = reapply_connection;
|
2012-02-15 17:31:37 -06:00
|
|
|
}
|
2014-09-08 10:15:02 -05:00
|
|
|
|
2016-10-02 18:22:50 +02:00
|
|
|
/*****************************************************************************/
|
2014-09-08 10:15:02 -05:00
|
|
|
|
2016-10-07 17:00:59 +02:00
|
|
|
#define NM_TYPE_BOND_DEVICE_FACTORY (nm_bond_device_factory_get_type ())
|
|
|
|
|
#define NM_BOND_DEVICE_FACTORY(obj) (G_TYPE_CHECK_INSTANCE_CAST ((obj), NM_TYPE_BOND_DEVICE_FACTORY, NMBondDeviceFactory))
|
2014-09-08 10:15:02 -05:00
|
|
|
|
|
|
|
|
static NMDevice *
|
2014-09-05 08:50:02 -05:00
|
|
|
create_device (NMDeviceFactory *factory,
|
|
|
|
|
const char *iface,
|
2016-01-10 15:13:20 +01:00
|
|
|
const NMPlatformLink *plink,
|
2014-09-05 08:50:02 -05:00
|
|
|
NMConnection *connection,
|
|
|
|
|
gboolean *out_ignore)
|
2014-09-08 10:15:02 -05:00
|
|
|
{
|
|
|
|
|
return (NMDevice *) g_object_new (NM_TYPE_DEVICE_BOND,
|
2016-01-08 16:11:41 +01:00
|
|
|
NM_DEVICE_IFACE, iface,
|
|
|
|
|
NM_DEVICE_DRIVER, "bonding",
|
|
|
|
|
NM_DEVICE_TYPE_DESC, "Bond",
|
|
|
|
|
NM_DEVICE_DEVICE_TYPE, NM_DEVICE_TYPE_BOND,
|
|
|
|
|
NM_DEVICE_LINK_TYPE, NM_LINK_TYPE_BOND,
|
|
|
|
|
NULL);
|
2014-09-08 10:15:02 -05:00
|
|
|
}
|
|
|
|
|
|
2014-09-17 14:17:30 -05:00
|
|
|
NM_DEVICE_FACTORY_DEFINE_INTERNAL (BOND, Bond, bond,
|
|
|
|
|
NM_DEVICE_FACTORY_DECLARE_LINK_TYPES (NM_LINK_TYPE_BOND)
|
|
|
|
|
NM_DEVICE_FACTORY_DECLARE_SETTING_TYPES (NM_SETTING_BOND_SETTING_NAME),
|
2016-10-07 16:05:43 +02:00
|
|
|
factory_class->create_device = create_device;
|
2016-09-29 13:49:01 +02:00
|
|
|
);
|