[Xen-devel] [XenPPC] change the system_state to SUSPEND on __do_suspend ?

[geyi <kudva@xxxxxxxxxxxxx>]

I got following warning message with dump_stack when saving the domain
to file:

BUG: sleeping function called from invalid context at
/work/linux/linux-ppc-2.6.hg/kernel/mutex.c:84
in_atomic():0, irqs_disabled():1
Call Trace:
[C000000003FC79D0] [C000000000010650] .show_stack+0x50/0x1cc (unreliable)
[C000000003FC7A80] [C00000000004C5A4] .__might_sleep+0xe4/0xf4
[C000000003FC7B10] [C00000000043A3EC] .mutex_lock+0x34/0x60
[C000000003FC7BB0] [C000000000034ECC] .im_free+0x2c/0xe4
[C000000003FC7C30] [C000000000032A18] .iounmap+0x3c/0x50
[C000000003FC7CA0] [C0000000000435F8] .arch_gnttab_suspend+0x10/0x28
[C000000003FC7D20] [C0000000002AF250] .gnttab_suspend+0x24/0x3c
[C000000003FC7DA0] [C0000000000447C4] .ppc_do_suspend+0xc0/0x314
[C000000003FC7E50] [C0000000002AFED8] .__do_suspend+0x30/0x4c
[C000000003FC7EE0] [C0000000000739BC] .kthread+0x158/0x1a8
[C000000003FC7F90] [C0000000000282AC] .kernel_thread+0x4c/0x68

This message is generated because the Iounmap is called after interrupt
disabled in __do_suspend. It could be removed by changing the
system_state to SYSTEM_SUSPEND_DISK. Is there any concern on modifying
the system_state in __do_suspend?



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel