* [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
@ 2016-02-06 13:50 Alex Estrin
[not found] ` <20160206135041.11630.77019.stgit-u2TXY/5TJkdZ7WVY1cDZ9q2pdiUAq4bhAL8bYrjMMd8@public.gmane.org>
0 siblings, 1 reply; 8+ messages in thread
From: Alex Estrin @ 2016-02-06 13:50 UTC (permalink / raw)
To: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
A narrow window for race condition still exist between
multicast join thread and *dev_flush workers.
A kernel crash caused by prolong erratic link state changes
was observed (most likely a faulty cabling):
[167275.656270] BUG: unable to handle kernel NULL pointer dereference at
0000000000000020
[167275.665973] IP: [<ffffffffa05f8f2e>] ipoib_mcast_join+0xae/0x1d0 [ib_ipoib]
[167275.674443] PGD 0
[167275.677373] Oops: 0000 [#1] SMP
...
[167275.977530] Call Trace:
[167275.982225] [<ffffffffa05f92f0>] ? ipoib_mcast_free+0x200/0x200 [ib_ipoib]
[167275.992024] [<ffffffffa05fa1b7>] ipoib_mcast_join_task+0x2a7/0x490
[ib_ipoib]
[167276.002149] [<ffffffff8109d5fb>] process_one_work+0x17b/0x470
[167276.010754] [<ffffffff8109e3cb>] worker_thread+0x11b/0x400
[167276.019088] [<ffffffff8109e2b0>] ? rescuer_thread+0x400/0x400
[167276.027737] [<ffffffff810a5aef>] kthread+0xcf/0xe0
Here was a hit spot:
ipoib_mcast_join() {
..............
rec.qkey = priv->broadcast->mcmember.qkey;
^^^^^^^
.....
}
Proposed patch should prevent multicast join task to continue
if link state change is detected.
Signed-off-by: Alex Estrin <alex.estrin-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
Changes from v1:
No need to lock again if error detected.
---
drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 14 ++++++++++++--
1 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
index 050dfa1..3ce2f0a 100644
--- a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
@@ -456,7 +456,7 @@ out_locked:
return status;
}
-static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
+static int ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
{
struct ipoib_dev_priv *priv = netdev_priv(dev);
struct ib_sa_multicast *multicast;
@@ -466,6 +466,9 @@ static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
ib_sa_comp_mask comp_mask;
int ret = 0;
+ if (!priv->broadcast)
+ return -EINVAL;
+
ipoib_dbg_mcast(priv, "joining MGID %pI6\n", mcast->mcmember.mgid.raw);
rec.mgid = mcast->mcmember.mgid;
@@ -539,6 +542,7 @@ static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
spin_unlock_irq(&priv->lock);
complete(&mcast->done);
}
+ return 0;
}
void ipoib_mcast_join_task(struct work_struct *work)
@@ -611,6 +615,11 @@ void ipoib_mcast_join_task(struct work_struct *work)
* and attached
*/
list_for_each_entry(mcast, &priv->multicast_list, list) {
+ if (!test_bit(IPOIB_FLAG_OPER_UP, &priv->flags)) {
+ mcast = NULL;
+ delay_until = 0;
+ goto out;
+ }
if (IS_ERR_OR_NULL(mcast->mc) &&
!test_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags) &&
(!test_bit(IPOIB_MCAST_FLAG_SENDONLY, &mcast->flags) ||
@@ -621,7 +630,8 @@ void ipoib_mcast_join_task(struct work_struct *work)
init_completion(&mcast->done);
set_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags);
spin_unlock_irq(&priv->lock);
- ipoib_mcast_join(dev, mcast);
+ if (ipoib_mcast_join(dev, mcast) != 0)
+ return;
spin_lock_irq(&priv->lock);
} else if (!delay_until ||
time_before(mcast->delay_until, delay_until))
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <20160206135041.11630.77019.stgit-u2TXY/5TJkdZ7WVY1cDZ9q2pdiUAq4bhAL8bYrjMMd8@public.gmane.org>
@ 2016-02-06 17:18 ` Leon Romanovsky
[not found] ` <20160206171828.GD8584-2ukJVAZIZ/Y@public.gmane.org>
2016-02-08 13:34 ` Erez Shitrit
1 sibling, 1 reply; 8+ messages in thread
From: Leon Romanovsky @ 2016-02-06 17:18 UTC (permalink / raw)
To: Alex Estrin; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
A number of very minor comments:
1. Please remove dot at the end of commit message title.
On Sat, Feb 06, 2016 at 08:50:41AM -0500, Alex Estrin wrote:
> A narrow window for race condition still exist between
> multicast join thread and *dev_flush workers.
> A kernel crash caused by prolong erratic link state changes
> was observed (most likely a faulty cabling):
>
> [167275.656270] BUG: unable to handle kernel NULL pointer dereference at
> 0000000000000020
> [167275.665973] IP: [<ffffffffa05f8f2e>] ipoib_mcast_join+0xae/0x1d0 [ib_ipoib]
> [167275.674443] PGD 0
> [167275.677373] Oops: 0000 [#1] SMP
> ...
> [167275.977530] Call Trace:
> [167275.982225] [<ffffffffa05f92f0>] ? ipoib_mcast_free+0x200/0x200 [ib_ipoib]
> [167275.992024] [<ffffffffa05fa1b7>] ipoib_mcast_join_task+0x2a7/0x490
> [ib_ipoib]
> [167276.002149] [<ffffffff8109d5fb>] process_one_work+0x17b/0x470
> [167276.010754] [<ffffffff8109e3cb>] worker_thread+0x11b/0x400
> [167276.019088] [<ffffffff8109e2b0>] ? rescuer_thread+0x400/0x400
> [167276.027737] [<ffffffff810a5aef>] kthread+0xcf/0xe0
> Here was a hit spot:
> ipoib_mcast_join() {
> ..............
> rec.qkey = priv->broadcast->mcmember.qkey;
> ^^^^^^^
> .....
> }
> Proposed patch should prevent multicast join task to continue
> if link state change is detected.
>
> Signed-off-by: Alex Estrin <alex.estrin-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> ---
>
> Changes from v1:
> No need to lock again if error detected.
> ---
> drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 14 ++++++++++++--
> 1 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
> index 050dfa1..3ce2f0a 100644
> --- a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
> +++ b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
> @@ -456,7 +456,7 @@ out_locked:
> return status;
> }
>
> -static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> +static int ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> {
> struct ipoib_dev_priv *priv = netdev_priv(dev);
> struct ib_sa_multicast *multicast;
> @@ -466,6 +466,9 @@ static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> ib_sa_comp_mask comp_mask;
> int ret = 0;
>
> + if (!priv->broadcast)
> + return -EINVAL;
> +
> ipoib_dbg_mcast(priv, "joining MGID %pI6\n", mcast->mcmember.mgid.raw);
>
> rec.mgid = mcast->mcmember.mgid;
> @@ -539,6 +542,7 @@ static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> spin_unlock_irq(&priv->lock);
> complete(&mcast->done);
> }
> + return 0;
2. This function declared as a void, it is an error to add "return 0".
> }
>
> void ipoib_mcast_join_task(struct work_struct *work)
> @@ -611,6 +615,11 @@ void ipoib_mcast_join_task(struct work_struct *work)
> * and attached
> */
> list_for_each_entry(mcast, &priv->multicast_list, list) {
> + if (!test_bit(IPOIB_FLAG_OPER_UP, &priv->flags)) {
> + mcast = NULL;
> + delay_until = 0;
> + goto out;
> + }
> if (IS_ERR_OR_NULL(mcast->mc) &&
> !test_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags) &&
> (!test_bit(IPOIB_MCAST_FLAG_SENDONLY, &mcast->flags) ||
> @@ -621,7 +630,8 @@ void ipoib_mcast_join_task(struct work_struct *work)
> init_completion(&mcast->done);
> set_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags);
> spin_unlock_irq(&priv->lock);
> - ipoib_mcast_join(dev, mcast);
> + if (ipoib_mcast_join(dev, mcast) != 0)
3. There is no need to compare with 0. Just use "if (ipoib...(...))"
construction.
> + return;
> spin_lock_irq(&priv->lock);
> } else if (!delay_until ||
> time_before(mcast->delay_until, delay_until))
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <20160206171828.GD8584-2ukJVAZIZ/Y@public.gmane.org>
@ 2016-02-08 12:23 ` Estrin, Alex
[not found] ` <F3529576D8E232409F431C309E29399328F6C7FE-8k97q/ur5Z1cIJlls4ac1rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
0 siblings, 1 reply; 8+ messages in thread
From: Estrin, Alex @ 2016-02-08 12:23 UTC (permalink / raw)
To: leon-2ukJVAZIZ/Y; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
Hi Leon,
> A number of very minor comments:
> 1. Please remove dot at the end of commit message title.
>
Ok. Thanks.
> 2. This function declared as a void, it is an error to add "return 0".
>
ipoib_mcast_join() is declared as int. Please see a very first change.
> -static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> +static int ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> 3. There is no need to compare with 0. Just use "if (ipoib...(...))"
> construction.
Ok. No problem.
Thanks,
Alex.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <F3529576D8E232409F431C309E29399328F6C7FE-8k97q/ur5Z1cIJlls4ac1rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
@ 2016-02-08 13:22 ` leon-2ukJVAZIZ/Y
0 siblings, 0 replies; 8+ messages in thread
From: leon-2ukJVAZIZ/Y @ 2016-02-08 13:22 UTC (permalink / raw)
To: Estrin, Alex; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
On Mon, Feb 08, 2016 at 12:23:07PM +0000, Estrin, Alex wrote:
> > 2. This function declared as a void, it is an error to add "return 0".
> >
> ipoib_mcast_join() is declared as int. Please see a very first change.
>
> > -static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> > +static int ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
>
You are right.
Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <20160206135041.11630.77019.stgit-u2TXY/5TJkdZ7WVY1cDZ9q2pdiUAq4bhAL8bYrjMMd8@public.gmane.org>
2016-02-06 17:18 ` Leon Romanovsky
@ 2016-02-08 13:34 ` Erez Shitrit
[not found] ` <CAAk-MO8b_bpb=cd0Ki7wu3sUSG_rC+fS0sxhmdn5cA-91WfR4g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
1 sibling, 1 reply; 8+ messages in thread
From: Erez Shitrit @ 2016-02-08 13:34 UTC (permalink / raw)
To: Alex Estrin; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
On Sat, Feb 6, 2016 at 3:50 PM, Alex Estrin <alex.estrin-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote:
> A narrow window for race condition still exist between
> multicast join thread and *dev_flush workers.
> A kernel crash caused by prolong erratic link state changes
> was observed (most likely a faulty cabling):
>
> [167275.656270] BUG: unable to handle kernel NULL pointer dereference at
> 0000000000000020
> [167275.665973] IP: [<ffffffffa05f8f2e>] ipoib_mcast_join+0xae/0x1d0 [ib_ipoib]
> [167275.674443] PGD 0
> [167275.677373] Oops: 0000 [#1] SMP
> ...
> [167275.977530] Call Trace:
> [167275.982225] [<ffffffffa05f92f0>] ? ipoib_mcast_free+0x200/0x200 [ib_ipoib]
> [167275.992024] [<ffffffffa05fa1b7>] ipoib_mcast_join_task+0x2a7/0x490
> [ib_ipoib]
> [167276.002149] [<ffffffff8109d5fb>] process_one_work+0x17b/0x470
> [167276.010754] [<ffffffff8109e3cb>] worker_thread+0x11b/0x400
> [167276.019088] [<ffffffff8109e2b0>] ? rescuer_thread+0x400/0x400
> [167276.027737] [<ffffffff810a5aef>] kthread+0xcf/0xe0
> Here was a hit spot:
> ipoib_mcast_join() {
> ..............
> rec.qkey = priv->broadcast->mcmember.qkey;
> ^^^^^^^
> .....
> }
> Proposed patch should prevent multicast join task to continue
> if link state change is detected.
>
> Signed-off-by: Alex Estrin <alex.estrin-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> ---
>
> Changes from v1:
> No need to lock again if error detected.
> ---
> drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 14 ++++++++++++--
> 1 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
> index 050dfa1..3ce2f0a 100644
> --- a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
> +++ b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
> @@ -456,7 +456,7 @@ out_locked:
> return status;
> }
>
> -static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> +static int ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> {
> struct ipoib_dev_priv *priv = netdev_priv(dev);
> struct ib_sa_multicast *multicast;
> @@ -466,6 +466,9 @@ static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> ib_sa_comp_mask comp_mask;
> int ret = 0;
>
> + if (!priv->broadcast)
> + return -EINVAL;
> +
Can you please elaborate what will avoid the other task that
invalidate the priv->broadcast (ipoib_mcast_dev_flush) to do it right
after that check?
> ipoib_dbg_mcast(priv, "joining MGID %pI6\n", mcast->mcmember.mgid.raw);
>
> rec.mgid = mcast->mcmember.mgid;
> @@ -539,6 +542,7 @@ static void ipoib_mcast_join(struct net_device *dev, struct ipoib_mcast *mcast)
> spin_unlock_irq(&priv->lock);
> complete(&mcast->done);
> }
> + return 0;
> }
>
> void ipoib_mcast_join_task(struct work_struct *work)
> @@ -611,6 +615,11 @@ void ipoib_mcast_join_task(struct work_struct *work)
> * and attached
> */
> list_for_each_entry(mcast, &priv->multicast_list, list) {
> + if (!test_bit(IPOIB_FLAG_OPER_UP, &priv->flags)) {
> + mcast = NULL;
> + delay_until = 0;
> + goto out;
> + }
> if (IS_ERR_OR_NULL(mcast->mc) &&
> !test_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags) &&
> (!test_bit(IPOIB_MCAST_FLAG_SENDONLY, &mcast->flags) ||
> @@ -621,7 +630,8 @@ void ipoib_mcast_join_task(struct work_struct *work)
> init_completion(&mcast->done);
> set_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags);
> spin_unlock_irq(&priv->lock);
> - ipoib_mcast_join(dev, mcast);
> + if (ipoib_mcast_join(dev, mcast) != 0)
> + return;
> spin_lock_irq(&priv->lock);
> } else if (!delay_until ||
> time_before(mcast->delay_until, delay_until))
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <CAAk-MO8b_bpb=cd0Ki7wu3sUSG_rC+fS0sxhmdn5cA-91WfR4g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2016-02-08 16:43 ` Estrin, Alex
[not found] ` <F3529576D8E232409F431C309E29399328F6C892-8k97q/ur5Z1cIJlls4ac1rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
0 siblings, 1 reply; 8+ messages in thread
From: Estrin, Alex @ 2016-02-08 16:43 UTC (permalink / raw)
To: Erez Shitrit; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 838 bytes --]
Hi Erez,
> Can you please elaborate what will avoid the other task that
> invalidate the priv->broadcast (ipoib_mcast_dev_flush) to do it right
> after that check?
I was considering check for IPOIB_FLAG_OPER_UP in mcast task loop
would be sufficient as its state is serialized by priv->lock:
> > list_for_each_entry(mcast, &priv->multicast_list, list) {
> > + if (!test_bit(IPOIB_FLAG_OPER_UP, &priv->flags)) {
And I can see your point now. We unlock before calling mcast_join().
Apparently check for OPER_UP flag should be added along priv->broadcast check
as it will ensure indication if mcast worker is competing with event handler worker.
Thanks,
Alex.
N§²æìr¸yúèØb²X¬¶Ç§vØ^)Þº{.nÇ+·¥{±Ù{ayº\x1dÊÚë,j\a¢f£¢·h»öì\x17/oSc¾Ú³9uÀ¦æåÈ&jw¨®\x03(éÝ¢j"ú\x1a¶^[m§ÿïêäz¹Þàþf£¢·h§~m
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <F3529576D8E232409F431C309E29399328F6C892-8k97q/ur5Z1cIJlls4ac1rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
@ 2016-02-09 7:18 ` Leon Romanovsky
[not found] ` <20160209071847.GA14741-2ukJVAZIZ/Y@public.gmane.org>
0 siblings, 1 reply; 8+ messages in thread
From: Leon Romanovsky @ 2016-02-09 7:18 UTC (permalink / raw)
To: Estrin, Alex
Cc: Erez Shitrit, linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
On Mon, Feb 08, 2016 at 04:43:29PM +0000, Estrin, Alex wrote:
> Hi Erez,
>
> > Can you please elaborate what will avoid the other task that
> > invalidate the priv->broadcast (ipoib_mcast_dev_flush) to do it right
> > after that check?
>
> I was considering check for IPOIB_FLAG_OPER_UP in mcast task loop
> would be sufficient as its state is serialized by priv->lock:
> > > list_for_each_entry(mcast, &priv->multicast_list, list) {
> > > + if (!test_bit(IPOIB_FLAG_OPER_UP, &priv->flags)) {
>
> And I can see your point now. We unlock before calling mcast_join().
> Apparently check for OPER_UP flag should be added along priv->broadcast check
> as it will ensure indication if mcast worker is competing with event handler worker.
Will you plan to respin it?
>
> Thanks,
> Alex.
>
> N?????r??y????b?X??ǧv?^?){.n?+????{??ٚ?{ay?\x1dʇڙ?,j\a??f???h???z?\x1e?w???\f???j:+v???w?j?m????\a????zZ+?????ݢj"??!
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v2] IB/ipoib: fix for rare multicast join race condition.
[not found] ` <20160209071847.GA14741-2ukJVAZIZ/Y@public.gmane.org>
@ 2016-02-09 12:06 ` Estrin, Alex
0 siblings, 0 replies; 8+ messages in thread
From: Estrin, Alex @ 2016-02-09 12:06 UTC (permalink / raw)
To: leon-2ukJVAZIZ/Y
Cc: Erez Shitrit, linux-rdma-u79uwXL29TY76Z2rM5mHXA, Doug Ledford
Hi Leon,
> Will you plan to respin it?
Yes, I'll resubmit shortly.
Thanks,
Alex.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-02-09 12:06 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-06 13:50 [PATCH v2] IB/ipoib: fix for rare multicast join race condition Alex Estrin
[not found] ` <20160206135041.11630.77019.stgit-u2TXY/5TJkdZ7WVY1cDZ9q2pdiUAq4bhAL8bYrjMMd8@public.gmane.org>
2016-02-06 17:18 ` Leon Romanovsky
[not found] ` <20160206171828.GD8584-2ukJVAZIZ/Y@public.gmane.org>
2016-02-08 12:23 ` Estrin, Alex
[not found] ` <F3529576D8E232409F431C309E29399328F6C7FE-8k97q/ur5Z1cIJlls4ac1rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2016-02-08 13:22 ` leon-2ukJVAZIZ/Y
2016-02-08 13:34 ` Erez Shitrit
[not found] ` <CAAk-MO8b_bpb=cd0Ki7wu3sUSG_rC+fS0sxhmdn5cA-91WfR4g-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-02-08 16:43 ` Estrin, Alex
[not found] ` <F3529576D8E232409F431C309E29399328F6C892-8k97q/ur5Z1cIJlls4ac1rfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2016-02-09 7:18 ` Leon Romanovsky
[not found] ` <20160209071847.GA14741-2ukJVAZIZ/Y@public.gmane.org>
2016-02-09 12:06 ` Estrin, Alex
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.