Hi Adrian, thanks for having a look. No, if you look into dfs_pri_detector.c, you'll find some words on the pulse and sequence pools (see DOC below). Actually, ETSI pulses even have a PRF of up to 4000 pulses per second, you'd go crazy allocating and freeing them continuously. With the pools every instance is allocated once during a detector's lifetime and is then provided to all pri_detectors commonly. The amount of allocated memory grows up to the maximum simultaneously used instances. This is limited by numerous means (sliding window, max tolerated PRF, max pulses per window, etc.). I did intensive memory allocation tracing when evaluating the detectors in user-space with valgrind and the absolute maximum memory load in worst-case (i.e. all DFS channels being flooded with pulse noise superposed with radar patterns) had an instance count for pulses and sequences of less than 1.5k, the maximum memory allocated does not exceed 30kB. While that last number might differ based on the used slab allocator, it is proven that the pool has a limited growth. I planned to add some allocation statistics in the debugfs later. Thanks, Zefir On 03/30/2012 08:04 AM, Adrian Chadd wrote: > Hiya, > > I know this is late, but I've been a bit busy elsewhere. :) > > Does pulse_queue_enqueue() always allocate a new entry? What situation > are pulse entries reused? > > You likely don't want to be allocating a new pulse entry via malloc() > for each pulse; it's going to be quite silly when you potentially > handle > 1,000 false positives a second. > > > Adrian > > > On 23 March 2012 08:34, Zefir Kurtisi <zefir.kurtisi@xxxxxxxxxxx> wrote: >> [...] >> + >> +/** >> + * DOC: Singleton Pulse and Sequence Pools >> + * >> + * Instances of pri_sequence and pulse_elem are kept in singleton pools to >> + * reduce the number of dynamic allocations. They are shared between all >> + * instances and grow up to the peak number of simultaneously used objects. >> + * >> + * Memory is freed after all references to the pools are released. >> + */ >> +static int singleton_pool_references; >> +static LIST_HEAD(pulse_pool); >> +static LIST_HEAD(pseq_pool); >> + >> [...] -- 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
