Re: [Xen-devel] [PATCH 1/3] add flag IRQF_NO_SUSPEND in 'struct irqaction'

[Jeremy Fitzhardinge <jeremy@xxxxxxxx>]

On 07/13/09 01:46, Yu, Ke wrote:
> Hi Jeremy,
>
> Forget to mention, this series of patches are targeting for pv_ops dom0 host 
> S3.
>
> We have tried the host S3 branch (xen-tip/dom0/acpi) and find host S3 will 
> hang at disable_nonboot_cpus. the root cause is that suspend_device_irqs () 
> unfortunately will disable the irq of XEN_CALL_FUNCTION_SINGLE_VECTOR IPI and 
> xen_timer_interrupt VIRQ, which however are required by disable_nonboot_cpus. 
> To fix this issue, flag IRQF_NO_SUSPEND is added to Xen IPI and IRQF_TIMER is 
> added to xen_timer_interrupt VIRQ, to notify suspend_device_irqs not to 
> disable these irqs when suspend.
>
> After applying this series of patches to xen-tip/dom0/acpi branch, and 
> rebasing xen-tip/dom0/acpi to xen-tip/master, the pv_ops dom0 host S3 is 
> working in our side.
>   

Thanks for debugging that.  Could you send me a patch?  Is the
IRQF_NO_SUSPEND flag something you've had to add?

Thanks,

    J

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