Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Julien Grall
On 15/05/2023 10:07, Michal Orzel wrote: Reviewed-by: Julien Grall Thanks, Bare in mind that Rahul responded in HTML so there will be a Thanks for the reminder! The patch is now committed. Cheers, -- Julien Grall

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Michal Orzel
On 15/05/2023 11:06, Julien Grall wrote: > > > On 15/05/2023 10:03, Michal Orzel wrote: >> Hi Julien, > > Hi Michal, > >> >> On 15/05/2023 10:56, Julien Grall wrote: >>> >>> >>> Hi, >>> >>> On 12/05/2023 15:35, Michal Orzel wrote: At the moment, even in case of a SMMU being I/O coherent

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Julien Grall
On 15/05/2023 10:03, Michal Orzel wrote: Hi Julien, Hi Michal, On 15/05/2023 10:56, Julien Grall wrote: Hi, On 12/05/2023 15:35, Michal Orzel wrote: At the moment, even in case of a SMMU being I/O coherent, we clean the updated PT as a result of not advertising the coherency feature.

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Michal Orzel
Hi Julien, On 15/05/2023 10:56, Julien Grall wrote: > > > Hi, > > On 12/05/2023 15:35, Michal Orzel wrote: >> At the moment, even in case of a SMMU being I/O coherent, we clean the >> updated PT as a result of not advertising the coherency feature. SMMUv3 >> coherency feature means that page ta

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Ayan Kumar Halder
On 15/05/2023 07:46, Michal Orzel wrote: Hi Ayan, Hi Michal, On 12/05/2023 18:59, Ayan Kumar Halder wrote: Hi Michal, On 12/05/2023 15:35, Michal Orzel wrote: At the moment, even in case of a SMMU being I/O coherent, we clean the updated PT as a result of not advertising the coherency fea

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Julien Grall
Hi, On 12/05/2023 15:35, Michal Orzel wrote: At the moment, even in case of a SMMU being I/O coherent, we clean the updated PT as a result of not advertising the coherency feature. SMMUv3 coherency feature means that page table walks, accesses to memory structures and queues are I/O coherent (re

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Rahul Singh
Hi Michal, On 12 May 2023, at 3:35 pm, Michal Orzel wrote: At the moment, even in case of a SMMU being I/O coherent, we clean the updated PT as a result of not advertising the coherency feature. SMMUv3 coherency feature means that page table walks, accesses to memory structures and queues are I

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-15 Thread Rahul Singh
Hi Michal, > On 12 May 2023, at 3:35 pm, Michal Orzel wrote: > > At the moment, even in case of a SMMU being I/O coherent, we clean the > updated PT as a result of not advertising the coherency feature. SMMUv3 > coherency feature means that page table walks, accesses to memory > structures and q

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-14 Thread Michal Orzel
Hi Ayan, On 12/05/2023 18:59, Ayan Kumar Halder wrote: > Hi Michal, > > On 12/05/2023 15:35, Michal Orzel wrote: >> At the moment, even in case of a SMMU being I/O coherent, we clean the >> updated PT as a result of not advertising the coherency feature. SMMUv3 >> coherency feature means that pag

Re: [PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-12 Thread Ayan Kumar Halder
Hi Michal, On 12/05/2023 15:35, Michal Orzel wrote: At the moment, even in case of a SMMU being I/O coherent, we clean the updated PT as a result of not advertising the coherency feature. SMMUv3 coherency feature means that page table walks, accesses to memory structures and queues are I/O coher

[PATCH 2/2] xen/arm: smmuv3: Advertise coherent table walk if supported

2023-05-12 Thread Michal Orzel
At the moment, even in case of a SMMU being I/O coherent, we clean the updated PT as a result of not advertising the coherency feature. SMMUv3 coherency feature means that page table walks, accesses to memory structures and queues are I/O coherent (refer ARM IHI 0070 E.A, 3.15). Follow the same st