merge: branch 'lr/ovs-slaves'

https://github.com/NetworkManager/NetworkManager/pull/155
This commit is contained in:
Lubomir Rintel 2018-07-10 13:13:21 +02:00
commit 8149859c02
16 changed files with 213 additions and 16 deletions

View file

@ -25,7 +25,7 @@
<!--
Slaves:
Array of object paths representing devices which are currently slaved to
Array of object paths representing devices which are currently enslaved to
this device.
-->
<property name="Slaves" type="ao" access="read"/>

View file

@ -25,7 +25,7 @@
<!--
Slaves:
Array of object paths representing devices which are currently slaved to
Array of object paths representing devices which are currently enslaved to
this device.
-->
<property name="Slaves" type="ao" access="read"/>

View file

@ -7,6 +7,16 @@
-->
<interface name="org.freedesktop.NetworkManager.Device.OvsBridge">
<!--
Slaves:
Array of object paths representing ports which are currently enslaved to
this bridge.
Since: 1.14
-->
<property name="Slaves" type="ao" access="read"/>
<!--
PropertiesChanged:
@properties: A dictionary mapping property names to variant boxed values

View file

@ -7,6 +7,16 @@
-->
<interface name="org.freedesktop.NetworkManager.Device.OvsPort">
<!--
Slaves:
Array of object paths representing interfaces which are currently enslaved to
this port.
Since: 1.14
-->
<property name="Slaves" type="ao" access="read"/>
<!--
PropertiesChanged:
@properties: A dictionary mapping property names to variant boxed values

View file

@ -25,7 +25,7 @@
<!--
Slaves:
Array of object paths representing devices which are currently slaved to
Array of object paths representing devices which are currently enslaved to
this device.
-->
<property name="Slaves" type="ao" access="read"/>

View file

@ -134,7 +134,7 @@ nm_device_bond_get_carrier (NMDeviceBond *device)
* nm_device_bond_get_slaves:
* @device: a #NMDeviceBond
*
* Gets the devices currently slaved to @device.
* Gets the devices currently enslaved to @device.
*
* Returns: (element-type NMDevice): the #GPtrArray containing
* #NMDevices that are slaves of @device. This is the internal
@ -333,7 +333,7 @@ nm_device_bond_class_init (NMDeviceBondClass *bond_class)
/**
* NMDeviceBond:slaves:
*
* The devices (#NMDevice) slaved to the bond device.
* The devices (#NMDevice) enslaved to the bond device.
*
* Since: 0.9.8
**/

View file

@ -142,7 +142,7 @@ nm_device_bridge_get_carrier (NMDeviceBridge *device)
* nm_device_bridge_get_slaves:
* @device: a #NMDeviceBridge
*
* Gets the devices currently slaved to @device.
* Gets the devices currently enslaved to @device.
*
* Returns: (element-type NMDevice): the #GPtrArray containing
* #NMDevices that are slaves of @device. This is the internal
@ -345,7 +345,7 @@ nm_device_bridge_class_init (NMDeviceBridgeClass *bridge_class)
/**
* NMDeviceBridge:slaves:
*
* The devices (#NMDevice) slaved to the bridge device.
* The devices (#NMDevice) enslaved to the bridge device.
*
* Since: 0.9.8
**/

View file

@ -86,7 +86,7 @@ nm_device_bond_get_carrier (NMDeviceBond *device)
* nm_device_bond_get_slaves:
* @device: a #NMDeviceBond
*
* Gets the devices currently slaved to @device.
* Gets the devices currently enslaved to @device.
*
* Returns: (element-type NMDevice): the #GPtrArray containing
* #NMDevices that are slaves of @device. This is the internal
@ -250,7 +250,7 @@ nm_device_bond_class_init (NMDeviceBondClass *bond_class)
/**
* NMDeviceBond:slaves: (type GPtrArray(NMDevice))
*
* The devices slaved to the bond device.
* The devices enslaved to the bond device.
**/
g_object_class_install_property
(object_class, PROP_SLAVES,

View file

@ -86,7 +86,7 @@ nm_device_bridge_get_carrier (NMDeviceBridge *device)
* nm_device_bridge_get_slaves:
* @device: a #NMDeviceBridge
*
* Gets the devices currently slaved to @device.
* Gets the devices currently enslaved to @device.
*
* Returns: (element-type NMDevice): the #GPtrArray containing
* #NMDevices that are slaves of @device. This is the internal
@ -255,7 +255,7 @@ nm_device_bridge_class_init (NMDeviceBridgeClass *bridge_class)
/**
* NMDeviceBridge:slaves: (type GPtrArray(NMDevice))
*
* The devices slaved to the bridge device.
* The devices enslaved to the bridge device.
**/
g_object_class_install_property
(object_class, PROP_SLAVES,

View file

@ -14,7 +14,7 @@
* Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
* Boston, MA 02110-1301 USA.
*
* Copyright 2017 Red Hat, Inc.
* Copyright 2017,2018 Red Hat, Inc.
*/
#include "nm-default.h"
@ -26,12 +26,21 @@
#include "nm-setting-ovs-bridge.h"
#include "nm-setting-ovs-port.h"
#include "nm-setting-connection.h"
#include "nm-core-internal.h"
enum {
PROP_0,
PROP_SLAVES,
LAST_PROP
};
/**
* NMDeviceOvsBridge:
*/
struct _NMDeviceOvsBridge {
NMDevice parent;
GPtrArray *slaves;
};
struct _NMDeviceOvsBridgeClass {
@ -42,6 +51,26 @@ G_DEFINE_TYPE (NMDeviceOvsBridge, nm_device_ovs_bridge, NM_TYPE_DEVICE)
/*****************************************************************************/
/**
* nm_device_ovs_bridge_get_slaves:
* @device: a #NMDeviceOvsBridge
*
* Gets the ports currently enslaved to @device.
*
* Returns: (element-type NMDevice): the #GPtrArray containing
* #NMDevices that are slaves of @device. This is the internal
* copy used by the device, and must not be modified.
*
* Since: 1.14
**/
const GPtrArray *
nm_device_ovs_bridge_get_slaves (NMDeviceOvsBridge *device)
{
g_return_val_if_fail (NM_IS_DEVICE_OVS_BRIDGE (device), FALSE);
return device->slaves;
}
static const char *
get_type_description (NMDevice *device)
{
@ -80,16 +109,67 @@ get_setting_type (NMDevice *device)
/*****************************************************************************/
static void
init_dbus (NMObject *object)
{
NMDeviceOvsBridge *device = NM_DEVICE_OVS_BRIDGE (object);
const NMPropertiesInfo property_info[] = {
{ NM_DEVICE_OVS_BRIDGE_SLAVES, &device->slaves, NULL, NM_TYPE_DEVICE },
{ NULL },
};
NM_OBJECT_CLASS (nm_device_ovs_bridge_parent_class)->init_dbus (object);
_nm_object_register_properties (object,
NM_DBUS_INTERFACE_DEVICE_OVS_BRIDGE,
property_info);
}
static void
get_property (GObject *object,
guint prop_id,
GValue *value,
GParamSpec *pspec)
{
NMDeviceOvsBridge *device = NM_DEVICE_OVS_BRIDGE (object);
switch (prop_id) {
case PROP_SLAVES:
g_value_take_boxed (value, _nm_utils_copy_object_array (nm_device_ovs_bridge_get_slaves (device)));
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
break;
}
}
static void
nm_device_ovs_bridge_init (NMDeviceOvsBridge *device)
{
}
static void
dispose (GObject *object)
{
NMDeviceOvsBridge *device = NM_DEVICE_OVS_BRIDGE (object);
g_clear_pointer (&device->slaves, g_ptr_array_unref);
G_OBJECT_CLASS (nm_device_ovs_bridge_parent_class)->dispose (object);
}
static void
nm_device_ovs_bridge_class_init (NMDeviceOvsBridgeClass *ovs_bridge_class)
{
GObjectClass *object_class = G_OBJECT_CLASS (ovs_bridge_class);
NMObjectClass *nm_object_class = NM_OBJECT_CLASS (ovs_bridge_class);
NMDeviceClass *device_class = NM_DEVICE_CLASS (ovs_bridge_class);
object_class->get_property = get_property;
object_class->dispose = dispose;
nm_object_class->init_dbus = init_dbus;
device_class->get_type_description = get_type_description;
device_class->connection_compatible = connection_compatible;
device_class->get_setting_type = get_setting_type;

View file

@ -14,7 +14,7 @@
* Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
* Boston, MA 02110-1301 USA.
*
* Copyright 2017 Red Hat, Inc.
* Copyright 2017,2018 Red Hat, Inc.
*/
#ifndef __NM_DEVICE_OVS_BRIDGE_H__
@ -35,11 +35,16 @@ G_BEGIN_DECLS
#define NM_IS_DEVICE_OVS_BRIDGE_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE ((klass), NM_TYPE_DEVICE_OVS_BRIDGE))
#define NM_DEVICE_OVS_BRIDGE_GET_CLASS(obj) (G_TYPE_INSTANCE_GET_CLASS ((obj), NM_TYPE_DEVICE_OVS_BRIDGE, NMDeviceOvsBridgeClass))
#define NM_DEVICE_OVS_BRIDGE_SLAVES "slaves"
typedef struct _NMDeviceOvsBridgeClass NMDeviceOvsBridgeClass;
NM_AVAILABLE_IN_1_10
GType nm_device_ovs_bridge_get_type (void);
NM_AVAILABLE_IN_1_14
const GPtrArray *nm_device_ovs_bridge_get_slaves (NMDeviceOvsBridge *device);
G_END_DECLS
#endif /* __NM_DEVICE_OVS_BRIDGE_H__ */

View file

@ -14,7 +14,7 @@
* Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
* Boston, MA 02110-1301 USA.
*
* Copyright 2017 Red Hat, Inc.
* Copyright 2017,2018 Red Hat, Inc.
*/
#include "nm-default.h"
@ -26,12 +26,21 @@
#include "nm-setting-ovs-port.h"
#include "nm-setting-ovs-port.h"
#include "nm-setting-connection.h"
#include "nm-core-internal.h"
enum {
PROP_0,
PROP_SLAVES,
LAST_PROP
};
/**
* NMDeviceOvsPort:
*/
struct _NMDeviceOvsPort {
NMDevice parent;
GPtrArray *slaves;
};
struct _NMDeviceOvsPortClass {
@ -42,6 +51,26 @@ G_DEFINE_TYPE (NMDeviceOvsPort, nm_device_ovs_port, NM_TYPE_DEVICE)
/*****************************************************************************/
/**
* nm_device_ovs_port_get_slaves:
* @device: a #NMDeviceOvsPort
*
* Gets the interfaces currently enslaved to @device.
*
* Returns: (element-type NMDevice): the #GPtrArray containing
* #NMDevices that are slaves of @device. This is the internal
* copy used by the device, and must not be modified.
*
* Since: 1.14
**/
const GPtrArray *
nm_device_ovs_port_get_slaves (NMDeviceOvsPort *device)
{
g_return_val_if_fail (NM_IS_DEVICE_OVS_PORT (device), FALSE);
return device->slaves;
}
static const char *
get_type_description (NMDevice *device)
{
@ -79,16 +108,67 @@ get_setting_type (NMDevice *device)
/*****************************************************************************/
static void
init_dbus (NMObject *object)
{
NMDeviceOvsPort *device = NM_DEVICE_OVS_PORT (object);
const NMPropertiesInfo property_info[] = {
{ NM_DEVICE_OVS_PORT_SLAVES, &device->slaves, NULL, NM_TYPE_DEVICE },
{ NULL },
};
NM_OBJECT_CLASS (nm_device_ovs_port_parent_class)->init_dbus (object);
_nm_object_register_properties (object,
NM_DBUS_INTERFACE_DEVICE_OVS_PORT,
property_info);
}
static void
get_property (GObject *object,
guint prop_id,
GValue *value,
GParamSpec *pspec)
{
NMDeviceOvsPort *device = NM_DEVICE_OVS_PORT (object);
switch (prop_id) {
case PROP_SLAVES:
g_value_take_boxed (value, _nm_utils_copy_object_array (nm_device_ovs_port_get_slaves (device)));
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
break;
}
}
static void
nm_device_ovs_port_init (NMDeviceOvsPort *device)
{
}
static void
dispose (GObject *object)
{
NMDeviceOvsPort *device = NM_DEVICE_OVS_PORT (object);
g_clear_pointer (&device->slaves, g_ptr_array_unref);
G_OBJECT_CLASS (nm_device_ovs_port_parent_class)->dispose (object);
}
static void
nm_device_ovs_port_class_init (NMDeviceOvsPortClass *ovs_port_class)
{
GObjectClass *object_class = G_OBJECT_CLASS (ovs_port_class);
NMObjectClass *nm_object_class = NM_OBJECT_CLASS (ovs_port_class);
NMDeviceClass *device_class = NM_DEVICE_CLASS (ovs_port_class);
object_class->get_property = get_property;
object_class->dispose = dispose;
nm_object_class->init_dbus = init_dbus;
device_class->get_type_description = get_type_description;
device_class->connection_compatible = connection_compatible;
device_class->get_setting_type = get_setting_type;

View file

@ -14,7 +14,7 @@
* Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
* Boston, MA 02110-1301 USA.
*
* Copyright 2017 Red Hat, Inc.
* Copyright 2017,2018 Red Hat, Inc.
*/
#ifndef __NM_DEVICE_OVS_PORT_H__
@ -35,11 +35,16 @@ G_BEGIN_DECLS
#define NM_IS_DEVICE_OVS_PORT_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE ((klass), NM_TYPE_DEVICE_OVS_PORT))
#define NM_DEVICE_OVS_PORT_GET_CLASS(obj) (G_TYPE_INSTANCE_GET_CLASS ((obj), NM_TYPE_DEVICE_OVS_PORT, NMDeviceOvsPortClass))
#define NM_DEVICE_OVS_PORT_SLAVES "slaves"
typedef struct _NMDeviceOvsPortClass NMDeviceOvsPortClass;
NM_AVAILABLE_IN_1_10
GType nm_device_ovs_port_get_type (void);
NM_AVAILABLE_IN_1_14
const GPtrArray *nm_device_ovs_port_get_slaves (NMDeviceOvsPort *device);
G_END_DECLS
#endif /* __NM_DEVICE_OVS_PORT_H__ */

View file

@ -120,7 +120,8 @@ nm_device_state_reason_check (NMDeviceStateReason reason)
#define NM_DEVICE_PARENT "parent"
/* the "slaves" property is internal in the parent class, but exposed
* by the derived classes NMDeviceBond, NMDeviceBridge and NMDeviceTeam. */
* by the derived classes NMDeviceBond, NMDeviceBridge, NMDeviceTeam,
* NMDeviceOvsBridge and NMDeviceOvsPort. */
#define NM_DEVICE_SLAVES "slaves" /* partially internal */
#define NM_DEVICE_TYPE_DESC "type-desc" /* Internal only */

View file

@ -134,6 +134,9 @@ nm_device_ovs_bridge_init (NMDeviceOvsBridge *self)
static const NMDBusInterfaceInfoExtended interface_info_device_ovs_bridge = {
.parent = NM_DEFINE_GDBUS_INTERFACE_INFO_INIT (
NM_DBUS_INTERFACE_DEVICE_OVS_BRIDGE,
.properties = NM_DEFINE_GDBUS_PROPERTY_INFOS (
NM_DEFINE_DBUS_PROPERTY_INFO_EXTENDED_READABLE ("Slaves", "ao", NM_DEVICE_SLAVES),
),
.signals = NM_DEFINE_GDBUS_SIGNAL_INFOS (
&nm_signal_info_property_changed_legacy,
),

View file

@ -180,6 +180,9 @@ nm_device_ovs_port_init (NMDeviceOvsPort *self)
static const NMDBusInterfaceInfoExtended interface_info_device_ovs_port = {
.parent = NM_DEFINE_GDBUS_INTERFACE_INFO_INIT (
NM_DBUS_INTERFACE_DEVICE_OVS_PORT,
.properties = NM_DEFINE_GDBUS_PROPERTY_INFOS (
NM_DEFINE_DBUS_PROPERTY_INFO_EXTENDED_READABLE ("Slaves", "ao", NM_DEVICE_SLAVES),
),
.signals = NM_DEFINE_GDBUS_SIGNAL_INFOS (
&nm_signal_info_property_changed_legacy,
),