On Thu, 2012-03-01 at 15:40 -0800, Javier Cardona wrote:
> Without this you won't see probe responses in an IBSS/hwsim network,
> which is convenient for testing.
>
> Signed-off-by: Javier Cardona <javier@xxxxxxxxxxx>
> ---
> drivers/net/wireless/mac80211_hwsim.c | 9 +++++++++
> 1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/mac80211_hwsim.c b/drivers/net/wireless/mac80211_hwsim.c
> index deed95f..e4b6ca6 100644
> --- a/drivers/net/wireless/mac80211_hwsim.c
> +++ b/drivers/net/wireless/mac80211_hwsim.c
> @@ -47,6 +47,8 @@ static bool fake_hw_scan;
> module_param(fake_hw_scan, bool, 0444);
> MODULE_PARM_DESC(fake_hw_scan, "Install fake (no-op) hw-scan handler");
>
> +static struct ieee80211_hw *txed_last_beacon;
That's definitely not right.
> /**
> * enum hwsim_regtest - the type of regulatory tests we offer
> *
> @@ -821,6 +823,7 @@ static void mac80211_hwsim_beacon_tx(void *arg, u8 *mac,
> hdr = (struct ieee80211_hdr_3addr *) skb->data;
> tsfpos = (__le64 *) (hdr + 1);
> put_unaligned(__mac80211_hwsim_get_tsf(data), tsfpos);
> + txed_last_beacon = hw;
>
> mac80211_hwsim_monitor_rx(hw, skb);
>
> @@ -1220,6 +1223,11 @@ static void mac80211_hwsim_sw_scan_complete(struct ieee80211_hw *hw)
> mutex_unlock(&hwsim->mutex);
> }
>
> +static int mac80211_hwsim_tx_last_beacon(struct ieee80211_hw *hw)
> +{
> + return (hw == txed_last_beacon);
> +}
And this is mostly a fancy way of saying "return 1" -- there's no beacon
backoff protocol in here, so why not just hardcode "return 1"?
johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
