From mboxrd@z Thu Jan  1 00:00:00 1970
From: Cong Wang <xiyou.wangcong@gmail.com>
Subject: Re: [Patch net v3] mlx5: force CHECKSUM_NONE for short ethernet frames
Date: Tue, 4 Dec 2018 12:35:34 -0800
Message-ID: <CAM_iQpUqhaJ0_hJZfZoJtu50K5bae0oR5f1jAUVR1F7id6YLzQ@mail.gmail.com>
References: <20181204061405.16539-1-xiyou.wangcong@gmail.com>
 <CANn89iK7Z12KwFoH_R=Z5zkUUcG7en44pdtL4GxQ3YiFvDdzAw@mail.gmail.com>
 <CAM_iQpUzVH9MkXTB5XNsQobodrN6ZKYj1RKNMcDNEMivZXfGfA@mail.gmail.com>
 <CANn89iK0j=2LYK=szVO+Fpg1-tX=wSz+ghZx8RnwZSEbxZjf5w@mail.gmail.com>
 <CAM_iQpXq6LrgcQ0i0Vr9yXouWZ7k3Ad4YMRwEVRoG2UUZehO+w@mail.gmail.com>
 <CANn89i+fhMxXpJY_tU24r7tQVRVm1XCP+1_nJSj7_RDYvvOLHg@mail.gmail.com> <CALzJLG_7cA0dOwsH+bXGnQ-WaT60LLoBNhq=LPnMBCs_wm81QA@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Cc: Eric Dumazet <edumazet@google.com>,
        Linux Kernel Network Developers <netdev@vger.kernel.org>,
        Saeed Mahameed <saeedm@mellanox.com>,
        Tariq Toukan <tariqt@mellanox.com>
To: saeedm@dev.mellanox.co.il
Return-path: <netdev-owner@vger.kernel.org>
Received: from mail-pg1-f194.google.com ([209.85.215.194]:33629 "EHLO
        mail-pg1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1725867AbeLDUfq (ORCPT
        <rfc822;netdev@vger.kernel.org>); Tue, 4 Dec 2018 15:35:46 -0500
Received: by mail-pg1-f194.google.com with SMTP id z11so7911371pgu.0
        for <netdev@vger.kernel.org>; Tue, 04 Dec 2018 12:35:45 -0800 (PST)
In-Reply-To: <CALzJLG_7cA0dOwsH+bXGnQ-WaT60LLoBNhq=LPnMBCs_wm81QA@mail.gmail.com>
Sender: netdev-owner@vger.kernel.org
List-ID: <netdev.vger.kernel.org>

On Tue, Dec 4, 2018 at 11:17 AM Saeed Mahameed
<saeedm@dev.mellanox.co.il> wrote:
>
> On Mon, Dec 3, 2018 at 11:52 PM Eric Dumazet <edumazet@google.com> wrote:
> >
> > On Mon, Dec 3, 2018 at 11:30 PM Cong Wang <xiyou.wangcong@gmail.com> wrote:
> > >
> > > On Mon, Dec 3, 2018 at 11:08 PM Eric Dumazet <edumazet@google.com> wrote:
> > > >
> > > > The hardware has probably validated the L3 & L4 checksum just fine.
> > > >
> > > > Note that if ip_summed is CHECKSUM_UNNECESSARY, the padding bytes (if any)
> > > > have no impact on the csum that has been verified by the NIC.
> > >
> > >
> > > Why? Why does the hardware validates L3/L4 checksum when it
> > > supplies a full-packet checksum? What's its point here?
> >
> > The point is that the driver author can decide what is best.
> >
> > For native IP+TCP or IP+UDP, the NIC has the ability to fully
> > understand the packet and fully validate the checksum.
>
> Also for Native IP4 and IP6 plain L3 packets.
> The Hardware validates the csum when it can, and always provides
> checksum complete for all packets.
> One of the reason to validate is that sometimes we want to skip
> checksum complete, but still leverage the hw validation,
> like in your patch :), or LRO case, or many other cases in other
> kernels/OSes/drivers.

This sounds wrong to me too.

If the HW already validates it, the software doesn't need to do it,
therefore must skip hw csum for performance gain.,


>
> So i agree with Eric, let's jump to checksum_unnecessary for short packets.

This is odd, if Eric is right, then we should completely get rid of
CHECKSUM_COMPLETE. Short packets are not exceptions.

I still don't understand why people including Eric kept fixing this
thing which could be just removed from the very beginning.
Sounds like nobody even looked into it until my patch.

Thanks.