Simplify some of the code for CQ handling slightly.

Signed-off-by: Roland Dreier <[EMAIL PROTECTED]>


--- linux-export.orig/drivers/infiniband/hw/mthca/mthca_cq.c    2005-03-03 
14:12:52.923433653 -0800
+++ linux-export/drivers/infiniband/hw/mthca/mthca_cq.c 2005-03-03 
14:12:53.538300187 -0800
@@ -150,9 +150,8 @@
 
 static inline struct mthca_cqe *cqe_sw(struct mthca_cq *cq, int i)
 {
-       struct mthca_cqe *cqe;
-       cqe = get_cqe(cq, i);
-       return (MTHCA_CQ_ENTRY_OWNER_HW & cqe->owner) ? NULL : cqe;
+       struct mthca_cqe *cqe = get_cqe(cq, i);
+       return MTHCA_CQ_ENTRY_OWNER_HW & cqe->owner ? NULL : cqe;
 }
 
 static inline struct mthca_cqe *next_cqe_sw(struct mthca_cq *cq)
@@ -378,7 +377,7 @@
        struct mthca_wq *wq;
        struct mthca_cqe *cqe;
        int wqe_index;
-       int is_error = 0;
+       int is_error;
        int is_send;
        int free_cqe = 1;
        int err = 0;
@@ -401,12 +400,9 @@
                dump_cqe(cqe);
        }
 
-       if ((cqe->opcode & MTHCA_ERROR_CQE_OPCODE_MASK) ==
-           MTHCA_ERROR_CQE_OPCODE_MASK) {
-               is_error = 1;
-               is_send = cqe->opcode & 1;
-       } else
-               is_send = cqe->is_send & 0x80;
+       is_error = (cqe->opcode & MTHCA_ERROR_CQE_OPCODE_MASK) ==
+               MTHCA_ERROR_CQE_OPCODE_MASK;
+       is_send  = is_error ? cqe->opcode & 0x01 : cqe->is_send & 0x80;
 
        if (!*cur_qp || be32_to_cpu(cqe->my_qpn) != (*cur_qp)->qpn) {
                if (*cur_qp) {

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to