geyi
2006-Oct-30 22:09 UTC
[Xen-devel] [XenPPC] change the system_state to SUSPEND on __do_suspend ?
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@lists.xensource.com http://lists.xensource.com/xen-devel