<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none"><!-- p { margin-top: 0px; margin-bottom: 0px; }--></style>
</head>
<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 16px; background-color: rgb(255, 255, 255);">
+x265-devel@ mailing list<br>
</p>
<div><br>
</div>
<p>Hi Gopi,<br>
</p>
<p><br>
</p>
<p style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 16px;">Thanks for your feedback. <span style="font-size: 12pt;">I will check the errors you reported.</span></p>
<p style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 16px;"><span style="font-size: 12pt;">I will fix all the issues</span><span style="font-size: 12pt;"> and re-submit </span><span style="font-size: 12pt;">the p2s patch for review</span><span style="font-size: 12pt;">.</span><br>
</p>
<div><span style="font-size: 12pt;"></span></div>
<span style="font-size: 12pt;">Could you please send me the exact cmake flags and the commands you used to run the smoke tests?</span>
<div>I want to make sure my testing covers the use cases you have seen failing.<br>
<span style="font-size: 12pt;"></span>
<p><br>
</p>
<p>I see the following cmake flags in <a href="https://github.com/videolan/x265/blob/master/build/linux/multilib.sh#L6">https://github.com/videolan/x265/blob/master/build/linux/multilib.sh#L6</a><br>
</p>
<p># cmake ../../../source -DHIGH_BIT_DEPTH=ON -DMAIN12=ON<br>
</p>
<p>With this configuration the current code in x265/source/common/aarch64 fails to build.<br>
</p>
<p>Would it be ok to remove the code in <span style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 16px; background-color: rgb(255, 255, 255);">x265/source/common/aarch64 and submit the p2s routines working with 8bit, 10bit, and 12bit configurations?</span><br>
</p>
<p><br>
</p>
<p>On my side I cleaned up aarch64/ipfilter8.S and have it complete with all the functions that x86_64 implements.<br>
</p>
<p>I will make sure ipfilter functions pass TestBench with and without <span style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 16px; background-color: rgb(255, 255, 255);">
HIGH_BIT_DEPTH</span> before I submit the patch for review.<br>
</p>
<p><br>
</p>
<p>Thanks,<br>
</p>
<p>Sebastian<br>
</p>
<div style="color: rgb(33, 33, 33);">
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Gopi Satykrishna Akisetty <gopi.satykrishna@multicorewareinc.com><br>
<b>Sent:</b> Monday, September 6, 2021 12:00 AM<br>
<b>To:</b> Pop, Sebastian<br>
<b>Cc:</b> Liwei Wang; Siva Viswanathan; Janani T E<br>
<b>Subject:</b> RE: [EXTERNAL] [x265] [arm64] Status and combined patch</font>
<div><br>
</div>
</div>
<div>
<div>
<div dir="ltr">Hi Pop Sebastian,
<div>Sorry for the late reply. We have been running some tests at our end and found that the patch is failing when HIGH_BIT_DEPTH is enabled i.e testbench is failing and also there are output changes with decoder errors in the smoke tests for HIGH_BIT_DEPTH
enabled builds. Can you check them at your end once.</div>
<div><br>
</div>
<div>Thanks,</div>
<div>Gopi.</div>
</div>
<br>
<div class="gmail_quote">
<div dir="ltr" class="gmail_attr">On Thu, Aug 19, 2021 at 2:44 AM <<a href="mailto:spop@amazon.com">spop@amazon.com</a>> wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex; border-left:1px solid rgb(204,204,204); padding-left:1ex">
Hello Gopi,<br>
<br>
Please see attached the first patch of the series.<br>
It ports p2s function.<br>
Please let me know if the format of the patch is fine.<br>
I will submit the next patches following your guidelines.<br>
<br>
Thanks,<br>
Sebastian<br>
<br>
<br>
On 8/18/21 6:46 AM, Gopi Satykrishna Akisetty wrote:<br>
> Hello Pop Sebastian,<br>
> Thanks for the contribution of ARM64 patches. Can you resend all the <br>
> final patches that have been reviewed by Min Chen over the development <br>
> mailing list, so that it is easier to check and commit each of them <br>
> individually instead of one big combined patch. Please include <br>
> performance numbers and the specs of the test machine used in the <br>
> patch as part of the commit message. Also send all these patches in a <br>
> sequence so that it is easier to apply them and check.<br>
><br>
> Thanks,<br>
> Gopi.<br>
><br>
<br>
</blockquote>
</div>
</div>
</div>
</div>
</div>
</body>
</html>