| Message ID | 1434443643-28477-1-git-send-email-zefir.kurtisi@neratec.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Delegated to: | Kalle Valo |
| Headers | show |
Zefir Kurtisi <zefir.kurtisi@neratec.com> writes: > In the PRI detector, after the current radar pulse > has been checked agains existing PRI sequences, it > is considered as part of a new potential sequence. > > Previously, the condition to accept a new sequence > was to have at least the same number of pulses as > the longest matching sequence. This was wrong, > since it led to duplicates of PRI sequences. > > This patch changes the acceptance criteria for new > potential sequences from 'at least' to 'more than' > the longest existing. > > Detection performance remains unaffected, while > the number of PRI sequences accounted at runtime > (and with it CPU load) is reduced by up to 50%. > > Signed-off-by: Zefir Kurtisi <zefir.kurtisi@neratec.com> Thanks, applied to ath.git.
diff --git a/drivers/net/wireless/ath/dfs_pri_detector.c b/drivers/net/wireless/ath/dfs_pri_detector.c index 1b5ad19..cc5c592 100644 --- a/drivers/net/wireless/ath/dfs_pri_detector.c +++ b/drivers/net/wireless/ath/dfs_pri_detector.c @@ -273,7 +273,7 @@ static bool pseq_handler_create_sequences(struct pri_detector *pde, tmp_false_count++; } } - if (ps.count < min_count) + if (ps.count <= min_count) /* did not reach minimum count, drop sequence */ continue;
In the PRI detector, after the current radar pulse has been checked agains existing PRI sequences, it is considered as part of a new potential sequence. Previously, the condition to accept a new sequence was to have at least the same number of pulses as the longest matching sequence. This was wrong, since it led to duplicates of PRI sequences. This patch changes the acceptance criteria for new potential sequences from 'at least' to 'more than' the longest existing. Detection performance remains unaffected, while the number of PRI sequences accounted at runtime (and with it CPU load) is reduced by up to 50%. Signed-off-by: Zefir Kurtisi <zefir.kurtisi@neratec.com> --- drivers/net/wireless/ath/dfs_pri_detector.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)