RE: [PATCH] drm/ttm: set bo->resv point to tbo->ttm_resv after individualize_resv

2017-11-07 Thread He, Roger
I guess this because you move the resv changing out of lock of bo->resv. Because at the beginning ttm_mem_evict_first may __ttm_bo_reserve(bo->resv) success, and then bo->resv has been changed by another thread. That is not matched and at this point bo->ttm_resv also may been freed already. And

RE: [PATCH 1/6] drm/ttm: add ttm_bo_pipeline_gutting

2018-03-07 Thread He, Roger
Patch 1,4,5: Acked-by: Roger He Patch 2,3,6: Reviewed-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Tuesday, March 06, 2018 10:44 PM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org S

RE: [PATCH 2/5] drm/ttm: keep a reference to transfer pipelined BOs

2018-03-26 Thread He, Roger
Reviewed-by: Roger He -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Friday, March 16, 2018 9:21 PM To: linaro-mm-...@lists.linaro.org; linux-me...@vger.kernel.org; dri-devel@lists.freedesktop.org; amd-...@lists.freede

RE: [PATCH 3/5] drm/ttm: remove the backing store if no placement is given

2018-03-26 Thread He, Roger
Acked-by: Roger He -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Sunday, March 25, 2018 6:58 PM To: linaro-mm-...@lists.linaro.org; linux-me...@vger.kernel.org; dri-devel@lists.freedesktop.org; amd-...@lists.freedeskt

RE: [radeon-alex:amd-staging-dkms-4.13 3272/3830] drivers/staging//vboxvideo/vbox_ttm.c:190:9: error: incompatible type for argument 2 of 'ttm_bo_move_memcpy'

2018-02-21 Thread He, Roger
:01 AM To: He, Roger Cc: kbuild-...@01.org; dri-devel@lists.freedesktop.org; Ma, Le ; Koenig, Christian Subject: [radeon-alex:amd-staging-dkms-4.13 3272/3830] drivers/staging//vboxvideo/vbox_ttm.c:190:9: error: incompatible type for argument 2 of 'ttm_bo_move_memcpy' t

RE: [PATCH] drm/ttm: check if free mem space is under the lower limit

2018-02-22 Thread He, Roger
-Original Message- From: Koenig, Christian Sent: Thursday, February 22, 2018 7:28 PM To: He, Roger ; dri-devel@lists.freedesktop.org Subject: Re: [PATCH] drm/ttm: check if free mem space is under the lower limit Am 22.02.2018 um 11:10 schrieb Roger He: > the free mem space and

RE: [PATCH] drm/ttm: check if free mem space is under the lower limit

2018-02-22 Thread He, Roger
-Original Message- From: Koenig, Christian Sent: Thursday, February 22, 2018 8:54 PM To: He, Roger ; dri-devel@lists.freedesktop.org Subject: Re: [PATCH] drm/ttm: check if free mem space is under the lower limit Am 22.02.2018 um 12:43 schrieb He, Roger: > > -Original M

RE: [PATCH 6/6] drm/ttm: drop persistent_swap_storage from ttm_bo_init and co

2018-02-22 Thread He, Roger
Series is: Reviewed-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Thursday, February 22, 2018 11:02 PM To: dri-devel@lists.freedesktop.org Subject: [PATCH 6/6] drm/ttm: drop persistent_swap_storage fro

RE: [PATCH 8/8] drm/bochs: remove the default ttm_tt_populate callbacks

2018-02-22 Thread He, Roger
Series is: Reviewed-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Thursday, February 22, 2018 7:16 PM To: dri-devel@lists.freedesktop.org Subject: [PATCH 8/8] drm/bochs: remove the default ttm_tt_popul

RE: [PATCH] drm/ttm: check if free mem space is under the lower limit

2018-02-22 Thread He, Roger
-Original Message- From: Alex Deucher [mailto:alexdeuc...@gmail.com] Sent: Thursday, February 22, 2018 10:06 PM To: He, Roger Cc: Koenig, Christian ; dri-devel@lists.freedesktop.org Subject: Re: [PATCH] drm/ttm: check if free mem space is under the lower limit On Thu, Feb 22, 2018 at

RE: [PATCH 4/4] drm/ttm: keep BOs reserved until end of eviction

2018-02-23 Thread He, Roger
looks good to me. Reviewed-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Tuesday, February 20, 2018 8:58 PM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org; linux-ker...@vger.kernel

RE: [PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout.

2018-02-23 Thread He, Roger
-Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Tuesday, February 20, 2018 8:58 PM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org; linux-ker...@vger.kernel.org Subject: [PATCH 3/4] drm/ttm: handle

RE: [PATCH 2/4] drm/amdgpu: use new ww_mutex_is_owned_by function

2018-02-23 Thread He, Roger
Reviewed-by: Roger He -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Tuesday, February 20, 2018 8:58 PM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org; linux-ker...@vger.kernel.org Subject: [PATCH 2

RE: [PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout.

2018-02-23 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Friday, February 23, 2018 8:06 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org; linux-ker...@vger.kernel.org Subject: Re: [PATCH 3/4] drm/ttm: handle already

RE: [PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout.

2018-02-23 Thread He, Roger
: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org; linux-ker...@vger.kernel.org Subject: Re: [PATCH 3/4] drm/ttm: handle already locked BOs during eviction and swapout. Am 23.02.2018 um 10:46 schrieb He, Roger: > > -Original Message- > From:

RE: [PATCH 2/2] drm/ttm: cleanup ttm_tt_create

2018-02-23 Thread He, Roger
Series is: Reviewed-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Friday, February 23, 2018 8:25 PM To: dri-devel@lists.freedesktop.org Subject: [PATCH 2/2] drm/ttm: cleanup ttm_tt_create Cleanup ttm

RE: [PATCH 4/5] drm/ttm: add ttm_sg_tt_init

2018-03-05 Thread He, Roger
Patch 1: Acked-by: Roger He Patch 2~5: Reviewed-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Monday, March 05, 2018 8:07 PM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org; Ben Ske

RE: [PATCH 1/2] drm/ttm: Allow page allocations w/o triggering OOM..

2018-01-15 Thread He, Roger
-Original Message- From: Andrey Grodzovsky [mailto:andrey.grodzov...@amd.com] Sent: Saturday, January 13, 2018 6:29 AM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org Cc: Koenig, Christian ; He, Roger ; Grodzovsky, Andrey Subject: [PATCH 1/2] drm/ttm: Allow page

RE: [PATCH 2/2] drm/amdgpu: Use new TTM flag to avoid OOM triggering.

2018-01-15 Thread He, Roger
-Original Message- From: Andrey Grodzovsky [mailto:andrey.grodzov...@amd.com] Sent: Saturday, January 13, 2018 6:29 AM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org Cc: Koenig, Christian ; He, Roger ; Grodzovsky, Andrey Subject: [PATCH 2/2] drm/amdgpu: Use new TTM

RE: [PATCH 2/2] drm/amdgpu: Use new TTM flag to avoid OOM triggering.

2018-01-16 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Tuesday, January 16, 2018 8:46 PM To: Grodzovsky, Andrey ; Koenig, Christian ; He, Roger ; dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org Subject: Re: [PATCH 2/2] drm/amdgpu: Use

RE: [PATCH v2 1/2] drm/ttm: Allow page allocations w/o triggering OOM..

2018-01-16 Thread He, Roger
, Christian Cc: He, Roger Subject: [PATCH v2 1/2] drm/ttm: Allow page allocations w/o triggering OOM.. This to allow drivers to choose to avoid OOM invocation and handle page allocation failures instead. v2: Remove extra new lines. Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/ttm/ttm_bo.c

RE: [PATCH] drm/ttm: add updated_glob_count in dma_page

2018-01-16 Thread He, Roger
-Original Message- From: Koenig, Christian Sent: Wednesday, January 17, 2018 3:37 PM To: He, Roger ; dri-devel@lists.freedesktop.org Cc: Grodzovsky, Andrey Subject: Re: [PATCH] drm/ttm: add updated_glob_count in dma_page Am 17.01.2018 um 06:54 schrieb Roger He: > add this for correc

RE: [PATCH 2/2] drm/ttm: Don't unreserve swapped BOs that were previously reserved

2018-01-18 Thread He, Roger
good catch! Thanks Roger(Hongbo.He) -Original Message- From: Kuehling, Felix Sent: Friday, January 19, 2018 12:56 AM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org; Koenig, Christian ; He, Roger Cc: Kuehling, Felix Subject: [PATCH 2/2] drm/ttm: Don't unre

RE: [RFC] Per file OOM badness

2018-01-18 Thread He, Roger
-Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Michal Hocko Sent: Friday, January 19, 2018 1:14 AM To: Grodzovsky, Andrey Cc: linux...@kvack.org; amd-...@lists.freedesktop.org; linux-ker...@vger.kernel.org; dri-devel@lists.freedesktop.org;

RE: [RFC] Per file OOM badness

2018-01-18 Thread He, Roger
Basically the idea is right to me. 1. But we need smaller granularity to control the contribution to OOM badness. Because when the TTM buffer resides in VRAM rather than evict to system memory, we should not take this account into badness. But I think it is not easy to implement. 2. If

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-01-29 Thread He, Roger
Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Michal Hocko Sent: Tuesday, January 30, 2018 12:31 AM To: He, Roger Cc: linux...@kvack.org; linux-ker...@vger.kernel.org; dri-devel@lists.freedesktop.org; Koenig, Christian Subject:

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-01-29 Thread He, Roger
wap size and control the swap size used by TTM very accurately. Thanks Roger(Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of He, Roger Sent: Tuesday, January 30, 2018 10:57 AM To: Michal Hocko Cc: linux...@kvack.org; linux-ker

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-01-30 Thread He, Roger
isk at all. Only special test case use more or probably that is intentional. Thanks Roger(Hongbo.He) -Original Message- From: Michal Hocko [mailto:mho...@kernel.org] Sent: Tuesday, January 30, 2018 8:29 PM To: Koenig, Christian Cc: He, Roger ; linux...@kvack.org; linux-ker...@vger.

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-01-31 Thread He, Roger
. Thanks Roger(Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Chunming Zhou Sent: Wednesday, January 31, 2018 3:15 PM To: He, Roger ; dri-devel@lists.freedesktop.org Cc: linux...@kvack.org; linux-ker...@vger.kernel.org; Koenig, Christ

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-01-31 Thread He, Roger
rve will ensure that. if (zone->used_mem > limit) goto out_unlock; Thanks Roger(Hongbo.He) -Original Message- From: Koenig, Christian Sent: Wednesday, January 31, 2018 4:13 PM To: He, Roger ; Zhou, David(ChunMing) ; dri-devel@lists.freedesktop

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-01-31 Thread He, Roger
Hi Michal: How about only EXPORT_SYMBOL_GPL(total_swap_pages) ? Thanks Roger(Hongbo.He) -Original Message- From: He, Roger Sent: Wednesday, January 31, 2018 1:52 PM To: 'Michal Hocko' ; Koenig, Christian Cc: linux...@kvack.org; linux-ker...@vger.kernel.org;

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-02-01 Thread He, Roger
o use more swap space as well. Thanks Roger(Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of He, Roger Sent: Thursday, February 01, 2018 1:48 PM To: Koenig, Christian ; Zhou, David(ChunMing) ; dri-devel@lists.freedesktop.org Cc:

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-02-01 Thread He, Roger
bigger than swap disk, so no swap for TTM is allowed at all. For now we work out an improved version based on get_nr_swap_pages(). Going to send out later. Thanks Roger(Hongbo.He) -Original Message- From: He, Roger Sent: Thursday, February 01, 2018 4:03 PM To: Koenig, Christian ; Zhou, David

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-02-01 Thread He, Roger
my machine, set it as 4G can work. But 4G also not work on test machine with 16GB system memory & 8GB swap disk. Thanks Roger(Hongbo.He) -Original Message- From: Koenig, Christian Sent: Friday, February 02, 2018 3:46 PM To: He, Roger ; Zhou, David(ChunMing) ; dri-devel@lists.freedesk

RE: [PATCH 1/5] drm/ttm: add max_swap_mem in ttm_mem_global

2018-02-02 Thread He, Roger
Need call si_swapinfo to fill those valules . void si_swapinfo(struct sysinfo *val) But that function is not exported as well. Thanks Roger(Hongbo.He) From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Chunming Zhou Sent: Friday, February 02, 2018 3:38 PM To: He

RE: [PATCH] mm/swap: add function get_total_swap_pages to expose total_swap_pages

2018-02-02 Thread He, Roger
mailist today. Thanks Roger(Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of He, Roger Sent: Friday, February 02, 2018 3:54 PM To: Koenig, Christian ; Zhou, David(ChunMing) ; dri-devel@lists.freedesktop.org Cc: linux.

RE: [PATCH 3/5] drm/radeon: remove extra TT unpopulated check

2018-02-04 Thread He, Roger
-Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Saturday, February 03, 2018 3:10 AM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: [PATCH 3/5] drm/radeon: remove extra TT unpopulated check

RE: [PATCH 5/5] drm/amdgpu: replace iova debugfs file with iomem

2018-02-04 Thread He, Roger
Patch1 & 2 & 4, Reviewed-by: Roger He Patch 5: Acked-by: Roger He -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Saturday, February 03, 2018 3:10 AM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.

RE: [PATCH 0/5] prevent OOM triggered by TTM

2018-02-06 Thread He, Roger
, because initially I also concern that but no better way at that time. Going to improve the patches. Thanks! -Original Message- From: Koenig, Christian Sent: Tuesday, February 06, 2018 5:52 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Cc: tho

RE: [PATCH 0/5] prevent OOM triggered by TTM

2018-02-07 Thread He, Roger
truct ttm_bo_device to let driver set according to its request. Thanks Roger(Hongbo.He) -Original Message- From: Thomas Hellstrom [mailto:tho...@shipmail.org] Sent: Wednesday, February 07, 2018 2:43 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Cc: K

RE: [PATCH 0/5] prevent OOM triggered by TTM

2018-02-07 Thread He, Roger
Sure, will make it turn off as default and make the limit configurable. Thanks Roger(Hongbo.He) -Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Wednesday, February 07, 2018 4:41 PM To: He, Roger ; Thomas Hellstrom ; amd-...@lists.freedesktop.org

RE: [PATCH 3/4] drm/ttm: add input parameter force_alloc for ttm_bo_evict_mm

2018-02-08 Thread He, Roger
I can't think of an use case when we don't want this to succeed. That is true. seems I can simplify more here. Thanks Roger(Hongbo.He) -Original Message- From: Koenig, Christian Sent: Thursday, February 08, 2018 8:58 PM To: He, Roger ; amd-...@lists.freedesktop.org;

RE: [PATCH 2/3] drm/ttm: add input parameter force_alloc for ttm_bo_force_list_clean

2018-02-09 Thread He, Roger
Ok. please ignore patch3 since I have some minor changes. Will send out later. Thanks Roger(Hongbo.He) -Original Message- From: Koenig, Christian Sent: Friday, February 09, 2018 5:38 PM To: He, Roger ; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 2/3] drm/ttm: add input parameter

Recall: [PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional

2017-11-10 Thread He, Roger
He, Roger would like to recall the message, "[PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional". ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Recall: [PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional

2017-11-10 Thread He, Roger
He, Roger would like to recall the message, "[PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional". ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Recall: [PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional

2017-11-10 Thread He, Roger
He, Roger would like to recall the message, "[PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional". ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

RE: [PATCH 6/7] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional

2017-11-10 Thread He, Roger
17 4:59 PM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org; He, Roger ; mic...@daenzer.net Subject: [PATCH 6/7] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional Needed for the next patch. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 52 +++

RE: [PATCH 3/4] drm/ttm: make unlocking in ttm_bo_cleanup_refs optional

2017-11-10 Thread He, Roger
Please see comments inline -Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Wednesday, November 08, 2017 11:00 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: [PATCH 3/4] drm/ttm: make unlocking in

RE: [PATCH 1/2] drm/ttm: fix ttm_mem_evict_first once more

2017-11-15 Thread He, Roger
Reviewed-by: Roger He Thanks Roger(Hongbo.He) -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Wednesday, November 15, 2017 8:32 PM To: amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: [PATCH 1/2] dr

RE: [PATCH 2/4] drm/ttm: add page order support in ttm_pages_put

2017-11-21 Thread He, Roger
From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Chunming Zhou Sent: Wednesday, November 22, 2017 2:02 PM To: dri-devel@lists.freedesktop.org Subject: Re: [PATCH 2/4] drm/ttm: add page order support in ttm_pages_put On 2017年11月22日 13:36, Roger He wrote: Change-I

RE: [PATCH 1/4] drm/ttm: add page order in page pool

2017-11-22 Thread He, Roger
-Original Message- From: Koenig, Christian Sent: Wednesday, November 22, 2017 3:48 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 1/4] drm/ttm: add page order in page pool Am 22.11.2017 um 06:36 schrieb Roger He: > to indic

RE: [PATCH 2/4] drm/ttm: add page order support in ttm_pages_put

2017-11-22 Thread He, Roger
Hi Christian: This is old patch, I already send new patches. And it is clean and simple. Please check that. Thanks Roger(Hongbo.He) -Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Wednesday, November 22, 2017 5:29 PM To: He, Roger ; amd

RE: [PATCH 3/5] drm/ttm: add page order support in ttm_pages_put

2017-11-22 Thread He, Roger
[mailto:ckoenig.leichtzumer...@gmail.com] Sent: Wednesday, November 22, 2017 6:01 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 3/5] drm/ttm: add page order support in ttm_pages_put That completely negates the advantage of setting write back on

RE: [PATCH 1/2] drm/ttm: swap consecutive allocated cached pages v2

2017-12-04 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Monday, December 04, 2017 8:13 PM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org; He, Roger ; ckoenig.leichtzumer...@gmail.com Subject: [PATCH 1/2] drm/ttm: swap consecutive

RE: [PATCH 2/2] drm/ttm: swap consecutive allocated pooled pages v3

2017-12-04 Thread He, Roger
Series is: Reviewed-by: Roger He Thanks Roger(Hongbo.He) -Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Monday, December 04, 2017 8:46 PM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org; He, Roger Subject: [PATCH 2/2] drm

RE: [PATCH 2/2] drm/ttm: swap consecutive allocated pooled pages v3

2017-12-04 Thread He, Roger
t - 1] == p - 1) { + swap(p, pages[count - 1]);// swap seems breaking the plist, after that iteration plist will not work. } pages[count++] = temp; + } -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of

RE: [PATCH] drm/ttm: swap consecutive allocated pooled pages v4

2017-12-05 Thread He, Roger
Reviewed-by: Roger He Thanks Roger(Hongbo.He) -Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Tuesday, December 05, 2017 8:55 PM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org; He, Roger Subject: [PATCH] drm/ttm: swap

RE: [PATCH 1/7] drm/ttm:fix incorrect calculate on shrink_pages

2017-12-12 Thread He, Roger
Reviewed-by: Roger He Thanks Roger(Hongbo.He) -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Monk Liu Sent: Tuesday, December 12, 2017 5:47 PM To: amd-...@lists.freedesktop.org Cc: Liu, Monk Subject: [PATCH 1/7] drm/ttm:fix incorrect calcul

RE: [PATCH 3/6] drm/ttm: use an ttm operation ctx for ttm_bo_move_xxx

2017-12-12 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Tuesday, December 12, 2017 6:34 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 3/6] drm/ttm: use an ttm operation ctx for ttm_bo_move_xxx Am

RE: [PATCH 4/6] drm/ttm: init locked again to prevent incorrect unlock

2017-12-12 Thread He, Roger
l.com] Sent: Tuesday, December 12, 2017 6:37 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 4/6] drm/ttm: init locked again to prevent incorrect unlock Am 12.12.2017 um 10:33 schrieb Roger He: > Change-Id: I8db51d843955f5db14bb4bbff892eaed

RE: [PATCH 5/6] drm/ttm: enable eviction for Per-VM-BO

2017-12-12 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Tuesday, December 12, 2017 6:40 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 5/6] drm/ttm: enable eviction for Per-VM-BO Am 12.12.2017 um 10

RE: [PATCH 1/6] drm/ttm: add on_alloc_stage and reservation into ttm_operation_ctx

2017-12-14 Thread He, Roger
, 2017 3:21 AM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 1/6] drm/ttm: add on_alloc_stage and reservation into ttm_operation_ctx Hi, I think this series is quite poorly documented. We should have a log message explaining the purpose of the

RE: [PATCH] drm/ttm: enable eviction for Per-VM-BO

2017-12-15 Thread He, Roger
-Original Message- From: Thomas Hellstrom [mailto:tho...@shipmail.org] Sent: Friday, December 15, 2017 2:25 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Cc: Koenig, Christian Subject: Re: [PATCH] drm/ttm: enable eviction for Per-VM-BO Hi. On 12/14

RE: [PATCH 3/3] drm/ttm: cleanup some old defines

2017-12-17 Thread He, Roger
The series is: Reviewed-by: Roger He Thanks Roger(Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Saturday, December 16, 2017 2:32 AM To: dri-devel@lists.freedesktop.org Subject: [PATCH 3/3] drm/ttm: cleanu

RE: [PATCH 3/7] drm/ttm: use an operation ctx for ttm_mem_global_alloc_page

2017-12-20 Thread He, Roger
-Original Message- From: Christian König [mailto:ckoenig.leichtzumer...@gmail.com] Sent: Wednesday, December 20, 2017 9:36 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re: [PATCH 3/7] drm/ttm: use an operation ctx for

RE: [PATCH 1/7] drm/ttm: call ttm_bo_swapout directly when ttm shrink

2017-12-21 Thread He, Roger
eviction and swapout here is the test always overalloction. Thanks Roger(Hongbo.He) -Original Message- From: Thomas Hellstrom [mailto:tho...@shipmail.org] Sent: Thursday, December 21, 2017 3:34 PM To: He, Roger ; amd-...@lists.freedesktop.org; dri-devel@lists.freedesktop.org Subject: Re

RE: [PATCH] drm/ttm: drop the spin in delayed delete if the trylock doesn't work

2017-12-21 Thread He, Roger
Reviewed-by: Roger He -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Friday, December 22, 2017 2:06 AM To: dri-devel@lists.freedesktop.org; amd-...@lists.freedesktop.org Subject: [PATCH] drm/ttm: drop the spin in delaye

RE: [PATCH 6/7] drm/ttm: add ttm_bo_evict_swapout_allowable to check bo is allowable to evict or swapout

2017-12-24 Thread He, Roger
-Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Thomas Hellstrom Sent: Thursday, December 21, 2017 3:51 PM To: dri-devel@lists.freedesktop.org Subject: Re: [PATCH 6/7] drm/ttm: add ttm_bo_evict_swapout_allowable to check bo is allowable to

RE: [PATCH 2/3] drm/ttm: add new function to check if bo is allowable to evict or swapout

2017-12-24 Thread He, Roger
+ Thomas -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Roger He Sent: Monday, December 25, 2017 2:08 PM To: dri-devel@lists.freedesktop.org Cc: He, Roger ; Koenig, Christian Subject: [PATCH 2/3] drm/ttm: add new function to check if bo

RE: [PATCH] staging: vboxvideo adapt to new TTM interface

2017-12-28 Thread He, Roger
Seems I have no permission to push the patch into amd-staging-drm-next. Needs Whitelisted. http://git.amd.com:8080/#/c/124051/1 anyone can help? Thanks Roger(Hongbo.He) -Original Message- From: Zhou, David(ChunMing) Sent: Thursday, December 28, 2017 12:24 PM To: He, Roger ; dri-devel

RE: [PATCH] drm/ttm: optimize errors checking and free _manager when finishing

2018-01-01 Thread He, Roger
-Original Message- From: Xiongwei Song [mailto:sxwj...@gmail.com] Sent: Sunday, December 31, 2017 7:40 PM To: Koenig, Christian ; He, Roger ; airl...@linux.ie Cc: dri-devel@lists.freedesktop.org; linux-ker...@vger.kernel.org Subject: [PATCH] drm/ttm: optimize errors checking and free

RE: [PATCH] staging: vboxvideo adapt to new TTM interface

2018-01-01 Thread He, Roger
+Alex Do you know who can help about it? Thanks Roger(Hongbo.He) -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of He, Roger Sent: Thursday, December 28, 2017 6:21 PM To: dri-devel@lists.freedesktop.org; Bridgman, John ; Gregory, Robert

RE: next/master build: 198 builds: 1 failed, 197 passed, 1 error, 148 warnings (next-20180110)

2018-01-10 Thread He, Roger
homas Hellstrom ; Dave Airlie ; He, Roger ; Maarten Lankhorst ; Francisco Jerez ; Viresh Kumar ; Shiraz Hashim ; Linus Walleij ; dri-devel Subject: Re: next/master build: 198 builds: 1 failed, 197 passed, 1 error, 148 warnings (next-20180110) On Wed, Jan 10, 2018 at 4:54 PM, Christian König

RE: [PATCH 1/2] drm/ttm: don't update global memory count for some special cases

2018-01-11 Thread He, Roger
It is the first step for the GTT size issue. Need other patches I am working on it. Thanks Roger(Hongbo.He) -Original Message- From: Zhou, David(ChunMing) Sent: Friday, January 12, 2018 2:46 PM To: He, Roger ; dri-devel@lists.freedesktop.org Cc: Koenig, Christian Subject: Re: [PATCH 1/2