Mesa (master): 44 new commits

Brian Paul brianp at kemper.freedesktop.org
Wed Jan 17 18:34:14 UTC 2018


URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f631a6ae8c951fc448618c1770f95ad93cac812b
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:58:58 2018 -0700

    glsl: remove unneeded extern "C" {} bracketing around Mesa includes
    
    The two headers already have the right extern "C" annotations.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7421e34dd69dfb8911e99b6004f8806707c323d1
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:53:29 2018 -0700

    mesa: move gl_external_samplers() to program.[ch]
    
    The function is only called from a couple places.  It doesn't make
    sense to have it in mtypes.h
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=6dc8896726b8fdfb69705dbe298e074a21cd05cf
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:57:57 2018 -0700

    st/mesa: include util/bitscan.h in st_glsl_to_tgsi_temprename.cpp
    
    And use "" instead of <> for including Mesa headers, as we do elsewhere.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=741d42347880835e34ee6214b637f0c00731b2ae
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:52:22 2018 -0700

    glsl: include util/bitscan.h in serialize.cpp
    
    Instead of relying on indirect inclusion of the header.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2f141462002290eb770b9d75995efd799aac48a0
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:49:09 2018 -0700

    util: include string.h in u_dynarray.h
    
    To get memset() prototype.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=02c0734adcb76fd7c0541d961b2c83196150ba2a
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:44:43 2018 -0700

    mesa: remove unneeded #includes of main/compiler.h
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7845397183aeb60c67b81facacc0b3f8d3cbe2d8
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 21:44:17 2018 -0700

    st/mesa: remove unneeded #includes of main/compiler.h
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=bafa33befd86c911fa2b5321a50da08a1963d28a
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 22:14:20 2018 -0700

    st/mesa: include main/compiler.h in st_cb_queryobj.c
    
    To get CPU_TO_LE32() macro.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7de0262f8fa24adad8ad88f1deca8afa1373c5c3
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 22:13:41 2018 -0700

    mesa: include util/macros.h in format_fallback.c
    
    To get definition of unreachable() macro.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=484ac243f6ffd488bc359e87927f408682bd23ab
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 22:12:49 2018 -0700

    mesa: include compiler.h in disk_cache.c
    
    Instead of indirect inclusion to get CPU_TO_LE32() macro.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ad00a7899396dc56abb4f813487bb19100551348
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 08:40:36 2018 -0700

    mesa/program: change validate_inputs() local var 'inputs' to GLbitfield64
    
    Both state->prog->info.inputs_read and state->InputsBound are GLbitfield64
    so it seems that the OR of those values should be of the same type.
    I'm not sure this fixes any actual issues though.
    
    Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d8306de4ac82685ff83626b56089e3c6c2d04d70
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 20:22:56 2018 -0700

    vbo: reindent vbo_attrib.h to use 3 spaces
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ef01d911ee2482d1dffa757db71512b8d2e17a4c
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 14:46:54 2018 -0700

    vbo: whitespace, formatting fixes in vbo_exec_api.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=18f4241b89b9f9869c89a291e0ab1d54e6e74fbf
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 14:28:01 2018 -0700

    vbo: add assertions, comments in vbo_exec_api.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5e8962b58f0d1791f45c0bd16fe87b457399780d
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 09:34:30 2018 -0700

    vbo: whitespace, formatting fixes in vbo_exec_draw.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f5140c1a6bf63f839070502dc4c9cca34c6955cc
Author: Brian Paul <brianp at vmware.com>
Date:   Tue Jan 16 09:31:01 2018 -0700

    vbo: use inputs_read var to simplify code
    
    v2: add some const qualifiers, per Ian.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e6e78d98a27f15892bda046b978b9db2deb9abd4
Author: Brian Paul <brianp at vmware.com>
Date:   Mon Jan 15 14:53:47 2018 -0700

    vbo: whitespace, formatting fixes in vbo_split_copy.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=380165d3994d53d245343dec2ae6b4ebd8fde6d2
Author: Brian Paul <brianp at vmware.com>
Date:   Mon Jan 15 14:48:07 2018 -0700

    vbo: use a new local 'array' variable in bind_vertex_list() loop
    
    Make the code a bit more concise.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=37863c38f3292e5a8c3ff0396fd92113a2a5b4e8
Author: Brian Paul <brianp at vmware.com>
Date:   Mon Jan 15 14:44:39 2018 -0700

    vbo: remove unneeded #includes in vbo_context.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=76a08eeeec08d5e52d88ccc4bfd594a311643163
Author: Brian Paul <brianp at vmware.com>
Date:   Mon Jan 15 14:41:18 2018 -0700

    vbo: whitespace, formatting fixes in vbo_context.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=95dec9097fb0a5cf5438f04511a52724ea545302
Author: Brian Paul <brianp at vmware.com>
Date:   Mon Jan 15 14:37:04 2018 -0700

    vbo: change vbo_context attribute map arrays to GLubyte
    
    The values will never be larger than VBO_ATTRIB_MAX (currently 44).
    
    v2: add STATIC_ASSERT to be sure VBO_ATTRIB_MAX can fit in ubyte,
    per Emil.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5d78440d5830e5f38ac4b3094aadcd436db18365
Author: Brian Paul <brianp at vmware.com>
Date:   Mon Jan 15 14:34:53 2018 -0700

    vbo: lift common code out of switch cases
    
    Both switch cases began with the same code.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
    Reviewed-by: Roland Scheidegger <sroland at vmware.com>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8e4efdc895eacca931b94a0c1fa01aae5c34463b
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 13:18:25 2018 -0700

    vbo: optimize some display list drawing (v2)
    
    The vbo_save_vertex_list structure records one or more glBegin/End
    primitives which all have the same vertex format.
    
    To draw these primitives, we setup the vertex array state, then
    issue the drawing command.  Before, the 'start' vertex was typically
    zero and we used the vertex array pointer to indicate where the
    vertex data starts.
    
    This patch checks if the vertex buffer offset is an exact multiple of
    the vertex size.  If so, that means we can use zero-based vertex array
    pointers and use the draw's start value to indicate where the vertex
    data starts.
    
    This means a series of display list drawing commands may have
    identical vertex array state.  This will get filtered out by the
    Gallium CSO module so we can issue a tight series of drawing commands
    without state changes to the device.
    
    Note that this also works for a series of glCallList commands (not
    just one list that contains multiple glBegin/End pairs).
    
    No Piglit or conform changes.
    
    v2: minor fixes suggested by Ian.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=4edc8fdcdc8df2a37930241b803383d46e6d7f5f
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 13:17:10 2018 -0700

    vbo: rewrite some code in playback_copy_to_current()
    
    I think this is a little easier to understand.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ad3b162b8165d9f72e7014671cf4ad64541474d6
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 10:28:14 2018 -0700

    vbo: add some comments in vbo_save_api.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ee86c346640367ddcb7ecea302c8e919e8207e27
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 10:21:48 2018 -0700

    vbo: rename some functions in vbo_save_api.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=efb5ce1c5ef5f150ffe257810020965a206a4d90
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 10:16:40 2018 -0700

    vbo: rename some functions in vbo_save_draw.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=2064c4e8144b401420fa9e0d603560baf9ef104c
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:57:29 2018 -0700

    vbo: add comment that vbo_save_vertex_list::buffer_offset is in bytes
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8b06acf642b183f8203876925639c6d6e2d5fe01
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:56:42 2018 -0700

    vbo: minor code simplification in _save_compile_vertex_list()
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=4acc8a0de3b136882483b25add0b8b21ceff9bb5
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:52:05 2018 -0700

    vbo: rename prim to prims
    
    Using a plural name makes it easier to see that this is an array and
    not a pointer to a single object.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=841d1839e2d4a0d2b0cf4ffcba26280570922356
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:48:17 2018 -0700

    vbo: removed unused ctx parameter for alloc_prim_store()
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=3442672df4d5c4b341bcc354affbd5c1d54e1589
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:30:42 2018 -0700

    vbo: rename vbo_save_context::buffer to buffer_map
    
    And move the field and improve comments.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=62fd12e5bc3fd8eaa4f6af115a021f9cddca2de9
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:28:37 2018 -0700

    vbo: remove unused vbo_save_context::count field
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1863b18ce64b5c8011bbbcffbac85cb1928a801c
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:27:16 2018 -0700

    vbo: s/GLuint/GLbitfield/ for vbo_save_context::replay_flags
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=65df0d41cf8ab7ef49830e0bc265e4e46cc7070c
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:23:37 2018 -0700

    vbo: rename vbo_save_vertex_list::count to vertex_count
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=b04195b9591604cbdf148781399c2253ccb73f14
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:18:32 2018 -0700

    vbo: rename vbo_save_vertex_store::buffer to buffer_map
    
    To match other parts of the VBO code and make things easier to understand.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=213bef62c31ed365253a1f3f6714290b8013051a
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:12:35 2018 -0700

    vbo: rename vbo_save_primitive_store::buffer to prims
    
    A little easier to understand.
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1e9ca36058693c304dc96a42c72451d4c6f38943
Author: Brian Paul <brianp at vmware.com>
Date:   Fri Jan 12 09:07:05 2018 -0700

    vbo: whitespace fixes in vbo_save.h
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c40e9034be31c085d96ea45381350381e2760c56
Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan 11 16:18:38 2018 -0700

    vbo: whitespace fixes in vbo_save_draw.c
    
    Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7027d9c1fd2a1bc0ef3cd523d407367f81a6eed5
Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan 11 11:17:45 2018 -0700

    svga: add num-commands-per-draw HUD query
    
    This query shows the ratio of total commands vs. drawing commands sent
    to the vgpu device.  This gives some idea of how many state changes
    are sent per draw call.  The closer the ratio is to 1.0, the better.
    
    Reviewed-by: Charmaine Lee <charmainel at vmware.com>
    Reviewed-by: Neha Bhende <bhenden at vmware.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=92840bd27683996d1de86fbc5b95be798f99796b
Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan 11 11:11:04 2018 -0700

    gallium/hud: Fix support for PIPE_DRIVER_QUERY_TYPE_FLOAT
    
    Evidently, nobody has used PIPE_DRIVER_QUERY_TYPE_FLOAT up to this
    point.  Adding a driver query of this type which returns the query
    value in pipe_query_result::f resulted in garbage output in the HUD.
    
    The problem is the pipe_query_result::f field was being accessed as
    through the u64 field and being added to the query_info::results_cumulative
    field.  This patch checks for PIPE_DRIVER_QUERY_TYPE_FLOAT in a few
    places and scales the float by 1000 before converting to uint64_t.
    
    Also, add some comments to explain the query_info::result_index field.
    
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=23a5fae317c40285956cee383b2ab774f84058d6
Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan 11 09:57:50 2018 -0700

    gallium/hud: remove uint64_t casts in sensor query_sti_load() function
    
    The hud_graph_add_value() function takes a double value, so just pass
    the current/critical values as-is since they're doubles.
    
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f774f2b52f56cb10fd7e4354e7c3861d370a58a8
Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan 11 09:56:12 2018 -0700

    gallium/hud: compute cpu load, percent with doubles
    
    The hud_graph_add_value() function takes a double precision value,
    so compute it that way.
    
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>

URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=541f569a193e6a8b426c7d0f4f6b7533db571305
Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan 11 09:50:42 2018 -0700

    gallium/hud: s/unsigned/enum pipe_query_type/
    
    Reviewed-by: Marek Olšák <marek.olsak at amd.com>




More information about the mesa-commit mailing list