<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style> </head> <body dir="ltr"> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Yes, correct.</div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> <br> </div> <div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"> Alex<br> </div> <div id="appendonsend"></div> <hr style="display:inline-block;width:98%" tabindex="-1"> <div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Huang, Ray <Ray.Huang@amd.com><br> <b>Sent:</b> Tuesday, February 11, 2020 9:35 AM<br> <b>To:</b> Deucher, Alexander <Alexander.Deucher@amd.com>; Christian König <ckoenig.leichtzumerken@gmail.com>; amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org><br> <b>Cc:</b> Pelloux-prayer, Pierre-eric <Pierre-eric.Pelloux-prayer@amd.com>; Olsak, Marek <Marek.Olsak@amd.com>; Liu, Aaron <Aaron.Liu@amd.com>; Tuikov, Luben <Luben.Tuikov@amd.com>; Koenig, Christian <Christian.Koenig@amd.com><br> <b>Subject:</b> RE: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests</font> <div> </div> </div> <style>  </style> <div lang="EN-US" link="#0563C1" vlink="#954F72"> <div class="x_WordSection1"> <p class="x_msipheader87abd423" style="margin:0in; margin-bottom:.0001pt"><span style="font-size:10.0pt; font-family:"Arial",sans-serif; color:#317100">[AMD Public Use]</span></p> <p class="x_MsoNormal"> </p> <p class="x_MsoNormal">Hi Alex,</p> <p class="x_MsoNormal"> </p> <p class="x_MsoNormal">MR = merge request? Should I create account to send merge request in the freedesktop gitlab?</p> <p class="x_MsoNormal"> </p> <p class="x_MsoNormal">Thanks,</p> <p class="x_MsoNormal">Ray</p> <p class="x_MsoNormal"> </p> <div> <div style="border:none; border-top:solid #E1E1E1 1.0pt; padding:3.0pt 0in 0in 0in"> <p class="x_MsoNormal"><b>From:</b> Deucher, Alexander <Alexander.Deucher@amd.com> <br> <b>Sent:</b> Tuesday, February 11, 2020 10:30 PM<br> <b>To:</b> Christian König <ckoenig.leichtzumerken@gmail.com>; Huang, Ray <Ray.Huang@amd.com>; amd-gfx@lists.freedesktop.org<br> <b>Cc:</b> Pelloux-prayer, Pierre-eric <Pierre-eric.Pelloux-prayer@amd.com>; Olsak, Marek <Marek.Olsak@amd.com>; Liu, Aaron <Aaron.Liu@amd.com>; Tuikov, Luben <Luben.Tuikov@amd.com>; Koenig, Christian <Christian.Koenig@amd.com><br> <b>Subject:</b> Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests</p> </div> </div> <p class="x_MsoNormal"> </p> <p style="margin:15.0pt"><span style="font-size:10.0pt; font-family:"Arial",sans-serif; color:#317100">[AMD Public Use]</span></p> <p class="x_MsoNormal"> </p> <div> <div> <p class="x_MsoNormal"><span style="font-size:12.0pt; color:black">Also, libdrm changes should go through a gitlab MR now.</span></p> </div> <div> <p class="x_MsoNormal"><span style="font-size:12.0pt; color:black"> </span></p> </div> <div> <p class="x_MsoNormal"><span style="font-size:12.0pt; color:black">Alex</span></p> </div> <div> <p class="x_MsoNormal"><span style="font-size:12.0pt; color:black"> </span></p> </div> <div class="x_MsoNormal" align="center" style="text-align:center"> <hr size="2" width="98%" align="center"> </div> <div id="x_divRplyFwdMsg"> <p class="x_MsoNormal"><b><span style="color:black">From:</span></b><span style="color:black"> Christian König <</span><a href="mailto:ckoenig.leichtzumerken@gmail.com">ckoenig.leichtzumerken@gmail.com</a><span style="color:black">><br> <b>Sent:</b> Tuesday, February 11, 2020 6:39 AM<br> <b>To:</b> Huang, Ray <</span><a href="mailto:Ray.Huang@amd.com">Ray.Huang@amd.com</a><span style="color:black">>; </span><a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a><span style="color:black"> <</span><a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a><span style="color:black">><br> <b>Cc:</b> Pelloux-prayer, Pierre-eric <</span><a href="mailto:Pierre-eric.Pelloux-prayer@amd.com">Pierre-eric.Pelloux-prayer@amd.com</a><span style="color:black">>; Olsak, Marek <</span><a href="mailto:Marek.Olsak@amd.com">Marek.Olsak@amd.com</a><span style="color:black">>; Liu, Aaron <</span><a href="mailto:Aaron.Liu@amd.com">Aaron.Liu@amd.com</a><span style="color:black">>; Tuikov, Luben <</span><a href="mailto:Luben.Tuikov@amd.com">Luben.Tuikov@amd.com</a><span style="color:black">>; Deucher, Alexander <</span><a href="mailto:Alexander.Deucher@amd.com">Alexander.Deucher@amd.com</a><span style="color:black">>; Koenig, Christian <</span><a href="mailto:Christian.Koenig@amd.com">Christian.Koenig@amd.com</a><span style="color:black">><br> <b>Subject:</b> Re: [PATCH libdrm 0/4] amdgpu: use amdgpu_cs_submit_raw2 for amdgpu tests</span> </p> <div> <p class="x_MsoNormal"> </p> </div> </div> <div> <div> <p class="x_MsoNormal" style="margin-bottom:12.0pt">Looks good on first glance, but Marek and/or Pierre can probably better <br> judge than me.<br> <br> Christian.<br> <br> Am 11.02.20 um 12:22 schrieb Huang Rui:<br> > So far, the amdgpu_cs_submit_raw2 is used for MesaGL, however the amdgpu tests<br> > still use the legacy interface. So we would like to make amdgpu tests verify the<br> > amdgpu_cs_submit_raw2 API.<br> ><br> > Thanks,<br> > Ray<br> ><br> > Huang Rui (4):<br> > amdgpu: use alloca for dependencies and sem_dependencies<br> > amdgpu: use amdgpu_cs_submit_raw2 in amdgpu_cs_submit<br> > amdgpu: remove the un-used chunk_array<br> > amdgpu: clean up the cs structure variable<br> ><br> > amdgpu/amdgpu_cs.c | 36 +++++++++++++-----------------------<br> > 1 file changed, 13 insertions(+), 23 deletions(-)<br> ></p> </div> </div> </div> </div> </div> </body> </html>