Re: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation failure

2019-01-24 Thread Szabolcs Fruhwald via iommu
m-driver-x86-ow...@vger.kernel.org > ow...@vger.kernel.org> On Behalf Of Szabolcs Fruhwald > > Sent: Wednesday, January 23, 2019 5:02 PM > > To: Stuart Hayes > > Cc: platform-driver-...@vger.kernel.org; Szabolcs Fruhwald > > Subject: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation fa

Re: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation failure

2019-01-24 Thread Andy Shevchenko
On Thu, Jan 24, 2019 at 10:31 PM Szabolcs Fruhwald wrote: > > (+iommu list for visibility and confirmation of the intended constant > externalization, see 2nd point below) > Absolutely, I thought so too. But, since the actual need to force > id-mapping comes from the lack of support for non-pci b

Re: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation failure

2019-01-24 Thread Szabolcs Fruhwald via iommu
Hi Andy, I took a quick look at arch_setup_pdev_archdata(), overridden it in dcdbas.c and it works well (despite it's being called twice, since it's called from platform_device_alloc and platform_device_register). However, as I am not super familiar with ELF weak method references, especially wit

Re: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation failure

2019-01-25 Thread Andy Shevchenko
On Fri, Jan 25, 2019 at 3:50 AM Szabolcs Fruhwald wrote: First of all, please do not top post! > I took a quick look at arch_setup_pdev_archdata(), overridden it in > dcdbas.c and it works well (despite it's being called twice, since > it's called from platform_device_alloc and platform_device_r

Re: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation failure

2019-01-25 Thread Robin Murphy
On 25/01/2019 11:58, Andy Shevchenko wrote: On Fri, Jan 25, 2019 at 3:50 AM Szabolcs Fruhwald wrote: First of all, please do not top post! I took a quick look at arch_setup_pdev_archdata(), overridden it in dcdbas.c and it works well (despite it's being called twice, since it's called from pl

Re: [PATCH] dcdbas: Fix Intel-IOMMU domain allocation failure

2019-01-28 Thread Szabolcs Fruhwald via iommu
On Fri, Jan 25, 2019 at 5:12 AM Robin Murphy wrote: > > On 25/01/2019 11:58, Andy Shevchenko wrote: > > On Fri, Jan 25, 2019 at 3:50 AM Szabolcs Fruhwald > > wrote: > > > > First of all, please do not top post! > > > >> I took a quick look at arch_setup_pdev_archdata(), overridden it in > >> dcd