netdev - [PATCH net-next 0/6] gve: Alloc before freeing when changing config

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: <20240122182632.1102721-1-shailend@google.com>
Date: Mon, 22 Jan 2024 18:26:26 +0000
From: Shailend Chand <shailend@...gle.com>
To: netdev@...r.kernel.org
Cc: davem@...emloft.net, kuba@...nel.org, Shailend Chand <shailend@...gle.com>
Subject: [PATCH net-next 0/6] gve: Alloc before freeing when changing config

Functions allocating resources did so directly into priv thus far. The
assumption doing that was that priv was not already holding references
to live resources.

When ring configuration is changed in any way from userspace, thus far
we relied on calling the ndo_stop and ndo_open callbacks in succession.
This meant that we teardown existing resources and rob the OS of
networking before we have successfully allocated resources for the new
config.

Correcting this requires us to perform allocations without editing priv.
That is what the "gve: Switch to config-aware..." patch does: it modifies
all the allocation paths so that they take a new configuration as input
and return references to newly allocated resources without modifying
priv or interfering with live resources in any way.

Having corrected the allocation paths so, the ndo open and close
callbacks are refactored to make available distinct functions for
allocating queue resources and starting or stopping them. This is then
put to use in the set_channels and set_features hooks in the last two
patches.

These changes have been tested by verifying the integrity of a stream of
integers while the driver is continuously reconfigured with ethtool.

Shailend Chand (6):
  gve: Define config structs for queue allocation
  gve: Refactor napi add and remove functions
  gve: Switch to config-aware queue allocation
  gve: Refactor gve_open and gve_close
  gve: Alloc before freeing when adjusting queues
  gve: Alloc before freeing when changing features

 drivers/net/ethernet/google/gve/gve.h        | 144 +++-
 drivers/net/ethernet/google/gve/gve_dqo.h    |  18 +-
 drivers/net/ethernet/google/gve/gve_main.c   | 854 ++++++++++++-------
 drivers/net/ethernet/google/gve/gve_rx.c     | 116 ++-
 drivers/net/ethernet/google/gve/gve_rx_dqo.c |  91 +-
 drivers/net/ethernet/google/gve/gve_tx.c     | 128 ++-
 drivers/net/ethernet/google/gve/gve_tx_dqo.c | 108 ++-
 drivers/net/ethernet/google/gve/gve_utils.c  |  31 +
 drivers/net/ethernet/google/gve/gve_utils.h  |   5 +
 9 files changed, 999 insertions(+), 496 deletions(-)

-- 
2.43.0.429.g432eaa2c6b-goog