Re: [PATCH v2 08/10] iommu/io-pgtable-arm: Rationalise TTBRn handling

2019-11-22 Thread Jordan Crouse
On Fri, Oct 25, 2019 at 07:08:37PM +0100, Robin Murphy wrote: > TTBR1 values have so far been redundant since no users implement any > support for split address spaces. Crucially, though, one of the main > reasons for wanting to do so is to be able to manage each half entirely > independently,

Re: [PATCH v2 08/10] iommu/io-pgtable-arm: Rationalise TTBRn handling

2019-11-04 Thread Robin Murphy
On 04/11/2019 18:36, Will Deacon wrote: On Mon, Oct 28, 2019 at 06:51:55PM +, Robin Murphy wrote: On 28/10/2019 15:09, Steven Price wrote: [...] --- a/drivers/iommu/io-pgtable-arm-v7s.c +++ b/drivers/iommu/io-pgtable-arm-v7s.c @@ -822,15 +822,13 @@ static struct io_pgtable

Re: [PATCH v2 08/10] iommu/io-pgtable-arm: Rationalise TTBRn handling

2019-11-04 Thread Will Deacon
On Mon, Oct 28, 2019 at 06:51:55PM +, Robin Murphy wrote: > On 28/10/2019 15:09, Steven Price wrote: > [...] > > > --- a/drivers/iommu/io-pgtable-arm-v7s.c > > > +++ b/drivers/iommu/io-pgtable-arm-v7s.c > > > @@ -822,15 +822,13 @@ static struct io_pgtable > > > *arm_v7s_alloc_pgtable(struct

Re: [PATCH v2 08/10] iommu/io-pgtable-arm: Rationalise TTBRn handling

2019-10-28 Thread Robin Murphy
On 28/10/2019 15:09, Steven Price wrote: [...] --- a/drivers/iommu/io-pgtable-arm-v7s.c +++ b/drivers/iommu/io-pgtable-arm-v7s.c @@ -822,15 +822,13 @@ static struct io_pgtable *arm_v7s_alloc_pgtable(struct io_pgtable_cfg *cfg, /* Ensure the empty pgd is visible before any actual TTBR

Re: [PATCH v2 08/10] iommu/io-pgtable-arm: Rationalise TTBRn handling

2019-10-28 Thread Steven Price
On 25/10/2019 19:08, Robin Murphy wrote: > TTBR1 values have so far been redundant since no users implement any > support for split address spaces. Crucially, though, one of the main > reasons for wanting to do so is to be able to manage each half entirely > independently, e.g. context-switching

[PATCH v2 08/10] iommu/io-pgtable-arm: Rationalise TTBRn handling

2019-10-25 Thread Robin Murphy
TTBR1 values have so far been redundant since no users implement any support for split address spaces. Crucially, though, one of the main reasons for wanting to do so is to be able to manage each half entirely independently, e.g. context-switching one set of mappings without disturbing the other.