> +/* > + * 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.