netdev - [PATCHv3 0/1] Fix deadlock problem and make performance better

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-next>] [day] [month] [year] [list]

Message-Id: <1567674942-5132-1-git-send-email-yanjun.zhu@oracle.com>
Date:   Thu,  5 Sep 2019 05:15:41 -0400
From:   Zhu Yanjun <yanjun.zhu@...cle.com>
To:     eric.dumazet@...il.com, davem@...emloft.net, netdev@...r.kernel.org
Subject: [PATCHv3 0/1] Fix deadlock problem and make performance better

When running with about 1Gbit/ses for very long time, running ifconfig
and netstat causes dead lock. These symptoms are similar to the
commit 5f6b4e14cada ("net: dsa: User per-cpu 64-bit statistics"). After
replacing network devices statistics with per-cpu 64-bit statistics,
the dead locks disappear even after very long time running with 1Gbit/sec.

V2->V3:
Based on David's advice, "Never use the inline keyword in foo.c files,
let the compiler decide.".

The inline keyword is removed from the functions nv_get_stats and
rx_missing_handler.

V1->V2:
Based on Eric's advice, "If the loops are ever restarted, the
storage->fields will have been modified multiple times.".

A similar change in the commit 5f6b4e14cada ("net: dsa: User per-cpu
64-bit statistics") is borrowed to fix the above problem.

Zhu Yanjun (1):
  forcedeth: use per cpu to collect xmit/recv statistics

 drivers/net/ethernet/nvidia/forcedeth.c | 143 ++++++++++++++++++++++----------
 1 file changed, 99 insertions(+), 44 deletions(-)

-- 
2.7.4