Bug 432652

This patch is a bug fix and will affect future SBs.

The bug is S3 resume failed when you put SATA disk 2nd port and set SATA
type as native IDE. The root cause is we have not correctly set
operating mode when resume. 

Is this sufficient to push it into R5.2?

We have an HP dx2450 with the SB700 that can be used for validation.

The system is in my office ready for testing.

Created attachment 295210 [details]
Proposed SATA IDE patch to fix resume issue

The patch for RHEL5.2 will have to include the pci_fixups_resume and
resume_early implementation.   I've attached the backport I have come 
up with so far.

Shane, 

Please look through this patch for completeness and provide the upstream patch 
set that adds the resume and resume_early implementation. Once I have the 
complete back port done I would ask you to test on RHEL5 as well.

Also, do we need resume_early and it's counterpart suspend_late?  Or does the 
latter have no bearing to the chipset resume issue?

Ideally we want the smallest patch set necessary to fix the issue.

Regards,
Bhavana

Russ, please add the master tracker.  Just when we thought we were done with R5.2...

Need the usual bullet items answered for an exception request post beta kernel:

1) business justification
2) patch status, is patch upstream
3) the intended patch attached to the bz or a clear pointer to said
4) current testing status
5) test plan

QE Can't ack without this data.

Hi Bhavana,
This backport of PCI_RESUME_FIXUP is well.

(In reply to comment #5)
> Shane, 
> 
> Please look through this patch for completeness and provide the upstream patch 
> set that adds the resume and resume_early implementation. Once I have the 
> complete back port done I would ask you to test on RHEL5 as well.
> 
> Also, do we need resume_early and it's counterpart suspend_late?  Or does the 
> latter have no bearing to the chipset resume issue?
> 
> Ideally we want the smallest patch set necessary to fix the issue.
> 
> Regards,
> Bhavana

upstream commit:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-
2.6.git;a=commit;h=05a7d22b9f02e7fe8fb5ff972b46743948139b3c

Created attachment 298778 [details]
ahci driver side fix patch

1) business justification
   We found the s3 resume failed issue exists in RHEL 5.2 beta. The patch is
for AMD SB600/SB700, which is already be seen in market. So it's very urgent
for us to have it be supported in RHEL5.2.

2) patch status, is patch upstream
   Upstream has accepted the patch, see Comment#9.

3) the intended patch attached to the bz or a clear pointer to said
   If we use the upstream patch, we needs back port pci_fixups_resume. But when
we back port it, we find it involve too much of power management subsystem
(resume_early). So I think it's better for redhat to back port
pci_fixups_resume.
   If pci_fixups_resume will not be back ported, we can also fix the issue on
ahci driver side. Please see the attached ahci driver side fix patch.

4) current testing status
   ahci driver side fix patch is under test by AMD QA, and will be finished in
next week.
5) test plan
   Do acpi test to check whether the issue is fixed.

Thanks Richard for the answering Martin's bulleted items.   

I want to clarify the patch that we want in RHEL5.2 a bit more.  We are
sensitive to the fact that R5.2 is post beta and want to make the least lines 
of change that would still fix the issue but not introduce a massive change.  
The upstream patch as is will need (as Richard described) resume early 
changes in the PM subsystems.  Instead we have chosen to use the existing 
AHCI resume function and adding the relevant code to that function. 

The brew build is being tested at AMD
http://brewweb.devel.redhat.com/brew/taskinfo?taskID=1222747

The 32-bit test have completed with successful results.  The 64-bit test results
will be available Monday.  I will submit the patch to RHML after the 64-bit
results are in as well.

If there are any more questions please let us know.

Posted to RHML on Mar 24.

Both 32-bit and 64-bit test results are done and the brew build works for all
SATA ports, both native IDE and Legacy IDE.    Please provide the BZ ACKs.

in kernel-2.6.18-88.el5
You can download this test kernel from http://people.redhat.com/dzickus/el5

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 the 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-2008-0314.html