netdev - [PATCH net 0/3] Revert openvswitch per-CPU storage

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: <20250610062631.1645885-1-gal@nvidia.com>
Date: Tue, 10 Jun 2025 09:26:28 +0300
From: Gal Pressman <gal@...dia.com>
To: "David S. Miller" <davem@...emloft.net>, Eric Dumazet
	<edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni
	<pabeni@...hat.com>, Andrew Lunn <andrew+netdev@...n.ch>,
	<netdev@...r.kernel.org>
CC: Aaron Conole <aconole@...hat.com>, Eelco Chaudron <echaudro@...hat.com>,
	Ilya Maximets <i.maximets@....org>, Simon Horman <horms@...nel.org>,
	Sebastian Andrzej Siewior <bigeasy@...utronix.de>, Clark Williams
	<clrkwllms@...nel.org>, Steven Rostedt <rostedt@...dmis.org>,
	<dev@...nvswitch.org>, <linux-rt-devel@...ts.linux.dev>, Gal Pressman
	<gal@...dia.com>
Subject: [PATCH net 0/3] Revert openvswitch per-CPU storage

This patch series reverts a set of changes that consolidated per-CPU
storage structures in the openvswitch module.

The original changes were intended to improve performance and reduce
complexity by merging three separate per-CPU structures into one, but
they have changed openvswitch to use static percpu allocations, and
exhausted the reserved chunk on module init.
This results in allocation of struct ovs_pcpu_storage (6488 bytes)
failure on ARM.

The reverts are applied in reverse order of the original commits.

Gal Pressman (3):
  Revert "openvswitch: Move ovs_frag_data_storage into the struct
    ovs_pcpu_storage"
  Revert "openvswitch: Use nested-BH locking for ovs_pcpu_storage"
  Revert "openvswitch: Merge three per-CPU structures into one"

 net/openvswitch/actions.c  | 86 ++++++++++++++++++++++++++++++++------
 net/openvswitch/datapath.c | 33 ++++-----------
 net/openvswitch/datapath.h | 52 ++---------------------
 3 files changed, 84 insertions(+), 87 deletions(-)

-- 
2.40.1