On 7/17/19 10:03 AM, Chuhong Yuan wrote: > Variable allocated by kvmalloc should not be freed by kfree. > Because it may be allocated by vmalloc. > So replace kfree with kvfree here. > > Signed-off-by: Chuhong Yuan <hsleste...@gmail.com> > --- Please add corresponding Fixes: tag, thanks ! > drivers/net/ethernet/mellanox/mlx5/core/health.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/health.c > b/drivers/net/ethernet/mellanox/mlx5/core/health.c > index 2fe6923f7ce0..9314777d99e3 100644 > --- a/drivers/net/ethernet/mellanox/mlx5/core/health.c > +++ b/drivers/net/ethernet/mellanox/mlx5/core/health.c > @@ -597,7 +597,7 @@ mlx5_fw_fatal_reporter_dump(struct > devlink_health_reporter *reporter, > err = devlink_fmsg_arr_pair_nest_end(fmsg); > > free_data: > - kfree(cr_data); > + kvfree(cr_data); > return err; > } > >