DM need to know wakeup reason when resume the VM, so vm_resume(char *name) is updated to vm_resume(char *name, int reason), in acrn_vm_ops.c Signed-off-by: Tao Yuhong <yuhong.tao@...> ---

With these changes, S3 feature works. changes in V2: 1.fixed get_sos_wakeup_reason() in acrnd.c 2.fixed handle_resume() in monitor.c changes in V3: 1.wakeup reason is unsigned

- 'reboot.c' defines 'warm_reboot()' only, which is deprecated. - remove the eXecution attribute for 'mttr.h' Signed-off-by: Yonghua Huang <yonghua.huang@...> --- hypervisor/Makefile

-- merge 'assert.h' & 'printf.h' into 'logmsg.h' -- merge 'shell.h' into 'console.h' Signed-off-by: Yonghua Huang <yonghua.huang@...> --- hypervisor/include/debug/assert.h | 21

- move local functions declarations to shell.c - remove 'name' field in 'struct shell' as i/o session s/w layer is deprecated. Signed-off-by: Yonghua Huang <yonghua.huang@...> ---

- to align the coding style in hypervisor Signed-off-by: Yonghua Huang <yonghua.huang@...> --- hypervisor/debug/shell.c | 2 +- hypervisor/debug/{shell_internal.h =>

That's good. Regards Yin, Fengwei

hi, Fengwei, I had a test based on Android GFX bench, there is no performance drop for offscreen & T-Rex test. -- Thanks Jason

vmexit_time and vmexit_cnt is used for debuging only. This patch encloses these debug specific code using #ifdef HV_DEBUG. Signed-off-by: Kaige Fu <kaige.fu@...> ---

Acked...

OK，will do it, and then submit a PR.

Add two files to do compile time assert. One is arch specific, and put in hypervisor/arch/x86/. The other one is common, and put in hypervisor/common/. If the statement is not true, there will be

Move the definitions of offset of fields in boot_ctx to the same header file that the struct boot_ctx is in, to reduce the possibility that some modification would make the offset inconsistant with

The patch series moves the definitions of offset of fields in boot_ctx to the same header file that the struct boot_ctx is in, to reduce the possibility that some modification would make the offset

In handle_resume(), wakeup_reason is updated before call ops->ops->resume(). Because ops->ops->resume() needs to know the latest wakeup reason. Signed-off-by: Tao Yuhong <yuhong.tao@...> ---

get_sos_wakeup_reason() runs into error branch without any error, so no wakeup reason will be returend. Signed-off-by: Tao Yuhong <yuhong.tao@...> --- tools/acrn-manager/acrnd.c | 2 +- 1 file

DM need to know wakeup reason when resume the VM, so vm_resume(char *name) is updated to vm_resume(char *name, int reason), in acrn_vm_ops.c Signed-off-by: Tao Yuhong <yuhong.tao@...> ---

With these changes, S3 feature works. changes in V2: 1.fixed get_sos_wakeup_reason() in acrnd.c 2.fixed handle_resume() in monitor.c yuhong.tao@... (3): tools: vm_resume() requires wakeup

Acked-by: Eddie Dong <eddie.dong@...>

LGTM.