On 19/02/2021 08:46, Jan Beulich wrote:
On 18.02.2021 18:41, Julien Grall wrote:
On 18/02/2021 17:04, Jan Beulich wrote:
On 18.02.2021 14:19, Julien Grall wrote:
On 18/02/2021 13:10, Jan Beulich wrote:
On 17.02.2021 17:29, Julien Grall wrote:
On 17/02/2021 15:13, Jan Beulich wrote:
On 18.02.2021 18:41, Julien Grall wrote:
>
>
> On 18/02/2021 17:04, Jan Beulich wrote:
>> On 18.02.2021 14:19, Julien Grall wrote:
>>>
>>>
>>> On 18/02/2021 13:10, Jan Beulich wrote:
On 17.02.2021 17:29, Julien Grall wrote:
> On 17/02/2021 15:13, Jan Beulich wrote:
>> On 17.02.2021
On 18/02/2021 17:04, Jan Beulich wrote:
On 18.02.2021 14:19, Julien Grall wrote:
On 18/02/2021 13:10, Jan Beulich wrote:
On 17.02.2021 17:29, Julien Grall wrote:
On 17/02/2021 15:13, Jan Beulich wrote:
On 17.02.2021 15:24, Julien Grall wrote:> --- a/xen/drivers/passthrough/x86/iommu.c>
On 18.02.2021 14:19, Julien Grall wrote:
>
>
> On 18/02/2021 13:10, Jan Beulich wrote:
>> On 17.02.2021 17:29, Julien Grall wrote:
>>> On 17/02/2021 15:13, Jan Beulich wrote:
On 17.02.2021 15:24, Julien Grall wrote:> ---
a/xen/drivers/passthrough/x86/iommu.c> +++
On 18/02/2021 13:10, Jan Beulich wrote:
On 17.02.2021 17:29, Julien Grall wrote:
On 17/02/2021 15:13, Jan Beulich wrote:
On 17.02.2021 15:24, Julien Grall wrote:> --- a/xen/drivers/passthrough/x86/iommu.c> +++
b/xen/drivers/passthrough/x86/iommu.c> @@ -149,6 +149,13 @@ int
On 17.02.2021 17:29, Julien Grall wrote:
> On 17/02/2021 15:13, Jan Beulich wrote:
>> On 17.02.2021 15:24, Julien Grall wrote:> ---
>> a/xen/drivers/passthrough/x86/iommu.c> +++
>> b/xen/drivers/passthrough/x86/iommu.c> @@ -149,6 +149,13 @@ int
>> arch_iommu_domain_init(struct domain *d)> >
Hi Jan,
On 17/02/2021 15:13, Jan Beulich wrote:
On 17.02.2021 15:24, Julien Grall wrote:> --- a/xen/drivers/passthrough/x86/iommu.c> +++
b/xen/drivers/passthrough/x86/iommu.c> @@ -149,6 +149,13 @@ int arch_iommu_domain_init(struct domain
*d)> > void arch_iommu_domain_destroy(struct domain
On 17.02.2021 15:24, Julien Grall wrote:> ---
a/xen/drivers/passthrough/x86/iommu.c> +++
b/xen/drivers/passthrough/x86/iommu.c> @@ -149,6 +149,13 @@ int
arch_iommu_domain_init(struct domain *d)> > void
arch_iommu_domain_destroy(struct domain *d)> {> +/*> + * There should
be not
From: Julien Grall
At the moment, we are assuming that only iommu_map() can allocate
IOMMU page-table.
Given the complexity of the IOMMU framework, it would be sensible to
have a check closer to the IOMMU allocator. This would avoid to leak
IOMMU page-tables again in the future.