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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




-----邮件原件-----
发件人: Leon Romanovsky <leon@xxxxxxxxxx> 
发送时间: 2020年12月21日 19:43
收件人: Yang, Xiao/杨 晓 <yangx.jy@xxxxxxxxxxxxxx>
抄送: linux-rdma@xxxxxxxxxxxxxxx
主题: Re: [PATCH v2] librdmacm: Make some functions report proper errno

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

Hi Leon,

Got it.  Thanks for your reply.

Best Regards,
Xiao Yang

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








[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux