Investigating failed suspend/resume T61

[John Baldwin]

On Tuesday, May 27, 2014 1:39:48 pm Sean Bruno wrote:
> On Tue, 2014-05-27 at 11:32 -0400, John Baldwin wrote:
> > On Friday, May 23, 2014 12:14:58 pm Sean Bruno wrote:
> > > Trying to figure out the failures on suspend resume for the T61 I have.
> > > I see a little acpi error at host startup, but I don't think its
> > > related.  However, I'm not sure what it means.
> > > 
> > > sean
> > > 
> > > ------
> > > 
> > > FreeBSD 11.0-CURRENT #1 r265820: Sat May 10 15:13:37 PDT 2014
> > >     sbruno at bruno:/usr/obj/usr/src/sys/BRUNO amd64
> > > FreeBSD clang version 3.4 (tags/RELEASE_34/final 197956) 20140216
> > > VT: running with driver "vga".
> > > CPU: Intel(R) Core(TM)2 Duo CPU     T7300  @ 2.00GHz (1995.04-MHz
> > > K8-class CPU)
> > >   Origin="GenuineIntel"  Id=0x6fa  Family=0x6  Model=0xf  Stepping=10
> > > 
> > > Features=0xbfebfbff<FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTRR,PGE,MCA,CMOV,PAT,PSE36,CLFLUSH,DTS,ACPI,MMX,FXSR,SSE,SSE2,SS,HTT,TM,PBE>
> > > 
> > > Features2=0xe3bd<SSE3,DTES64,MON,DS_CPL,VMX,EST,TM2,SSSE3,CX16,xTPR,PDCM>
> > >   AMD Features=0x20100800<SYSCALL,NX,LM>
> > >   AMD Features2=0x1<LAHF>
> > >   TSC: P-state invariant, performance statistics
> > > real memory  = 2147483648 (2048 MB)
> > > avail memory = 2007138304 (1914 MB)
> > > Event timer "LAPIC" quality 400
> > > ACPI APIC Table: <LENOVO TP-7L   >
> > > FreeBSD/SMP: Multiprocessor System Detected: 2 CPUs
> > > FreeBSD/SMP: 1 package(s) x 2 core(s)
> > >  cpu0 (BSP): APIC ID:  0
> > >  cpu1 (AP): APIC ID:  1
> > > ACPI BIOS Warning (bug): 32/64X length mismatch in FADT/Gpe1Block: 0/32
> > > (20130823/tbfadt-601)
> > > ACPI BIOS Warning (bug): Optional FADT field Gpe1Block has zero address
> > > or length: 0x000000000000102C/0x0 (20130823/tbfadt-630)
> > 
> > It might be related as Gpe1Block describes a register set that IIRC is used
> > to enter sleep states.  Can you put your acpidump -t somewhere?  (No need
> > for -d as this is in the FADT, not the DSDT.)
> > 
> 
> 
> Here --> http://people.freebsd.org/~sbruno/T61_acpidump.txt

Ah, so the warning is due to the fact that the 'FACP' table has 'X_GPE1_BLOCK'
but no 'GPE1_BLOCK'.  (Note how it has both 'GPE0_BLOCK' and 'X_GPE0_BLOCK' 
which say the same thing.)  Try this workaround to quiet the warning.  I've
no idea if it will help at all with suspend/resume.

Index: sys/contrib/dev/acpica/components/tables/tbfadt.c
===================================================================
--- tbfadt.c	(revision 266442)
+++ tbfadt.c	(working copy)
@@ -601,6 +601,10 @@ AcpiTbValidateFadt (
             ACPI_BIOS_WARNING ((AE_INFO,
                 "32/64X length mismatch in FADT/%s: %u/%u",
                 Name, ACPI_MUL_8 (Length), Address64->BitWidth));
+	    if (Length == 0)
+	    {
+		Length = ACPI_DIV_8 (Address64->BitWidth);
+	    }
         }
 
         if (FadtInfoTable[i].Type & ACPI_FADT_REQUIRED)


-- 
John Baldwin