Re: 回复: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-29 Thread Christian König
ian) ; Grodzovsky, Andrey ; Christian König ; dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price Cc: Zhang, Andy ; Jiang, Jerry (SW) Subject: Re: 回复: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak Hi Monk, I

RE: 回复: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-29 Thread Liu, Monk
> /Monk > > > -邮件原件- > 发件人: Koenig, Christian > 发送时间: 2021年3月26日 17:06 > 收件人: Zhang, Jack (Jian) ; Grodzovsky, Andrey > ; Christian König > ; dri-de...@lists.freedesktop.org; > amd-gfx@lists.freedesktop.org; Liu, Monk ; Deng, > Emily ; Rob Herring ; Tome

Re: 回复: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-26 Thread Christian König
ry much, and please understand our painful here /Monk -邮件原件- 发件人: Koenig, Christian 发送时间: 2021年3月26日 17:06 收件人: Zhang, Jack (Jian) ; Grodzovsky, Andrey ; Christian König ; dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Viz

回复: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-26 Thread Liu, Monk
; From: Grodzovsky, Andrey > Sent: Friday, March 26, 2021 12:32 AM > To: Zhang, Jack (Jian) ; Christian König > ; dri-de...@lists.freedesktop.org; > amd-gfx@lists.freedesktop.org; Koenig, Christian > ; Liu, Monk ; Deng, Emily > ; Rob Herring ; Tomeu Vizoso > ; Steven Price >

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-26 Thread Steven Price
) ; dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Koenig, Christian ; Grodzovsky, Andrey ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso Subject: Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak On 15/03/2021 05:23, Zhang, Jack (Jian) wrote: [AMD Public Use] Hi, Rob/To

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-26 Thread Christian König
Original Message- >> From: Zhang, Jack (Jian) >> Sent: Tuesday, March 16, 2021 3:20 PM >> To: dri-de...@lists.freedesktop.org <mailto:dri-de...@lists.freedesktop.org>; amd-gfx@lists.freedesktop.org <mailto:amd-gfx@lists.freedesktop.org>

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-25 Thread Zhang, Jack (Jian)
sts.freedesktop.org; Koenig, Christian ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price Subject: Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak There are a few issues here like - how u handle non guilty singnaled jobs in drm_sched_stop, currently looks lik

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-25 Thread Zhang, Jack (Jian)
, Christian ; Grodzovsky, Andrey ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso Subject: Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak On 15/03/2021 05:23, Zhang, Jack (Jian) wrote: > [AMD Public Use] > > Hi, Rob/Tomeu/Steven, > > Would you please help to re

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-25 Thread Andrey Grodzovsky
ruct drm_sched_entity  *entity;     struct dma_fence_cb cb; + }; *From:* Grodzovsky, Andrey *Sent:* Friday, March 19, 2021 12:17 AM *To:* Zhang, Jack (Jian) ; Christian König ; dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Koenig, Christian ; Liu, Monk ; Deng

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-25 Thread Zhang, Jack (Jian)
12:17 AM To: Zhang, Jack (Jian) ; Christian König ; dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Koenig, Christian ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price Subject: Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak On

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-22 Thread Steven Price
-gfx@lists.freedesktop.org; Koenig, Christian ; Grodzovsky, Andrey ; Liu, Monk ; Deng, Emily Cc: Zhang, Jack (Jian) Subject: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak re-insert Bailing jobs to avoid memory leak. V2: move re-insert step to drm/scheduler logic V3: add

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-18 Thread Andrey Grodzovsky
nk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price ; Grodzovsky, Andrey *Subject:* Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak [AMD Official Use Only - Internal Distribution Only] [AMD Official Use Only - Internal Distribution Only] Hi,Andrey, Go

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-18 Thread Zhang, Jack (Jian)
p.org; Koenig, Christian ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price ; Grodzovsky, Andrey Subject: Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak [AMD Official Use Only - Internal Distribution Only] [AMD Official Use Only - Internal Distributi

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-17 Thread Zhang, Jack (Jian)
...@lists.freedesktop.org ; amd-gfx@lists.freedesktop.org ; Koenig, Christian ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price Subject: Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak I actually have a race condition concern here - see bellow - On

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-17 Thread Andrey Grodzovsky
) Sent: Tuesday, March 16, 2021 3:20 PM To: dri-de...@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Koenig, Christian ; Grodzovsky, Andrey ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price Subject: RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

Re: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-17 Thread Christian König
enig, Christian ; Grodzovsky, Andrey ; Liu, Monk ; Deng, Emily ; Rob Herring ; Tomeu Vizoso ; Steven Price Subject: RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak [AMD Public Use] Ping -Original Message- From: Zhang, Jack (Jian) Sent: Monday, March 15, 2021 1:24 PM To: J

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-16 Thread Zhang, Jack (Jian)
meu Vizoso ; Steven Price Subject: RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak [AMD Public Use] Hi, Rob/Tomeu/Steven, Would you please help to review this patch for panfrost driver? Thanks, Jack Zhang -Original Message- From: Jack Zhang Sent: Monday, March 15,

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-16 Thread Zhang, Jack (Jian)
; Steven Price Subject: RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak [AMD Public Use] Hi, Rob/Tomeu/Steven, Would you please help to review this patch for panfrost driver? Thanks, Jack Zhang -Original Message- From: Jack Zhang Sent: Monday, March 15, 2021 1:21 PM To

RE: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-14 Thread Zhang, Jack (Jian)
; Grodzovsky, Andrey ; Liu, Monk ; Deng, Emily Cc: Zhang, Jack (Jian) Subject: [PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak re-insert Bailing jobs to avoid memory leak. V2: move re-insert step to drm/scheduler logic V3: add panfrost's return value for bailing jobs in ca

[PATCH v3] drm/scheduler re-insert Bailing job to avoid memleak

2021-03-14 Thread Jack Zhang
re-insert Bailing jobs to avoid memory leak. V2: move re-insert step to drm/scheduler logic V3: add panfrost's return value for bailing jobs in case it hits the memleak issue. Signed-off-by: Jack Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 +++- drivers/gpu/drm/amd/amdgpu/amdgpu_jo