From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AE53EC433E0 for ; Wed, 1 Jul 2020 11:12:09 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7BCFC2067D for ; Wed, 1 Jul 2020 11:12:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="hx51FY+I" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730161AbgGALMI (ORCPT ); Wed, 1 Jul 2020 07:12:08 -0400 Received: from us-smtp-2.mimecast.com ([205.139.110.61]:52689 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1730054AbgGALMI (ORCPT ); Wed, 1 Jul 2020 07:12:08 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1593601925; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=Yexujkd7auN9ITgPnwA4kXggvTeorMC1CWNzXOrWnvM=; b=hx51FY+InFYWb/b2mFr5WLS/uXogj8g60xIVikBVxUDik2o3QdsQ5tR5UrEmSPQ+ZI4iU5 zaYmKu/8wywh2GkGQ/fKYXoo8C2WRILj7TwMYMVbBFjMhT6i82G5actO6Smhx2RwyszOTb wWgHQ0IqXbCxy0NVkUi9Cvj4pqAmemk= Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-446-VFS_VbANMmaUqY9tPewVCQ-1; Wed, 01 Jul 2020 07:12:03 -0400 X-MC-Unique: VFS_VbANMmaUqY9tPewVCQ-1 Received: by mail-wm1-f72.google.com with SMTP id y204so11663535wmd.2 for ; Wed, 01 Jul 2020 04:12:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=Yexujkd7auN9ITgPnwA4kXggvTeorMC1CWNzXOrWnvM=; b=QdlGWPHPzJT7E2qdFwbAj4KTnOwipXVQ3ByUCNe1JqQxkMMJK1jw/DZq2YXfovLwoP uDvDzsGnqp8KSw1/H+S6ISNh4926tbW3hQkyBOzfbwLstz+AgH1zYSQvRJU8cbcdovkX 11tnaH0Os/qxaKQAWhAN46/2zHX7szJN0BnMMcD5i1MGp6Ad2KR+TZnseBElX6Upd2Qz qj6z69NwtyzaCsP+J5n9m6MFDhe+NFKvS3vg16ZFJmHaDrRZ+DzuEyqHSa/YtXPR3XB6 vH7yetYaJ/tPnlA88R24BLOCfGQ2de8Jas3l5kAZixbP3ckKda3LIJTqPA2XiQ/vW5tc 3QRg== X-Gm-Message-State: AOAM530CdlcPOf97O5SrgoV0SYwJ7LhBhWVusSf0qenHUSjzFj9rToFO 576IYdjKFfPl1y4Bm6UHYZPzOWic5rKib1fLxFPaKFQu1+BhcUkx9eI4UAEK05XPUWniu/wcqwM AnC9CCPfwyNsmTGXf X-Received: by 2002:a1c:398b:: with SMTP id g133mr25236184wma.76.1593601922472; Wed, 01 Jul 2020 04:12:02 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxQMUaayG0vDg3RNtuezDlgxdvof9uLCFzLvN6HKEyQRgj3zoZo7nehIzUAAPphqwvJnljolw== X-Received: by 2002:a1c:398b:: with SMTP id g133mr25236149wma.76.1593601922091; Wed, 01 Jul 2020 04:12:02 -0700 (PDT) Received: from redhat.com (bzq-79-182-31-92.red.bezeqint.net. [79.182.31.92]) by smtp.gmail.com with ESMTPSA id 140sm1930942wmb.15.2020.07.01.04.11.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Jul 2020 04:12:01 -0700 (PDT) Date: Wed, 1 Jul 2020 07:11:57 -0400 From: "Michael S. Tsirkin" To: Eugenio Perez Martin Cc: Konrad Rzeszutek Wilk , linux-kernel@vger.kernel.org, kvm list , virtualization@lists.linux-foundation.org, netdev@vger.kernel.org, Jason Wang Subject: Re: [PATCH RFC v8 02/11] vhost: use batched get_vq_desc version Message-ID: <20200701071041-mutt-send-email-mst@kernel.org> References: <20200611113404.17810-1-mst@redhat.com> <20200611113404.17810-3-mst@redhat.com> <20200611152257.GA1798@char.us.oracle.com> <20200622114622-mutt-send-email-mst@kernel.org> <20200622122546-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Wed, Jul 01, 2020 at 12:43:09PM +0200, Eugenio Perez Martin wrote: > On Tue, Jun 23, 2020 at 6:15 PM Eugenio Perez Martin > wrote: > > > > On Mon, Jun 22, 2020 at 6:29 PM Michael S. Tsirkin wrote: > > > > > > On Mon, Jun 22, 2020 at 06:11:21PM +0200, Eugenio Perez Martin wrote: > > > > On Mon, Jun 22, 2020 at 5:55 PM Michael S. Tsirkin wrote: > > > > > > > > > > On Fri, Jun 19, 2020 at 08:07:57PM +0200, Eugenio Perez Martin wrote: > > > > > > On Mon, Jun 15, 2020 at 2:28 PM Eugenio Perez Martin > > > > > > wrote: > > > > > > > > > > > > > > On Thu, Jun 11, 2020 at 5:22 PM Konrad Rzeszutek Wilk > > > > > > > wrote: > > > > > > > > > > > > > > > > On Thu, Jun 11, 2020 at 07:34:19AM -0400, Michael S. Tsirkin wrote: > > > > > > > > > As testing shows no performance change, switch to that now. > > > > > > > > > > > > > > > > What kind of testing? 100GiB? Low latency? > > > > > > > > > > > > > > > > > > > > > > Hi Konrad. > > > > > > > > > > > > > > I tested this version of the patch: > > > > > > > https://lkml.org/lkml/2019/10/13/42 > > > > > > > > > > > > > > It was tested for throughput with DPDK's testpmd (as described in > > > > > > > http://doc.dpdk.org/guides/howto/virtio_user_as_exceptional_path.html) > > > > > > > and kernel pktgen. No latency tests were performed by me. Maybe it is > > > > > > > interesting to perform a latency test or just a different set of tests > > > > > > > over a recent version. > > > > > > > > > > > > > > Thanks! > > > > > > > > > > > > I have repeated the tests with v9, and results are a little bit different: > > > > > > * If I test opening it with testpmd, I see no change between versions > > > > > > > > > > > > > > > OK that is testpmd on guest, right? And vhost-net on the host? > > > > > > > > > > > > > Hi Michael. > > > > > > > > No, sorry, as described in > > > > http://doc.dpdk.org/guides/howto/virtio_user_as_exceptional_path.html. > > > > But I could add to test it in the guest too. > > > > > > > > These kinds of raw packets "bursts" do not show performance > > > > differences, but I could test deeper if you think it would be worth > > > > it. > > > > > > Oh ok, so this is without guest, with virtio-user. > > > It might be worth checking dpdk within guest too just > > > as another data point. > > > > > > > Ok, I will do it! > > > > > > > > * If I forward packets between two vhost-net interfaces in the guest > > > > > > using a linux bridge in the host: > > > > > > > > > > And here I guess you mean virtio-net in the guest kernel? > > > > > > > > Yes, sorry: Two virtio-net interfaces connected with a linux bridge in > > > > the host. More precisely: > > > > * Adding one of the interfaces to another namespace, assigning it an > > > > IP, and starting netserver there. > > > > * Assign another IP in the range manually to the other virtual net > > > > interface, and start the desired test there. > > > > > > > > If you think it would be better to perform then differently please let me know. > > > > > > > > > Not sure why you bother with namespaces since you said you are > > > using L2 bridging. I guess it's unimportant. > > > > > > > Sorry, I think I should have provided more context about that. > > > > The only reason to use namespaces is to force the traffic of these > > netperf tests to go through the external bridge. To test netperf > > different possibilities than the testpmd (or pktgen or others "blast > > of frames unconditionally" tests). > > > > This way, I make sure that is the same version of everything in the > > guest, and is a little bit easier to manage cpu affinity, start and > > stop testing... > > > > I could use a different VM for sending and receiving, but I find this > > way a faster one and it should not introduce a lot of noise. I can > > test with two VM if you think that this use of network namespace > > introduces too much noise. > > > > Thanks! > > > > > > > > > > > > > - netperf UDP_STREAM shows a performance increase of 1.8, almost > > > > > > doubling performance. This gets lower as frame size increase. > > Regarding UDP_STREAM: > * with event_idx=on: The performance difference is reduced a lot if > applied affinity properly (manually assigning CPU on host/guest and > setting IRQs on guest), making them perform equally with and without > the patch again. Maybe the batching makes the scheduler perform > better. > > > > > > > - rests of the test goes noticeably worse: UDP_RR goes from ~6347 > > > > > > transactions/sec to 5830 > > * Regarding UDP_RR, TCP_STREAM, and TCP_RR, proper CPU pinning makes > them perform similarly again, only a very small performance drop > observed. It could be just noise. > ** All of them perform better than vanilla if event_idx=off, not sure > why. I can try to repeat them if you suspect that can be a test > failure. > > * With testpmd and event_idx=off, if I send from the VM to host, I see > a performance increment especially in small packets. The buf api also > increases performance compared with only batching: Sending the minimum > packet size in testpmd makes pps go from 356kpps to 473 kpps. Sending > 1024 length UDP-PDU makes it go from 570kpps to 64 kpps. > > Something strange I observe in these tests: I get more pps the bigger > the transmitted buffer size is. Not sure why. > > ** Sending from the host to the VM does not make a big change with the > patches in small packets scenario (minimum, 64 bytes, about 645 > without the patch, ~625 with batch and batch+buf api). If the packets > are bigger, I can see a performance increase: with 256 bits, it goes > from 590kpps to about 600kpps, and in case of 1500 bytes payload it > gets from 348kpps to 528kpps, so it is clearly an improvement. > > * with testpmd and event_idx=on, batching+buf api perform similarly in > both directions. > > All of testpmd tests were performed with no linux bridge, just a > host's tap interface ( in xml), with a > testpmd txonly and another in rxonly forward mode, and using the > receiving side packets/bytes data. Guest's rps, xps and interrupts, > and host's vhost threads affinity were also tuned in each test to > schedule both testpmd and vhost in different processors. > > I will send the v10 RFC with the small changes requested by Stefan and Jason. > > Thanks! > OK so there's a chance you are seeing effects of an aggressive power management. which tuned profile are you using? It might be helpful to disable PM/frequency scaling. > > > > > > > > > > > > > > > OK so it seems plausible that we still have a bug where an interrupt > > > > > is delayed. That is the main difference between pmd and virtio. > > > > > Let's try disabling event index, and see what happens - that's > > > > > the trickiest part of interrupts. > > > > > > > > > > > > > Got it, will get back with the results. > > > > > > > > Thank you very much! > > > > > > > > > > > > > > > > > > > > - TCP_STREAM goes from ~10.7 gbps to ~7Gbps > > > > > > - TCP_RR from 6223.64 transactions/sec to 5739.44 > > > > > > > >