diff options
| author | Thomas Haller <thaller@redhat.com> | 2015-10-16 11:52:30 +0200 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2015-10-19 12:24:39 +0200 |
| commit | 8a6afe92c18dfefa91d057518be00b0127720fa5 (patch) | |
| tree | 80fbb3bbf10403e07d74cd094184daafdea919cc | |
| parent | 571f0dba38a55cda33bae799dda49d5cb66c96ec (diff) | |
platform: fix link-detection for s390 CTC-type devices
Commit 02c6a9334335d3ef32c6cc8fafc6cea235c80ffc (rh#1212118)
introduced detection of CTC devices based on the driver name.
That was broken a while ago while refactoring.
Related: https://bugzilla.redhat.com/show_bug.cgi?id=1212118
https://bugzilla.redhat.com/show_bug.cgi?id=1272974
Fixes: ddaea22332907c05222cbec1e1b4365689fbda9f
| -rw-r--r-- | src/platform/nm-linux-platform.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/platform/nm-linux-platform.c b/src/platform/nm-linux-platform.c index 8b7a617e3a..ccd9fd0868 100644 --- a/src/platform/nm-linux-platform.c +++ b/src/platform/nm-linux-platform.c @@ -945,18 +945,18 @@ link_extract_type (NMPlatform *platform, struct rtnl_link *rtnllink, gboolean *c gs_free char *anycast_mask = NULL; gs_free char *devtype = NULL; - if (arptype == 256) { - /* Some s390 CTC-type devices report 256 for the encapsulation type - * for some reason, but we need to call them Ethernet. - */ - if (!g_strcmp0 (driver, "ctcm")) - return NM_LINK_TYPE_ETHERNET; - } - /* Fallback OVS detection for kernel <= 3.16 */ if (nmp_utils_ethtool_get_driver_info (ifname, &driver, NULL, NULL)) { if (!g_strcmp0 (driver, "openvswitch")) return NM_LINK_TYPE_OPENVSWITCH; + + if (arptype == 256) { + /* Some s390 CTC-type devices report 256 for the encapsulation type + * for some reason, but we need to call them Ethernet. + */ + if (!g_strcmp0 (driver, "ctcm")) + return NM_LINK_TYPE_ETHERNET; + } } sysfs_path = g_strdup_printf ("/sys/class/net/%s", ifname); |