wlan: Fix arp offload filtering issue with static ip.
When an ap is roved out and roved in, while the DUT in suspended
state, the arp offload filter is not set. The arp filter is
reconfigured, when the ipv4 notifier is invoked and only when flag
sus_res_mcastbcast_filter_valid is set. This flag is set when
power state callback is invoked. There could be scenarios
where ipv4 notifier is invoked and the power state cal back is
invoked later or the ipv4 is invoked and the device is suspended
later leading the arp offload filter unset.
To address these, set sus_res_mcastbcast_filter_valid flag and
restore value of configuredMcastBcastFilter to sus_res_mcastbcast_filter
when ipv4 notifier is invoked.
Change-Id: I70c900dbf95da83ace2a264b034de42e97732b1d
CRs-Fixed: 593533
diff --git a/CORE/HDD/src/wlan_hdd_early_suspend.c b/CORE/HDD/src/wlan_hdd_early_suspend.c
index 9550b8b..efe7413 100644
--- a/CORE/HDD/src/wlan_hdd_early_suspend.c
+++ b/CORE/HDD/src/wlan_hdd_early_suspend.c
@@ -627,9 +627,16 @@
return;
}
+ if ( VOS_FALSE == pHddCtx->sus_res_mcastbcast_filter_valid)
+ {
+ pHddCtx->sus_res_mcastbcast_filter =
+ pHddCtx->configuredMcastBcastFilter;
+ pHddCtx->sus_res_mcastbcast_filter_valid = VOS_TRUE;
+ }
+
if ((eConnectionState_Associated ==
(WLAN_HDD_GET_STATION_CTX_PTR(pAdapter))->conn_info.connState)
- && (VOS_TRUE == pHddCtx->sus_res_mcastbcast_filter_valid))
+ && (pHddCtx->hdd_wlan_suspended))
{
// This invocation being part of the IPv6 registration callback,
// we are passing second parameter as 2 to avoid registration
@@ -873,9 +880,16 @@
return;
}
+ if ( VOS_FALSE == pHddCtx->sus_res_mcastbcast_filter_valid)
+ {
+ pHddCtx->sus_res_mcastbcast_filter =
+ pHddCtx->configuredMcastBcastFilter;
+ pHddCtx->sus_res_mcastbcast_filter_valid = VOS_TRUE;
+ }
+
if ((eConnectionState_Associated ==
(WLAN_HDD_GET_STATION_CTX_PTR(pAdapter))->conn_info.connState)
- && (VOS_TRUE == pHddCtx->sus_res_mcastbcast_filter_valid))
+ && (pHddCtx->hdd_wlan_suspended))
{
// This invocation being part of the IPv4 registration callback,
// we are passing second parameter as 2 to avoid registration