linux-kernel - Re: [PATCH] xsk: ixgbe: solve the soft interrupt 100% CPU usage when xdp rx traffic congestion

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sat, 11 Jul 2020 11:14:21 -0700
From:   Jakub Kicinski <kuba@...nel.org>
To:     Yahui Chen <goodluckwillcomesoon@...il.com>
Cc:     Jeff Kirsher <jeffrey.t.kirsher@...el.com>,
        "David S. Miller" <davem@...emloft.net>,
        Alexei Starovoitov <ast@...nel.org>,
        Daniel Borkmann <daniel@...earbox.net>,
        Jesper Dangaard Brouer <hawk@...nel.org>,
        John Fastabend <john.fastabend@...il.com>,
        intel-wired-lan@...ts.osuosl.org, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org, bpf@...r.kernel.org,
        steven.zou@...el.com
Subject: Re: [PATCH] xsk: ixgbe: solve the soft interrupt 100% CPU usage
 when xdp rx traffic congestion

On Sat, 11 Jul 2020 18:10:38 +0800 Yahui Chen wrote:
> 2. If the wakeup mechanism is used, that is, use the
> `XDP_UMEM_USES_NEED_WAKEUP` flag. This method takes advantage of the
> interrupt delay function of ixgbe skillfully, thus solving the problem
> that the si CPU is always 100%. However, it will cause other problems.
> The port-level flow control will be triggered on 82599, and the pause
> frame will be sent to the upstream sender. This will affect the other
> packet receiving queues of the network card, resulting in the packet
> receiving rate of all queues dropping to 10Kpps.

To me the current behavior sounds correct.. if you don't want pause
frames to be generated you have to disable them completely. The point 
of pause frames is to prevent drops.