Skip to content

Commit

Permalink
scsi-disk: fix bug in scsi_block_new_request() introduced by commit 1…
Browse files Browse the repository at this point in the history
…37745c

This patch fixes a bug in scsi_block_new_request() that was introduced
by commit 137745c. If the host cache
is used - i.e. if BDRV_O_NOCACHE is _not_ set - the 'break' statement
needs to be executed to 'fall back' to SG_IO.

Cc: qemu-stable@nongnu.org
Signed-off-by: Ulrich Obergfell <uobergfe@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
  • Loading branch information
rh-ulrich-o authored and bonzini committed Jun 18, 2014
1 parent 4525c13 commit 2fe5a9f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hw/scsi/scsi-disk.c
Expand Up @@ -2526,7 +2526,7 @@ static SCSIRequest *scsi_block_new_request(SCSIDevice *d, uint32_t tag,
* ones (such as WRITE SAME or EXTENDED COPY, etc.). So, without
* O_DIRECT everything must go through SG_IO.
*/
if (bdrv_get_flags(s->qdev.conf.bs) & BDRV_O_NOCACHE) {
if (!(bdrv_get_flags(s->qdev.conf.bs) & BDRV_O_NOCACHE)) {
break;
}

Expand Down

0 comments on commit 2fe5a9f

Please sign in to comment.