Bug 1038945 - make scsi-testsuite pass
Summary: make scsi-testsuite pass
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: qemu-kvm
Version: 7.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
: ---
Assignee: Miroslav Rezanina
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On: 825188
Blocks: 833566
TreeView+ depends on / blocked
 
Reported: 2013-12-06 08:32 UTC by xhan
Modified: 2014-03-13 06:56 UTC (History)
15 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of: 825188
Environment:
Last Closed: 2014-01-15 11:19:39 UTC
Target Upstream Version:


Attachments (Terms of Use)
Fix for scsi-testsuite on RHEL7 (1.42 KB, patch)
2014-01-15 11:17 UTC, Miroslav Rezanina
no flags Details | Diff

Comment 1 Paolo Bonzini 2013-12-06 08:46:32 UTC
Looks like the output of sg_* changed from RHEL6 to RHEL7.  Just patch the 00?.out files to remove the lines that were removed.

Comment 4 Miroslav Rezanina 2014-01-15 11:17:30 UTC
Created attachment 850443 [details]
Fix for scsi-testsuite on RHEL7

RHEL-7 sg3_utils do not print following lines:

Sense Information:
sense buffer empty

With this patch all tests pass (run in guest on qemu-kvm-1.5.3-36.el7)

Comment 5 Sibiao Luo 2014-03-13 06:56:43 UTC
(In reply to Miroslav Rezanina from comment #4)
> Created attachment 850443 [details]
> Fix for scsi-testsuite on RHEL7
> RHEL-7 sg3_utils do not print following lines:
> 
> Sense Information:
> sense buffer empty
Yes, thanks for your kindly reminds and patch. I will make a note to our TCMS case.
> With this patch all tests pass (run in guest on qemu-kvm-1.5.3-36.el7)

I also test it pass with your instructions.

host info:
# uname -r && rpm -q qemu-kvm-rhev
3.10.0-100.el7.x86_64
qemu-kvm-rhev-1.5.3-50.el7.x86_64

scsi-testsuite]# ./check
PLATFORM      -- Linux/x86_64 localhost 3.10.0-100.el7.x86_64

001 0s ...
002 0s ...
003 0s ...
004 0s ...
005 0s ...
006 0s ...
007 0s ...
Passed all 7 tests

Best Regards,
sluo


Note You need to log in before you can comment on or make changes to this bug.