[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2] xen-netfront: Fix Rx stall during network stress and OOM
On 01/29/2017 03:09 PM, Boris Ostrovsky wrote: Thanks for catching this. In my testing I did not notice this - mostly because of the nature of the workload in my testing.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(). 2. It tickles a latent bug during resume where the timer triggers before we re-connect. The trouble is that we now try to dereference queue->rx.sring which is NULL since we disconnect in netfront_resume(). (Curiously, I only observe it with 32-bit guests)I think we may hit this bug after removing the timer as well. We call RING_PUSH_REQUESTS_AND_CHECK_NOTIFY soon after, which also dereference queue->rx.sring. Thanks, Vineeth _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |