Commit a8a0a133 authored by Dan Williams's avatar Dan Williams

isci: pare back error messsages

The messages emitted from task.c and some from request.c likely
duplicate (in a less undertandable way) what is reported by the
midlayer.
Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>
parent dc00c8b6
...@@ -2350,7 +2350,7 @@ static void isci_task_save_for_upper_layer_completion( ...@@ -2350,7 +2350,7 @@ static void isci_task_save_for_upper_layer_completion(
/* No notification to libsas because this request is /* No notification to libsas because this request is
* already in the abort path. * already in the abort path.
*/ */
dev_warn(&host->pdev->dev, dev_dbg(&host->pdev->dev,
"%s: Aborted - task = %p, response=%d (%d), status=%d (%d)\n", "%s: Aborted - task = %p, response=%d (%d), status=%d (%d)\n",
__func__, __func__,
task, task,
...@@ -2373,7 +2373,7 @@ static void isci_task_save_for_upper_layer_completion( ...@@ -2373,7 +2373,7 @@ static void isci_task_save_for_upper_layer_completion(
case isci_perform_error_io_completion: case isci_perform_error_io_completion:
/* Use sas_task_abort */ /* Use sas_task_abort */
dev_warn(&host->pdev->dev, dev_dbg(&host->pdev->dev,
"%s: Error - task = %p, response=%d (%d), status=%d (%d)\n", "%s: Error - task = %p, response=%d (%d), status=%d (%d)\n",
__func__, __func__,
task, task,
...@@ -2385,7 +2385,7 @@ static void isci_task_save_for_upper_layer_completion( ...@@ -2385,7 +2385,7 @@ static void isci_task_save_for_upper_layer_completion(
break; break;
default: default:
dev_warn(&host->pdev->dev, dev_dbg(&host->pdev->dev,
"%s: Unknown - task = %p, response=%d (%d), status=%d (%d)\n", "%s: Unknown - task = %p, response=%d (%d), status=%d (%d)\n",
__func__, __func__,
task, task,
...@@ -2710,7 +2710,7 @@ static void isci_request_io_request_complete(struct isci_host *ihost, ...@@ -2710,7 +2710,7 @@ static void isci_request_io_request_complete(struct isci_host *ihost,
default: default:
/* Catch any otherwise unhandled error codes here. */ /* Catch any otherwise unhandled error codes here. */
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: invalid completion code: 0x%x - " "%s: invalid completion code: 0x%x - "
"isci_request = %p\n", "isci_request = %p\n",
__func__, completion_status, request); __func__, completion_status, request);
...@@ -3164,7 +3164,7 @@ static enum sci_status isci_smp_request_build(struct isci_request *ireq) ...@@ -3164,7 +3164,7 @@ static enum sci_status isci_smp_request_build(struct isci_request *ireq)
status = sci_io_request_construct_smp(dev, ireq, task); status = sci_io_request_construct_smp(dev, ireq, task);
if (status != SCI_SUCCESS) if (status != SCI_SUCCESS)
dev_warn(&ireq->isci_host->pdev->dev, dev_dbg(&ireq->isci_host->pdev->dev,
"%s: failed with status = %d\n", "%s: failed with status = %d\n",
__func__, __func__,
status); status);
...@@ -3219,7 +3219,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost, ...@@ -3219,7 +3219,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost,
status = sci_io_request_construct(ihost, idev, request); status = sci_io_request_construct(ihost, idev, request);
if (status != SCI_SUCCESS) { if (status != SCI_SUCCESS) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: failed request construct\n", "%s: failed request construct\n",
__func__); __func__);
return SCI_FAILURE; return SCI_FAILURE;
...@@ -3238,7 +3238,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost, ...@@ -3238,7 +3238,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost,
status = isci_request_stp_request_construct(request); status = isci_request_stp_request_construct(request);
break; break;
default: default:
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: unknown protocol\n", __func__); "%s: unknown protocol\n", __func__);
return SCI_FAILURE; return SCI_FAILURE;
} }
...@@ -3302,7 +3302,7 @@ int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *ide ...@@ -3302,7 +3302,7 @@ int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *ide
status = isci_io_request_build(ihost, ireq, idev); status = isci_io_request_build(ihost, ireq, idev);
if (status != SCI_SUCCESS) { if (status != SCI_SUCCESS) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: request_construct failed - status = 0x%x\n", "%s: request_construct failed - status = 0x%x\n",
__func__, __func__,
status); status);
...@@ -3335,7 +3335,7 @@ int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *ide ...@@ -3335,7 +3335,7 @@ int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *ide
if (status != SCI_SUCCESS && if (status != SCI_SUCCESS &&
status != SCI_FAILURE_REMOTE_DEVICE_RESET_REQUIRED) { status != SCI_FAILURE_REMOTE_DEVICE_RESET_REQUIRED) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: failed request start (0x%x)\n", "%s: failed request start (0x%x)\n",
__func__, status); __func__, status);
spin_unlock_irqrestore(&ihost->scic_lock, flags); spin_unlock_irqrestore(&ihost->scic_lock, flags);
......
...@@ -104,7 +104,7 @@ static void isci_task_refuse(struct isci_host *ihost, struct sas_task *task, ...@@ -104,7 +104,7 @@ static void isci_task_refuse(struct isci_host *ihost, struct sas_task *task,
/* No notification because this request is already in the /* No notification because this request is already in the
* abort path. * abort path.
*/ */
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: Aborted - task = %p, response=%d, " "%s: Aborted - task = %p, response=%d, "
"status=%d\n", "status=%d\n",
__func__, task, response, status); __func__, task, response, status);
...@@ -112,7 +112,7 @@ static void isci_task_refuse(struct isci_host *ihost, struct sas_task *task, ...@@ -112,7 +112,7 @@ static void isci_task_refuse(struct isci_host *ihost, struct sas_task *task,
case isci_perform_error_io_completion: case isci_perform_error_io_completion:
/* Use sas_task_abort */ /* Use sas_task_abort */
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: Error - task = %p, response=%d, " "%s: Error - task = %p, response=%d, "
"status=%d\n", "status=%d\n",
__func__, task, response, status); __func__, task, response, status);
...@@ -121,7 +121,7 @@ static void isci_task_refuse(struct isci_host *ihost, struct sas_task *task, ...@@ -121,7 +121,7 @@ static void isci_task_refuse(struct isci_host *ihost, struct sas_task *task,
break; break;
default: default:
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: isci task notification default case!", "%s: isci task notification default case!",
__func__); __func__);
sas_task_abort(task); sas_task_abort(task);
...@@ -374,7 +374,7 @@ static int isci_task_execute_tmf(struct isci_host *ihost, ...@@ -374,7 +374,7 @@ static int isci_task_execute_tmf(struct isci_host *ihost,
status = sci_controller_start_task(ihost, idev, ireq); status = sci_controller_start_task(ihost, idev, ireq);
if (status != SCI_TASK_SUCCESS) { if (status != SCI_TASK_SUCCESS) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: start_io failed - status = 0x%x, request = %p\n", "%s: start_io failed - status = 0x%x, request = %p\n",
__func__, __func__,
status, status,
...@@ -604,7 +604,7 @@ static void isci_terminate_request_core(struct isci_host *ihost, ...@@ -604,7 +604,7 @@ static void isci_terminate_request_core(struct isci_host *ihost,
* being aborted. * being aborted.
*/ */
if (status != SCI_SUCCESS) { if (status != SCI_SUCCESS) {
dev_err(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: sci_controller_terminate_request" "%s: sci_controller_terminate_request"
" returned = 0x%x\n", " returned = 0x%x\n",
__func__, status); __func__, status);
...@@ -662,7 +662,7 @@ static void isci_terminate_request_core(struct isci_host *ihost, ...@@ -662,7 +662,7 @@ static void isci_terminate_request_core(struct isci_host *ihost,
if (!termination_completed) { if (!termination_completed) {
dev_err(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: *** Timeout waiting for " "%s: *** Timeout waiting for "
"termination(%p/%p)\n", "termination(%p/%p)\n",
__func__, io_request_completion, __func__, io_request_completion,
...@@ -853,7 +853,7 @@ static int isci_task_send_lu_reset_sata(struct isci_host *ihost, ...@@ -853,7 +853,7 @@ static int isci_task_send_lu_reset_sata(struct isci_host *ihost,
ret = isci_task_execute_tmf(ihost, idev, &tmf, ISCI_SRST_TIMEOUT_MS); ret = isci_task_execute_tmf(ihost, idev, &tmf, ISCI_SRST_TIMEOUT_MS);
if (ret != TMF_RESP_FUNC_COMPLETE) { if (ret != TMF_RESP_FUNC_COMPLETE) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: Assert SRST failed (%p) = %x", "%s: Assert SRST failed (%p) = %x",
__func__, idev, ret); __func__, idev, ret);
...@@ -897,7 +897,7 @@ int isci_task_lu_reset(struct domain_device *domain_device, u8 *lun) ...@@ -897,7 +897,7 @@ int isci_task_lu_reset(struct domain_device *domain_device, u8 *lun)
*/ */
if (!isci_device || if (!isci_device ||
isci_device_is_reset_pending(isci_host, isci_device)) { isci_device_is_reset_pending(isci_host, isci_device)) {
dev_warn(&isci_host->pdev->dev, dev_dbg(&isci_host->pdev->dev,
"%s: No dev (%p), or " "%s: No dev (%p), or "
"RESET PENDING: domain_device=%p\n", "RESET PENDING: domain_device=%p\n",
__func__, isci_device, domain_device); __func__, isci_device, domain_device);
...@@ -970,7 +970,7 @@ static void isci_abort_task_process_cb( ...@@ -970,7 +970,7 @@ static void isci_abort_task_process_cb(
*/ */
if ((old_request->status != aborted) if ((old_request->status != aborted)
&& (old_request->status != completed)) && (old_request->status != completed))
dev_err(&old_request->isci_host->pdev->dev, dev_dbg(&old_request->isci_host->pdev->dev,
"%s: Bad request status (%d): tmf=%p, old_request=%p\n", "%s: Bad request status (%d): tmf=%p, old_request=%p\n",
__func__, old_request->status, tmf, old_request); __func__, old_request->status, tmf, old_request);
break; break;
...@@ -988,7 +988,7 @@ static void isci_abort_task_process_cb( ...@@ -988,7 +988,7 @@ static void isci_abort_task_process_cb(
break; break;
default: default:
dev_err(&old_request->isci_host->pdev->dev, dev_dbg(&old_request->isci_host->pdev->dev,
"%s: Bad cb_state (%d): tmf=%p, old_request=%p\n", "%s: Bad cb_state (%d): tmf=%p, old_request=%p\n",
__func__, cb_state, tmf, old_request); __func__, cb_state, tmf, old_request);
break; break;
...@@ -1046,7 +1046,7 @@ int isci_task_abort_task(struct sas_task *task) ...@@ -1046,7 +1046,7 @@ int isci_task_abort_task(struct sas_task *task)
* SCSI error handler thread to escalate to LUN reset * SCSI error handler thread to escalate to LUN reset
*/ */
if (sas_protocol_ata(task->task_proto)) { if (sas_protocol_ata(task->task_proto)) {
dev_warn(&isci_host->pdev->dev, dev_dbg(&isci_host->pdev->dev,
" task %p is for a STP/SATA device;" " task %p is for a STP/SATA device;"
" returning TMF_RESP_FUNC_FAILED\n" " returning TMF_RESP_FUNC_FAILED\n"
" to cause a LUN reset...\n", task); " to cause a LUN reset...\n", task);
...@@ -1176,7 +1176,7 @@ int isci_task_abort_task(struct sas_task *task) ...@@ -1176,7 +1176,7 @@ int isci_task_abort_task(struct sas_task *task)
ISCI_ABORT_TASK_TIMEOUT_MS); ISCI_ABORT_TASK_TIMEOUT_MS);
if (ret != TMF_RESP_FUNC_COMPLETE) if (ret != TMF_RESP_FUNC_COMPLETE)
dev_err(&isci_host->pdev->dev, dev_dbg(&isci_host->pdev->dev,
"%s: isci_task_send_tmf failed\n", "%s: isci_task_send_tmf failed\n",
__func__); __func__);
} }
...@@ -1395,7 +1395,7 @@ static int isci_smp_execute_task(struct isci_host *ihost, ...@@ -1395,7 +1395,7 @@ static int isci_smp_execute_task(struct isci_host *ihost,
if (res) { if (res) {
del_timer(&task->timer); del_timer(&task->timer);
dev_err(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: executing SMP task failed:%d\n", "%s: executing SMP task failed:%d\n",
__func__, res); __func__, res);
goto ex_err; goto ex_err;
...@@ -1404,12 +1404,12 @@ static int isci_smp_execute_task(struct isci_host *ihost, ...@@ -1404,12 +1404,12 @@ static int isci_smp_execute_task(struct isci_host *ihost,
wait_for_completion(&task->completion); wait_for_completion(&task->completion);
res = -ECOMM; res = -ECOMM;
if ((task->task_state_flags & SAS_TASK_STATE_ABORTED)) { if ((task->task_state_flags & SAS_TASK_STATE_ABORTED)) {
dev_err(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: smp task timed out or aborted\n", "%s: smp task timed out or aborted\n",
__func__); __func__);
isci_task_abort_task(task); isci_task_abort_task(task);
if (!(task->task_state_flags & SAS_TASK_STATE_DONE)) { if (!(task->task_state_flags & SAS_TASK_STATE_DONE)) {
dev_err(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: SMP task aborted and not done\n", "%s: SMP task aborted and not done\n",
__func__); __func__);
goto ex_err; goto ex_err;
...@@ -1432,7 +1432,7 @@ static int isci_smp_execute_task(struct isci_host *ihost, ...@@ -1432,7 +1432,7 @@ static int isci_smp_execute_task(struct isci_host *ihost,
res = -EMSGSIZE; res = -EMSGSIZE;
break; break;
} else { } else {
dev_err(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: task to dev %016llx response: 0x%x " "%s: task to dev %016llx response: 0x%x "
"status 0x%x\n", __func__, "status 0x%x\n", __func__,
SAS_ADDR(dev->sas_addr), SAS_ADDR(dev->sas_addr),
...@@ -1526,7 +1526,7 @@ static void isci_wait_for_smp_phy_reset(struct isci_remote_device *idev, int phy ...@@ -1526,7 +1526,7 @@ static void isci_wait_for_smp_phy_reset(struct isci_remote_device *idev, int phy
tmo = deadline - jiffies; tmo = deadline - jiffies;
if (res) { if (res) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: iteration %d, phase %d:" "%s: iteration %d, phase %d:"
" SMP error=%d, time_remaining=%lu\n", " SMP error=%d, time_remaining=%lu\n",
__func__, iteration, phy_state, res, tmo); __func__, iteration, phy_state, res, tmo);
...@@ -1578,7 +1578,7 @@ static int isci_reset_device(struct isci_host *ihost, ...@@ -1578,7 +1578,7 @@ static int isci_reset_device(struct isci_host *ihost,
if (status != SCI_SUCCESS) { if (status != SCI_SUCCESS) {
spin_unlock_irqrestore(&ihost->scic_lock, flags); spin_unlock_irqrestore(&ihost->scic_lock, flags);
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: sci_remote_device_reset(%p) returned %d!\n", "%s: sci_remote_device_reset(%p) returned %d!\n",
__func__, idev, status); __func__, idev, status);
...@@ -1619,7 +1619,7 @@ static int isci_reset_device(struct isci_host *ihost, ...@@ -1619,7 +1619,7 @@ static int isci_reset_device(struct isci_host *ihost,
} }
if (status != SCI_SUCCESS) { if (status != SCI_SUCCESS) {
dev_warn(&ihost->pdev->dev, dev_dbg(&ihost->pdev->dev,
"%s: sci_remote_device_reset_complete(%p) " "%s: sci_remote_device_reset_complete(%p) "
"returned %d!\n", __func__, idev, status); "returned %d!\n", __func__, idev, status);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment