BXE: minor changes
authorBarret Rhoden <brho@cs.berkeley.edu>
Mon, 12 Jan 2015 23:10:05 +0000 (18:10 -0500)
committerBarret Rhoden <brho@cs.berkeley.edu>
Tue, 3 Feb 2015 15:12:26 +0000 (10:12 -0500)
DELAY = udelay.

Regarding the __predicts, I've heard they are a mixed bag for performance (via
waterman@).  I'd want to test it out before adding it blindly.

kern/drivers/net/bxe/bxe.c
kern/drivers/net/bxe/bxe_stats.c
kern/drivers/net/bxe/ecore_init_ops.h
kern/drivers/net/bxe/ecore_sp.h

index d52c602..bb51712 100644 (file)
@@ -3119,7 +3119,7 @@ bxe_update_sge_prod(struct bxe_adapter          *sc,
 
     /* now update the prod */
     for (i = first_elem; i != last_elem; i = RX_SGE_NEXT_MASK_ELEM(i)) {
-        if (__predict_true(fp->sge_mask[i])) {
+        if (fp->sge_mask[i]) {
             break;
         }
 
@@ -3450,7 +3450,7 @@ next_rx:
         bd_prod_fw = RX_BD_NEXT(bd_prod_fw);
 
         /* pass the frame to the stack */
-        if (__predict_true(m != NULL)) {
+        if (m != NULL) {
             if_inc_counter(ifp, IFCOUNTER_IPACKETS, 1);
             rx_pkts++;
             if_input(ifp, m);
@@ -3542,7 +3542,7 @@ bxe_free_tx_pkt(struct bxe_adapter    *sc,
 #endif
 
     /* free the mbuf */
-    if (__predict_true(tx_buf->m != NULL)) {
+    if (tx_buf->m != NULL) {
         m_freem(tx_buf->m);
         fp->eth_q_stats.mbuf_alloc_tx--;
     } else {
@@ -14311,7 +14311,7 @@ bxe_media_detect(struct bxe_adapter *sc)
         break;
     }
 #endif
-    asset(0);
+    assert(0);
 }
 
 #define GET_FIELD(value, fname)                     \
@@ -16215,7 +16215,7 @@ bxe_set_qm_cid_count(struct bxe_adapter *sc)
         cid_count += CNIC_CID_MAX;
     }
 
-    return (roundup(cid_count, QM_CID_ROUND));
+    return (ROUNDUP(cid_count, QM_CID_ROUND));
 }
 
 static void
index ae9d5d9..12de401 100644 (file)
@@ -28,6 +28,7 @@
 
 #include "bxe.h"
 #include "bxe_stats.h"
+#include <time.h>      /* udelay */
 
 #ifdef __i386__
 #define BITS_PER_LONG 32
@@ -241,7 +242,7 @@ bxe_stats_comp(struct bxe_adapter *sc)
         }
 
         cnt--;
-        DELAY(1000);
+        udelay(1000);
     }
 
     return (1);
index fbd50be..b0bd521 100644 (file)
 #ifndef ECORE_INIT_OPS_H
 #define ECORE_INIT_OPS_H
 
-
-
-
-
-
-
-
-
-
 static int ecore_gunzip(struct bxe_adapter *sc, const uint8_t *zbuf, int len);
 static void ecore_reg_wr_ind(struct bxe_adapter *sc, uint32_t addr, uint32_t val);
 static void ecore_write_dmae_phys_len(struct bxe_adapter *sc,
@@ -88,7 +79,7 @@ static void ecore_init_fill(struct bxe_adapter *sc, uint32_t addr, int fill,
        ECORE_MEMSET(GUNZIP_BUF(sc), (uint8_t)fill, buf_len);
 
        for (i = 0; i < len; i += buf_len32) {
-               uint32_t cur_len = min(buf_len32, len - i);
+               uint32_t cur_len = MIN(buf_len32, len - i);
 
                ecore_write_big_buf(sc, addr + i*4, cur_len, wb);
        }
@@ -119,7 +110,7 @@ static void ecore_init_wr_64(struct bxe_adapter *sc, uint32_t addr,
        /* 64 bit value is in a blob: first low DWORD, then high DWORD */
        data64 = HILO_U64((*(data + 1)), (*data));
 
-       len64 = min((uint32_t)(FW_BUF_SIZE/8), len64);
+       len64 = MIN((uint32_t)(FW_BUF_SIZE/8), len64);
        for (i = 0; i < len64; i++) {
                uint64_t *pdata = ((uint64_t *)(GUNZIP_BUF(sc))) + i;
 
@@ -127,7 +118,7 @@ static void ecore_init_wr_64(struct bxe_adapter *sc, uint32_t addr,
        }
 
        for (i = 0; i < len; i += buf_len32) {
-               uint32_t cur_len = min(buf_len32, len - i);
+               uint32_t cur_len = MIN(buf_len32, len - i);
 
                ecore_write_big_buf_wb(sc, addr + i*4, cur_len);
        }
index 3e92e52..8af4352 100644 (file)
@@ -54,6 +54,7 @@
 #include "ecore_fw_defs.h"
 #include "ecore_hsi.h"
 #include "ecore_reg.h"
+#include <time.h>      /* udelay */
 
 struct bxe_adapter;
 typedef bus_addr_t ecore_dma_addr_t; /* expected to be 64 bit wide */
@@ -138,11 +139,11 @@ typedef _Bool bool;
 #define ECORE_CPU_TO_LE16(x) htole16(x)
 #define ECORE_CPU_TO_LE32(x) htole32(x)
 
-#define ECORE_WAIT(_s, _t) DELAY(1000)
-#define ECORE_MSLEEP(_t)   DELAY((_t) * 1000)
+#define ECORE_WAIT(_s, _t) udelay(1000)
+#define ECORE_MSLEEP(_t)   udelay((_t) * 1000)
 
-#define ECORE_LIKELY(x)   __predict_true(x)
-#define ECORE_UNLIKELY(x) __predict_false(x)
+#define ECORE_LIKELY(x)   (x)
+#define ECORE_UNLIKELY(x) (x)
 
 #define ECORE_ZALLOC(_size, _flags, _sc) \
        kzmalloc(_size, KMALLOC_WAIT) /*M_TEMP, (M_NOWAIT | M_ZERO))*/
@@ -247,15 +248,15 @@ extern unsigned long bxe_debug;
 
 #define ECORE_DBG_BREAK_IF(exp)     \
     if (bxe_debug & BXE_DEBUG_ECORE_DBG_BREAK_IF) \
-        printf("%s (%s,%d)\n", __FUNCTION__, __FILE__, __LINE__);
+        printk("%s (%s,%d)\n", __FUNCTION__, __FILE__, __LINE__);
 
 #define ECORE_BUG(exp)     \
     if (bxe_debug & BXE_DEBUG_ECORE_BUG) \
-        printf("%s (%s,%d)\n", __FUNCTION__, __FILE__, __LINE__);
+        printk("%s (%s,%d)\n", __FUNCTION__, __FILE__, __LINE__);
 
 #define ECORE_BUG_ON(exp)     \
     if (bxe_debug & BXE_DEBUG_ECORE_BUG_ON) \
-        printf("%s (%s,%d)\n", __FUNCTION__, __FILE__, __LINE__);
+        printk("%s (%s,%d)\n", __FUNCTION__, __FILE__, __LINE__);
 
 
 #endif /* #ifdef ECORE_STOP_ON_ERROR */