sfc: Reverse initial buffer table allocation to allow for later resizing
authorBen Hutchings <bhutchings@solarflare.com>
Tue, 21 Feb 2012 23:22:00 +0000 (23:22 +0000)
committerBen Hutchings <bhutchings@solarflare.com>
Wed, 22 Feb 2012 20:48:08 +0000 (20:48 +0000)
We have a very simple way of allocating buffer table entries to
queues, which is just to take the next one available.  The extra
channels are the highest numbered channels but they need to be
allocated the lowest entries so that the traffic channels can be
allocated new entries without any collisions.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
drivers/net/ethernet/sfc/efx.c

index ac571cf144853c77d50dd9081a1886d0324bbf9f..c7004dcb58380bb2c7e2518755041160235be4f7 100644 (file)
@@ -585,7 +585,12 @@ static int efx_probe_channels(struct efx_nic *efx)
        /* Restart special buffer allocation */
        efx->next_buffer_table = 0;
 
-       efx_for_each_channel(channel, efx) {
+       /* Probe channels in reverse, so that any 'extra' channels
+        * use the start of the buffer table. This allows the traffic
+        * channels to be resized without moving them or wasting the
+        * entries before them.
+        */
+       efx_for_each_channel_rev(channel, efx) {
                rc = efx_probe_channel(channel);
                if (rc) {
                        netif_err(efx, probe, efx->net_dev,