Merge ~lihow731/network-manager:ubuntu/focal into network-manager:ubuntu/focal

Proposed by Leon Liao
Status: Merged
Merged at revision: b566e7b7e0fad691e11992caf150e3265b6c9faf
Proposed branch: ~lihow731/network-manager:ubuntu/focal
Merge into: network-manager:ubuntu/focal
Diff against target: 54 lines (+32/-0)
3 files modified
debian/changelog (+6/-0)
debian/patches/add-the-NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE-status-check.patch (+25/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Iain Lane Pending
Review via email: mp+390544@code.launchpad.net

This proposal supersedes a proposal from 2020-08-07.

Commit message

In some cases, the wow is not configured and the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE is set (for to disable management of wake-on-LAN in NetworkManager).

The network manager only checks the NM_SETTING_WIRELESS_WAKE_ON_WLAN_NONE bit.
But, the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE does not be check.
So, the management of wake-on-LAN still is done by NetworkManager.

Description of the change

This patch was sent to freedesktop and merged into ubuntu/master branch (in commit e7b44ef4c80907346ec7492a09c45277459924fc).
https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/597

For OEM project (Focal), we need this patch. So, please help view this patch for ubuntu/focal branch.

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote : Posted in a previous version of this proposal

Thanks. Can you please add some DEP-3 patch headers to the patch? Author, Description, Origin, Forwarded, Applied-Upstream at least.

Please could you expand the commit / changelog message to say what the *problem* as well as the solution is?

And it would be nice if you could submit to Debian too on Salsa, so (1) more people can benefit from the fix, and (2) it will be less delta.

Thanks again!

https://dep-team.pages.debian.net/deps/dep3/
https://salsa.debian.org/utopia-team/network-manager/

review: Needs Fixing
Revision history for this message
Leon Liao (lihow731) wrote :

@Iain,

I completed the DEP-3 patch header, please help to view it.

And, I found this patch already merged into ubuntu/master.
So, I resubmit this MP for ubuntu/focal branch.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 6692d9e..c1e89ca 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,9 @@
6+network-manager (1.22.10-1ubuntu2.2) UNRELEASED; urgency=medium
7+
8+ * platform: add the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE status check
9+
10+ -- Li-Hao Liao (Leon Liao) <leon.liao@canonical.com> Tue, 25 Aug 2020 15:29:09 +0800
11+
12 network-manager (1.22.10-1ubuntu2.1) focal; urgency=medium
13
14 * debian/rules:
15diff --git a/debian/patches/add-the-NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE-status-check.patch b/debian/patches/add-the-NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE-status-check.patch
16new file mode 100644
17index 0000000..82deca3
18--- /dev/null
19+++ b/debian/patches/add-the-NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE-status-check.patch
20@@ -0,0 +1,25 @@
21+Description: platform: add the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE status check
22+ In some cases, the wow is not configured and the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE is set. So, add the NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE status check.
23+Author: Li-Hao Liao (Leon Liao) <leon.liao@canonical.com>
24+Origin: upstream, https://gitlab.freedesktop.org/NetworkManager/NetworkManager
25+Forwarded: not-needed
26+Applied-Upstream: https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/commit/3ba31aad5817547058c9d04dd4f690c6bd488cb4
27+Last-Update: 2020-08-10
28+---
29+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
30+Index: network-manager/src/platform/nm-linux-platform.c
31+===================================================================
32+--- network-manager.orig/src/platform/nm-linux-platform.c
33++++ network-manager/src/platform/nm-linux-platform.c
34+@@ -8196,7 +8196,10 @@ link_get_wake_on_lan (NMPlatform *platfo
35+ if (!wifi_data)
36+ return FALSE;
37+
38+- return nm_wifi_utils_get_wake_on_wlan (wifi_data) != NM_SETTING_WIRELESS_WAKE_ON_WLAN_NONE;
39++ return !NM_IN_SET (nm_wifi_utils_get_wake_on_wlan (wifi_data),
40++ NM_SETTING_WIRELESS_WAKE_ON_WLAN_NONE,
41++ NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE);
42++
43+ } else
44+ return FALSE;
45+ }
46diff --git a/debian/patches/series b/debian/patches/series
47index f815073..b7f16d9 100644
48--- a/debian/patches/series
49+++ b/debian/patches/series
50@@ -4,4 +4,5 @@ Don-t-setup-Sleep-Monitor-if-not-booted-with-systemd.patch
51 # Ubuntu patches
52 Provide-access-to-some-of-NM-s-interfaces-to-whoopsie.patch
53 Update-dnsmasq-parameters.patch
54+add-the-NM_SETTING_WIRELESS_WAKE_ON_WLAN_IGNORE-status-check.patch
55

Subscribers

People subscribed via source and target branches

to all changes: