Let IB core distribute cache update events
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest3 patches 5 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/3 | IB/core: Let IB core distribute cache update eventsdrivers/ drivers/ drivers/ +1 more | +69 | 4 replies | |
| 2/3 | IB/core: Cut down single member ib_cache structuredrivers/ include/ | +17 | 0 replies | |
| 3/3 | IB/core: Do not notify GID change event of an unregistered devicedrivers/ | +1 | 1 reply |
Reviews trailers per patch
All 3 patches have a Reviewed-by or Acked-by.
| Patch | Daniel Jurgens | Leon Romanovsky |
|---|---|---|
| 1/3 | R-b | |
| 2/3 | R-b | |
| 3/3 | R-b | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH rdma-next 0/3] Let IB core distribute cache update events
From: Leon Romanovsky <[email protected]>
From Parav,
Currently when low level driver notifies Pkey, GID, port change events, they are notified to the registered handlers in the order they are registered.
IB core and other ULPs such as IPoIB are interested in GID, LID, Pkey change events. Since all GID query done by ULPs is serviced by IB core, IB core is yet to update the GID cache when IPoIB queries the GID, resulting into not updating IPoIB address. Detailed flow is shown in the patch-1.
Hence, all events which require cache update are handled first by the IB core. Once cache update work is completed, IB core distributes the event to subscriber clients.
This is tested with opensm's /etc/rdma/opensm.conf subnet_prefix configuration update where before the update
$ ip link show dev ib0
ib0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 2044 qdisc pfifo_fast
state UP mode DEFAULT group default qlen 256
link/infiniband
80:00:01:07:fe:80:00:00:00:00:00:00:24:8a:07:03:00:b3:d1:12 brd
00:ff:ff:ff:ff:12:40:1b:ff:ff:00:00:00:00:00:00:ff:ff:ff:ffAnd after the subnet prefix update:
ib0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 2044 qdisc pfifo_fast state UP mode DEFAULT group default qlen 256
link/infiniband
80:00:01:07:fe:80:00:00:00:00:00:02:24:8a:07:03:00:b3:d1:12 brd 00:ff:ff:ff:ff:12:40:1b:ff:ff:00:00:00:00:00:00:ff:ff:ff:ff
Patch-1 fixes the race condition between GID users and GID cache update Patch-2 eliminates single entry structure
Patch-3 simplifies the code to not generate the event for unregistered device
Parav Pandit (3):
IB/core: Let IB core distribute cache update events IB/core: Cut down single member ib_cache structure IB/core: Do not notify GID change event of an unregistered device
drivers/infiniband/core/cache.c | 132 ++++++++++++++-------------- drivers/infiniband/core/core_priv.h | 3 + drivers/infiniband/core/device.c | 26 ++++-- include/rdma/ib_verbs.h | 8 +- 4 files changed, 87 insertions(+), 82 deletions(-)
--
2.20.1