diff options
author | Dan Williams <dan.j.williams@intel.com> | 2011-07-01 15:07:25 -0400 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2011-07-03 07:04:52 -0400 |
commit | a8a0a133b03c6863d0f77229d19befca4de905fa (patch) | |
tree | 387efebbe86ba9b8510fd1374ca3221bb42dbb42 /drivers/scsi/isci/request.c | |
parent | dc00c8b6940aa10ab1ce6a4d10b1bfe7b848781b (diff) |
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: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/scsi/isci/request.c')
-rw-r--r-- | drivers/scsi/isci/request.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c index 33c8ed1741e6..a46e07ac789f 100644 --- a/drivers/scsi/isci/request.c +++ b/drivers/scsi/isci/request.c | |||
@@ -2350,7 +2350,7 @@ static void isci_task_save_for_upper_layer_completion( | |||
2350 | /* No notification to libsas because this request is | 2350 | /* No notification to libsas because this request is |
2351 | * already in the abort path. | 2351 | * already in the abort path. |
2352 | */ | 2352 | */ |
2353 | dev_warn(&host->pdev->dev, | 2353 | dev_dbg(&host->pdev->dev, |
2354 | "%s: Aborted - task = %p, response=%d (%d), status=%d (%d)\n", | 2354 | "%s: Aborted - task = %p, response=%d (%d), status=%d (%d)\n", |
2355 | __func__, | 2355 | __func__, |
2356 | task, | 2356 | task, |
@@ -2373,7 +2373,7 @@ static void isci_task_save_for_upper_layer_completion( | |||
2373 | 2373 | ||
2374 | case isci_perform_error_io_completion: | 2374 | case isci_perform_error_io_completion: |
2375 | /* Use sas_task_abort */ | 2375 | /* Use sas_task_abort */ |
2376 | dev_warn(&host->pdev->dev, | 2376 | dev_dbg(&host->pdev->dev, |
2377 | "%s: Error - task = %p, response=%d (%d), status=%d (%d)\n", | 2377 | "%s: Error - task = %p, response=%d (%d), status=%d (%d)\n", |
2378 | __func__, | 2378 | __func__, |
2379 | task, | 2379 | task, |
@@ -2385,7 +2385,7 @@ static void isci_task_save_for_upper_layer_completion( | |||
2385 | break; | 2385 | break; |
2386 | 2386 | ||
2387 | default: | 2387 | default: |
2388 | dev_warn(&host->pdev->dev, | 2388 | dev_dbg(&host->pdev->dev, |
2389 | "%s: Unknown - task = %p, response=%d (%d), status=%d (%d)\n", | 2389 | "%s: Unknown - task = %p, response=%d (%d), status=%d (%d)\n", |
2390 | __func__, | 2390 | __func__, |
2391 | task, | 2391 | task, |
@@ -2710,7 +2710,7 @@ static void isci_request_io_request_complete(struct isci_host *ihost, | |||
2710 | 2710 | ||
2711 | default: | 2711 | default: |
2712 | /* Catch any otherwise unhandled error codes here. */ | 2712 | /* Catch any otherwise unhandled error codes here. */ |
2713 | dev_warn(&ihost->pdev->dev, | 2713 | dev_dbg(&ihost->pdev->dev, |
2714 | "%s: invalid completion code: 0x%x - " | 2714 | "%s: invalid completion code: 0x%x - " |
2715 | "isci_request = %p\n", | 2715 | "isci_request = %p\n", |
2716 | __func__, completion_status, request); | 2716 | __func__, completion_status, request); |
@@ -3164,7 +3164,7 @@ static enum sci_status isci_smp_request_build(struct isci_request *ireq) | |||
3164 | 3164 | ||
3165 | status = sci_io_request_construct_smp(dev, ireq, task); | 3165 | status = sci_io_request_construct_smp(dev, ireq, task); |
3166 | if (status != SCI_SUCCESS) | 3166 | if (status != SCI_SUCCESS) |
3167 | dev_warn(&ireq->isci_host->pdev->dev, | 3167 | dev_dbg(&ireq->isci_host->pdev->dev, |
3168 | "%s: failed with status = %d\n", | 3168 | "%s: failed with status = %d\n", |
3169 | __func__, | 3169 | __func__, |
3170 | status); | 3170 | status); |
@@ -3219,7 +3219,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost, | |||
3219 | status = sci_io_request_construct(ihost, idev, request); | 3219 | status = sci_io_request_construct(ihost, idev, request); |
3220 | 3220 | ||
3221 | if (status != SCI_SUCCESS) { | 3221 | if (status != SCI_SUCCESS) { |
3222 | dev_warn(&ihost->pdev->dev, | 3222 | dev_dbg(&ihost->pdev->dev, |
3223 | "%s: failed request construct\n", | 3223 | "%s: failed request construct\n", |
3224 | __func__); | 3224 | __func__); |
3225 | return SCI_FAILURE; | 3225 | return SCI_FAILURE; |
@@ -3238,7 +3238,7 @@ static enum sci_status isci_io_request_build(struct isci_host *ihost, | |||
3238 | status = isci_request_stp_request_construct(request); | 3238 | status = isci_request_stp_request_construct(request); |
3239 | break; | 3239 | break; |
3240 | default: | 3240 | default: |
3241 | dev_warn(&ihost->pdev->dev, | 3241 | dev_dbg(&ihost->pdev->dev, |
3242 | "%s: unknown protocol\n", __func__); | 3242 | "%s: unknown protocol\n", __func__); |
3243 | return SCI_FAILURE; | 3243 | return SCI_FAILURE; |
3244 | } | 3244 | } |
@@ -3302,7 +3302,7 @@ int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *ide | |||
3302 | 3302 | ||
3303 | status = isci_io_request_build(ihost, ireq, idev); | 3303 | status = isci_io_request_build(ihost, ireq, idev); |
3304 | if (status != SCI_SUCCESS) { | 3304 | if (status != SCI_SUCCESS) { |
3305 | dev_warn(&ihost->pdev->dev, | 3305 | dev_dbg(&ihost->pdev->dev, |
3306 | "%s: request_construct failed - status = 0x%x\n", | 3306 | "%s: request_construct failed - status = 0x%x\n", |
3307 | __func__, | 3307 | __func__, |
3308 | status); | 3308 | status); |
@@ -3335,7 +3335,7 @@ int isci_request_execute(struct isci_host *ihost, struct isci_remote_device *ide | |||
3335 | 3335 | ||
3336 | if (status != SCI_SUCCESS && | 3336 | if (status != SCI_SUCCESS && |
3337 | status != SCI_FAILURE_REMOTE_DEVICE_RESET_REQUIRED) { | 3337 | status != SCI_FAILURE_REMOTE_DEVICE_RESET_REQUIRED) { |
3338 | dev_warn(&ihost->pdev->dev, | 3338 | dev_dbg(&ihost->pdev->dev, |
3339 | "%s: failed request start (0x%x)\n", | 3339 | "%s: failed request start (0x%x)\n", |
3340 | __func__, status); | 3340 | __func__, status); |
3341 | spin_unlock_irqrestore(&ihost->scic_lock, flags); | 3341 | spin_unlock_irqrestore(&ihost->scic_lock, flags); |