Re: [PATCH] IB/ipoib: Initialize ipoib_neigh list properly
You are correct that it needs to be initialized but I think you must have made an error in applying the patch. If you look again at the latest version of the patch this line is present. https://patchwork.kernel.org/patch/120013/ On Tue, 2010-08-24 at 20:51 -0700, Ira Weiny wrote: I applied Ralph's fix dangling pointer references to ipoib_neigh and ipoib_path patch to our local RHEL based kernel and experienced crashes in ipoib_neigh_cleanup. It turns out ipoib_neigh-list was not initialized properly. So the following code from Ralph's patch caused issues. if (ipoib_cm_get(neigh)) ipoib_cm_destroy_tx(ipoib_cm_get(neigh)); Looking at Rolands upstream kernel it appears the same is true upstream. The patch below initializes ipoib_neigh-list correctly. Signed-off-by: Ira Weiny wei...@llnl.gov --- drivers/infiniband/ulp/ipoib/ipoib_main.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index b4b2257..fa38ede 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -882,6 +882,7 @@ struct ipoib_neigh *ipoib_neigh_alloc(struct neighbour *neighbour, if (!neigh) return NULL; + INIT_LIST_HEAD(neigh-list); neigh-neighbour = neighbour; neigh-dev = dev; memset(neigh-dgid.raw, 0, sizeof (union ib_gid)); -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] IB/ipoib: Initialize ipoib_neigh list properly
On Mon, 30 Aug 2010 14:56:43 -0700 Ralph Campbell ralph.campb...@qlogic.com wrote: You are correct that it needs to be initialized but I think you must have made an error in applying the patch. If you look again at the latest version of the patch this line is present. https://*patchwork.kernel.org/patch/120013/ Yep, :-( I must have made a mistake in applying the patch. It did not apply to our old kernel so I had to do some stuff by hand... :-( Sorry for the noise, Ira On Tue, 2010-08-24 at 20:51 -0700, Ira Weiny wrote: I applied Ralph's fix dangling pointer references to ipoib_neigh and ipoib_path patch to our local RHEL based kernel and experienced crashes in ipoib_neigh_cleanup. It turns out ipoib_neigh-list was not initialized properly. So the following code from Ralph's patch caused issues. if (ipoib_cm_get(neigh)) ipoib_cm_destroy_tx(ipoib_cm_get(neigh)); Looking at Rolands upstream kernel it appears the same is true upstream. The patch below initializes ipoib_neigh-list correctly. Signed-off-by: Ira Weiny wei...@llnl.gov --- drivers/infiniband/ulp/ipoib/ipoib_main.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index b4b2257..fa38ede 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -882,6 +882,7 @@ struct ipoib_neigh *ipoib_neigh_alloc(struct neighbour *neighbour, if (!neigh) return NULL; + INIT_LIST_HEAD(neigh-list); neigh-neighbour = neighbour; neigh-dev = dev; memset(neigh-dgid.raw, 0, sizeof (union ib_gid)); -- Ira Weiny Math Programmer/Computer Scientist Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] IB/ipoib: Initialize ipoib_neigh list properly
I applied Ralph's fix dangling pointer references to ipoib_neigh and ipoib_path patch to our local RHEL based kernel and experienced crashes in ipoib_neigh_cleanup. It turns out ipoib_neigh-list was not initialized properly. So the following code from Ralph's patch caused issues. if (ipoib_cm_get(neigh)) ipoib_cm_destroy_tx(ipoib_cm_get(neigh)); Looking at Rolands upstream kernel it appears the same is true upstream. The patch below initializes ipoib_neigh-list correctly. Signed-off-by: Ira Weiny wei...@llnl.gov --- drivers/infiniband/ulp/ipoib/ipoib_main.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index b4b2257..fa38ede 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -882,6 +882,7 @@ struct ipoib_neigh *ipoib_neigh_alloc(struct neighbour *neighbour, if (!neigh) return NULL; + INIT_LIST_HEAD(neigh-list); neigh-neighbour = neighbour; neigh-dev = dev; memset(neigh-dgid.raw, 0, sizeof (union ib_gid)); -- 1.5.4.5 -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html