Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests
Looks fine to me as well. Just as Christian said, would have to be approved by Marek. Regards, Luben On 2020-02-11 9:57 a.m., Deucher, Alexander wrote: > Yes, correct. > > Alex > -- > *From:* Huang, Ray > *Sent:* Tuesday, February 11, 2020 9:35 AM > *To:* Deucher, Alexander ; Christian König > ; amd-gfx@lists.freedesktop.org > > *Cc:* Pelloux-prayer, Pierre-eric ; > Olsak, Marek ; Liu, Aaron ; Tuikov, > Luben ; Koenig, Christian > *Subject:* RE: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for > amdgpu tests > > > [AMD Public Use] > > > > Hi Alex, > > > > MR = merge request? Should I create account to send merge request in the > freedesktop gitlab? > > > > Thanks, > > Ray > > > > *From:* Deucher, Alexander > *Sent:* Tuesday, February 11, 2020 10:30 PM > *To:* Christian König ; Huang, Ray > ; amd-gfx@lists.freedesktop.org > *Cc:* Pelloux-prayer, Pierre-eric ; > Olsak, Marek ; Liu, Aaron ; Tuikov, > Luben ; Koenig, Christian > *Subject:* Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for > amdgpu tests > > > > [AMD Public Use] > > > > Also, libdrm changes should go through a gitlab MR now. > > > > Alex > > > > -- > > *From:*Christian König <mailto:ckoenig.leichtzumer...@gmail.com>> > *Sent:* Tuesday, February 11, 2020 6:39 AM > *To:* Huang, Ray mailto:ray.hu...@amd.com>>; > amd-gfx@lists.freedesktop.org > <mailto:amd-gfx@lists.freedesktop.org> <mailto:amd-gfx@lists.freedesktop.org>> > *Cc:* Pelloux-prayer, Pierre-eric <mailto:pierre-eric.pelloux-pra...@amd.com>>; Olsak, Marek > mailto:marek.ol...@amd.com>>; Liu, Aaron > mailto:aaron....@amd.com>>; Tuikov, Luben > mailto:luben.tui...@amd.com>>; Deucher, Alexander > mailto:alexander.deuc...@amd.com>>; Koenig, > Christian mailto:christian.koe...@amd.com>> > *Subject:* Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for > amdgpu tests > > > > Looks good on first glance, but Marek and/or Pierre can probably better > judge than me. > > Christian. > > Am 11.02.20 um 12:22 schrieb Huang Rui: >> So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu >> tests >> still use the legacy interface. So we would like to make amdgpu tests verify >> the >> amdgpu_cs_submit_raw2 API. >> >> Thanks, >> Ray >> >> Huang Rui (4): >> amdgpu: use alloca for dependencies and sem_dependencies >> amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit >> amdgpu: remove the un-used chunk_array >> amdgpu: clean up the cs structure variable >> >> amdgpu/amdgpu_cs.c | 36 +--- >> 1 file changed, 13 insertions(+), 23 deletions(-) >> > ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests
Yes, correct. Alex From: Huang, Ray Sent: Tuesday, February 11, 2020 9:35 AM To: Deucher, Alexander ; Christian König ; amd-gfx@lists.freedesktop.org Cc: Pelloux-prayer, Pierre-eric ; Olsak, Marek ; Liu, Aaron ; Tuikov, Luben ; Koenig, Christian Subject: RE: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests [AMD Public Use] Hi Alex, MR = merge request? Should I create account to send merge request in the freedesktop gitlab? Thanks, Ray From: Deucher, Alexander Sent: Tuesday, February 11, 2020 10:30 PM To: Christian König ; Huang, Ray ; amd-gfx@lists.freedesktop.org Cc: Pelloux-prayer, Pierre-eric ; Olsak, Marek ; Liu, Aaron ; Tuikov, Luben ; Koenig, Christian Subject: Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests [AMD Public Use] Also, libdrm changes should go through a gitlab MR now. Alex From: Christian König mailto:ckoenig.leichtzumer...@gmail.com>> Sent: Tuesday, February 11, 2020 6:39 AM To: Huang, Ray mailto:ray.hu...@amd.com>>; amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> mailto:amd-gfx@lists.freedesktop.org>> Cc: Pelloux-prayer, Pierre-eric mailto:pierre-eric.pelloux-pra...@amd.com>>; Olsak, Marek mailto:marek.ol...@amd.com>>; Liu, Aaron mailto:aaron@amd.com>>; Tuikov, Luben mailto:luben.tui...@amd.com>>; Deucher, Alexander mailto:alexander.deuc...@amd.com>>; Koenig, Christian mailto:christian.koe...@amd.com>> Subject: Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests Looks good on first glance, but Marek and/or Pierre can probably better judge than me. Christian. Am 11.02.20 um 12:22 schrieb Huang Rui: > So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu tests > still use the legacy interface. So we would like to make amdgpu tests verify > the > amdgpu_cs_submit_raw2 API. > > Thanks, > Ray > > Huang Rui (4): >amdgpu: use alloca for dependencies and sem_dependencies >amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit >amdgpu: remove the un-used chunk_array >amdgpu: clean up the cs structure variable > > amdgpu/amdgpu_cs.c | 36 +--- > 1 file changed, 13 insertions(+), 23 deletions(-) > ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
RE: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests
[AMD Public Use] Hi Alex, MR = merge request? Should I create account to send merge request in the freedesktop gitlab? Thanks, Ray From: Deucher, Alexander Sent: Tuesday, February 11, 2020 10:30 PM To: Christian König ; Huang, Ray ; amd-gfx@lists.freedesktop.org Cc: Pelloux-prayer, Pierre-eric ; Olsak, Marek ; Liu, Aaron ; Tuikov, Luben ; Koenig, Christian Subject: Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests [AMD Public Use] Also, libdrm changes should go through a gitlab MR now. Alex From: Christian König mailto:ckoenig.leichtzumer...@gmail.com>> Sent: Tuesday, February 11, 2020 6:39 AM To: Huang, Ray mailto:ray.hu...@amd.com>>; amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> mailto:amd-gfx@lists.freedesktop.org>> Cc: Pelloux-prayer, Pierre-eric mailto:pierre-eric.pelloux-pra...@amd.com>>; Olsak, Marek mailto:marek.ol...@amd.com>>; Liu, Aaron mailto:aaron@amd.com>>; Tuikov, Luben mailto:luben.tui...@amd.com>>; Deucher, Alexander mailto:alexander.deuc...@amd.com>>; Koenig, Christian mailto:christian.koe...@amd.com>> Subject: Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests Looks good on first glance, but Marek and/or Pierre can probably better judge than me. Christian. Am 11.02.20 um 12:22 schrieb Huang Rui: > So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu tests > still use the legacy interface. So we would like to make amdgpu tests verify > the > amdgpu_cs_submit_raw2 API. > > Thanks, > Ray > > Huang Rui (4): >amdgpu: use alloca for dependencies and sem_dependencies >amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit >amdgpu: remove the un-used chunk_array >amdgpu: clean up the cs structure variable > > amdgpu/amdgpu_cs.c | 36 +--- > 1 file changed, 13 insertions(+), 23 deletions(-) > ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests
[AMD Public Use] Also, libdrm changes should go through a gitlab MR now. Alex From: Christian König Sent: Tuesday, February 11, 2020 6:39 AM To: Huang, Ray ; amd-gfx@lists.freedesktop.org Cc: Pelloux-prayer, Pierre-eric ; Olsak, Marek ; Liu, Aaron ; Tuikov, Luben ; Deucher, Alexander ; Koenig, Christian Subject: Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests Looks good on first glance, but Marek and/or Pierre can probably better judge than me. Christian. Am 11.02.20 um 12:22 schrieb Huang Rui: > So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu tests > still use the legacy interface. So we would like to make amdgpu tests verify > the > amdgpu_cs_submit_raw2 API. > > Thanks, > Ray > > Huang Rui (4): >amdgpu: use alloca for dependencies and sem_dependencies >amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit >amdgpu: remove the un-used chunk_array >amdgpu: clean up the cs structure variable > > amdgpu/amdgpu_cs.c | 36 +--- > 1 file changed, 13 insertions(+), 23 deletions(-) > ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests
Looks good on first glance, but Marek and/or Pierre can probably better judge than me. Christian. Am 11.02.20 um 12:22 schrieb Huang Rui: So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu tests still use the legacy interface. So we would like to make amdgpu tests verify the amdgpu_cs_submit_raw2 API. Thanks, Ray Huang Rui (4): amdgpu: use alloca for dependencies and sem_dependencies amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit amdgpu: remove the un-used chunk_array amdgpu: clean up the cs structure variable amdgpu/amdgpu_cs.c | 36 +--- 1 file changed, 13 insertions(+), 23 deletions(-) ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests
So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu tests still use the legacy interface. So we would like to make amdgpu tests verify the amdgpu_cs_submit_raw2 API. Thanks, Ray Huang Rui (4): amdgpu: use alloca for dependencies and sem_dependencies amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit amdgpu: remove the un-used chunk_array amdgpu: clean up the cs structure variable amdgpu/amdgpu_cs.c | 36 +--- 1 file changed, 13 insertions(+), 23 deletions(-) -- 2.7.4 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
