liquidio: add debug error messages to report command timeout

Add timeout error message in lio_process_ordered_list().  Add host failure
status in existing error message in if_cfg_callback().

Signed-off-by: Rick Farrington <ricardo.farrington@cavium.com>
Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/cavium/liquidio/lio_main.c b/drivers/net/ethernet/cavium/liquidio/lio_main.c
index b2d6478ffbd9c4acd7cb5bde0068700b3c14e3f9..afa173de1042bf1f0213bb0d8fd6078f7c6b8062 100644 (file)
--- a/drivers/net/ethernet/cavium/liquidio/lio_main.c
+++ b/drivers/net/ethernet/cavium/liquidio/lio_main.c
@@ -2225,8 +2225,8 @@ static void if_cfg_callback(struct octeon_device *oct,
 
        oct = lio_get_device(ctx->octeon_id);
        if (resp->status)
-               dev_err(&oct->pci_dev->dev, "nic if cfg instruction failed. Status: %llx\n",
-                       CVM_CAST64(resp->status));
+               dev_err(&oct->pci_dev->dev, "nic if cfg instruction failed. Status: 0x%llx (0x%08x)\n",
+                       CVM_CAST64(resp->status), status);
        WRITE_ONCE(ctx->cond, 1);
 
        snprintf(oct->fw_info.liquidio_firmware_version, 32, "%s",

diff --git a/drivers/net/ethernet/cavium/liquidio/response_manager.c b/drivers/net/ethernet/cavium/liquidio/response_manager.c
index 1d987d7cd7bd6a69b61e826b9944045c9676f247..9456333397233e314c9ec57b0cfe7cfcfb77a46d 100644 (file)
--- a/drivers/net/ethernet/cavium/liquidio/response_manager.c
+++ b/drivers/net/ethernet/cavium/liquidio/response_manager.c
@@ -107,6 +107,8 @@ int lio_process_ordered_list(struct octeon_device *octeon_dev,
                        }
                } else if (force_quit || (sc->timeout &&
                        time_after(jiffies, (unsigned long)sc->timeout))) {
+                       dev_err(&octeon_dev->pci_dev->dev, "%s: cmd failed, timeout (%ld, %ld)\n",
+                               __func__, (long)jiffies, (long)sc->timeout);
                        status = OCTEON_REQUEST_TIMEOUT;
                }