From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: Latest net-next kernel 4.19.0+ Date: Tue, 30 Oct 2018 10:32:39 -0700 Message-ID: References: <59d5657c-ea0a-7b64-d5ff-5b55eb4fcccf@itcare.pl> <1e954663-ed05-4f33-4384-db880844f9d1@gmail.com> <68f25a28-b79e-d3ae-6eef-50c354ad63ae@gmail.com> <76dfbbda-d7f1-b13a-5921-c12c3b0f8e3e@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Cc: =?UTF-8?Q?Pawe=C5=82_Staszewski?= , dmichail@google.com, Linux Kernel Network Developers To: Eric Dumazet Return-path: Received: from mail-pl1-f194.google.com ([209.85.214.194]:44108 "EHLO mail-pl1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726532AbeJaC1P (ORCPT ); Tue, 30 Oct 2018 22:27:15 -0400 Received: by mail-pl1-f194.google.com with SMTP id s5-v6so4241571plq.11 for ; Tue, 30 Oct 2018 10:32:52 -0700 (PDT) In-Reply-To: <76dfbbda-d7f1-b13a-5921-c12c3b0f8e3e@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Oct 30, 2018 at 7:16 AM Eric Dumazet wrote= : > > > > On 10/30/2018 01:09 AM, Pawe=C5=82 Staszewski wrote: > > > > > > W dniu 30.10.2018 o 08:29, Eric Dumazet pisze: > >> > >> On 10/29/2018 11:09 PM, Dimitris Michailidis wrote: > >> > >>> Indeed this is a bug. I would expect it to produce frequent errors > >>> though as many odd-length > >>> packets would trigger it. Do you have RXFCS? Regardless, how > >>> frequently do you see the problem? > >>> > >> Old kernels (before 88078d98d1bb) were simply resetting ip_summed to C= HECKSUM_NONE > >> > >> And before your fix (commit d55bef5059dd057bd), mlx5 bug was canceling= the bug you fixed. > >> > >> So we now need to also fix mlx5. > >> > >> And of course use skb_header_pointer() in mlx5e_get_fcs() as I mention= ed earlier, > >> plus __get_unaligned_cpu32() as you hinted. > >> > >> > >> > >> > > > > No RXFCS Same with Pawel, RXFCS is disabled by default. > > > > And this trace is rly frequently like once per 3/4 seconds > > like below: > > [28965.776864] vlan1490: hw csum failure > > Might be vlan related. Unlike Pawel's case, we don't use vlan at all, maybe this is why we see it much less frequently than Pawel. Also, it is probably not specific to mlx5, as there is another report which is probably a non-mlx5 driver. Thanks.