On Tue, Apr 17, 2012 at 11:27:01AM +0800, Jason Wang wrote:On 04/16/2012 09:39 PM, Michael S. Tsirkin wrote:Sounds OK. BTW how do you test this?On Mon, Apr 16, 2012 at 04:28:10PM +0800, Jason Wang wrote:Right, so do you think it makes sense that we only restart pollingDoes same thing happen if we get an error from copy from user?On 04/16/2012 03:16 PM, Michael S. Tsirkin wrote:-EFAULT, vhost>On Mon, Apr 16, 2012 at 02:08:33PM +0800, Jason Wang wrote:> This issue is when guest driver is able to hit the>>Currently, we restart tx polling unconditionally when sendmsg()>Why is this a problem?
>>fails. This would cause unnecessary wakeups of vhost wokers as it's
>>only needed when the socket send buffer were exceeded.
discard the the descriptor and restart the polling. This would wake
vhost thread and repeat the loop again which waste cpu.
on -EAGAIN or -ENOBUFS?