From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Ricardo Leitner Subject: Re: [PATCH net-next 0/5] virtio-net: Add SCTP checksum offload support Date: Tue, 17 Apr 2018 22:33:33 -0300 Message-ID: <20180418013333.GO4716@localhost.localdomain> References: <20180402134006.10111-1-vyasevic@redhat.com> <20180402144730.GA6001@localhost.localdomain> <6bc762f6-d6fb-5471-2893-a888cce199f9@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Vladislav Yasevich , netdev@vger.kernel.org, linux-sctp@vger.kernel.org, virtualization@lists.linux-foundation.org, mst@redhat.com, jasowang@redhat.com, nhorman@tuxdriver.com, Xin Long To: Vlad Yasevich Return-path: Received: from mail-qt0-f196.google.com ([209.85.216.196]:33696 "EHLO mail-qt0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751248AbeDRBm6 (ORCPT ); Tue, 17 Apr 2018 21:42:58 -0400 Content-Disposition: inline In-Reply-To: <6bc762f6-d6fb-5471-2893-a888cce199f9@redhat.com> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Apr 17, 2018 at 04:35:18PM -0400, Vlad Yasevich wrote: > On 04/02/2018 10:47 AM, Marcelo Ricardo Leitner wrote: > > On Mon, Apr 02, 2018 at 09:40:01AM -0400, Vladislav Yasevich wrote: > >> Now that we have SCTP offload capabilities in the kernel, we can add > >> them to virtio as well. First step is SCTP checksum. > > > > Thanks. > > > >> As for GSO, the way sctp GSO is currently implemented buys us nothing > >> in added support to virtio. To add true GSO, would require a lot of > >> re-work inside of SCTP and would require extensions to the virtio > >> net header to carry extra sctp data. > > > > Can you please elaborate more on this? Is this because SCTP GSO relies > > on the gso skb format for knowing how to segment it instead of having > > a list of sizes? > > > > it's mainly because all the true segmentation, placing data into chunks, > has already happened. All that GSO does is allow for higher bundling > rate between VMs. If that is all SCTP GSO ever going to do, that fine, > but the goal is to do real GSO eventually and potentially reduce the > amount of memory copying we are doing. > If we do that, any current attempt at GSO in virtio would have to be > depricated and we'd need GSO2 or something like that. > > This is why, after doing the GSO support, I decided not to include it. Gotcha. I don't think it will ever go further than what we have now. Placing data into chunks later is not really feasible/wanted, especially now with stream schedulers and idata chunks. Doesn't seem worth the hassle... we would have to support things like, "segment half of this message plus a third of this other one from that other stream." (in case it's round robin). Marcelo