Part of the Khronos Group
OpenGL.org

The Industry's Foundation for High Performance Graphics

from games to virtual reality, mobile phones to supercomputers

Results 1 to 9 of 9

Thread: Drastic performance drop with random index

  1. #1
    Junior Member Regular Contributor Kopelrativ's Avatar
    Join Date
    Apr 2011
    Posts
    214

    Drastic performance drop with random index

    I am using a multi sample access to a bitmap that contains world position (in a deferred shader). The idea is to calculate the number of samples that are closer to the camera than a reference point (implementing Screen Space Ambient Occlusion). To get some better distribution on the samples, I use a Poisson disk distribution. However, instead of simple indexing the same subset of the Poisson disk list every time, I want to use a random subset. But when I do that, the shader suddenly falls in performance by a factor of 10. Why is that, and is there anything good I can do to it?

    The shader sub section looks as follows. There are two ways I compute the "ind" used to get a Poisson disk value from an array. One of them adds some noise (random) by using the x value of the world position. And it one will the break performance beyond repair. 'screen' is a vec2 addressing the current screen position being analyzed.
    Code :
    int num = 0;
    worldPos = texture(posTex, screen);
    float ref = distance(UBOCamera.xyz, worldPos);
    for (int i=0; i<SIZE; i++) {
        int ind = (i+int(worldPos.x*100)) % 64;
        // int ind = i;
        vec2 sampleInd = screen + (2.0*poissonDisk[ind]-1.0)*SCALE;
        vec3 sample = texture(posTex, sampleInd).xyz;
        float dist = distance(UBOCamera.xyz, sample);
        if (dist < ref) num++;
    }
    The only explanation I can come up with myself would be that the loop is different for every pixel, making it harder to optimize.

  2. #2
    Advanced Member Frequent Contributor
    Join Date
    Apr 2009
    Posts
    590
    What is the hardware? Also, out of my own curiosity can you post the sampler types? In addition what is the filtering on each of those samplers?

  3. #3
    Junior Member Regular Contributor Kopelrativ's Avatar
    Join Date
    Apr 2011
    Posts
    214
    It is:

    Vendor: ATI Technologies Inc.
    Renderer: AMD Radeon HD 6570
    Version: 4.2.11627 Compatibility Profile/Debug Context

    The samplers are sampler2D. The texture is configured as
    Code :
        glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA32F, width, height, 0, GL_RGBA, GL_FLOAT, NULL);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);

    I'll get back with timing data, but using any kind of random makes the shader time approximately go from 1 ms to 10 ms.

  4. #4
    Advanced Member Frequent Contributor
    Join Date
    Apr 2009
    Posts
    590
    There are 3 possible causes that I can speculate:
    1. One possibly glich bugger being that conditional increment of num dependent upon that varying index and triple texture lookup... I'd thing the GLSL compiler would make turn that if into a conditional expression, but see if changing "if(dist<ref) num++" to "num+=step(dist, ref)" helps. The GLSL compiler _should_ catch it.. so I don't think it would help.
    2. It might just comes down to that the AMD hardware does not like the random read access of a sampler in a fragment shader... there is a pretty long chain of texture look ups going on. I don't buy this explanation either.
    3. It might also come down to that you are accessing that array, poissonDisc[] with an index that cannot be determined at compile time in the much slower shader. Is that array a uniform array, const array or regular variable computed locally? If poissonDisc[] is a uniform array, I suggest that instead of placing those values into a uniform to place them into a texture and make the array access into a texture lookup. Possibly an unfiltered 1D texture or a texture buffer object. This might make the two times much closer.

  5. #5
    Junior Member Regular Contributor Kopelrativ's Avatar
    Join Date
    Apr 2011
    Posts
    214
    Thanks for your comments!

    See below the complete source code of the vertex shader and the fragment shader. It is by far not yet a working algorithm for SSAO, but I have to understand the performance issue before I can go on. I now tested it on another PC, getting similar problems:

    Vendor: NVIDIA Corporation
    Renderer: NVS 4200M/PCI/SSE2
    Version: 4.2.0
    GLSL: 4.20 NVIDIA via Cg compiler
    OpenGL context version parsed by GLFW: 4.2.0

    Notice the line with "ind = i % 64". If it is commented, the shader takes 7.6ms to execute. With the line included, it takes 881ms! I measure the execution time with a query of GL_TIME_ELAPSED. So there is the same problem with Nvidia, and even worse (though both graphic cards are in the lower performance segment). Funny thing is, it looks like the GLSL compiler decides it can't predict what happens at the presence of the %-operator. If I instead do "ind = i+1", there is no performance loss. But that isn't much of randomness :-)

    Using "num += int(step(dist, ref));" didn't produce any difference, as you suspected. The measured times are very stable.

    I have another shader for using shadowmaps with oversampling. It had the same problem when I used random samples, so I went for using a very large set of samples instead. Obviously not satisfactorily, as it takes a lot of time.


    Code :
    // This vertex shader will only draw two triangles, giving a full screen.
    // The vertex input is 0,0 in one corner and 1,1 in the other.
    static const GLchar *vertexShaderSource[] = {
        "#version 330\n", // This corresponds to OpenGL 3.3
        "layout(location = 0) in vec2 vertex;\n",
        "out vec2 screen;\n",             // Screen coordinate
        "void main(void)\n",
        "{\n",
        "    gl_Position = vec4(vertex*2-1, 0, 1);\n", // Transform from interval 0 to 1, to interval -1 to 1.
        "   screen = vertex;\n",  // Copy position to the fragment shader. Only x and y is needed.
        "}\n",
    };
     
     
    static const GLchar *fragmentShaderSource[] = {
        "#version 330\n", // This corresponds to OpenGL 3.3
        "layout(std140) uniform GlobalData {"
        "    mat4 UBOProjectionMatrix;"
        "    mat4 UBOProjectionviewMatrix;"
        "    mat4 UBOViewMatrix;"
        "    vec4 UBOCamera;"
        "    float UBOViewingDistance;"
        "    int UBOPerformance;"
        "    int UBODynamicshadows;"
        "};\n"
        "uniform sampler2D posTex;\n",     // World position
        "uniform sampler2D normalTex;\n",  // Normals
        "in vec2 screen;\n",               // The screen position
        "layout(location = 0) out float light;\n",
     
     
        "vec4 worldPos;\n",
        "vec4 normal;\n",
     
     
        "const vec2 poissonDisk[64] = vec2[]("
        "  vec2( 0.282571, 0.023957 ),"
        "  vec2( 0.792657, 0.945738 ),"
        "  vec2( 0.922361, 0.411756 ),"
        "  vec2( 0.165838, 0.552995 ),"
        "  vec2( 0.566027, 0.216651 ),"
        "  vec2( 0.335398, 0.783654 ),"
        "  vec2( 0.0190741, 0.318522 ),"
        "  vec2( 0.647572, 0.581896 ),"
        "  vec2( 0.916288, 0.0120243 ),"
        "  vec2( 0.0278329, 0.866634 ),"
        "  vec2( 0.398053, 0.4214 ),"
        "  vec2( 0.00289926, 0.051149 ),"
        "  vec2( 0.517624, 0.989044 ),"
        "  vec2( 0.963744, 0.719901 ),"
        "  vec2( 0.76867, 0.018128 ),"
        "  vec2( 0.684194, 0.167302 ),"
        "  vec2( 0.727103, 0.410871 ),"
        "  vec2( 0.557482, 0.724143 ),"
        "  vec2( 0.483352, 0.0527055 ),"
        "  vec2( 0.162877, 0.351482 ),"
        "  vec2( 0.959716, 0.180578 ),"
        "  vec2( 0.140355, 0.112003 ),"
        "  vec2( 0.796228, 0.223365 ),"
        "  vec2( 0.187048, 0.787225 ),"
        "  vec2( 0.55446, 0.35612 ),"
        "  vec2( 0.449965, 0.640522 ),"
        "  vec2( 0.438917, 0.194769 ),"
        "  vec2( 0.791253, 0.565325 ),"
        "  vec2( 0.719718, 0.794794 ),"
        "  vec2( 0.0651875, 0.708609 ),"
        "  vec2( 0.641987, 0.0233772 ),"
        "  vec2( 0.376415, 0.944243 ),"
        "  vec2( 0.827723, 0.723258 ),"
        "  vec2( 0.968627, 0.884518 ),"
        "  vec2( 0.263405, 0.458968 ),"
        "  vec2( 0.985717, 0.559587 ),"
        "  vec2( 0.0616169, 0.468612 ),"
        "  vec2( 0.159154, 0.934782 ),"
        "  vec2( 0.287301, 0.284768 ),"
        "  vec2( 0.550066, 0.849391 ),"
        "  vec2( 0.353587, 0.003296 ),"
        "  vec2( 0.000671407, 0.582507 ),"
        "  vec2( 0.850459, 0.461989 ),"
        "  vec2( 0.526139, 0.640126 ),"
        "  vec2( 0.786889, 0.487686 ),"
        "  vec2( 0.164129, 0.02472 ),"
        "  vec2( 0.517075, 0.90933 ),"
        "  vec2( 0.316111, 0.663564 ),"
        "  vec2( 0.09476, 0.895749 ),"
        "  vec2( 0.298288, 0.195318 ),"
        "  vec2( 0.427229, 0.7828 ),"
        "  vec2( 0.734764, 0.266152 ),"
        "  vec2( 0.0816065, 0.965972 ),"
        "  vec2( 0.698935, 0.646352 ),"
        "  vec2( 0.281899, 0.355144 ),"
        "  vec2( 0.871334, 0.303171 ),"
        "  vec2( 0.138249, 0.661214 ),"
        "  vec2( 0.202399, 0.252449 ),"
        "  vec2( 0.0734275, 0.399853 ),"
        "  vec2( 0.786767, 0.660268 ),"
        "  vec2( 0.933744, 0.508621 ),"
        "  vec2( 0.398236, 0.0509049 ),"
        "  vec2( 0.500473, 0.130253 ),"
        "  vec2( 0.0332957, 0.526292 )"
        ");"
     
     
     
        "void main(void)\n",
        "{\n",
        "    normal = texture(normalTex, screen);\n", // Not used
        "    worldPos = texture(posTex, screen);\n",
        "    float ref = distance(UBOCamera.xyz, worldPos.xyz);"
        "    int num = 0;"
        "    const int SIZE=20;"
        "    const float p = 1.0/1900;\n" // Size of one pixel
        "    for (int i=0; i<SIZE; i++) {"
        "        int ind = i;"
        //"        ind = i % 64;"
        "        vec2 sampleInd = screen + (2.0*poissonDisk[ind]-1.0)*p*10;"
        "        vec3 sample = texture(posTex, sampleInd).xyz;\n"
        "        float dist = distance(UBOCamera.xyz, sample);"
        "        if (dist < ref) num++;\n"
        "    }"
        "    if (num > SIZE*0.7)"
        "        light = 0.5;\n"
        "    else {"
        "        discard; return;"
        "    }"
        // "   light = worldPos.a;\n",
        "}\n",
    };

  6. #6
    Advanced Member Frequent Contributor
    Join Date
    Apr 2009
    Posts
    590
    I strongly advise to change that array poissonDisc into a 1d texture (make the repeat mode repeat and likely since is is supposed to represent a function, mske its filter linear) also doing it this way will let you sample much more finely than 60.

  7. #7
    Junior Member Regular Contributor Kopelrativ's Avatar
    Join Date
    Apr 2011
    Posts
    214
    Quote Originally Posted by kRogue View Post
    I strongly advise to change that array poissonDisc into a 1d texture (make the repeat mode repeat and likely since is is supposed to represent a function, mske its filter linear) also doing it this way will let you sample much more finely than 60.
    Thanks for the suggestion, I did some further testing on the ATI system. Using the following:
    Code :
        "        int ind = i;"
        "        vec2 sampleInd = screen + (2.0*poissonDisk[ind]-1.0)*p*10;"
        "        vec3 sample = texture(posTex, sampleInd).xyz;\n"
    I got 5.45ms.
    Using:
    Code :
        "        int ind = i + int(worldPos.x*12.32);"
        "        vec2 sampleInd = screen + (2.0*poissonDisk[ind%64]-1.0)*p*10;"
        "        vec3 sample = texture(posTex, sampleInd).xyz;\n"
    Makes it 23ms.
    Using:
    Code :
        "vec2 rand(vec2 a, vec2 b) {"
        "    return fract(a*10.23 + b*123.1232);"
        "}"
     
     
    ...
        "        vec2 sampleInd = screen + rand(worldPos.xy, normal.xy)*p*10;"

    makes it 2.9ms! Which would confirm your suspicion that it is the access of the possionDisk[] that is the problem. A little counter intuitive, I would expect a table lookup to be quicker than a function call. Though the function call is probably inline expanded. I now consider skipping the use of poissonDisk completely.

  8. #8
    Junior Member Regular Contributor Kopelrativ's Avatar
    Join Date
    Apr 2011
    Posts
    214
    Quote Originally Posted by Kopelrativ View Post
    Code :
        "vec2 rand(vec2 a, vec2 b) {"
        "    return fract(a*10.23 + b*123.1232);"
        "}"
     
     
    ...
        "        vec2 sampleInd = screen + rand(worldPos.xy, normal.xy)*p*10;"
    Just to preempt a comment, I realised this random function is invariant for a given pixel, and isn't much of a help for multi sampling as currently used.

  9. #9
    Junior Member Regular Contributor
    Join Date
    Dec 2009
    Posts
    206
    How is the performance of your shader if you bind a 1x1 texture to posTex ? That should eliminate the overhead of the non-uniform texture accesses and you can estimate how costly the poissonDisk access is.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •