Bug 701621

Summary: suspend test failed on RHEL6
Product: [Retired] Red Hat Hardware Certification Program Reporter: Caspar Zhang <czhang>
Component: Test Suite (tests)Assignee: Greg Nichols <gnichols>
Status: CLOSED ERRATA QA Contact: Red Hat Kernel QE team <kernel-qe>
Severity: high Docs Contact:
Priority: high    
Version: 1.3CC: qcai, rlandry, ykun
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
When resuming from hibernating, the change of one kernel message from RHEL5 to RHEL6 caused suspend test to fail in v7 1.2, this issue has been fixed in v7 1.3, now suspend test passes on both RHEL5 and RHEL6.
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-05-09 16:14:30 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
suspend patch accounting for suspend logging changes in RHEL6
none
more fix on function key none

Description Caspar Zhang 2011-05-03 11:57:36 UTC
Description of problem:

pm-utils changed the words when hibernating:

swsusp: Need to copy

to "PM: Need to copy xxxx pages"

which will cause hibernate test fail on RHEL6.

Version-Release number of selected component (if applicable):
v7 1.3 R41

How reproducible:


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


Expected results:


Additional info:

Comment 1 Caspar Zhang 2011-05-03 13:48:33 UTC
correction, it's kernel behavior, not pm-utils: kernel/power/snapshot.c

Anyway, a fix would be quick.

Comment 2 Rob Landry 2011-05-03 13:51:00 UTC
Can the error/run logs be linked/attached?

It looks like that's strings matched in the test code, so I would expect the above listed change to cause the hibernate check to fail.  Greg, does that sound accurate?

Comment 3 Greg Nichols 2011-05-03 14:37:38 UTC
Created attachment 496548 [details]
suspend patch accounting for suspend logging changes in RHEL6

Comment 5 Caspar Zhang 2011-05-03 14:49:02 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:
When resuming from hibernating, the change of one kernel message from RHEL5 to RHEL6 caused suspend test to fail in v7 1.2, this issue has been fixed in v7 1.3, now suspend test passes on both RHEL5 and RHEL6.

Comment 6 Caspar Zhang 2011-05-03 15:15:02 UTC
--- /usr/share/v7/tests/suspend/suspend.py	2011-05-03 22:55:42.000000000 +0800
+++ suspend.py	2011-05-03 15:16:36.000000000 +0800
@@ -20,6 +20,7 @@
 from v7.tags import Constants, DeviceClass, TestTag
 from v7.hal import Hal
 from v7.controller import Controller
+from v7.redhatrelease import RedHatRelease
 
 class SuspendTest(Test):        
     
@@ -156,7 +157,7 @@
             print "Error: unknown suspend method %s" % method
             return False
 
-        if source == Constants.OSCommand:
+        if source == Constants.OSCommand or RedHatRelease().getVersion() == 6:
             suspendMessage = "Disabling non-boot CPUs"
         if not suspendMessage in log:
             print "Error: could not verify suspend"
@@ -164,10 +165,11 @@
         # otherwise
         print "Verified suspend"
         resumeMessage = "Resuming computer"
-        if source == Constants.OSCommand:
+        if source == Constants.OSCommand or RedHatRelease().getVersion() == 6:
             resumeMessage = "Enabling non-boot CPUs"
         if not resumeMessage in log:
             print "Error: could not verify resume"
+            return False
             
         # check that they used the correct method
         self.verifiedResumeMethod = Constants.memory

Comment 7 Caspar Zhang 2011-05-03 15:22:19 UTC
Created attachment 496565 [details]
more fix on function key

Comment 9 errata-xmlrpc 2011-05-09 16:14:30 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

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