cxgb4: correct formatting of MSI-X interrupt names
authorDimitris Michailidis <dm@chelsio.com>
Tue, 14 Dec 2010 21:36:50 +0000 (21:36 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 16 Dec 2010 21:16:02 +0000 (13:16 -0800)
The last byte of the buffer for MSI-X names could not be used due to a
bogus -1.  Also do not explicitly clear the last byte, snprintf will do
the right thing.

Signed-off-by: Dimitris Michailidis <dm@chelsio.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/cxgb4/cxgb4_main.c

index 089b753b8a2fe253f69757ce663f83e3774a8c49..7e26fe5efbbf2d7c290d2d47353f399d2b08fb1d 100644 (file)
@@ -522,39 +522,32 @@ static irqreturn_t t4_nondata_intr(int irq, void *cookie)
  */
 static void name_msix_vecs(struct adapter *adap)
 {
-       int i, j, msi_idx = 2, n = sizeof(adap->msix_info[0].desc) - 1;
+       int i, j, msi_idx = 2, n = sizeof(adap->msix_info[0].desc);
 
        /* non-data interrupts */
        snprintf(adap->msix_info[0].desc, n, "%s", adap->name);
-       adap->msix_info[0].desc[n] = 0;
 
        /* FW events */
        snprintf(adap->msix_info[1].desc, n, "%s-FWeventq", adap->name);
-       adap->msix_info[1].desc[n] = 0;
 
        /* Ethernet queues */
        for_each_port(adap, j) {
                struct net_device *d = adap->port[j];
                const struct port_info *pi = netdev_priv(d);
 
-               for (i = 0; i < pi->nqsets; i++, msi_idx++) {
+               for (i = 0; i < pi->nqsets; i++, msi_idx++)
                        snprintf(adap->msix_info[msi_idx].desc, n, "%s-Rx%d",
                                 d->name, i);
-                       adap->msix_info[msi_idx].desc[n] = 0;
-               }
        }
 
        /* offload queues */
-       for_each_ofldrxq(&adap->sge, i) {
-               snprintf(adap->msix_info[msi_idx].desc, n, "%s-ofld%d",
+       for_each_ofldrxq(&adap->sge, i)
+               snprintf(adap->msix_info[msi_idx++].desc, n, "%s-ofld%d",
                         adap->name, i);
-               adap->msix_info[msi_idx++].desc[n] = 0;
-       }
-       for_each_rdmarxq(&adap->sge, i) {
-               snprintf(adap->msix_info[msi_idx].desc, n, "%s-rdma%d",
+
+       for_each_rdmarxq(&adap->sge, i)
+               snprintf(adap->msix_info[msi_idx++].desc, n, "%s-rdma%d",
                         adap->name, i);
-               adap->msix_info[msi_idx++].desc[n] = 0;
-       }
 }
 
 static int request_msix_queue_irqs(struct adapter *adap)