On 01/30/2017 08:47 AM, Vineeth Remanan Pillai wrote:
On 01/29/2017 03:09 PM, Boris Ostrovsky wrote:
There are couple of problems with this patch.
1. The 'if' clause now evaluates to true on pretty much every call to
xennet_alloc_rx_buffers().
Thanks for catching this. In my testing I did not notice this - mostly
because of the nature of the workload in my testing.
I am working on a patch to revert to the old behavior and solve the Rx
stall issue by scheduling the timer if any of the following
conditions are true:
- unconsumed requests + new requests < NET_RX_SLOTS_MIN (old behavior)
- skb allocations fail
Will send out the patch by next week after I can do some testing.
Thanks,
Vineeth
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel