armbian/linux-rockchip
0
0
Fork 0
mirror of https://github.com/armbian/linux-rockchip.git synced 2026-01-06 11:08:10 -08:00
Code Issues Packages Projects Releases Wiki Activity

ANDROID: GKI: fix up abi breakage from "cfg80211: fix management registrations locking"

Browse Source 
Commit 4c22227e39 ("cfg80211: fix management registrations locking")
in 5.10.77, which is commit 09b1d5dc6c ("cfg80211: fix management
registrations locking") upstream, moved the mgmt_registrations_lock out
of struct wireless_dev.  This did not change the structure size, but
caused a CRC change in the structure pointer, which tripped the abi
checker.

So restore the variable, and comment that this is not used anymore.  No
driver should ever be touching this so it is safe.

Note, using a __GENKSYMS__ hack would work, as the overall structure
size remains the same when this variable is removed, BUT the warning
that the field is gone is reported by libabigail correctly, which would
require a .xml update, so it's not worth it.

Bug: 161946584
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Change-Id: Ibcfe743833860e6f75611d7ebed1036a77be0a69
This commit is contained in:
Greg Kroah-Hartman
2021-11-03 10:28:52 +01:00
parent a7d87a08b4
commit 5b87b8d669
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
include/net/cfg80211.h
View File

@@ -5305,6 +5305,13 @@ struct wireless_dev {
u32 identifier;
struct list_head mgmt_registrations;
/*
* ANDROID: mgmt_registrations_lock was restored to preserve ABI in
* 5.10.77 due to backport of 09b1d5dc6ce1 ("cfg80211: fix management
* registrations locking") but it is not used for anything so do not
* touch this variable!
*/
spinlock_t mgmt_registrations_lock;
u8 mgmt_registrations_need_update:1;
struct mutex mtx;