Re: [PATCH net-next rfc V2 0/2] basic busy polling support for vhost_net
On 10/30/2015 07:58 PM, Jason Wang wrote: > > On 10/29/2015 04:45 PM, Jason Wang wrote: >> Hi all: >> >> This series tries to add basic busy polling for vhost net. The idea is >> simple: at the end of tx processing, busy polling for new tx added >> descriptor and rx receive socket for a while. The maximum number of >> time (in us) could be spent on busy polling was specified through >> module parameter. >> >> Test were done through: >> >> - 50 us as busy loop timeout >> - Netperf 2.6 >> - Two machines with back to back connected mlx4 >> - Guest with 8 vcpus and 1 queue >> >> Result shows very huge improvement on both tx (at most 158%) and rr >> (at most 53%) while rx is as much as in the past. Most cases the cpu >> utilization is also improved: >> > Just notice there's something wrong in the setup. So the numbers are > incorrect here. Will re-run and post correct number here. > > Sorry. Here's the updated testing result: 1) 1 vcpu 1 queue: TCP_RR size/session/+thu%/+normalize% 1/ 1/0%/ -25% 1/50/ +12%/0% 1/ 100/ +12%/ +1% 1/ 200/ +9%/ -1% 64/ 1/ +3%/ -21% 64/50/ +8%/0% 64/ 100/ +7%/0% 64/ 200/ +9%/0% 256/ 1/ +1%/ -25% 256/50/ +7%/ -2% 256/ 100/ +6%/ -2% 256/ 200/ +4%/ -2% 512/ 1/ +2%/ -19% 512/50/ +5%/ -2% 512/ 100/ +3%/ -3% 512/ 200/ +6%/ -2% 1024/ 1/ +2%/ -20% 1024/50/ +3%/ -3% 1024/ 100/ +5%/ -3% 1024/ 200/ +4%/ -2% Guest RX size/session/+thu%/+normalize% 64/ 1/ -4%/ -5% 64/ 4/ -3%/ -10% 64/ 8/ -3%/ -5% 512/ 1/ +15%/ +1% 512/ 4/ -5%/ -5% 512/ 8/ -2%/ -4% 1024/ 1/ -5%/ -16% 1024/ 4/ -2%/ -5% 1024/ 8/ -6%/ -6% 2048/ 1/ +10%/ +5% 2048/ 4/ -8%/ -4% 2048/ 8/ -1%/ -4% 4096/ 1/ -9%/ -11% 4096/ 4/ +1%/ -1% 4096/ 8/ +1%/0% 16384/ 1/ +20%/ +11% 16384/ 4/0%/ -3% 16384/ 8/ +1%/0% 65535/ 1/ +36%/ +13% 65535/ 4/ -10%/ -9% 65535/ 8/ -3%/ -2% Guest TX size/session/+thu%/+normalize% 64/ 1/ -7%/ -16% 64/ 4/ -14%/ -23% 64/ 8/ -9%/ -20% 512/ 1/ -62%/ -56% 512/ 4/ -62%/ -56% 512/ 8/ -61%/ -53% 1024/ 1/ -66%/ -61% 1024/ 4/ -77%/ -73% 1024/ 8/ -73%/ -67% 2048/ 1/ -74%/ -75% 2048/ 4/ -77%/ -74% 2048/ 8/ -72%/ -68% 4096/ 1/ -65%/ -68% 4096/ 4/ -66%/ -63% 4096/ 8/ -62%/ -57% 16384/ 1/ -25%/ -28% 16384/ 4/ -28%/ -17% 16384/ 8/ -24%/ -10% 65535/ 1/ -17%/ -14% 65535/ 4/ -22%/ -5% 65535/ 8/ -25%/ -9% - obvious improvement on TCP_RR (at most 12%) - improvement on guest RX - huge decreasing on Guest TX (at most -75%), this is probably because virtio-net driver suffers from buffer bloat by orphaning skb before transmission. The faster vhost it is, the smaller packet it could produced. To reduce the impact on this, turning off gso in guest can result the following result: size/session/+thu%/+normalize% 64/ 1/ +3%/ -11% 64/ 4/ +4%/ -10% 64/ 8/ +4%/ -10% 512/ 1/ +2%/ +5% 512/ 4/0%/ -1% 512/ 8/0%/0% 1024/ 1/ +11%/0% 1024/ 4/0%/ -1% 1024/ 8/ +3%/ +1% 2048/ 1/ +4%/ -1% 2048/ 4/ +8%/ +3% 2048/ 8/0%/ -1% 4096/ 1/ +4%/ -1% 4096/ 4/ +1%/0% 4096/ 8/ +2%/0% 16384/ 1/ +2%/ -2% 16384/ 4/ +3%/ +1% 16384/ 8/0%/ -1% 65535/ 1/ +9%/ +7% 65535/ 4/0%/ -3% 65535/ 8/ -1%/ -1% 2) 8 vcpus 1 queue: TCP_RR size/session/+thu%/+normalize% 1/ 1/ +5%/ -14% 1/50/ +2%/ +1% 1/ 100/0%/ -1% 1/ 200/0%/0% 64/ 1/0%/ -25% 64/50/ +5%/ +5% 64/ 100/0%/0% 64/ 200/0%/ -1% 256/ 1/0%/ -30% 256/50/0%/0% 256/ 100/ -2%/ -2% 256/ 200/0%/0% 512/ 1/ +1%/ -23% 512/50/ +1%/ +1% 512/ 100/ +1%/0% 512/ 200/ +1%/ +1% 1024/ 1/ +1%/ -23% 1024/50/ +5%/ +5% 1024/ 100/0%/ -1% 1024/ 200/0%/0% Guest RX size/session/+thu%/+normalize% 64/ 1/ +1%/ +1% 64/ 4/ -2%/ +1% 64/ 8/ +6%/ +19% 512/ 1/ +5%/ -7% 512/ 4/ -4%/ -4% 512/ 8/0%/0% 1024/ 1/ +1%/ +2% 1024/ 4/ -2%/ -2% 1024/ 8/ -1%/ +7% 2048/ 1/ +8%/ -2% 2048/ 4/0%/ +5% 2048/ 8/ -1%/ +13% 4096/ 1/ -1%/ +2% 4096/ 4/0%/ +6% 4096/ 8/ -2%/ +15% 16384/ 1/ -1%/0% 16384/ 4/ -2%/ -1% 16384/ 8/ -2%/ +2% 65535/ 1/ -2%/0% 65535/ 4/ -3%/ -3% 65535/ 8/ -2%/ +2% Guest TX size/session/+thu%/+normalize% 64/ 1/ +6%/
Re: [PATCH net-next rfc V2 0/2] basic busy polling support for vhost_net
On 10/29/2015 04:45 PM, Jason Wang wrote: > Hi all: > > This series tries to add basic busy polling for vhost net. The idea is > simple: at the end of tx processing, busy polling for new tx added > descriptor and rx receive socket for a while. The maximum number of > time (in us) could be spent on busy polling was specified through > module parameter. > > Test were done through: > > - 50 us as busy loop timeout > - Netperf 2.6 > - Two machines with back to back connected mlx4 > - Guest with 8 vcpus and 1 queue > > Result shows very huge improvement on both tx (at most 158%) and rr > (at most 53%) while rx is as much as in the past. Most cases the cpu > utilization is also improved: > Just notice there's something wrong in the setup. So the numbers are incorrect here. Will re-run and post correct number here. Sorry. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH net-next rfc V2 0/2] basic busy polling support for vhost_net
Hi all: This series tries to add basic busy polling for vhost net. The idea is simple: at the end of tx processing, busy polling for new tx added descriptor and rx receive socket for a while. The maximum number of time (in us) could be spent on busy polling was specified through module parameter. Test were done through: - 50 us as busy loop timeout - Netperf 2.6 - Two machines with back to back connected mlx4 - Guest with 8 vcpus and 1 queue Result shows very huge improvement on both tx (at most 158%) and rr (at most 53%) while rx is as much as in the past. Most cases the cpu utilization is also improved: Guest TX: size/session/+thu%/+normalize% 64/ 1/ +17%/ +6% 64/ 4/ +9%/ +17% 64/ 8/ +34%/ +21% 512/ 1/ +48%/ +40% 512/ 4/ +31%/ +20% 512/ 8/ +39%/ +22% 1024/ 1/ +158%/ +99% 1024/ 4/ +20%/ +11% 1024/ 8/ +40%/ +18% 2048/ 1/ +108%/ +74% 2048/ 4/ +21%/ +7% 2048/ 8/ +32%/ +14% 4096/ 1/ +94%/ +77% 4096/ 4/ +7%/ -6% 4096/ 8/ +9%/ -4% 16384/ 1/ +33%/ +9% 16384/ 4/ +10%/ -6% 16384/ 8/ +19%/ +2% 65535/ 1/ +15%/ -6% 65535/ 4/ +8%/ -9% 65535/ 8/ +14%/0% Guest RX: size/session/+thu%/+normalize% 64/ 1/ -3%/ -3% 64/ 4/ +4%/ +20% 64/ 8/ -1%/ -1% 512/ 1/ +20%/ +12% 512/ 4/ +1%/ +3% 512/ 8/0%/ -5% 1024/ 1/ +9%/ -2% 1024/ 4/0%/ +5% 1024/ 8/ +1%/0% 2048/ 1/0%/ +3% 2048/ 4/ -2%/ +3% 2048/ 8/ -1%/ -3% 4096/ 1/ -8%/ +3% 4096/ 4/0%/ +2% 4096/ 8/0%/ +5% 16384/ 1/ +3%/0% 16384/ 4/ +2%/ +2% 16384/ 8/0%/ +13% 65535/ 1/0%/ +3% 65535/ 4/ +2%/ -1% 65535/ 8/ +1%/ +14% TCP_RR: size/session/+thu%/+normalize% 1/ 1/ +8%/ -6% 1/50/ +18%/ +15% 1/ 100/ +22%/ +19% 1/ 200/ +25%/ +23% 64/ 1/ +2%/ -19% 64/50/ +46%/ +39% 64/ 100/ +47%/ +39% 64/ 200/ +50%/ +44% 512/ 1/0%/ -28% 512/50/ +50%/ +44% 512/ 100/ +53%/ +47% 512/ 200/ +51%/ +58% 1024/ 1/ +3%/ -14% 1024/50/ +45%/ +37% 1024/ 100/ +53%/ +49% 1024/ 200/ +48%/ +55% Changes from V1: - Add a comment for vhost_has_work() to explain why it could be lockless - Add param description for busyloop_timeout - Split out the busy polling logic into a new helper - Check and exit the loop when there's a pending signal - Disable preemption during busy looping to make sure lock_clock() was correctly used. Todo: - Make the busyloop timeout could be configure per VM through ioctl. Please review. Thanks Jason Wang (2): vhost: introduce vhost_has_work() vhost_net: basic polling support drivers/vhost/net.c | 54 +++ drivers/vhost/vhost.c | 7 +++ drivers/vhost/vhost.h | 1 + 3 files changed, 58 insertions(+), 4 deletions(-) -- 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html