diff options
| author | D. Wythe <alibuda@linux.alibaba.com> | 2023-02-02 16:26:42 +0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2023-02-04 09:48:19 +0000 |
| commit | aff7bfed9097435ea38de919befbe2d7771a3e87 (patch) | |
| tree | c16906013868a13495e55a16f63946d7d4f5f4bc /net/smc/smc_core.h | |
| parent | 4da687448de7b51dcca9d024c6049fa223f5363f (diff) | |
net/smc: replace mutex rmbs_lock and sndbufs_lock with rw_semaphore
It's clear that rmbs_lock and sndbufs_lock are aims to protect the
rmbs list or the sndbufs list.
During connection establieshment, smc_buf_get_slot() will always
be invoked, and it only performs read semantics in rmbs list and
sndbufs list.
Based on the above considerations, we replace mutex with rw_semaphore.
Only smc_buf_get_slot() use down_read() to allow smc_buf_get_slot()
run concurrently, other part use down_write() to keep exclusive
semantics.
Signed-off-by: D. Wythe <alibuda@linux.alibaba.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/smc/smc_core.h')
| -rw-r--r-- | net/smc/smc_core.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/smc/smc_core.h b/net/smc/smc_core.h index bd7198b72dc1..08b457c2d294 100644 --- a/net/smc/smc_core.h +++ b/net/smc/smc_core.h @@ -252,9 +252,9 @@ struct smc_link_group { unsigned short vlan_id; /* vlan id of link group */ struct list_head sndbufs[SMC_RMBE_SIZES];/* tx buffers */ - struct mutex sndbufs_lock; /* protects tx buffers */ + struct rw_semaphore sndbufs_lock; /* protects tx buffers */ struct list_head rmbs[SMC_RMBE_SIZES]; /* rx buffers */ - struct mutex rmbs_lock; /* protects rx buffers */ + struct rw_semaphore rmbs_lock; /* protects rx buffers */ u8 id[SMC_LGR_ID_SIZE]; /* unique lgr id */ struct delayed_work free_work; /* delayed freeing of an lgr */ |