Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-10-03 Thread James Morse
Hi Catalin, Mark, On 25/09/18 15:06, Mark Rutland wrote: > On Tue, Sep 25, 2018 at 05:53:16PM +0800, Jun Yao wrote: >> On Mon, Sep 24, 2018 at 06:19:36PM +0100, Mark Rutland wrote: >>> I've pushed a branch with the cleanups I requested [1] folded in. >>> >>> I'm still a bit worried about the pgd l

Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-25 Thread Catalin Marinas
On Tue, Sep 25, 2018 at 03:06:25PM +0100, Mark Rutland wrote: > On Tue, Sep 25, 2018 at 05:53:16PM +0800, Jun Yao wrote: > > On Mon, Sep 24, 2018 at 06:19:36PM +0100, Mark Rutland wrote: > > > I've pushed a branch with the cleanups I requested [1] folded in. > > > > > > I'm still a bit worried abo

Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-25 Thread Mark Rutland
On Tue, Sep 25, 2018 at 05:53:16PM +0800, Jun Yao wrote: > On Mon, Sep 24, 2018 at 06:19:36PM +0100, Mark Rutland wrote: > > I've pushed a branch with the cleanups I requested [1] folded in. > > > > I'm still a bit worried about the pgd lock, but otherwise I think this > > is sound. I intend to th

Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-25 Thread Jun Yao
Hi Mark, On Mon, Sep 24, 2018 at 06:19:36PM +0100, Mark Rutland wrote: > Hi, > > On Mon, Sep 17, 2018 at 12:43:27PM +0800, Jun Yao wrote: > > Version 5 changes: > > 1. Correct spelling and indentation errors[1]. > > 2. Update init_mm.pgd by assembly[2]. > > 3. Simplify set_p?d() by in

Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-25 Thread Jun Yao
Hi James, On Fri, Sep 21, 2018 at 11:26:39PM +0100, James Morse wrote: > Hi Jun, > > On 09/17/2018 05:43 AM, Jun Yao wrote: > > Version 5 changes: > > 1. Correct spelling and indentation errors[1]. > > 2. Update init_mm.pgd by assembly[2]. > > 3. Simplify set_p?d() by introducing set_

Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-24 Thread Mark Rutland
Hi, On Mon, Sep 17, 2018 at 12:43:27PM +0800, Jun Yao wrote: > Version 5 changes: > 1. Correct spelling and indentation errors[1]. > 2. Update init_mm.pgd by assembly[2]. > 3. Simplify set_p?d() by introducing set_swapper_pgd()[3]. > 4. Reduce unnecessary tlbi for every wri

Re: [PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-21 Thread James Morse
Hi Jun, On 09/17/2018 05:43 AM, Jun Yao wrote: Version 5 changes: 1. Correct spelling and indentation errors[1]. 2. Update init_mm.pgd by assembly[2]. 3. Simplify set_p?d() by introducing set_swapper_pgd()[3]. 4. Reduce unnecessary tlbi for every write to swapper_

[PATCH v5 0/6] Move swapper_pg_dir to rodata section.

2018-09-16 Thread Jun Yao
Version 5 changes: 1. Correct spelling and indentation errors[1]. 2. Update init_mm.pgd by assembly[2]. 3. Simplify set_p?d() by introducing set_swapper_pgd()[3]. 4. Reduce unnecessary tlbi for every write to swapper_pg_dir during paging_init()[3]. v4: ht