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=-7.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 250D1C64E7B for ; Wed, 2 Dec 2020 13:25:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C14E2217A0 for ; Wed, 2 Dec 2020 13:25:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727466AbgLBNZK (ORCPT ); Wed, 2 Dec 2020 08:25:10 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60872 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726567AbgLBNZJ (ORCPT ); Wed, 2 Dec 2020 08:25:09 -0500 Received: from mail-il1-x143.google.com (mail-il1-x143.google.com [IPv6:2607:f8b0:4864:20::143]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9812BC0613CF for ; Wed, 2 Dec 2020 05:24:23 -0800 (PST) Received: by mail-il1-x143.google.com with SMTP id f5so1500988ilj.9 for ; Wed, 02 Dec 2020 05:24:23 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=Dv8ZiKgBcnrmHIpIdvmlabal0p64cd22xU1Edp1tW/Q=; b=ScPpKJHfvj+cLFIbyJZHKhcxkLb1V/F1eF8VrmPqvgZNdU4PvfX5CHT2oAM+eYI9ou Tnx43RsMsmqsy/5rAg6/XY38z9Zqf1AmvZ5MI9IPvw+D1mYPuX+h9J/14N9c4FtYBKe6 5UhoIjk/+0DWu/zVupOxZyxjVue3QTfKDohfRPwtVfmX8J9Bl2EQ0tBLvvN9+EPPQ/OC IHKi3sw5Jn1KUXW0gF7wR5Bcbn8/7v9iVzrRAgGdLovSyP88Tp2+AcGJDa/XFRS0B4jH o0s5eUyKjYW1fDjvm07vl+sfIopVjAZs+1fVmrrhYatfhInYC0mkfr7F6zjwwJo5uV/S wLSw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=Dv8ZiKgBcnrmHIpIdvmlabal0p64cd22xU1Edp1tW/Q=; b=Dn13FVJ119H0NFLcvbEWycCJHiMUQ3HDvgyFksIX8us8quSwtkSrvAhsCV0KvAPy3K B5cWSnOH4RqxVKkNVlOpFFTRusQl8nPQl9OASfFgD1JkUghuNuHefthIfGs2xaUNp0jY Yf7YVcWl38dFq6x7tbY4MrhE2I8ow4ivAGeTRidIvn9lZr9w4CVhDQ9+spgaAaR89vpo sxusYuiGm9UZE5+1n6hBVIba8ErTpXa08hmfm9EibiSMnQ5e7oY5yni3Ah7/7LJwBk3i qb5l7spdDVXd2P8fVZumw9EDg/KCwsuw/+ca05ghblaOiEhoeTcoBdi66QtM0nKcrhxK MAPw== X-Gm-Message-State: AOAM5301cfr5o8xr5pKq84TSHLccW6rDYOx8ZdM16YnBoQtpa5TBQq+a HilQqoAD7QFjqhzxr1SwG0gDyQ== X-Google-Smtp-Source: ABdhPJyA9HGXtcZqRuPGy5PU3kC1yKYFQgCr0Ya1Q48vab3miN+IIwUyIhQD8EqqZjwczpMEjL10Lw== X-Received: by 2002:a92:a154:: with SMTP id v81mr2398764ili.85.1606915462869; Wed, 02 Dec 2020 05:24:22 -0800 (PST) Received: from [172.22.22.26] (c-73-185-129-58.hsd1.mn.comcast.net. [73.185.129.58]) by smtp.googlemail.com with ESMTPSA id u11sm913650iol.51.2020.12.02.05.24.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 02 Dec 2020 05:24:22 -0800 (PST) Subject: Re: [PATCH net-next 2/2] net: ipa: add support for inline checksum offload To: Jakub Kicinski Cc: davem@davemloft.net, evgreen@chromium.org, cpratapa@codeaurora.org, bjorn.andersson@linaro.org, subashab@codeaurora.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: <20201201004143.27569-1-elder@linaro.org> <20201201004143.27569-3-elder@linaro.org> <20201201181319.41091a37@kicinski-fedora-pc1c0hjn.DHCP.thefacebook.com> From: Alex Elder Message-ID: Date: Wed, 2 Dec 2020 07:24:21 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.5.0 MIME-Version: 1.0 In-Reply-To: <20201201181319.41091a37@kicinski-fedora-pc1c0hjn.DHCP.thefacebook.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On 12/1/20 8:13 PM, Jakub Kicinski wrote: >> To function, the rmnet driver must also add support for this new >> "inline" checksum offload. The changes implementing this will be >> submitted soon. > We don't usually merge half of a feature. Why not wait until all > support is in place? > > Do I understand right that it's rmnet that will push the csum header? > This change seems to only reserve space for it and request the feature > at init.. You are correct. The IPA hardware needs to be programmed to perform the computation and verify that the checksum in the header matches what it computes (for AP RX offload), or insert it into the header (for AP TX offload). That's what this patch handles. The RMNet driver is responsible for stripping the offload header off on RX, and acting on what it says (i.e., whether hardware is able to state the checksum is good). And on TX it inserts an offload header that says what to checksum and where to put it in the packet. It's totally fine not to merge this until we have the whole package ready, I understand. I'll see what I can do to get that done quickly. Thanks Jakub. -Alex