Re: [Xen-devel] [PATCH 7/7] x86/IOMMU: initialize iommu_ops in vendor-independent code

2019-04-05 Thread Woods, Brian
On 3/28/19 9:54 AM, Jan Beulich wrote: > Move this into iommu_hardware_setup() and make that function non- > inline. Move its declaration into common code. > > Signed-off-by: Jan Beulich > Acked-by: Brian Woods ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH 7/7] x86/IOMMU: initialize iommu_ops in vendor-independent code

2019-04-02 Thread Andrew Cooper
On 29/03/2019 09:15, Jan Beulich wrote: On 28.03.19 at 18:50, wrote: >> On 28/03/2019 14:54, Jan Beulich wrote: >>> --- a/xen/drivers/passthrough/x86/iommu.c >>> +++ b/xen/drivers/passthrough/x86/iommu.c >>> @@ -26,6 +26,19 @@ >>> const struct iommu_init_ops *__initdata iommu_init_ops; >>>

Re: [Xen-devel] [PATCH 7/7] x86/IOMMU: initialize iommu_ops in vendor-independent code

2019-04-01 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, March 28, 2019 10:55 PM > > Move this into iommu_hardware_setup() and make that function non- > inline. Move its declaration into common code. > > Signed-off-by: Jan Beulich Reviewed-by: Kevin Tian

Re: [Xen-devel] [PATCH 7/7] x86/IOMMU: initialize iommu_ops in vendor-independent code

2019-03-29 Thread Jan Beulich
>>> On 28.03.19 at 18:50, wrote: > On 28/03/2019 14:54, Jan Beulich wrote: >> --- a/xen/drivers/passthrough/x86/iommu.c >> +++ b/xen/drivers/passthrough/x86/iommu.c >> @@ -26,6 +26,19 @@ >> const struct iommu_init_ops *__initdata iommu_init_ops; >> struct iommu_ops __read_mostly iommu_ops; >>

Re: [Xen-devel] [PATCH 7/7] x86/IOMMU: initialize iommu_ops in vendor-independent code

2019-03-28 Thread Andrew Cooper
On 28/03/2019 14:54, Jan Beulich wrote: > --- a/xen/drivers/passthrough/x86/iommu.c > +++ b/xen/drivers/passthrough/x86/iommu.c > @@ -26,6 +26,19 @@ > const struct iommu_init_ops *__initdata iommu_init_ops; > struct iommu_ops __read_mostly iommu_ops; > > +int __init iommu_hardware_setup(void)

[Xen-devel] [PATCH 7/7] x86/IOMMU: initialize iommu_ops in vendor-independent code

2019-03-28 Thread Jan Beulich
Move this into iommu_hardware_setup() and make that function non- inline. Move its declaration into common code. Signed-off-by: Jan Beulich --- a/xen/drivers/passthrough/amd/pci_amd_iommu.c +++ b/xen/drivers/passthrough/amd/pci_amd_iommu.c @@ -31,7 +31,6 @@ static bool_t __read_mostly