Re: [PATCH rdma-next v1 1/3] RDMA/core: Share driver structure size with core

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

 



On Tue, Jan 29, 2019 at 09:25:47AM +0200, Leon Romanovsky wrote:
> From: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> 
> Add new macros to be used in drivers while registering ops structure
> and IB/core while calling allocation routines, so drivers won't need
> to perform kzalloc/kfree in their paths.
> 
> The change in allocation stage allows us to initialize common fields
> prior to calling to drivers (e.g. restrack).
> 
> Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
>  drivers/infiniband/core/device.c |  2 ++
>  include/rdma/ib_verbs.h          | 13 +++++++++++++
>  2 files changed, 15 insertions(+)
> 
> diff --git a/drivers/infiniband/core/device.c b/drivers/infiniband/core/device.c
> index d13491b416dc..c332398376fc 100644
> +++ b/drivers/infiniband/core/device.c
> @@ -1221,6 +1221,8 @@ void ib_set_device_ops(struct ib_device *dev, const struct ib_device_ops *ops)
>  				(ptr)->name = ops->name;                       \
>  	} while (0)
>  
> +#define SET_OBJ_SIZE(ptr, name) SET_DEVICE_OP(ptr, size_##name)
> +
>  	SET_DEVICE_OP(dev_ops, add_gid);
>  	SET_DEVICE_OP(dev_ops, advise_mr);
>  	SET_DEVICE_OP(dev_ops, alloc_dm);
> diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h
> index d54c87640f89..4423de3b3738 100644
> +++ b/include/rdma/ib_verbs.h
> @@ -2264,6 +2264,19 @@ struct ib_counters_read_attr {
>  
>  struct uverbs_attr_bundle;
>  
> +#define INIT_RDMA_OBJ_SIZE(ib_struct, drv_struct, member)                      \
> +	.size_##ib_struct =                                                    \
> +		(sizeof(struct drv_struct) +                                   \
> +		 BUILD_BUG_ON_ZERO(offsetof(struct drv_struct, member)) +      \
> +		 BUILD_BUG_ON_ZERO(                                            \
> +			 !__same_type(((struct drv_struct *)NULL)->member,     \
> +				      struct ib_struct)))

This should still be the container_of version I noted before..

Jason




[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