2015-09-29 10:30:12

by Zefir Kurtisi

[permalink] [raw]
Subject: [PATCH] ath: fix DFS timestamp wraparound reset condition

The DFS pattern detector ought to reset the
detector lines when a pulse is added with
lower time stamp than the previous (which
indicates a TSF restart).

This did not work so far and is fixed with
this patch.

The modification does not change detection
performance within the driver, since it
only ensures early reset (which is later
performed by the PRI detectors anyway).
It is relevant for synthetic tests and
statistical evaluations, where millions
of pulse patterns are processed and an
early reset helps reducing load.

Signed-off-by: Zefir Kurtisi <[email protected]>
---
drivers/net/wireless/ath/dfs_pattern_detector.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/dfs_pattern_detector.c b/drivers/net/wireless/ath/dfs_pattern_detector.c
index 656ce42..b52c209 100644
--- a/drivers/net/wireless/ath/dfs_pattern_detector.c
+++ b/drivers/net/wireless/ath/dfs_pattern_detector.c
@@ -290,10 +290,10 @@ dpd_add_pulse(struct dfs_pattern_detector *dpd, struct pulse_event *event)
if (cd == NULL)
return false;

- dpd->last_pulse_ts = event->ts;
/* reset detector on time stamp wraparound, caused by TSF reset */
if (event->ts < dpd->last_pulse_ts)
dpd_reset(dpd);
+ dpd->last_pulse_ts = event->ts;

/* do type individual pattern matching */
for (i = 0; i < dpd->num_radar_types; i++) {
--
2.5.0



2015-10-09 08:47:59

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] ath: fix DFS timestamp wraparound reset condition

Zefir Kurtisi <[email protected]> writes:

> The DFS pattern detector ought to reset the
> detector lines when a pulse is added with
> lower time stamp than the previous (which
> indicates a TSF restart).
>
> This did not work so far and is fixed with
> this patch.
>
> The modification does not change detection
> performance within the driver, since it
> only ensures early reset (which is later
> performed by the PRI detectors anyway).
> It is relevant for synthetic tests and
> statistical evaluations, where millions
> of pulse patterns are processed and an
> early reset helps reducing load.
>
> Signed-off-by: Zefir Kurtisi <[email protected]>

Applied, thanks.

--
Kalle Valo