Re: [PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-25 Thread Zhenzhong Duan
On 2013-08-24 01:15, Konrad Rzeszutek Wilk wrote: On Thu, Aug 22, 2013 at 03:14:34PM -0600, Bjorn Helgaas wrote: On Mon, Aug 5, 2013 at 1:21 AM, Zhenzhong Duan wrote: xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask in dom0. It's better to do the same in

Re: [PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-25 Thread Zhenzhong Duan
On 2013-08-24 01:15, Konrad Rzeszutek Wilk wrote: On Thu, Aug 22, 2013 at 03:14:34PM -0600, Bjorn Helgaas wrote: On Mon, Aug 5, 2013 at 1:21 AM, Zhenzhong Duan zhenzhong.d...@oracle.com wrote: xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask in dom0. It's better to

Re: [PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-23 Thread Konrad Rzeszutek Wilk
On Thu, Aug 22, 2013 at 03:14:34PM -0600, Bjorn Helgaas wrote: > On Mon, Aug 5, 2013 at 1:21 AM, Zhenzhong Duan > wrote: > > xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask > > in dom0. It's better to do the same in default_restore_msi_irqs for > > baremetal. > > > >

Re: [PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-23 Thread Konrad Rzeszutek Wilk
On Thu, Aug 22, 2013 at 03:14:34PM -0600, Bjorn Helgaas wrote: On Mon, Aug 5, 2013 at 1:21 AM, Zhenzhong Duan zhenzhong.d...@oracle.com wrote: xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask in dom0. It's better to do the same in default_restore_msi_irqs for

Re: [PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-22 Thread Bjorn Helgaas
On Mon, Aug 5, 2013 at 1:21 AM, Zhenzhong Duan wrote: > xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask > in dom0. It's better to do the same in default_restore_msi_irqs for baremetal. > > Move restore of mask in default_restore_msi_irqs, this could avoid mask >

Re: [PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-22 Thread Bjorn Helgaas
On Mon, Aug 5, 2013 at 1:21 AM, Zhenzhong Duan zhenzhong.d...@oracle.com wrote: xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask in dom0. It's better to do the same in default_restore_msi_irqs for baremetal. Move restore of mask in default_restore_msi_irqs, this could

[PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-05 Thread Zhenzhong Duan
xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask in dom0. It's better to do the same in default_restore_msi_irqs for baremetal. Move restore of mask in default_restore_msi_irqs, this could avoid mask restored twice in dom0, and the logic for baremetal keep same. First

[PATCH 2/3 v2] Refactor msi/msix restore code Part2

2013-08-05 Thread Zhenzhong Duan
xen_initdom_restore_msi_irqs trigger a hypercall to restore addr/data/mask in dom0. It's better to do the same in default_restore_msi_irqs for baremetal. Move restore of mask in default_restore_msi_irqs, this could avoid mask restored twice in dom0, and the logic for baremetal keep same. First