Thanks.
On 2019/2/1 14:59, Qing Xia wrote:
> In the first loop, gfp_flags will be modified to high_order_gfp_flags,
> and there will be no chance to change back to low_order_gfp_flags.
>
> Fixes: e7f63771 ("ION: Sys_heap: Add cached pool to spead up cached buffer
> alloc")
> Signed-off-by: Qing
On 2018/1/27 1:48, Liam Mark wrote:
> Since commit 204f672255c2 ("staging: android: ion: Use CMA APIs directly")
> the CMA API is now used directly and therefore the allocated memory is no
> longer automatically zeroed.
>
> Explicitly zero CMA allocated memory to ensure that no data is exposed
On 2018/1/9 18:43, Zeng Tao wrote:
> This issue is introduced by the commit ("ION: Sys_heap:
> Add cached pool to spead up cached buffer alloc"), the gfp_mask low
> order pool is overlapped by the high order inside the loop, so the
> gfp_mask of all pools are set to high_order_gfp_flags.
>
It's useful to show the current memory in detail when alloc failed.
And, there may be a lot of high order alloc failed, just show memory
when an order 0 alloc failed.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
---
drivers/staging/android/ion/ion_system_heap.c | 2 +-
1 file chan
Greg,
I checkout your staging tree[1].
Not find this patch. Can you take it thanks!
[1]git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
On 2016/6/5 23:02, Greg KH wrote:
> On Sun, Jun 05, 2016 at 04:51:23PM +0800, Chen Feng wrote:
>> Hi Greg,
>>
>> Ca
;vma_vm_mm pointer will be compared to current->mm.
>
> Let the binder take a reference to the mm_struct to avoid this.
>
> v2: use the right refcounter
>
> Fixes: a906d6931f3ccaf7de805643190765ddd7378e27
> Signed-off-by: Jann Horn <ja...@google.com>
The idea is good, define the heap ids in header file is inconvenient.
But if we query the heaps information from user-space.
It need to maintain this ids and name userspace one by one. The code may
be complicated in different module user-space.
In android, the gralloc and other lib will all use
Hi Greg,
Can you take this patch?
Thanks
On 2016年05月24日 06:32, Laura Abbott wrote:
> On 05/18/2016 08:03 PM, Chen Feng wrote:
>> Add ion cached pool in system heap. This patch add a cached pool
>> in system heap. It has a great improvement of alloc for cached
>> buffer.
>
zeroed before going to pool
v2: Add cached param in pool to distinguish wheather need to flush
cache at a fresh alloc.
Rework the shrink function.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Xia Qing <saberlily@hisilicon.com>
Reviewed-by: Fu
Hi Laura,
On 2016/5/10 7:50, Laura Abbott wrote:
> On 05/09/2016 01:37 AM, Chen Feng wrote:
>> Add ion cached pool in system heap.
>>
>> Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
>> Signed-off-by: Xia Qing <saberlily@hisilicon.com>
>>
Hi Greg,
Please see my v1 version for detail.
Thank you.
On 2016年05月09日 17:02, Greg KH wrote:
> On Mon, May 09, 2016 at 04:37:34PM +0800, Chen Feng wrote:
>> Add ion cached pool in system heap.
>>
>> Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
>> Si
Add ion cached pool in system heap. This patch add a cached pool
in system heap. It has a great improvement of alloc for cached
buffer.
v1: Makes the cached buffer zeroed before going to pool
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Xia Qing <saberlily@hisi
Add ion cached pool in system heap.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Xia Qing <saberlily@hisilicon.com>
Reviewed-by: Fu Jun <oliver...@hisilicon.com>
---
drivers/staging/android/ion/ion_system_heap.c | 145 +-
Sorry,the order here is not right, please ignore this patch.
I will RESEND it.
On 2016/5/9 14:24, Chen Feng wrote:
> Add ion cached pool in system heap.
>
> Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
> Signed-off-by: Xia Qing <saberlily@hisilicon.com>
Add ion cached pool in system heap.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Xia Qing <saberlily@hisilicon.com>
---
drivers/staging/android/ion/ion_system_heap.c | 120 +-
1 file changed, 79 insertions(+), 41 deletions(-)
Makes the ion buffer always alloced from page pool, no matter
it's cached or not. In this way, it can improve the efficiency
of it.
Currently, there is no difference from cached or non-cached buffer
for the page pool.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
---
drivers/s
Hi greg,
On 2016/3/12 1:12, Greg KH wrote:
> On Fri, Mar 11, 2016 at 05:05:45PM +0800, Chen Feng wrote:
>>
>>
>> On 2016/3/11 15:30, yalin wang wrote:
>>>
>>>> On Mar 11, 2016, at 15:23, Lu Bing <albert.lub...@hisilicon.com> wrote:
>&g
" in tasksize.
>> Refer oom_kill.c,we add pte also.
>>
>> Reviewed-by: Chen Feng <puck.c...@hisilicon.com>
>> Reviewed-by: Fu Jun <oliver...@hisilicon.com>
>> Reviewed-by: Xu YiPing <xuyip...@hisilicon.com>
>> Reviewed-by: Yu DongBin <yu
Add android kernel team.
On 2016/2/1 14:04, Chen Feng wrote:
> Sanity check at binder ioctl function,
> Only allow the shared mm_struct to use the same binder-object
> to do binder operate.
>
> And add proc->vma_vm_mm = current->mm at the open function.
> The libbin
dition to proc->vma_vm_mm
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Wei Dong <weido...@hisilicon.com>
Signed-off-by: Junmin Zhao <zhaojun...@huawei.com>
Reviewed-by: Zhuangluan Su <suzhuangl...@hisilicon.com>
---
drivers/android/binder.c | 5 +
1 fi
->tsk->mm.
If the thread exit the tsk->mm will be NULL.
proc->tsk->mm != current->mm && proc->tsk->mm
So only allow the shared mm_struct to use the same binder-object and
check the existence of mm_struct.
V2: Fix compile error for error commit
Signed-off-by: Ch
->tsk->mm.
If the thread exit the tsk->mm will be NULL.
proc->tsk->mm != current->tsk->mm && proc->tsk->mm
So only allow the shared mm_struct to use the same binder-object and
check the existence of mm_struct.
Signed-off-by: Chen Feng <puck.c...@hisilico
pgsteal_kswapd_normal 7071043
pgscan_kswapd_dma 10428250
pgscan_kswapd_normal 37840094
With this change the reclaim ratio has greatly improved
18.9% -> 72.5%
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Lu bing <albert.lub...@hisilicon.com>
---
drivers/stag
~PTE_RDONLY;
else
pte_val(pte) |= PTE_RDONLY;
So with the dirty bit,it can improve the efficiency
and donnot need to handle memory fault when use access.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Wei Dong <weido...@hi
pgsteal_kswapd_normal 7071043
pgscan_kswapd_dma 10428250
pgscan_kswapd_normal 37840094
With this change the reclaim ratio has greatly improved
18.9% -> 72.5%
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Lu bing <albert.lub...@hisilicon.com>
---
drivers/stag
Add ion support for hi6220 SoC platform.
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Yu Dongbin <yudong...@hisilicon.com>
---
drivers/staging/android/ion/Kconfig| 7 +
drivers/staging/android/ion/Makefile | 1 +
drivers/staging
Add ION node to enable ION on hi6220 SoC platform
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Yu Dongbin <yudong...@hisilicon.com>
---
arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 1 +
arch/arm64/boot/dts/hisilicon/hi6220-ion.dtsi | 20 +
Documentation for hi6220 SoC ION node
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Yu Dongbin <yudong...@hisilicon.com>
---
.../devicetree/bindings/staging/ion/hi6220-ion.txt | 31 ++
1 file changed, 31 insertions(+)
create mode 100644 D
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Yu Dongbin <yudong...@hisilicon.com>
---
drivers/staging/android/ion/Kconfig| 7 +
drivers/staging/android/ion/Makefile | 1 +
drivers/staging/android/ion/hisilicon/Kconfig |
Add ION node to enable ION on hi6220 SoC platform
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Yu Dongbin <yudong...@hisilicon.com>
---
arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 1 +
arch/arm64/boot/dts/hisilicon/hi6220-ion.dtsi | 23
Documentation for hi6220 SoC ION node
Signed-off-by: Chen Feng <puck.c...@hisilicon.com>
Signed-off-by: Yu Dongbin <yudong...@hisilicon.com>
---
.../devicetree/bindings/staging/ion/hi6220-ion.txt | 27 ++
1 file changed, 27 insertions(+)
create mode 100644 D
31 matches
Mail list logo