Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-02-20 Thread Kani, Toshi
On Tue, 2018-02-20 at 14:54 +0530, Chintan Pandya wrote: > > On 12/28/2017 4:54 PM, Hanjun Guo wrote: > > From: Hanjun Guo > > > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > > but leave P4D/PUD/PMD unchanged, also will not free the memory of

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-02-20 Thread Kani, Toshi
On Tue, 2018-02-20 at 14:54 +0530, Chintan Pandya wrote: > > On 12/28/2017 4:54 PM, Hanjun Guo wrote: > > From: Hanjun Guo > > > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > > but leave P4D/PUD/PMD unchanged, also will not free the memory of page > > tables. > > >

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-02-20 Thread Chintan Pandya
On 12/28/2017 4:54 PM, Hanjun Guo wrote: From: Hanjun Guo When we using iounmap() to free the 4K mapping, it just clear the PTEs but leave P4D/PUD/PMD unchanged, also will not free the memory of page tables. This will cause issues on ARM64 platform (not sure if other

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-02-20 Thread Chintan Pandya
On 12/28/2017 4:54 PM, Hanjun Guo wrote: From: Hanjun Guo When we using iounmap() to free the 4K mapping, it just clear the PTEs but leave P4D/PUD/PMD unchanged, also will not free the memory of page tables. This will cause issues on ARM64 platform (not sure if other archs have the same

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-01-08 Thread Kani, Toshi
On Sat, 2018-01-06 at 17:46 +0800, Hanjun Guo wrote: > On 2018/1/6 6:15, Kani, Toshi wrote: > > On Thu, 2017-12-28 at 19:24 +0800, Hanjun Guo wrote: > > > From: Hanjun Guo > > > > > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > > > but leave

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-01-08 Thread Kani, Toshi
On Sat, 2018-01-06 at 17:46 +0800, Hanjun Guo wrote: > On 2018/1/6 6:15, Kani, Toshi wrote: > > On Thu, 2017-12-28 at 19:24 +0800, Hanjun Guo wrote: > > > From: Hanjun Guo > > > > > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > > > but leave P4D/PUD/PMD unchanged,

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-01-06 Thread Hanjun Guo
On 2018/1/6 6:15, Kani, Toshi wrote: > On Thu, 2017-12-28 at 19:24 +0800, Hanjun Guo wrote: >> From: Hanjun Guo >> >> When we using iounmap() to free the 4K mapping, it just clear the PTEs >> but leave P4D/PUD/PMD unchanged, also will not free the memory of page >> tables.

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-01-06 Thread Hanjun Guo
On 2018/1/6 6:15, Kani, Toshi wrote: > On Thu, 2017-12-28 at 19:24 +0800, Hanjun Guo wrote: >> From: Hanjun Guo >> >> When we using iounmap() to free the 4K mapping, it just clear the PTEs >> but leave P4D/PUD/PMD unchanged, also will not free the memory of page >> tables. >> >> This will cause

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-01-05 Thread Kani, Toshi
On Thu, 2017-12-28 at 19:24 +0800, Hanjun Guo wrote: > From: Hanjun Guo > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > but leave P4D/PUD/PMD unchanged, also will not free the memory of page > tables. > > This will cause issues on ARM64

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2018-01-05 Thread Kani, Toshi
On Thu, 2017-12-28 at 19:24 +0800, Hanjun Guo wrote: > From: Hanjun Guo > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > but leave P4D/PUD/PMD unchanged, also will not free the memory of page > tables. > > This will cause issues on ARM64 platform (not sure if other

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2017-12-29 Thread Hanjun Guo
oops, the title of this patch is wrong, should be: ioremap: skip setting up huge I/O mappings when p4d/pud/pmd is zero On 2017/12/28 19:24, Hanjun Guo wrote: > From: Hanjun Guo > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > but leave

Re: [RFC patch] ioremap: don't set up huge I/O mappings when p4d/pud/pmd is zero

2017-12-29 Thread Hanjun Guo
oops, the title of this patch is wrong, should be: ioremap: skip setting up huge I/O mappings when p4d/pud/pmd is zero On 2017/12/28 19:24, Hanjun Guo wrote: > From: Hanjun Guo > > When we using iounmap() to free the 4K mapping, it just clear the PTEs > but leave P4D/PUD/PMD unchanged, also