Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 27 Jul 2016 20:43:08 +0000 (UTC)
From:      =?UTF-8?Q?Imre_Vad=c3=a1sz?= <ivadasz@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r303413 - head/sys/dev/iwm
Message-ID:  <201607272043.u6RKh8P5093441@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ivadasz
Date: Wed Jul 27 20:43:08 2016
New Revision: 303413
URL: https://svnweb.freebsd.org/changeset/base/303413

Log:
  [iwm] Fix inverted logic in iwm_tx().
  
      The PROT_REQUIRE flag in should be set for data frames above a certain
      length, but we were setting it for !data frames above a certain length,
      which makes no sense at all.
  
      Taken-From: OpenBSD, Linux iwlwifi
  
  Approved by:	adrian (mentor)
  Obtained from:	DragonFlyBSD git 8cc03924a36c572c2908e659e624f44636dc2b33
  Differential Revision:	https://reviews.freebsd.org/D7323

Modified:
  head/sys/dev/iwm/if_iwm.c

Modified: head/sys/dev/iwm/if_iwm.c
==============================================================================
--- head/sys/dev/iwm/if_iwm.c	Wed Jul 27 20:37:32 2016	(r303412)
+++ head/sys/dev/iwm/if_iwm.c	Wed Jul 27 20:43:08 2016	(r303413)
@@ -3344,7 +3344,7 @@ iwm_tx(struct iwm_softc *sc, struct mbuf
 		flags |= IWM_TX_CMD_FLG_ACK;
 	}
 
-	if (type != IEEE80211_FC0_TYPE_DATA
+	if (type == IEEE80211_FC0_TYPE_DATA
 	    && (totlen + IEEE80211_CRC_LEN > vap->iv_rtsthreshold)
 	    && !IEEE80211_IS_MULTICAST(wh->i_addr1)) {
 		flags |= IWM_TX_CMD_FLG_PROT_REQUIRE;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201607272043.u6RKh8P5093441>