Commit ce953204 authored by Bob Copeland's avatar Bob Copeland Committed by Johannes Berg
Browse files

mac80211: mesh: only get tsf if we need it 



The local TSF timer is used to compute the timing offset between
mesh peers on beacon reception.  However, asking the device for
the TSF is not very accurate, so we prefer to use rx->mactime
if available.  In the latter case, calling drv_get_tsf() just
adds more delay into the RX path, so skip it if we can.

Signed-off-by: default avatarBob Copeland <bob@cozybit.com>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent b176e629

net/mac80211/mesh_sync.c

+14 −14
Original line number Diff line number Diff line
@@ -92,11 +92,19 @@ static void mesh_sync_offset_rx_bcn_presp(struct ieee80211_sub_if_data *sdata, 
	if (stype != IEEE80211_STYPE_BEACON)

		return;



	/* The current tsf is a first approximation for the timestamp

	 * for the received beacon.  Further down we try to get a

	 * better value from the rx_status->mactime field if

	 * available. Also we have to call drv_get_tsf() before

	 * entering the rcu-read section.*/

	/*

	 * Get time when timestamp field was received.  If we don't

	 * have rx timestamps, then use current tsf as an approximation.

	 * drv_get_tsf() must be called before entering the rcu-read

	 * section.

	 */

	if (ieee80211_have_rx_timestamp(rx_status))

		t_r = ieee80211_calculate_rx_timestamp(local, rx_status,

						       24 + 12 +

						       elems->total_len +

						       FCS_LEN,

						       24);

	else

		t_r = drv_get_tsf(local, sdata);



	rcu_read_lock();
@@ -117,14 +125,6 @@ static void mesh_sync_offset_rx_bcn_presp(struct ieee80211_sub_if_data *sdata, 
		goto no_sync;

	}



	if (ieee80211_have_rx_timestamp(rx_status))

		/* time when timestamp field was received */

		t_r = ieee80211_calculate_rx_timestamp(local, rx_status,

						       24 + 12 +

						       elems->total_len +

						       FCS_LEN,

						       24);



	/* Timing offset calculation (see 13.13.2.2.2) */

	t_t = le64_to_cpu(mgmt->u.beacon.timestamp);

	sta->t_offset = t_t - t_r;

CRA Git | Maintained and supported by SUSTech CRA and CCSE