On Fri, 18 August 2006 13:31:19 +0200, Jan-Bernd Themann wrote:
>
> +     if (queue->current_q_offset > queue->queue_length) {
> +             queue->current_q_offset -= queue->pagesize;
> +             retvalue = NULL;
> +     }
> +     else if ((((u64) retvalue) & (EHEA_PAGESIZE-1)) != 0) {

        } else if (((u64) retvalue) & (EHEA_PAGESIZE-1)) {

> +             if (hret < H_SUCCESS) {

Do you have a reason to keep H_SUCCESS?

> +     if(qp_attr->rq_count > 1)
> +             hw_queue_dtor(&qp->hw_rqueue2);
> +     if(qp_attr->rq_count > 2)

Small amount of whitespace damage.

Jörn

-- 
Write programs that do one thing and do it well. Write programs to work
together. Write programs to handle text streams, because that is a
universal interface.
-- Doug MacIlroy
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to