Bug 736679

Summary: v7 storage test: improper dt options for raw I/O test
Product: [Retired] Red Hat Hardware Certification Program Reporter: WANG Chao <chaowang>
Component: Test Suite (tests)Assignee: Greg Nichols <gnichols>
Status: CLOSED ERRATA QA Contact: Caspar Zhang <czhang>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.1CC: chaowang, czhang, fge, gbai, qcai, rlandry, ruyang, ykun
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
In v7 1.3, storage test enabled aio option in both buffered I/O testing and raw I/O testing, which was incorrect since raw I/O should be unbuffered. This issue has been fixed in v7 1.4, now v7 storage test uses "flags=direct" option in raw I/O testing instead of using aio.
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-11-08 15:43:05 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
patch fix none

Description WANG Chao 2011-09-08 12:20:09 UTC
Description of problem:
After code review of /usr/share/v7/tests/storage/storage.py, I found that
dt has a global option 'enable=aio' and I think that is improper for raw I/O test.
raw I/O test should be non-buffered, need to directly write to the block layer.And aio option use memory cache, and may confuse test result.I think remove this option and just with 'flags=direct' is more proper

Version-Release number of selected component (if applicable):
14

How reproducible:
100%

Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 WANG Chao 2011-09-08 12:26:42 UTC
Created attachment 522103 [details]
patch fix

This patch also fix:
bug 736638 , bug 736666 and bug 736675

Comment 4 Caspar Zhang 2011-10-21 15:25:29 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
In v7 1.3, storage test enabled aio option in both buffered I/O testing and raw I/O testing, which was incorrect since raw I/O should be unbuffered. This issue has been fixed in v7 1.4, now v7 storage test uses "flags=direct" option in raw I/O testing instead of using aio.

Comment 5 Caspar Zhang 2011-11-02 10:31:10 UTC
Recent tests passed. Move to "VERIFIED"

Comment 6 errata-xmlrpc 2011-11-08 15:43:05 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2011-1436.html

Comment 7 errata-xmlrpc 2011-11-08 18:33:09 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2011-1436.html