> +/*
> + * Check driver override. If driver passes a value use it, otherwise we
> use our
> + * own value.
> + */
> +#define CDR(rdi, x) \
> +     rdi->ibdev.x = rdi->ibdev.x ? : rvt_ ##x

This is an extremely obscure name.

No one will be able to look at this:

> +     CDR(rdi, alloc_pd);
> +     CDR(rdi, dealloc_pd);

And have a clue what is happening without searching for the macro.

Reply via email to