[ofw] [patch][IBAL] move sanity checks to the beginning of the function to prevent crash.

[Hefty, Sean]

> One question, why is it that when creating a QP, both a send and recv CQs
> are required? Otherwise the create QP call fails.
> Why is not possible to have a send-only QP or a receive only QP?

This should be possible, and the checks in ibal are in error.

We want something like:

	if ((qp_attr.sq_depth && !qp_attr.send_cq) ||
		(qp_attr.rq_depth && !qp_attr.recv_cq))
			return error;