* [PATCH for-next v2] IB/mlx4: Convert rej_tmout radix-tree to XArray
@ 2020-10-09 14:24 Håkon Bugge
2020-10-09 15:35 ` Jason Gunthorpe
0 siblings, 1 reply; 3+ messages in thread
From: Håkon Bugge @ 2020-10-09 14:24 UTC (permalink / raw)
To: Yishai Hadas, Jason Gunthorpe, Doug Ledford, linux-rdma
Fixes: 227a0e142e37 ("IB/mlx4: Add support for REJ due to timeout")
Signed-off-by: Håkon Bugge <haakon.bugge@oracle.com>
---
v1 -> v2:
* Fixed SHA and excess blank line in commit message
* Added explicit locking in alloc_rej_tmout() and
lookup_rej_tmout_slave()
---
drivers/infiniband/hw/mlx4/cm.c | 92 ++++++++++++++++++------------------
drivers/infiniband/hw/mlx4/mlx4_ib.h | 4 +-
2 files changed, 48 insertions(+), 48 deletions(-)
diff --git a/drivers/infiniband/hw/mlx4/cm.c b/drivers/infiniband/hw/mlx4/cm.c
index 0ce4b5a..4668a7d 100644
--- a/drivers/infiniband/hw/mlx4/cm.c
+++ b/drivers/infiniband/hw/mlx4/cm.c
@@ -58,9 +58,7 @@ struct rej_tmout_entry {
int slave;
u32 rem_pv_cm_id;
struct delayed_work timeout;
- struct radix_tree_root *rej_tmout_root;
- /* Points to the mutex protecting this radix-tree */
- struct mutex *lock;
+ struct xarray *xa_rej_tmout;
};
struct cm_generic_msg {
@@ -350,9 +348,7 @@ static void rej_tmout_timeout(struct work_struct *work)
struct rej_tmout_entry *item = container_of(delay, struct rej_tmout_entry, timeout);
struct rej_tmout_entry *deleted;
- mutex_lock(item->lock);
- deleted = radix_tree_delete_item(item->rej_tmout_root, item->rem_pv_cm_id, NULL);
- mutex_unlock(item->lock);
+ deleted = xa_cmpxchg(item->xa_rej_tmout, item->rem_pv_cm_id, item, NULL, 0);
if (deleted != item)
pr_debug("deleted(%p) != item(%p)\n", deleted, item);
@@ -363,18 +359,21 @@ static void rej_tmout_timeout(struct work_struct *work)
static int alloc_rej_tmout(struct mlx4_ib_sriov *sriov, u32 rem_pv_cm_id, int slave)
{
struct rej_tmout_entry *item;
- int sts;
+ struct rej_tmout_entry *old;
+ int ret = 0;
+
+ xa_lock(&sriov->xa_rej_tmout);
+ item = xa_load(&sriov->xa_rej_tmout, (unsigned long)rem_pv_cm_id);
- mutex_lock(&sriov->rej_tmout_lock);
- item = radix_tree_lookup(&sriov->rej_tmout_root, (unsigned long)rem_pv_cm_id);
- mutex_unlock(&sriov->rej_tmout_lock);
if (item) {
- if (IS_ERR(item))
- return PTR_ERR(item);
- /* If a retry, adjust delayed work */
- mod_delayed_work(system_wq, &item->timeout, CM_CLEANUP_CACHE_TIMEOUT);
- return 0;
+ if (xa_err(item))
+ ret = xa_err(item);
+ else
+ /* If a retry, adjust delayed work */
+ mod_delayed_work(system_wq, &item->timeout, CM_CLEANUP_CACHE_TIMEOUT);
+ goto err_or_exists;
}
+ xa_unlock(&sriov->xa_rej_tmout);
item = kmalloc(sizeof(*item), GFP_KERNEL);
if (!item)
@@ -383,39 +382,42 @@ static int alloc_rej_tmout(struct mlx4_ib_sriov *sriov, u32 rem_pv_cm_id, int sl
INIT_DELAYED_WORK(&item->timeout, rej_tmout_timeout);
item->slave = slave;
item->rem_pv_cm_id = rem_pv_cm_id;
- item->rej_tmout_root = &sriov->rej_tmout_root;
- item->lock = &sriov->rej_tmout_lock;
+ item->xa_rej_tmout = &sriov->xa_rej_tmout;
- mutex_lock(&sriov->rej_tmout_lock);
- sts = radix_tree_insert(&sriov->rej_tmout_root, (unsigned long)rem_pv_cm_id, item);
- mutex_unlock(&sriov->rej_tmout_lock);
- if (sts)
- goto err_insert;
+ old = xa_cmpxchg(&sriov->xa_rej_tmout, (unsigned long)rem_pv_cm_id, NULL, item, GFP_KERNEL);
+ if (old) {
+ pr_debug("Non-null old entry (%p) or error (%d) when inserting\n", old, xa_err(old));
+ kfree(item);
+ return xa_err(old);
+ }
schedule_delayed_work(&item->timeout, CM_CLEANUP_CACHE_TIMEOUT);
return 0;
-err_insert:
- kfree(item);
- return sts;
+err_or_exists:
+ xa_unlock(&sriov->xa_rej_tmout);
+ return ret;
}
static int lookup_rej_tmout_slave(struct mlx4_ib_sriov *sriov, u32 rem_pv_cm_id)
{
struct rej_tmout_entry *item;
+ int slave;
- mutex_lock(&sriov->rej_tmout_lock);
- item = radix_tree_lookup(&sriov->rej_tmout_root, (unsigned long)rem_pv_cm_id);
- mutex_unlock(&sriov->rej_tmout_lock);
+ xa_lock(&sriov->xa_rej_tmout);
+ item = xa_load(&sriov->xa_rej_tmout, (unsigned long)rem_pv_cm_id);
- if (!item || IS_ERR(item)) {
+ if (!item || xa_err(item)) {
pr_debug("Could not find slave. rem_pv_cm_id 0x%x error: %d\n",
- rem_pv_cm_id, (int)PTR_ERR(item));
- return !item ? -ENOENT : PTR_ERR(item);
+ rem_pv_cm_id, xa_err(item));
+ slave = !item ? -ENOENT : xa_err(item);
+ } else {
+ slave = item->slave;
}
+ xa_unlock(&sriov->xa_rej_tmout);
- return item->slave;
+ return slave;
}
int mlx4_ib_demux_cm_handler(struct ib_device *ibdev, int port, int *slave,
@@ -483,34 +485,34 @@ void mlx4_ib_cm_paravirt_init(struct mlx4_ib_dev *dev)
INIT_LIST_HEAD(&dev->sriov.cm_list);
dev->sriov.sl_id_map = RB_ROOT;
xa_init_flags(&dev->sriov.pv_id_table, XA_FLAGS_ALLOC);
- mutex_init(&dev->sriov.rej_tmout_lock);
- INIT_RADIX_TREE(&dev->sriov.rej_tmout_root, GFP_KERNEL);
+ xa_init(&dev->sriov.xa_rej_tmout);
}
-static void rej_tmout_tree_cleanup(struct mlx4_ib_sriov *sriov, int slave)
+static void rej_tmout_xa_cleanup(struct mlx4_ib_sriov *sriov, int slave)
{
- struct radix_tree_iter iter;
+ struct rej_tmout_entry *item;
bool flush_needed = false;
- __rcu void **slot;
+ unsigned long id;
int cnt = 0;
- mutex_lock(&sriov->rej_tmout_lock);
- radix_tree_for_each_slot(slot, &sriov->rej_tmout_root, &iter, 0) {
- struct rej_tmout_entry *item = *slot;
-
+ xa_lock(&sriov->xa_rej_tmout);
+ xa_for_each(&sriov->xa_rej_tmout, id, item) {
if (slave < 0 || slave == item->slave) {
mod_delayed_work(system_wq, &item->timeout, 0);
flush_needed = true;
++cnt;
}
}
- mutex_unlock(&sriov->rej_tmout_lock);
+ xa_unlock(&sriov->xa_rej_tmout);
if (flush_needed) {
flush_scheduled_work();
- pr_debug("Deleted %d entries in radix_tree for slave %d during cleanup\n",
- slave, cnt);
+ pr_debug("Deleted %d entries in xarray for slave %d during cleanup\n",
+ cnt, slave);
}
+
+ if (slave < 0)
+ WARN_ON(!xa_empty(&sriov->xa_rej_tmout));
}
/* slave = -1 ==> all slaves */
@@ -581,5 +583,5 @@ void mlx4_ib_cm_paravirt_clean(struct mlx4_ib_dev *dev, int slave)
kfree(map);
}
- rej_tmout_tree_cleanup(sriov, slave);
+ rej_tmout_xa_cleanup(sriov, slave);
}
diff --git a/drivers/infiniband/hw/mlx4/mlx4_ib.h b/drivers/infiniband/hw/mlx4/mlx4_ib.h
index 2ab83ed..d8add5f 100644
--- a/drivers/infiniband/hw/mlx4/mlx4_ib.h
+++ b/drivers/infiniband/hw/mlx4/mlx4_ib.h
@@ -495,9 +495,7 @@ struct mlx4_ib_sriov {
spinlock_t id_map_lock;
struct rb_root sl_id_map;
struct list_head cm_list;
- /* Protects the radix-tree */
- struct mutex rej_tmout_lock;
- struct radix_tree_root rej_tmout_root;
+ struct xarray xa_rej_tmout;
};
struct gid_cache_context {
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH for-next v2] IB/mlx4: Convert rej_tmout radix-tree to XArray
2020-10-09 14:24 [PATCH for-next v2] IB/mlx4: Convert rej_tmout radix-tree to XArray Håkon Bugge
@ 2020-10-09 15:35 ` Jason Gunthorpe
2020-10-09 17:47 ` Håkon Bugge
0 siblings, 1 reply; 3+ messages in thread
From: Jason Gunthorpe @ 2020-10-09 15:35 UTC (permalink / raw)
To: Håkon Bugge; +Cc: Yishai Hadas, Doug Ledford, linux-rdma
On Fri, Oct 09, 2020 at 04:24:42PM +0200, Håkon Bugge wrote:
> Fixes: 227a0e142e37 ("IB/mlx4: Add support for REJ due to timeout")
> Signed-off-by: Håkon Bugge <haakon.bugge@oracle.com>
> ---
> drivers/infiniband/hw/mlx4/cm.c | 92 ++++++++++++++++++------------------
> drivers/infiniband/hw/mlx4/mlx4_ib.h | 4 +-
> 2 files changed, 48 insertions(+), 48 deletions(-)
Applied to for-next, thanks
Jason
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH for-next v2] IB/mlx4: Convert rej_tmout radix-tree to XArray
2020-10-09 15:35 ` Jason Gunthorpe
@ 2020-10-09 17:47 ` Håkon Bugge
0 siblings, 0 replies; 3+ messages in thread
From: Håkon Bugge @ 2020-10-09 17:47 UTC (permalink / raw)
To: Jason Gunthorpe; +Cc: Yishai Hadas, Doug Ledford, OFED mailing list
> On 9 Oct 2020, at 17:35, Jason Gunthorpe <jgg@nvidia.com> wrote:
>
> On Fri, Oct 09, 2020 at 04:24:42PM +0200, Håkon Bugge wrote:
>> Fixes: 227a0e142e37 ("IB/mlx4: Add support for REJ due to timeout")
>> Signed-off-by: Håkon Bugge <haakon.bugge@oracle.com>
>> ---
>> drivers/infiniband/hw/mlx4/cm.c | 92 ++++++++++++++++++------------------
>> drivers/infiniband/hw/mlx4/mlx4_ib.h | 4 +-
>> 2 files changed, 48 insertions(+), 48 deletions(-)
>
> Applied to for-next, thanks
Thxs
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-10-09 17:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-09 14:24 [PATCH for-next v2] IB/mlx4: Convert rej_tmout radix-tree to XArray Håkon Bugge
2020-10-09 15:35 ` Jason Gunthorpe
2020-10-09 17:47 ` Håkon Bugge
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.