wl12xx: Simplify TX blocks accounting

The total number of TX memory blocks may change when the dynamic memory
option is enabled. The current implementation only tracks the available
memory blocks, which over-complicates TX blocks accounting.

By tracking the number of allocated blocks, calculation of the number of
available blocks becomes simpler and cleaner. It simply equals the total
number of TX memory blocks minus the allocated ones.

Also, remove some unnecessary castings and use union member accesses
instead.

Signed-off-by: Ido Yariv <ido@wizery.com>
Signed-off-by: Arik Nemtsov <arik@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>

diff --git a/drivers/net/wireless/wl12xx/acx.c b/drivers/net/wireless/wl12xx/acx.c
index e005aa40ef84539538a9e6c4ad3442bc20872080..b277947400b5f68489bbdb82b61daf98d17c7d82 100644 (file)
--- a/drivers/net/wireless/wl12xx/acx.c
+++ b/drivers/net/wireless/wl12xx/acx.c
@@ -1081,8 +1081,6 @@ int wl1271_acx_init_mem_config(struct wl1271 *wl)
        wl1271_debug(DEBUG_TX, "available tx blocks: %d",
                     wl->tx_blocks_available);
 
-       wl->tx_new_total = wl->tx_blocks_available;
-
        return 0;
 }
 

diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c
index 7dce24c0b331b8dae4f934e57babb79f781e38de..1feb9551ef8faa9a2f148531bc8fd7a97e2555c9 100644 (file)
--- a/drivers/net/wireless/wl12xx/main.c
+++ b/drivers/net/wireless/wl12xx/main.c
@@ -622,7 +622,7 @@ static void wl1271_fw_status(struct wl1271 *wl,
        struct wl1271_fw_common_status *status = &full_status->common;
        struct timespec ts;
        u32 old_tx_blk_count = wl->tx_blocks_available;
-       u32 total = 0;
+       u32 freed_blocks = 0;
        int i;
 
        if (wl->bss_type == BSS_TYPE_AP_BSS) {
@@ -631,15 +631,6 @@ static void wl1271_fw_status(struct wl1271 *wl,
        } else {
                wl1271_raw_read(wl, FW_STATUS_ADDR, status,
                                sizeof(struct wl1271_fw_sta_status), false);
-
-               /* Update tx total blocks change */
-               wl->tx_total_diff +=
-                       ((struct wl1271_fw_sta_status *)status)->tx_total -
-                       wl->tx_new_total;
-
-               /* Update total tx blocks */
-               wl->tx_new_total =
-                       ((struct wl1271_fw_sta_status *)status)->tx_total;
        }
 
        wl1271_debug(DEBUG_IRQ, "intr: 0x%x (fw_rx_counter = %d, "
@@ -651,34 +642,38 @@ static void wl1271_fw_status(struct wl1271 *wl,
 
        /* update number of available TX blocks */
        for (i = 0; i < NUM_TX_QUEUES; i++) {
-               total += le32_to_cpu(status->tx_released_blks[i]) -
-                       wl->tx_blocks_freed[i];
+               freed_blocks += le32_to_cpu(status->tx_released_blks[i]) -
+                               wl->tx_blocks_freed[i];
 
                wl->tx_blocks_freed[i] =
                        le32_to_cpu(status->tx_released_blks[i]);
-
        }
 
-       /*
-        * By adding the freed blocks to tx_total_diff we are actually
-        * moving them to the RX pool.
-        */
-       wl->tx_total_diff += total;
+       wl->tx_allocated_blocks -= freed_blocks;
+
+       if (wl->bss_type == BSS_TYPE_AP_BSS) {
+               /* Update num of allocated TX blocks per link and ps status */
+               wl1271_irq_update_links_status(wl, &full_status->ap);
+               wl->tx_blocks_available += freed_blocks;
+       } else {
+               int avail = full_status->sta.tx_total - wl->tx_allocated_blocks;
 
-       /* if we have positive difference, add the blocks to the TX pool */
-       if (wl->tx_total_diff >= 0) {
-               wl->tx_blocks_available += wl->tx_total_diff;
-               wl->tx_total_diff = 0;
+               /*
+                * The FW might change the total number of TX memblocks before
+                * we get a notification about blocks being released. Thus, the
+                * available blocks calculation might yield a temporary result
+                * which is lower than the actual available blocks. Keeping in
+                * mind that only blocks that were allocated can be moved from
+                * TX to RX, tx_blocks_available should never decrease here.
+                */
+               wl->tx_blocks_available = max((int)wl->tx_blocks_available,
+                                             avail);
        }
 
        /* if more blocks are available now, tx work can be scheduled */
        if (wl->tx_blocks_available > old_tx_blk_count)
                clear_bit(WL1271_FLAG_FW_TX_BUSY, &wl->flags);
 
-       /* for AP update num of allocated TX blocks per link and ps status */
-       if (wl->bss_type == BSS_TYPE_AP_BSS)
-               wl1271_irq_update_links_status(wl, &full_status->ap);
-
        /* update the host-chipset time offset */
        getnstimeofday(&ts);
        wl->time_offset = (timespec_to_ns(&ts) >> 10) -
@@ -1495,6 +1490,7 @@ static void __wl1271_op_remove_interface(struct wl1271 *wl)
        wl->psm_entry_retry = 0;
        wl->power_level = WL1271_DEFAULT_POWER_LEVEL;
        wl->tx_blocks_available = 0;
+       wl->tx_allocated_blocks = 0;
        wl->tx_results_count = 0;
        wl->tx_packets_count = 0;
        wl->tx_security_last_seq = 0;

diff --git a/drivers/net/wireless/wl12xx/tx.c b/drivers/net/wireless/wl12xx/tx.c
index 67245ad396ea3deb998e5a97556959f9d1327dcf..7a3339fd34158a8ebad55906ba9a03ff43ead2b1 100644 (file)
--- a/drivers/net/wireless/wl12xx/tx.c
+++ b/drivers/net/wireless/wl12xx/tx.c
@@ -203,6 +203,7 @@ static int wl1271_tx_allocate(struct wl1271 *wl, struct sk_buff *skb, u32 extra,
                desc->id = id;
 
                wl->tx_blocks_available -= total_blocks;
+               wl->tx_allocated_blocks += total_blocks;
 
                if (wl->bss_type == BSS_TYPE_AP_BSS)
                        wl->links[hlid].allocated_blks += total_blocks;

diff --git a/drivers/net/wireless/wl12xx/wl12xx.h b/drivers/net/wireless/wl12xx/wl12xx.h
index 1b430d2aec4ec678e33598c7d5b946363baf3ff9..9ccbcfdd08028c676257b8f2a2589c051e18acaf 100644 (file)
--- a/drivers/net/wireless/wl12xx/wl12xx.h
+++ b/drivers/net/wireless/wl12xx/wl12xx.h
@@ -418,10 +418,8 @@ struct wl1271 {
        /* Accounting for allocated / available TX blocks on HW */
        u32 tx_blocks_freed[NUM_TX_QUEUES];
        u32 tx_blocks_available;
+       u32 tx_allocated_blocks;
        u32 tx_results_count;
-       /* Indicates how many memory blocks should be moved to the RX pool */
-       int tx_total_diff;
-       u32 tx_new_total;
 
        /* Transmitted TX packets counter for chipset interface */
        u32 tx_packets_count;