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:
Created attachment 522103 [details] patch fix This patch also fix: bug 736638 , bug 736666 and bug 736675
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.
Recent tests passed. Move to "VERIFIED"
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