sfc: Fix check for failure of MC_CMD_FLUSH_RX_QUEUES
authorBen Hutchings <bhutchings@solarflare.com>
Tue, 11 Sep 2012 17:25:13 +0000 (18:25 +0100)
committerBen Hutchings <bhutchings@solarflare.com>
Sat, 1 Dec 2012 00:26:11 +0000 (00:26 +0000)
efx_mcdi_rpc_start() returns a negative value on error or zero on
success.  However one caller that can't properly handle failure then
does WARN_ON(rc > 0).  Change it to WARN_ON(rc < 0).

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

index aea43cb..e1967fb 100644 (file)
@@ -1216,7 +1216,7 @@ int efx_mcdi_flush_rxqs(struct efx_nic *efx)
 
        rc = efx_mcdi_rpc(efx, MC_CMD_FLUSH_RX_QUEUES, (u8 *)qid,
                          count * sizeof(*qid), NULL, 0, NULL);
-       WARN_ON(rc > 0);
+       WARN_ON(rc < 0);
 
        kfree(qid);