RE: [Patch v4 07/12] net: mana: Export Work Queue functions for use by RDMA driver

From: Long Li
Date: Tue Jul 12 2022 - 15:16:43 EST


> Subject: RE: [Patch v4 07/12] net: mana: Export Work Queue functions for use
> by RDMA driver
>
> > From: longli@xxxxxxxxxxxxxxxxx <longli@xxxxxxxxxxxxxxxxx>
> > Sent: Wednesday, June 15, 2022 7:07 PM @@ -125,6 +125,7 @@ int
> > mana_gd_send_request(struct gdma_context *gc,
> > u32 req_len, const void *req,
> >
> > return mana_hwc_send_request(hwc, req_len, req, resp_len, resp); }
> > +EXPORT_SYMBOL(mana_gd_send_request);
> Can we use EXPORT_SYMBOL_GPL?
>
> > @@ -715,9 +715,10 @@ static int mana_create_wq_obj(struct
> > mana_port_context *apc,
> > out:
> > return err;
> > }
> > +EXPORT_SYMBOL_GPL(mana_create_wq_obj);
>

Will fix this in v5.

> Well, here we use EXPORT_SYMBOL_GPL. If there is a rule to decide which one
> should be used, please add a comment.
>
> In general, the patch looks good to me.
>
> Reviewed-by: Dexuan Cui <decui@xxxxxxxxxxxxx>