Re: [PATCH v2] librdmacm: Make some functions report proper errno

[Leon Romanovsky <leon@kernel.org>]

On Mon, Dec 21, 2020 at 09:26:34AM +0800, Xiao Yang wrote:
> On 2020/12/16 17:55, Leon Romanovsky wrote:
> > On Wed, Dec 16, 2020 at 05:51:46PM +0800, Xiao Yang wrote:
> > > Hi Leon,
> > >
> > > Thanks for your quick reply. :-)
> > > I have done the same change on three
> > > functions(ucma_get_device,ucma_create_cqs, rdma_create_qp_ex).
> > >
> > > On 2020/12/16 17:22, Xiao Yang wrote:
> > > > Some functions reports fixed ENOMEM when getting any failure, so
> > > > it's hard for user to know which actual error happens on them.
> > > >
> > > > Fixes(ucma_get_device):
> > > > 2ffda7f29913 ("librdmacm: Only allocate verbs resources when needed")
> > > > 191c9346f335 ("librdmacm: Reference count access to verbs context")
> > > > Fixes(ucma_create_cqs):
> > > > f8f1335ad8d8 ("librdmacm: make CQs optional for rdma_create_qp")
> > > > 9e33488e8e50 ("librdmacm: fix all calls to set errno")
> > > > Fixes(rdma_create_qp_ex):
> > > > d2efdede11f7 ("r4019: Add support for userspace RDMA connection management abstraction (CMA)")
> > > > 4e33a4109a62 ("librdmacm: returns errors from the library consistently")
> > > > 995eb0c90c1a ("rdmacm: Add support for XRC QPs")
> > > For every function, I am not sure which one is an exact commit so just
> > > attach all related commits ids.
> > No problem, I'll try to sort it out now.
> Hi Leon,
>
> Sorry to bother you.
> Is there anything blocking the patch? :-)

Nothing, we are in the middle of Christmas vacation here,
so everything takes more time than usual.

Thanks

>
> Best Regards,
> Xiao Yang
> > Thanks
> >
> >
> > .
> >
>
>
>