From: Haiyang Zhang <[email protected]> Date: Tue, 3 Nov 2015 18:49:05 +0000
>> -----Original Message----- >> From: David Miller [mailto:[email protected]] >> Sent: Tuesday, November 3, 2015 1:20 PM >> To: Haiyang Zhang <[email protected]> >> Cc: [email protected]; KY Srinivasan <[email protected]>; >> [email protected]; [email protected] >> Subject: Re: [PATCH net-next] net: increase LL_MAX_HEADER if HYPERV_NET >> is enabled >> >> From: Haiyang Zhang <[email protected]> >> Date: Tue, 3 Nov 2015 17:34:47 +0000 >> >> > But we still keep this busy return in our code, just for "weird corner >> > cases". >> >> The queue_stopped condition must be precise. > > The only case we return NETDEV_TX_BUSY is when the outgoing ring buffer is > full, > which almost never happens because we stop the xmit queue if ring is <10% > available. I don't think you understand. You must perform the queue stop operation such that it is impossible for your ->ndo_start_xmit() method to be invoked in a way such that you cannot transmit the SKB given to you immediately. It's quite tiring to keep trying to explain this over and over repeatedly. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
