Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-23 Thread Willem de Bruijn
; Michael S. Tsirkin ; > > virtualizat...@lists.linux-foundation.org; Lilijun (Jerry) > > ; chenchanghu ; > > xudingke ; huangbin (J) > > > > Subject: Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg > > fails > > > > > > On

RE: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-23 Thread wangyunjian
erry) > ; chenchanghu ; > xudingke ; huangbin (J) > > Subject: Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg > fails > > > On 2020/12/22 下午10:24, Willem de Bruijn wrote: > > On Mon, Dec 21, 2020 at 11:41 PM Jason Wang > wrote: > >> > >

Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-22 Thread Jason Wang
On 2020/12/22 下午10:24, Willem de Bruijn wrote: On Mon, Dec 21, 2020 at 11:41 PM Jason Wang wrote: On 2020/12/22 上午7:07, Willem de Bruijn wrote: On Wed, Dec 16, 2020 at 3:20 AM wangyunjian wrote: From: Yunjian Wang Currently we break the loop and wake up the vhost_worker when sendmsg fail

RE: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-22 Thread wangyunjian
erry) > ; chenchanghu ; > xudingke ; huangbin (J) > > Subject: Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg > fails > > > On 2020/12/22 上午7:07, Willem de Bruijn wrote: > > On Wed, Dec 16, 2020 at 3:20 AM wangyunjian > wrote: > >> From: Yun

Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-22 Thread Willem de Bruijn
On Mon, Dec 21, 2020 at 11:41 PM Jason Wang wrote: > > > On 2020/12/22 上午7:07, Willem de Bruijn wrote: > > On Wed, Dec 16, 2020 at 3:20 AM wangyunjian wrote: > >> From: Yunjian Wang > >> > >> Currently we break the loop and wake up the vhost_worker when > >> sendmsg fails. When the worker wakes u

Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-21 Thread Jason Wang
On 2020/12/22 上午7:07, Willem de Bruijn wrote: On Wed, Dec 16, 2020 at 3:20 AM wangyunjian wrote: From: Yunjian Wang Currently we break the loop and wake up the vhost_worker when sendmsg fails. When the worker wakes up again, we'll meet the same error. The patch is based on the assumption th

Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-21 Thread Willem de Bruijn
On Wed, Dec 16, 2020 at 3:20 AM wangyunjian wrote: > > From: Yunjian Wang > > Currently we break the loop and wake up the vhost_worker when > sendmsg fails. When the worker wakes up again, we'll meet the > same error. The patch is based on the assumption that such error cases always return EAGAI

Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-16 Thread Jason Wang
On 2020/12/16 下午5:23, Michael S. Tsirkin wrote: On Wed, Dec 16, 2020 at 04:20:37PM +0800, wangyunjian wrote: From: Yunjian Wang Currently we break the loop and wake up the vhost_worker when sendmsg fails. When the worker wakes up again, we'll meet the same error. This will cause high CPU load

RE: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-16 Thread wangyunjian
; > Lilijun (Jerry) ; chenchanghu > ; xudingke ; huangbin (J) > > Subject: Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg > fails > > On Wed, Dec 16, 2020 at 04:20:37PM +0800, wangyunjian wrote: > > From: Yunjian Wang > > > > Currently we

Re: [PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-16 Thread Michael S. Tsirkin
On Wed, Dec 16, 2020 at 04:20:37PM +0800, wangyunjian wrote: > From: Yunjian Wang > > Currently we break the loop and wake up the vhost_worker when > sendmsg fails. When the worker wakes up again, we'll meet the > same error. This will cause high CPU load. To fix this issue, > we can skip this de

[PATCH net v2 2/2] vhost_net: fix high cpu load when sendmsg fails

2020-12-16 Thread wangyunjian
From: Yunjian Wang Currently we break the loop and wake up the vhost_worker when sendmsg fails. When the worker wakes up again, we'll meet the same error. This will cause high CPU load. To fix this issue, we can skip this description by ignoring the error. When we exceeds sndbuf, the return value