diff mbox

[02/17] atapi: clear sense code

Message ID 1414569232-21357-3-git-send-email-hare@suse.de
State New
Headers show

Commit Message

Hannes Reinecke Oct. 29, 2014, 7:53 a.m. UTC 
The sense code needs to be cleared after REQUEST SENSE.

Signed-off-by: Hannes Reinecke <hare@suse.de>
---
 hw/ide/atapi.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Stefan Hajnoczi Oct. 29, 2014, 2:50 p.m. UTC | #1 
On Wed, Oct 29, 2014 at 08:53:37AM +0100, Hannes Reinecke wrote:
> The sense code needs to be cleared after REQUEST SENSE.
> 
> Signed-off-by: Hannes Reinecke <hare@suse.de>
> ---
>  hw/ide/atapi.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c
> index 10218df..fca9174 100644
> --- a/hw/ide/atapi.c
> +++ b/hw/ide/atapi.c
> @@ -611,9 +611,7 @@ static void cmd_request_sense(IDEState *s, uint8_t *buf)
>      buf[7] = 10;
>      buf[12] = s->asc;
>  
> -    if (s->sense_key == UNIT_ATTENTION) {
> -        s->sense_key = NO_SENSE;
> -    }
> +    s->sense_key = NO_SENSE;
>  
>      ide_atapi_cmd_reply(s, 18, max_len);
>  }

I'm not a SCSI expert, but this patch seems justified.  I skimmed
through SPC, SAM, and ATA/ATAPI-4 sections on REQUEST SENSE and sense
data.

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
diff mbox

Patch

diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c
index 10218df..fca9174 100644
--- a/hw/ide/atapi.c
+++ b/hw/ide/atapi.c
@@ -611,9 +611,7 @@  static void cmd_request_sense(IDEState *s, uint8_t *buf)
     buf[7] = 10;
     buf[12] = s->asc;
 
-    if (s->sense_key == UNIT_ATTENTION) {
-        s->sense_key = NO_SENSE;
-    }
+    s->sense_key = NO_SENSE;
 
     ide_atapi_cmd_reply(s, 18, max_len);
 }