mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: alexander.levin@verizon.com
To: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"stable@vger.kernel.org" <stable@vger.kernel.org>
Cc: Masashi Honma <masashi.honma@gmail.com>,
	Johannes Berg <johannes.berg@intel.com>,
	alexander.levin@verizon.com
Subject: [PATCH AUTOSEL for 4.9 33/56] mac80211: Remove invalid flag operations in mesh TSF synchronization
Date: Wed, 15 Nov 2017 02:45:42 +0000	[thread overview]
Message-ID: <20171115024521.5884-33-alexander.levin@verizon.com> (raw)
In-Reply-To: <20171115024521.5884-1-alexander.levin@verizon.com>

From: Masashi Honma <masashi.honma@gmail.com>

[ Upstream commit 76f43b4c0a9337af22827d78de4f2b8fd5328489 ]

mesh_sync_offset_adjust_tbtt() implements Extensible synchronization
framework ([1] 13.13.2 Extensible synchronization framework). It shall
not operate the flag "TBTT Adjusting subfield" ([1] 8.4.2.100.8 Mesh
Capability), since it is used only for MBCA ([1] 13.13.4 Mesh beacon
collision avoidance, see 13.13.4.4.3 TBTT scanning and adjustment
procedures for detail). So this patch remove the flag operations.

[1] IEEE Std 802.11 2012

Signed-off-by: Masashi Honma <masashi.honma@gmail.com>
[remove adjusting_tbtt entirely, since it's now unused]
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
---
 net/mac80211/ieee80211_i.h |  1 -
 net/mac80211/mesh.c        |  3 ---
 net/mac80211/mesh_sync.c   | 11 -----------
 3 files changed, 15 deletions(-)

diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
index 34c2add2c455..03dbc6bd8598 100644
--- a/net/mac80211/ieee80211_i.h
+++ b/net/mac80211/ieee80211_i.h
@@ -681,7 +681,6 @@ struct ieee80211_if_mesh {
 	const struct ieee80211_mesh_sync_ops *sync_ops;
 	s64 sync_offset_clockdrift_max;
 	spinlock_t sync_offset_lock;
-	bool adjusting_tbtt;
 	/* mesh power save */
 	enum nl80211_mesh_power_mode nonpeer_pm;
 	int ps_peers_light_sleep;
diff --git a/net/mac80211/mesh.c b/net/mac80211/mesh.c
index 50e1b7f78bd4..5c67a696e046 100644
--- a/net/mac80211/mesh.c
+++ b/net/mac80211/mesh.c
@@ -279,8 +279,6 @@ int mesh_add_meshconf_ie(struct ieee80211_sub_if_data *sdata,
 	/* Mesh PS mode. See IEEE802.11-2012 8.4.2.100.8 */
 	*pos |= ifmsh->ps_peers_deep_sleep ?
 			IEEE80211_MESHCONF_CAPAB_POWER_SAVE_LEVEL : 0x00;
-	*pos++ |= ifmsh->adjusting_tbtt ?
-			IEEE80211_MESHCONF_CAPAB_TBTT_ADJUSTING : 0x00;
 	*pos++ = 0x00;
 
 	return 0;
@@ -850,7 +848,6 @@ int ieee80211_start_mesh(struct ieee80211_sub_if_data *sdata)
 	ifmsh->mesh_cc_id = 0;	/* Disabled */
 	/* register sync ops from extensible synchronization framework */
 	ifmsh->sync_ops = ieee80211_mesh_sync_ops_get(ifmsh->mesh_sp_id);
-	ifmsh->adjusting_tbtt = false;
 	ifmsh->sync_offset_clockdrift_max = 0;
 	set_bit(MESH_WORK_HOUSEKEEPING, &ifmsh->wrkq_flags);
 	ieee80211_mesh_root_setup(ifmsh);
diff --git a/net/mac80211/mesh_sync.c b/net/mac80211/mesh_sync.c
index faca22cd02b5..75608c07dc7b 100644
--- a/net/mac80211/mesh_sync.c
+++ b/net/mac80211/mesh_sync.c
@@ -123,7 +123,6 @@ static void mesh_sync_offset_rx_bcn_presp(struct ieee80211_sub_if_data *sdata,
 	 */
 
 	if (elems->mesh_config && mesh_peer_tbtt_adjusting(elems)) {
-		clear_sta_flag(sta, WLAN_STA_TOFFSET_KNOWN);
 		msync_dbg(sdata, "STA %pM : is adjusting TBTT\n",
 			  sta->sta.addr);
 		goto no_sync;
@@ -172,11 +171,9 @@ static void mesh_sync_offset_adjust_tbtt(struct ieee80211_sub_if_data *sdata,
 					 struct beacon_data *beacon)
 {
 	struct ieee80211_if_mesh *ifmsh = &sdata->u.mesh;
-	u8 cap;
 
 	WARN_ON(ifmsh->mesh_sp_id != IEEE80211_SYNC_METHOD_NEIGHBOR_OFFSET);
 	WARN_ON(!rcu_read_lock_held());
-	cap = beacon->meshconf->meshconf_cap;
 
 	spin_lock_bh(&ifmsh->sync_offset_lock);
 
@@ -190,21 +187,13 @@ static void mesh_sync_offset_adjust_tbtt(struct ieee80211_sub_if_data *sdata,
 			  "TBTT : kicking off TBTT adjustment with clockdrift_max=%lld\n",
 			  ifmsh->sync_offset_clockdrift_max);
 		set_bit(MESH_WORK_DRIFT_ADJUST, &ifmsh->wrkq_flags);
-
-		ifmsh->adjusting_tbtt = true;
 	} else {
 		msync_dbg(sdata,
 			  "TBTT : max clockdrift=%lld; too small to adjust\n",
 			  (long long)ifmsh->sync_offset_clockdrift_max);
 		ifmsh->sync_offset_clockdrift_max = 0;
-
-		ifmsh->adjusting_tbtt = false;
 	}
 	spin_unlock_bh(&ifmsh->sync_offset_lock);
-
-	beacon->meshconf->meshconf_cap = ifmsh->adjusting_tbtt ?
-			IEEE80211_MESHCONF_CAPAB_TBTT_ADJUSTING | cap :
-			~IEEE80211_MESHCONF_CAPAB_TBTT_ADJUSTING & cap;
 }
 
 static const struct sync_method sync_methods[] = {
-- 
2.11.0

  parent reply	other threads:[~2017-11-15  2:59 UTC|newest]

Thread overview: 83+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-15  2:45 [PATCH AUTOSEL for 4.9 01/56] ACPICA: Resources: Not a valid resource if buffer length too long alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 02/56] RDS: make message size limit compliant with spec alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 05/56] PCI: Apply _HPX settings only to relevant devices alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 04/56] RDS: RDMA: fix the ib_map_mr_sg_zbva() argument alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 03/56] RDS: RDMA: return appropriate error on rdma map failures alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 07/56] clk: sunxi-ng: A31: Fix spdif clock register alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 08/56] clk: sunxi-ng: fix PLL_CPUX adjusting on A33 alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 06/56] drm/sun4i: Fix a return value in case of error alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 11/56] fscrypt: use ENOTDIR when setting encryption policy on nondirectory alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 09/56] dmaengine: zx: set DMA_CYCLIC cap_mask bit alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 12/56] net: Allow IP_MULTICAST_IF to set index to L3 slave alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 10/56] fscrypt: use ENOKEY when file cannot be created w/o key alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 17/56] adm80211: return an error if adm8211_alloc_rings() fails alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 14/56] net: 3com: typhoon: typhoon_init_one: fix incorrect return values alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 16/56] rt2800: set minimum MPDU and PSDU lengths to sane values alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 13/56] net: 3com: typhoon: typhoon_init_one: make return values more specific alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 15/56] drm/armada: Fix compile fail alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 18/56] mwifiex: sdio: fix use after free issue for save_adapter alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 20/56] ath10k: ignore configuring the incorrect board_id alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 21/56] ath10k: fix potential memory leak in ath10k_wmi_tlv_op_pull_fw_stats() alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 19/56] ath10k: fix incorrect txpower set by P2P_DEVICE interface alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 22/56] pinctrl: sirf: atlas7: Add missing 'of_node_put()' alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 23/56] bnxt_en: Set default completion ring for async events alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 24/56] ath10k: set CTS protection VDEV param only if VDEV is up alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 25/56] ALSA: hda - Apply ALC269_FIXUP_NO_SHUTUP on HDA_FIXUP_ACT_PROBE alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 27/56] drm: Apply range restriction after color adjustment when allocation alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 28/56] drm/915: Parsing the missed out DTD fields from the VBT alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 26/56] gpio: mockup: dynamically allocate memory for chip name alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 29/56] clk: qcom: ipq4019: Add all the frequencies for apss cpu alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 30/56] drm/atomic: Unconditionally call prepare_fb alexander.levin
2017-11-15  2:45 ` alexander.levin [this message]
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 31/56] ASoC: da7218: Set DAI output pin high impedance when not in use alexander.levin
2017-11-15 12:19   ` Mark Brown
2017-11-15 16:43     ` alexander.levin
2017-11-15 16:55       ` Adam Thomson
2017-11-16 10:46         ` Mark Brown
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 32/56] drm/mediatek: don't use drm_put_dev alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 35/56] drm/i915: Protect DSPARB registers with a spinlock alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 37/56] adm80211: add checks for dma mapping errors alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 34/56] mac80211: Suppress NEW_PEER_CANDIDATE event if no room alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 36/56] drm/i915: Fix the level 0 max_wm hack on VLV/CHV alexander.levin
2017-11-15 11:08   ` Ville Syrjälä
2017-11-15 16:44     ` alexander.levin
2017-11-15 17:03       ` Ville Syrjälä
2017-11-17 11:28         ` Jani Nikula
2017-11-17 12:41           ` Greg KH
2017-11-17 12:53             ` Ville Syrjälä
2017-11-17 12:59               ` Greg KH
2017-11-17 13:13                 ` [Intel-gfx] " Emil Velikov
2017-11-17 13:58                   ` Greg KH
2017-11-17 13:01             ` Jani Nikula
2017-11-17 13:57               ` Greg KH
2017-11-18  3:15               ` alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 41/56] drm/i915: Protect dev_priv->atomic_cdclk_freq with all the crtc locks alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 39/56] staging: iio: cdc: fix improper return value alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 38/56] iio: light: " alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 40/56] drm/i915: Assert no external observers when unwind a failed request alloc alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 43/56] spi: SPI_FSL_DSPI should depend on HAS_DMA alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 45/56] netfilter: nft_queue: use raw_smp_processor_id() alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 42/56] drm/i915/bxt: Correct dual-link MIPI port control alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 44/56] ASoC: dwc: Fix PIO mode initialization alexander.levin
2017-11-15 10:54   ` Mark Brown
2017-11-15 12:46     ` Jose Abreu
2017-11-15 13:20       ` Mark Brown
2017-11-15 14:06         ` Jose Abreu
2017-11-15 16:41           ` alexander.levin
2017-11-16 10:51             ` Mark Brown
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 46/56] netfilter: nf_tables: fix oob access alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 48/56] crypto: marvell - Copy IVDIG before launching partial DMA ahash requests alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 49/56] btrfs: return the actual error value from from btrfs_uuid_tree_iterate alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 47/56] ASoC: rsnd: don't double free kctrl alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 51/56] s390/kbuild: enable modversions for symbols exported from asm alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 50/56] ASoC: wm_adsp: Don't overrun firmware file buffer when reading region data alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 53/56] [media] cec: CEC_MSG_GIVE_FEATURES should abort for CEC version < 2 alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 52/56] [media] cec: when canceling a message, don't overwrite old status info alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 56/56] xen: xenbus driver must not accept invalid transaction ids alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 54/56] [media] cec: update log_addr[] before finishing configuration alexander.levin
2017-11-15  2:45 ` [PATCH AUTOSEL for 4.9 55/56] nvmet: fix KATO offset in Set Features alexander.levin
     [not found] ` <94F2FBAB4432B54E8AACC7DFDE6C92E37E619757@ORSMSX110.amr.corp.intel.com>
2017-11-15 16:39   ` [PATCH AUTOSEL for 4.9 01/56] ACPICA: Resources: Not a valid resource if buffer length too long alexander.levin
2017-11-15 17:05     ` Schmauss, Erik
2017-11-17  0:56       ` alexander.levin
2017-11-20 17:22         ` Schmauss, Erik
2017-11-22 23:24           ` alexander.levin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171115024521.5884-33-alexander.levin@verizon.com \
    --to=alexander.levin@verizon.com \
    --cc=johannes.berg@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=masashi.honma@gmail.com \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®