Thread (2 messages) read the whole thread 2 messages, 2 authors, 2022-08-11

Re: [PATCH net v2] net/tls: Use RCU API to access tls_ctx->netdev

From: patchwork-bot+netdevbpf@kernel.org
Date: 2022-08-11 06:10:28

Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski [off-list ref]:

On Wed, 10 Aug 2022 11:16:02 +0300 you wrote:
Currently, tls_device_down synchronizes with tls_device_resync_rx using
RCU, however, the pointer to netdev is stored using WRITE_ONCE and
loaded using READ_ONCE.

Although such approach is technically correct (rcu_dereference is
essentially a READ_ONCE, and rcu_assign_pointer uses WRITE_ONCE to store
NULL), using special RCU helpers for pointers is more valid, as it
includes additional checks and might change the implementation
transparently to the callers.

[...]
Here is the summary with links:
  - [net,v2] net/tls: Use RCU API to access tls_ctx->netdev
    https://git.kernel.org/netdev/net/c/94ce3b64c62d

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help