Re: [PATCH rdma-core] verbs: Remove comp_mask checks from verbs.h

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

 



On Thu, Dec 21, 2017 at 07:42:16AM +0200, Leon Romanovsky wrote:

> Both these functions are performing their checks of cq_attr->comp_mask,
> but return different errno. I think that it is perfect timing to start
> organize and unite our return errors. They should return the same errno.

Bugs bugs everywhere.. This is going to be another patch. I would drop
the errno set into check_comp_mask and change all comp_mask users to
call it, and probably drop the _RESERVED uglyness too, but another
patch.

> > In the case of ibv_queryrt_values_ex, there is no check in the function
> > op, which is an ABI mistake. So move the check there.
> 
> ibv_queryrt_values_ex -> ibv_query_rt_values_ex

K.. Are you going to collect the loose patches on the list? Can you
fix the typo when you grab this one :)

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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