On Mon, Dec 03, 2012 at 01:15:01PM +0800, Jason Wang wrote:
quoted
quoted
+
quoted
quoted
+ /* Work struct for refilling if we run low on memory. */
quoted
quoted
+ struct delayed_work refill;
quoted
quoted
I can't really see the justificaiton for a refill per queue. Just have
quoted
one work iterate all the queues if it happens, unless it happens often
quoted
(in which case, we need to look harder at this anyway).
But during this kind of iteration, we may need enable/disable the napi
regardless of whether the receive queue has lots to be refilled. This may add
extra latency.
We are running from the timer, so latency is not a concern I think.
--
MST