igb: fix anoying type mismatch warning on rx/tx queue sizing
authorLinus Torvalds <torvalds@linux-foundation.org>
Tue, 6 Jan 2009 02:47:12 +0000 (18:47 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 6 Jan 2009 02:47:12 +0000 (18:47 -0800)
When using "min()", the types of both sides should match.  With the cpu
mask changes, the type of num_online_cpus() will now depend on config
options. Use "min_t()" with an explicit type instead.

And make the rx/tx case look the same too, just for sanity.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/net/igb/igb_main.c

index 022794e579c7367f326aaa886128579d042b18d4..b82b0fb2056c6122f5ad0be865fe6b6988d82c44 100644 (file)
@@ -1457,8 +1457,8 @@ static int __devinit igb_sw_init(struct igb_adapter *adapter)
 
        /* Number of supported queues. */
        /* Having more queues than CPUs doesn't make sense. */
-       adapter->num_rx_queues = min((u32)IGB_MAX_RX_QUEUES, (u32)num_online_cpus());
-       adapter->num_tx_queues = min(IGB_MAX_TX_QUEUES, num_online_cpus());
+       adapter->num_rx_queues = min_t(u32, IGB_MAX_RX_QUEUES, num_online_cpus());
+       adapter->num_tx_queues = min_t(u32, IGB_MAX_TX_QUEUES, num_online_cpus());
 
        /* This call may decrease the number of queues depending on
         * interrupt mode. */