>>> @@ -1869,6 +1867,17 @@ int ath_rx_tasklet(struct ath_softc *sc, int flush, 
>>> bool hp)
>>>                    unlikely(tsf_lower - rs.rs_tstamp > 0x10000000))
>>>                        rxs->mactime += 0x100000000ULL;
>>>
>>> +               if ((rs.rs_status & ATH9K_RXERR_PHY) &&
>>> +                   (rs.rs_phyerr == ATH9K_PHYERR_RADAR)) {
>>> +                       /* DFS: check for radar pulse */
>>> +                       ath9k_dfs_process_phyerr(sc, hdr, &rs, 
>>> rxs->mactime);
>>> +               }
>>> +
>>> +               retval = ath9k_rx_skb_preprocess(common, hw, hdr, &rs,
>>> +                                                rxs, &decrypt_error);
>>> +               if (retval)
>>> +                       goto requeue_drop_frag;
>>
>> does the above fixes any bug related to mactime displayed in sniffer
>>
>>
>
> Hi Mohammed,
>
> no, like all other modifications within this patch the above does not fix 
> anything (at least not intentionally) ;)
>
> What it does is moving the calculation of rxs->mactime before 
> ath9k_rx_skb_preprocess(), since the mactime is needed to stamp the radar 
> event. So, the five lines marked as removals are actually moved further down 
> which shouldn't have any side effects.

sure, thanks a lot for your explanation ;)

>
>
> Thanks
> Zefir
>



-- 
shafi
_______________________________________________
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel

Reply via email to