On Wed, Apr 7, 2010 at 11:39 AM, Dan Carpenter <error27@xxxxxxxxx> wrote: > The intent here was to check the "mfrpl->mapped_page_list" allocation. > We checked "mfrpl->ibfrpl.page_list" earlier. > > Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> > > diff --git a/drivers/infiniband/hw/mlx4/mr.c b/drivers/infiniband/hw/mlx4/mr.c > index 56147b2..1d27b9a 100644 > --- a/drivers/infiniband/hw/mlx4/mr.c > +++ b/drivers/infiniband/hw/mlx4/mr.c > @@ -240,7 +240,7 @@ struct ib_fast_reg_page_list *mlx4_ib_alloc_fast_reg_page_list(struct ib_device > mfrpl->mapped_page_list = dma_alloc_coherent(&dev->dev->pdev->dev, > size, &mfrpl->map, > GFP_KERNEL); > - if (!mfrpl->ibfrpl.page_list) > + if (!mfrpl->mapped_page_list) > goto err_free; > > WARN_ON(mfrpl->map & 0x3f); Hello Dan, Have you already had the time to check whether this fix should be backported to already released kernels, and if so, to which kernel versions ? The easiest way to ask for backporting is to add "Cc: <stable@xxxxxxxxxx>" in a patch. Bart. -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
