On Mon, Aug 21, 2023 at 10:18?AM Shawn.Shao <shawn.shao at
jaguarmicro.com> wrote:>
> If the shared_buffer allocation fails, need to unregister mgmt_dev first.
>
Please add a Fixes tag.
Other than this
Acked-by: Jason Wang <jasowang at redhat.com>
Thanks
> Signed-off-by: Shawn.Shao <shawn.shao at jaguarmicro.com>
> ---
>  drivers/vdpa/vdpa_sim/vdpa_sim_blk.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
b/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
> index 00d7d72713be..b3a3cb165795 100644
> --- a/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
> +++ b/drivers/vdpa/vdpa_sim/vdpa_sim_blk.c
> @@ -499,12 +499,13 @@ static int __init vdpasim_blk_init(void)
>                                          GFP_KERNEL);
>                 if (!shared_buffer) {
>                         ret = -ENOMEM;
> -                       goto parent_err;
> +                       goto mgmt_dev_err;
>                 }
>         }
>
>         return 0;
> -
> +mgmt_dev_err:
> +       vdpa_mgmtdev_unregister(&mgmt_dev);
>  parent_err:
>         device_unregister(&vdpasim_blk_mgmtdev);
>         return ret;
> --
> 2.34.1
>