<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 03/09/2015 04:34 PM, chen wrote:<br>
    </div>
    <blockquote
      cite="mid:3be8505c.a69.14c00e37ccb.Coremail.chenm003@163.com"
      type="cite">
      <div
        style="line-height:1.7;color:#000000;font-size:14px;font-family:arial">
        <div><br>
           </div>
        At 2015-03-10 07:31:13,dave <a class="moz-txt-link-rfc2396E" href="mailto:dtyx265@gmail.com"><dtyx265@gmail.com></a> wrote:<br>
        <blockquote id="isReplyContent" style="margin: 0px 0px 0px
          0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204,
          204); border-left-width: 1px; border-left-style: solid;">
          <div class="moz-cite-prefix">On 03/09/2015 04:09 PM, chen
            wrote:<br>
          </div>
          <blockquote
            cite="mid:376e7908.a61.14c00cd34b9.Coremail.chenm003@163.com"
            type="cite">
            <div style="color: rgb(0, 0, 0); line-height: 1.7;
              font-family: arial; font-size: 14px;">
              <div><br>
                 </div>
              At 2015-03-10 07:06:26,dave <a moz-do-not-send="true"
                class="moz-txt-link-rfc2396E"
                href="mailto:dtyx265@gmail.com"><dtyx265@gmail.com></a>
              wrote:<br>
              <blockquote id="isReplyContent" style="margin: 0px 0px 0px
                0.8ex; padding-left: 1ex; border-left-color: rgb(204,
                204, 204); border-left-width: 1px; border-left-style:
                solid;">
                <div class="moz-cite-prefix">On 03/09/2015 03:23 PM,
                  chen wrote:<br>
                </div>
                <blockquote
                  cite="mid:4ad6a641.a50.14c00a2f84b.Coremail.chenm003@163.com"
                  type="cite">
                  <div style="color: rgb(0, 0, 0); line-height: 1.7;
                    font-family: arial; font-size: 14px;">
                    <div><br>
                       </div>
                    <pre>At 2015-03-10 05:24:09,<a class="moz-txt-link-abbreviated" href="mailto:dtyx265@gmail.com" moz-do-not-send="true">dtyx265@gmail.com</a> wrote:
># HG changeset patch
># User David T Yuen <a class="moz-txt-link-rfc2396E" href="mailto:dtyx265@gmail.com" moz-do-not-send="true"><dtyx265@gmail.com></a>
># Date 1425935926 25200
># Node ID 4d1d54d28cb1635448ceec764c793d1b37cef7a4
># Parent  ef383507c21ca704b58e759e440f4ae2e177c499
>asm:intra pred planar32 sse2 high bit
>
>This replaces c code for systems using ssse3 to sse2 processors
>The code is backported from intrapred planar32 sse4
>
>Unlike the sse4 high bit version which operates with 32-bit values
>this version uses 16-bits to calculate the prediction.  16-bits
>is just enough for 10 bit luma depth, or at least it's enough for
>the testbench.  Anything more and this primitive could overflow.
>
>./test/TestBench --testbench intrapred | grep intra_planar_32x32
>intra_planar_32x32  13.80x   8289.69         114379.01
>
>diff -r ef383507c21c -r 4d1d54d28cb1 source/common/x86/asm-primitives.cpp
>--- a/source/common/x86/asm-primitives.cpp  Mon Mar 09 13:50:57 2015 -0700
>+++ b/source/common/x86/asm-primitives.cpp  Mon Mar 09 14:18:46 2015 -0700
>@@ -876,6 +876,7 @@
>         p.cu[BLOCK_4x4].intra_pred[PLANAR_IDX] = x265_intra_pred_planar4_sse2;
>         p.cu[BLOCK_8x8].intra_pred[PLANAR_IDX] = x265_intra_pred_planar8_sse2;
>         p.cu[BLOCK_16x16].intra_pred[PLANAR_IDX] = x265_intra_pred_planar16_sse2;
>+        p.cu[BLOCK_32x32].intra_pred[PLANAR_IDX] = x265_intra_pred_planar32_sse2;

>         p.cu[BLOCK_4x4].sse_ss = x265_pixel_ssd_ss_4x4_mmx2;
>         ALL_LUMA_CU(sse_ss, pixel_ssd_ss, sse2);
>diff -r ef383507c21c -r 4d1d54d28cb1 source/common/x86/intrapred16.asm
>--- a/source/common/x86/intrapred16.asm     Mon Mar 09 13:50:57 2015 -0700
>+++ b/source/common/x86/intrapred16.asm     Mon Mar 09 14:18:46 2015 -0700
>@@ -65,6 +65,10 @@
> pw_planar16_1:        dw 15, 15, 15, 15, 15, 15, 15, 15
> pd_planar32_1:        dd 31, 31, 31, 31

>+pw_planar32_1:        dw 31, 31, 31, 31, 31, 31, 31, 31
>+pw_planar32_L:        dw 31, 30, 29, 28, 27, 26, 25, 24
>+pw_planar32_H:        dw 23, 22, 21, 20, 19, 18, 17, 16
>+
> const planar32_table
> %assign x 31
> %rep 8
>@@ -86,12 +90,15 @@
> cextern pw_4
> cextern pw_8
> cextern pw_16
>+cextern pw_32
> cextern pw_1023
> cextern pd_16
> cextern pd_32
> cextern pw_4096
> cextern multiL
> cextern multiH
>+cextern multiH2
>+cextern multiH3
> cextern multi_2Row
> cextern pw_swap
> cextern pb_unpackwq1
>@@ -575,6 +582,133 @@
>     INTRA_PRED_PLANAR_16 15
>     RET

>+;---------------------------------------------------------------------------------------
>+; void intra_pred_planar(pixel* dst, intptr_t dstStride, pixel*srcPix, int, int filter)
>+;---------------------------------------------------------------------------------------
>+INIT_XMM sse2
>+cglobal intra_pred_planar32, 3,3,13
>+    movd            m3, [r2 + 66]               ; topRight   = above[32]
>+
>+    pshuflw         m3, m3, 0x00
>+    pshufd          m3, m3, 0x44
>+
>+    pmullw          m0, m3, [multiL]            ; (x + 1) * topRight
>+    pmullw          m1, m3, [multiH]            ; (x + 1) * topRight
>+    pmullw          m2, m3, [multiH2]           ; (x + 1) * topRight
>+    pmullw          m3, [multiH3]               ; (x + 1) * topRight
>+
>+    movd            m6, [r2 + 194]               ; bottomLeft = left[32]
>+    pshuflw         m6, m6, 0x00
>+    pshufd          m6, m6, 0x44
>+    mova            m5, m6
>+    paddw           m5, [pw_32]
>+
>+    paddw           m0, m5
>+    paddw           m1, m5
>+    paddw           m2, m5
>+    paddw           m3, m5
>+    mova            m8, m6
>+    mova            m9, m6
>+    mova            m10, m6
>+
>+    movu            m4, [r2 + 2]
>+    psubw           m8, m4
>+    pmullw          m4, [pw_planar32_1]
>+    paddw           m0, m4
>+
>+    movu            m5, [r2 + 18]
>+    psubw           m9, m5
>+    pmullw          m5, [pw_planar32_1]
>+    paddw           m1, m5
>+
>+    movu            m4, [r2 + 34]
>+    psubw           m10, m4
>+    pmullw          m4, [pw_planar32_1]
>+    paddw           m2, m4
>+
>+    movu            m5, [r2 + 50]
>+    psubw           m6, m5
>+    pmullw          m5, [pw_planar32_1]
the constant use more than 2 times, need buffer into register</pre>
                    <pre>>+    paddw           m3, m5
>+
>+%macro PROCESS 1
>+    pmullw          m5, %1, [pw_planar32_L]
>+    pmullw          m11, %1, [pw_planar32_H]
</pre>
                    <pre>buffer into register</pre>
                  </div>
                </blockquote>
                <p> Buffering these constants in registers slows
                  performance on my machine.<br>
                  <br>
                  ./test/TestBench --testbench intrapred | grep
                  intra_planar_32x32<br>
                  intra_planar_32x32    12.01x      9521.82     
                   114371.95<br>
                  <br>
                  And no, I didn't set the registers inside the PROCESS
                  macro...</p>
                <p>[MC] your code work on x64 platform, you have at
                  least two free register to buffer these constant, it
                  reduce code size and improve performance</p>
              </blockquote>
            </div>
          </blockquote>
          <p>     mova            m13, [pw_planar32_L]<br>
                mova            m14, [pw_planar32_H]<br>
                mova            m15, [pw_planar16_0]<br>
            %macro PROCESS 1<br>
                pmullw          m5, %1, m13 ;[pw_planar32_L]<br>
                pmullw          m11, %1, m14 ;[pw_planar32_H]<br>
                paddw           m5, m0<br>
                paddw           m11, m1<br>
                psrlw           m5, 6<br>
                psrlw           m11, 6<br>
                movu            [r0], m5<br>
                movu            [r0 + 16], m11<br>
            <br>
                pmullw          m5, %1, m15 ;[pw_planar16_0]<br>
            <br>
            This runs slower<br>
          </p>
          <p>[MC] just 4 cycles? could you test the really encoder
            speed? the really have lower cache hit rate.</p>
        </blockquote>
      </div>
    </blockquote>
    Interesting.  Performance is almost identical<br>
    <br>
    original code<br>
    <br>
    /x265 -I 1 --input ~/Videos/bridge-close-cif/bridge-close.y4m -o
    bridge-close.y4m<br>
    y4m  [info]: 352x288 fps 30/1 i420p8 frames 0 - 1999 of 2000<br>
    x265 [info]: HEVC encoder version 1.5+162-4d1d54d28cb1<br>
    x265 [info]: build info [Linux][GCC 4.7.2][64 bit] 16bpp<br>
    x265 [info]: using cpu capabilities: MMX2 SSE2Slow SlowCTZ<br>
    x265 [info]: Main 10 profile, Level-2 (Main tier)<br>
    x265 [info]: Thread pool created using 2 threads<br>
    x265 [info]: frame threads / pool features       : 1 / wpp(5 rows)<br>
    x265 [info]: Internal bit depth                  : 10<br>
    x265 [info]: Coding QT: max CU size, min CU size : 64 / 8<br>
    x265 [info]: Residual QT: max TU size, max depth : 32 / 1 inter / 1
    intra<br>
    x265 [info]: ME / range / subpel / merge         : hex / 57 / 2 / 2<br>
    x265 [info]: Keyframe min / max / scenecut       : 1 / 1 / 40<br>
    x265 [info]: Lookahead / bframes / badapt        : 20 / 0 / 0<br>
    x265 [info]: b-pyramid / weightp / weightb / refs: 0 / 1 / 0 / 3<br>
    x265 [info]: Rate Control / AQ-Strength / CUTree : CRF-28.0 / 1.0 /
    1<br>
    x265 [info]: tools: rd=3 psy-rd=0.30 deblock sao signhide tmvp <br>
    x265 [info]: frame I:   2000, Avg QP:33.08  kb/s:
    973.45                        <br>
    x265 [info]: global :   2000, Avg QP:33.08  kb/s: 973.45  <br>
    x265 [info]: consecutive B-frames: 100.0% <br>
    <br>
    encoded 2000 frames in 414.26s (4.83 fps), 973.45 kb/s<br>
    <br>
    and using registers to hold constants<br>
    <br>
    ./x265 -I 1 --input ~/Videos/bridge-close-cif/bridge-close.y4m -o
    bridge-close.y4m<br>
    y4m  [info]: 352x288 fps 30/1 i420p8 frames 0 - 1999 of 2000<br>
    x265 [info]: HEVC encoder version 1.5+162-4d1d54d28cb1<br>
    x265 [info]: build info [Linux][GCC 4.7.2][64 bit] 16bpp<br>
    x265 [info]: using cpu capabilities: MMX2 SSE2Slow SlowCTZ<br>
    x265 [info]: Main 10 profile, Level-2 (Main tier)<br>
    x265 [info]: Thread pool created using 2 threads<br>
    x265 [info]: frame threads / pool features       : 1 / wpp(5 rows)<br>
    x265 [info]: Internal bit depth                  : 10<br>
    x265 [info]: Coding QT: max CU size, min CU size : 64 / 8<br>
    x265 [info]: Residual QT: max TU size, max depth : 32 / 1 inter / 1
    intra<br>
    x265 [info]: ME / range / subpel / merge         : hex / 57 / 2 / 2<br>
    x265 [info]: Keyframe min / max / scenecut       : 1 / 1 / 40<br>
    x265 [info]: Lookahead / bframes / badapt        : 20 / 0 / 0<br>
    x265 [info]: b-pyramid / weightp / weightb / refs: 0 / 1 / 0 / 3<br>
    x265 [info]: Rate Control / AQ-Strength / CUTree : CRF-28.0 / 1.0 /
    1<br>
    x265 [info]: tools: rd=3 psy-rd=0.30 deblock sao signhide tmvp <br>
    x265 [info]: frame I:   2000, Avg QP:33.08  kb/s:
    973.45                        <br>
    x265 [info]: global :   2000, Avg QP:33.08  kb/s: 973.45  <br>
    x265 [info]: consecutive B-frames: 100.0% <br>
    <br>
    encoded 2000 frames in 414.28s (4.83 fps), 973.45 kb/s<br>
    <br>
    The closest I could find to forcing intra planar32 to be used is -I
    1<br>
    <blockquote
      cite="mid:3be8505c.a69.14c00e37ccb.Coremail.chenm003@163.com"
      type="cite">
      <div
        style="line-height:1.7;color:#000000;font-size:14px;font-family:arial">
        <blockquote id="isReplyContent" style="margin: 0px 0px 0px
          0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204,
          204); border-left-width: 1px; border-left-style: solid;">
          <p><br>
            ./test/TestBench --testbench intrapred | grep
            intra_planar_32x32<br>
            intra_planar_32x32    12.04x      9517.47       114580.26<br>
            <br>
            over 1200 worse<br>
          </p>
          <blockquote
            cite="mid:376e7908.a61.14c00cd34b9.Coremail.chenm003@163.com"
            type="cite">
            <div style="color: rgb(0, 0, 0); line-height: 1.7;
              font-family: arial; font-size: 14px;">
              <blockquote id="isReplyContent" style="margin: 0px 0px 0px
                0.8ex; padding-left: 1ex; border-left-color: rgb(204,
                204, 204); border-left-width: 1px; border-left-style:
                solid;">
                <p><br>
                </p>
                <blockquote
                  cite="mid:4ad6a641.a50.14c00a2f84b.Coremail.chenm003@163.com"
                  type="cite">
                  <div style="color: rgb(0, 0, 0); line-height: 1.7;
                    font-family: arial; font-size: 14px;">
                    <pre>>+    paddw           m5, m0
>+    paddw           m11, m1
>+    psrlw           m5, 6
>+    psrlw           m11, 6
>+    movu            [r0], m5
>+    movu            [r0 + 16], m11
>+
>+    pmullw          m5, %1, [pw_planar16_0]
>+    pmullw          %1, [pw_planar8_0]
>+    paddw           m5, m2
>+    paddw           %1, m3
>+    psrlw           m5, 6
>+    psrlw           %1, 6
>+    movu            [r0 + 32], m5
>+    movu            [r0 + 48], %1
>+%endmacro
>+
>+%macro  INCREMENT 0
>+    paddw           m2, m10
>+    paddw           m3, m6
>+    paddw           m0, m8
>+    paddw           m1, m9
>+    lea             r0, [r0 + r1 * 2]
</pre>
                    <pre>we have free register, so may buffer r1*2 and replace by ADD to avoid execute port bottleneck</pre>
                  </div>
                </blockquote>
                This provides a small performance improvement, I will
                submit a new patch.<br>
                <blockquote
                  cite="mid:4ad6a641.a50.14c00a2f84b.Coremail.chenm003@163.com"
                  type="cite">
                  <div style="color: rgb(0, 0, 0); line-height: 1.7;
                    font-family: arial; font-size: 14px;">
                    <pre>>+%endmacro
>+
>+%assign x 0
>+%rep 4
>+    movu            m4, [r2 + 130 + x * 16]     ;130 = 32*sizeof(pixel)*2 + 1*sizeof(pixel)
>+
>+    pshuflw         m7, m4, 0x00
>+    pshufd          m7, m7, 0x44
>+    pshuflw         m12, m4, 0x55
>+    pshufd          m12, m12, 0x44
>+
>+    PROCESS m7
>+    INCREMENT
>+    PROCESS m12
>+    INCREMENT
>+
>+    pshuflw         m7, m4, 0xAA
>+    pshufd          m7, m7, 0x44
>+    pshuflw         m12, m4, 0xFF
>+    pshufd          m12, m12, 0x44
>+
>+    PROCESS m7
>+    INCREMENT
>+    PROCESS m12
>+    INCREMENT
>+
>+    pshufhw         m7, m4, 0x00
>+    pshufd          m7, m7, 0xEE
>+    pshufhw         m12, m4, 0x55
>+    pshufd          m12, m12, 0xEE
>+
>+    PROCESS m7
>+    INCREMENT
>+    PROCESS m12
>+    INCREMENT
>+
>+    pshufhw         m7, m4, 0xAA
>+    pshufd          m7, m7, 0xEE
>+    pshufhw         m12, m4, 0xFF
>+    pshufd          m12, m12, 0xEE
>+
>+    PROCESS m7
>+    INCREMENT
>+    PROCESS m12
>+
>+    %if x < 3
>+    INCREMENT
>+    %endif
>+%assign x x+1
>+%endrep
>+    RET
>+
> ;-----------------------------------------------------------------------------------
> ; void intra_pred_dc(pixel* dst, intptr_t dstStride, pixel* above, int, int filter)
> ;-----------------------------------------------------------------------------------
</pre>
                  </div>
                </blockquote>
              </blockquote>
            </div>
          </blockquote>
        </blockquote>
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
x265-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a>
<a class="moz-txt-link-freetext" href="https://mailman.videolan.org/listinfo/x265-devel">https://mailman.videolan.org/listinfo/x265-devel</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>