Re: [RFC] libibverbs IB Device Memory support

[Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1042 bytes --]

On Mon, Jun 05, 2017 at 11:20:58AM -0600, Jason Gunthorpe wrote:
> On Wed, May 10, 2017 at 02:25:23PM +0300, ahmad omary wrote:
>
> > struct ibv_dm *ibv_alloc_dm(struct ibv_context *context,
> >    struct ibv_alloc_dm_attr *dm_attr);
> >
> > struct ibv_alloc_dm_attr{
> > size_t length;
> > uint32_t comp_mask; /*enable future extensions*/
> > }
>
> Can we please stop this madness where every function call needs to
> accept a struct?
>
> It is perfectly fine to add new function calls if new arguments are
> needed someday.

IMHO, generally speaking, it is better and cleaner to have similar functions
signatures for all APIs which is exported to user.

>
> > int ibv_memcpy_dm(struct ibv_dm *dm, struct ibv_memcpy_dm_attr *
> > memcpy_dm_attr);
>
> To be clear, this is has to be a sycall, right?

Yes

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]