On Thu, Feb 9, 2017 at 4:00 AM, Tariq Toukan <ttoukan.li...@gmail.com> wrote:
> Yes. > > It can be fixed with this: > > diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c > b/drivers/net/ethernet/mellanox/mlx4/en_rx.c > index 6854a19087ed..d97ee69393f0 100644 > --- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c > +++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c > @@ -94,8 +94,8 @@ static void mlx4_en_free_frag(const struct mlx4_en_priv > *priv, > dma_unmap_page(priv->ddev, frag->dma, > PAGE_SIZE, priv->dma_dir); > put_page(frag->page); > - frag->page = NULL; > } > + memset(frag, 0, sizeof(*frag)); > } > Oh nice, and this is slow path (mlx4_en_free_frag() wont be called anymore in fast path after my patches) Thanks a lot Tariq for tracking this down.
