Message ID | 1379696386-29573-1-git-send-email-malahal@us.ibm.com |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Fri, 2013-09-20 at 11:59 -0500, Malahal Naineni wrote: > Also changed a log message to indicate that memory was not allocated > instead of memory not available! [] > diff --git a/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c b/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c [] > @@ -1274,11 +1274,13 @@ void ql_mpi_reset_work(struct work_struct *work) > return; > } > > - if (!ql_core_dump(qdev, qdev->mpi_coredump)) { > - netif_err(qdev, drv, qdev->ndev, "Core is dumped!\n"); > - qdev->core_is_dumped = 1; > - queue_delayed_work(qdev->workqueue, > - &qdev->mpi_core_to_log, 5 * HZ); > + if (qdev->mpi_coredump) { > + if (!ql_core_dump(qdev, qdev->mpi_coredump)) { > + netif_err(qdev, drv, qdev->ndev, "Core is dumped!\n"); > + qdev->core_is_dumped = 1; > + queue_delayed_work(qdev->workqueue, > + &qdev->mpi_core_to_log, 5 * HZ); > + } This can be done without adding another indentation level if (qdev->mpi_coredump && !ql_core_dump(qdev, qdev->mpi_coredump)) { -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Joe Perches <joe@perches.com> Date: Fri, 20 Sep 2013 11:08:51 -0700 > On Fri, 2013-09-20 at 11:59 -0500, Malahal Naineni wrote: >> Also changed a log message to indicate that memory was not allocated >> instead of memory not available! > [] >> diff --git a/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c b/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c > [] >> @@ -1274,11 +1274,13 @@ void ql_mpi_reset_work(struct work_struct *work) >> return; >> } >> >> - if (!ql_core_dump(qdev, qdev->mpi_coredump)) { >> - netif_err(qdev, drv, qdev->ndev, "Core is dumped!\n"); >> - qdev->core_is_dumped = 1; >> - queue_delayed_work(qdev->workqueue, >> - &qdev->mpi_core_to_log, 5 * HZ); >> + if (qdev->mpi_coredump) { >> + if (!ql_core_dump(qdev, qdev->mpi_coredump)) { >> + netif_err(qdev, drv, qdev->ndev, "Core is dumped!\n"); >> + qdev->core_is_dumped = 1; >> + queue_delayed_work(qdev->workqueue, >> + &qdev->mpi_core_to_log, 5 * HZ); >> + } > > This can be done without adding another indentation level > > if (qdev->mpi_coredump && > !ql_core_dump(qdev, qdev->mpi_coredump)) { Agreed. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/qlogic/qlge/qlge_dbg.c b/drivers/net/ethernet/qlogic/qlge/qlge_dbg.c index 10093f0..6bc5db7 100644 --- a/drivers/net/ethernet/qlogic/qlge/qlge_dbg.c +++ b/drivers/net/ethernet/qlogic/qlge/qlge_dbg.c @@ -740,8 +740,8 @@ int ql_core_dump(struct ql_adapter *qdev, struct ql_mpi_coredump *mpi_coredump) int i; if (!mpi_coredump) { - netif_err(qdev, drv, qdev->ndev, "No memory available\n"); - return -ENOMEM; + netif_err(qdev, drv, qdev->ndev, "No memory allocated\n"); + return -EINVAL; } /* Try to get the spinlock, but dont worry if diff --git a/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c b/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c index ff2bf8a..99a3e07 100644 --- a/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c +++ b/drivers/net/ethernet/qlogic/qlge/qlge_mpi.c @@ -1274,11 +1274,13 @@ void ql_mpi_reset_work(struct work_struct *work) return; } - if (!ql_core_dump(qdev, qdev->mpi_coredump)) { - netif_err(qdev, drv, qdev->ndev, "Core is dumped!\n"); - qdev->core_is_dumped = 1; - queue_delayed_work(qdev->workqueue, - &qdev->mpi_core_to_log, 5 * HZ); + if (qdev->mpi_coredump) { + if (!ql_core_dump(qdev, qdev->mpi_coredump)) { + netif_err(qdev, drv, qdev->ndev, "Core is dumped!\n"); + qdev->core_is_dumped = 1; + queue_delayed_work(qdev->workqueue, + &qdev->mpi_core_to_log, 5 * HZ); + } } ql_soft_reset_mpi_risc(qdev); }
Also changed a log message to indicate that memory was not allocated instead of memory not available! Signed-off-by: Malahal Naineni <malahal@us.ibm.com> --- drivers/net/ethernet/qlogic/qlge/qlge_dbg.c | 4 ++-- drivers/net/ethernet/qlogic/qlge/qlge_mpi.c | 12 +++++++----- 2 files changed, 9 insertions(+), 7 deletions(-)