Bug 663240

Created attachment 468767 [details]
pwrtest-rhel6

virtio-win version:
1.1.16

Update:
Same result in virtio-win-1.1.17.

DTM log:
 Start Test 1/11/2011 3:24:07.228 PM PwrTest
Message 1/11/2011 3:24:07.228 PM
Error 1/11/2011 3:24:07.228 PM ERROR: System does not support any sleep with RTC wakeup enable!
File:   d:\5429\base\power\tools\pwrtest\exe\sleep.cpp Line: 1789
Error Type:   NT_STATUS
Error Code:   0xc0000001
Error Text:   Error 0xc0000001
End Test 1/11/2011 3:24:08.228 PM PwrTest
Result:   Fail
Repro:   pwrtest /sleep /c:4 /d:300 /s:all

According to Vadim it is a bios bug. Moving to seabios, assigning to Gleb.

(In reply to comment #3)
> Update:
> Same result in virtio-win-1.1.17.

Could you confirm the version of packages on the host?

Especially:
- kernel
- qemu-kvm
- seabios
- vgabios

(In reply to comment #5)
> (In reply to comment #3)
> > Update:
> > Same result in virtio-win-1.1.17.
> 
> Could you confirm the version of packages on the host?
> 
> Especially:
> - kernel
> - qemu-kvm
> - seabios
> - vgabios

kernel-2.6.32-95.el6.x86_64
qemu-kvm-0.12.1.2-2.129.el6.x86_64
seabios-0.5.1-3.el6.x86_64
vgabios-0.6b-3.4.el6.noarch

What I would like to know it why it worked before. We already signed drivers and didn't see the error. So what changed?

Vadim you had a patch ready. Can you send it?

(In reply to comment #7)
> What I would like to know it why it worked before. We already signed drivers
> and didn't see the error. So what changed?
> 
> Vadim you had a patch ready. Can you send it?

Seems it did not work before. We passed whql and got the signatures because it is not a blocker. It is a failed child job though the parent(main) job got passed.

(In reply to comment #7)
> What I would like to know it why it worked before. We already signed drivers
> and didn't see the error. So what changed?
> 
> Vadim you had a patch ready. Can you send it?

When applying the following changes, we can pass S4 test case.
But I have never tried to run the entire test.

for RHEL5:
diff --git a/bios/rombios32.c b/bios/rombios32.c
index 8928233..08fc369 100755
--- a/bios/rombios32.c
+++ b/bios/rombios32.c
@@ -1566,8 +1566,8 @@ void acpi_bios_init(void)
     fadt->plvl3_lat = cpu_to_le16(0xfff); // C3 state not supported
     fadt->gpe0_blk = cpu_to_le32(0xafe0);
     fadt->gpe0_blk_len = 4;
-    /* WBINVD + PROC_C1 + SLP_BUTTON + FIX_RTC */
-    fadt->flags = cpu_to_le32((1 << 0) | (1 << 2) | (1 << 5) | (1 <<
6));
+    /* WBINVD + PROC_C1 + SLP_BUTTON + FIX_RTC + RTC_S4*/
+    fadt->flags = cpu_to_le32((1 << 0) | (1 << 2) | (1 << 5) | (1 << 6)
| (1<<7));
     acpi_build_table_header((struct acpi_table_header *)fadt, "FACP",
                             sizeof(*fadt), 1);

for RHEL6:
diff --git a/src/acpi.c b/src/acpi.c
index 1962632..ae542e5 100644
--- a/src/acpi.c
+++ b/src/acpi.c
@@ -336,8 +336,8 @@ build_fadt(int bdf)
     fadt->plvl3_lat = cpu_to_le16(0xfff); // C3 state not supported
     fadt->gpe0_blk = cpu_to_le32(0xafe0);
     fadt->gpe0_blk_len = 4;
-    /* WBINVD + PROC_C1 + SLP_BUTTON + FIX_RTC */
-    fadt->flags = cpu_to_le32((1 << 0) | (1 << 2) | (1 << 5) | (1 <<
6));
+    /* WBINVD + PROC_C1 + SLP_BUTTON + FIX_RTC + RTC_S4*/
+    fadt->flags = cpu_to_le32((1 << 0) | (1 << 2) | (1 << 5) | (1 << 6)
| (1<<7));
 
     build_header((void*)fadt, FACP_SIGNATURE, sizeof(*fadt), 1);

Verified pass on seabios-0.6.1.2-3.el6.
Run this job again after update the seabios to the fixed version. pass.

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-0564.html

I launch "pwrtest /sleep /c:4 /d:300 /s:all" on win7dbg x64, host rhel6.1 with seabios-0.6.1.2-3.el6 and I still get the error.

My kvm line is:

/usr/libexec/qemu-kvm -M rhel6.1.0 -enable-kvm -m 4096 -smp 1,sockets=1,cores=1,threads=1 -name Win7x64-qxlw7 -uuid d03c4eb4-19bc-e4dc-dcb8-9faaba1abc46 -nodefconfig -nodefaults -monitor stdio -rtc base=utc -boot c -drive file=/dev/vg_kvm/debian,if=none,id=drive-ide0-0-0,format=raw,cache=none,aio=threads -device ide-drive,bus=ide.0,unit=0,drive=drive-ide0-0-0,id=ide0-0-0 -drive if=none,media=cdrom,id=drive-ide0-1-0,readonly=on,format=raw -device ide-drive,bus=ide.1,unit=0,drive=drive-ide0-1-0,id=ide0-1-0 -netdev tap,fd=20,id=hostnet0 -device rtl8139,netdev=hostnet0,id=net0,mac=52:54:00:d0:4d:60,bus=pci.0,addr=0x3 -chardev socket,host=localhost,port=4555,id=charserial0 -device isa-serial,chardev=charserial0,id=serial0 -usb -spice port=5909,addr=127.0.0.1,disable-ticketing -vga qxl -global qxl-vga.vram_size=9437184 -device intel-hda,id=sound0,bus=pci.0,addr=0x4 -device hda-duplex,id=sound0-codec0,bus=sound0.0,cad=0 -device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x5 -snapshot

Am I missing some arguments or some update?

thanks