{"push_id":1381426383,"size":206,"distinct_size":0,"ref":"refs/heads/i965-fp64-gen7-ivb-scalar-vec4","head":"7a7697d0e3a725576c76f9b242063ddae366d537","before":"81f4c51c295bc5743c4532f33f8ac48a610e24bb","commits":[{"sha":"ab92480272bc34d943d28506cc36759e141b4371","author":{"name":"Jason Ekstrand","email":"8d9da6ec6f8b6f23bae58d38147b2bea1269b4d6@intel.com"},"message":"intel/blorp: Rename compile_nir_shader to compile_fs\n\nSigned-off-by: Jason Ekstrand <jason@jlekstrand.net>\nReviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/ab92480272bc34d943d28506cc36759e141b4371"},{"sha":"43dadb6edd5e3e3e10b1198184a9f75556edad49","author":{"name":"Jason Ekstrand","email":"8d9da6ec6f8b6f23bae58d38147b2bea1269b4d6@intel.com"},"message":"intel/blorp: Rework our usage of ralloc when compiling shaders\n\nPreviously, we were creating the shader with a NULL ralloc context and then\ntrusting in blorp_compile_fs to clean it up.  The only problem was that\nblorp_compile_fs didn't clean up its context properly so we were leaking.\nWhen I went to fix that, I realized that it couldn't because it has to\nreturn the shader binary which is allocated off of that context and used by\nthe caller.  The solution is to make blorp_compile_fs take a ralloc\ncontext, allocate the nir_shaders directly off that context, and clean it\nall up in whatever function creates the shader and calls blorp_compile_fs.\n\nSigned-off-by: Jason Ekstrand <jason@jlekstrand.net>\nReviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>\nReviewed-by: Timothy Arceri <timothy.arceri@collabora.com>\nCc: \"12.0, 13.0\" <mesa-stable@lists.freedesktop.org>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/43dadb6edd5e3e3e10b1198184a9f75556edad49"},{"sha":"c06480390b68c7118b0628f0a2702e6a8c67e1e0","author":{"name":"Eduardo Lima Mitev","email":"c4ce3d882000cbb875558b52d58f0fea852b719a@igalia.com"},"message":"drivers/meta: Accept GL_TEXTURE_3D as target for tex image decompression\n\nAn assert is currently raised, preventing decompression of a texture image into\na GL_TEXTURE_3D target. I have not found any spec wording that would explain\nthis, or implementation detail that would prevent it. And in any case, the\ndriver should not cause a crash upon user input arguments.\n\nFixes most failing subcases in CTS tests:\n* GL44-CTS.gtf32.GL3Tests.packed_pixels.packed_pixels_pixelstore\n* GL45-CTS.gtf32.GL3Tests.packed_pixels.packed_pixels_pixelstore\n\nThese tests were crashing the driver before. Now they just fail, but due\nto an unrelated issue affecting 2 out of the 45 test subcases.\n\nNo regressions observed against piglit or CTS-GL.\n\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/c06480390b68c7118b0628f0a2702e6a8c67e1e0"},{"sha":"5857c3082e9f84db12cf9e503666503848242344","author":{"name":"Timothy Arceri","email":"2ddafe7dbd3e23500c275f83e0c6a411ad072887@collabora.com"},"message":"intel/blorp: remove stale comment\n\nReviewed-by: Jason Ekstrand <jason@jlekstrand.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/5857c3082e9f84db12cf9e503666503848242344"},{"sha":"7d059bdfb96a59043000e9d8c378ca077f473774","author":{"name":"Timothy Arceri","email":"2ddafe7dbd3e23500c275f83e0c6a411ad072887@collabora.com"},"message":"i965: use memory context when creating passthrough tcs\n\nReviewed-by: Jason Ekstrand <jason@jlekstrand.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/7d059bdfb96a59043000e9d8c378ca077f473774"},{"sha":"b677b99db5c48ffd1eeef538b962080ac5fd65d9","author":{"name":"Eduardo Lima Mitev","email":"c4ce3d882000cbb875558b52d58f0fea852b719a@igalia.com"},"message":"vulkan/wsi/x11: Fix behavior of vkGetPhysicalDeviceSurfacePresentModesKHR\n\nx11_surface_get_present_modes() is currently asserting that the number of\nelements in pPresentModeCount must be greater than or equal to the number\nof present modes available. This is buggy because pPresentModeCount\nelements are later copied from the internal modes' array, so if\npPresentModeCount is greater, it will overflow it.\n\nOn top of that, this assertion violates the spec. From the Vulkan 1.0\n(revision 32, with KHR extensions), page 581 of the PDF:\n\n    \"If the value of pPresentModeCount is less than the number of\n     presentation modes supported, at most pPresentModeCount values will be\n     written. If pPresentModeCount is smaller than the number of\n     presentation modes supported for the given surface, VK_INCOMPLETE\n     will be returned instead of VK_SUCCESS to indicate that not all the\n     available values were returned.\"\n\nSo, the correct behavior is: if pPresentModeCount is greater than the\ninternal number of formats, it is clamped to that many present modes. But\nif it is lesser than that, then pPresentModeCount elements are copied,\nand the call returns VK_INCOMPLETE.\n\nThis fix is similar (but simpler and more readable) than the one I provided\nin 750d8cad72a for vkGetPhysicalDeviceSurfaceFormatsKHR, which was suffering\nfrom the same problem.\n\nReviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/b677b99db5c48ffd1eeef538b962080ac5fd65d9"},{"sha":"129da274261b6e79f459e24428591f137bf92ed1","author":{"name":"Eduardo Lima Mitev","email":"c4ce3d882000cbb875558b52d58f0fea852b719a@igalia.com"},"message":"vulkan/wsi/x11: Smplify implementation of vkGetPhysicalDeviceSurfaceFormatsKHR\n\nThis patch simplifies x11_surface_get_formats(). It is actually just a\nreadability improvement over the patch I provided earlier this week\n(750d8cad72).\n\nReviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/129da274261b6e79f459e24428591f137bf92ed1"},{"sha":"910bc4d12cdc2b5743ad379fddee45c8b0112762","author":{"name":"Brian Paul","email":"496e153a9fb909fb693e8592e6df5f4bc45bdb28@vmware.com"},"message":"mesa: rename gl_vertex_array_object::VertexBinding to BufferBinding\n\nTo be a little more understandable.\n\nReviewed-by: Anuj Phogat <anuj.phogat@gmail.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/910bc4d12cdc2b5743ad379fddee45c8b0112762"},{"sha":"7a4ba9f16e5eb42c2df33376c606427bdfc5b0c4","author":{"name":"Brian Paul","email":"496e153a9fb909fb693e8592e6df5f4bc45bdb28@vmware.com"},"message":"mesa: update comment on vertex_attrib_binding()\n\nWas missed in an earlier renaming patch.\n\nReviewed-by: Anuj Phogat <anuj.phogat@gmail.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/7a4ba9f16e5eb42c2df33376c606427bdfc5b0c4"},{"sha":"161db1335b8dd1f3e2a9abbe82d3b8921d2da314","author":{"name":"Brian Paul","email":"496e153a9fb909fb693e8592e6df5f4bc45bdb28@vmware.com"},"message":"mesa: code clean-up in _mesa_update_vao_client_arrays()\n\nInit vars where declared, use const qualifiers.\n\nReviewed-by: Anuj Phogat <anuj.phogat@gmail.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/161db1335b8dd1f3e2a9abbe82d3b8921d2da314"},{"sha":"c538846e313c43858fb118db012b592513c0ed29","author":{"name":"Brian Paul","email":"496e153a9fb909fb693e8592e6df5f4bc45bdb28@vmware.com"},"message":"mesa: rename gl_client_array -> gl_vertex_array\n\nThe term \"client array\" is a legacy thing dating back to the pre-VBO\nera when _all_ vertex arrays lived in client memory.\n\nNowadays, it only contains vertex array state which is derived from\ngl_array_attributes and gl_vertex_buffer_binding.  It's used by the\nVBO module and some drivers.\n\nReviewed-by: Anuj Phogat <anuj.phogat@gmail.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/c538846e313c43858fb118db012b592513c0ed29"},{"sha":"e6aeeace6953a7007d98082e3f44bff40a44106d","author":{"name":"Kenneth Graunke","email":"e8e0e12ced052a7b04539107b6dfbe9787d38bd2@whitecape.org"},"message":"glsl: Improve accuracy of alpha scaling in advanced blend lowering.\n\nWhen blending with GL_COLORBURN_KHR and these colors:\n\n   dst = <0.372549027, 0.372549027, 0.372549027, 0.372549027>\n   src = <0.09375, 0.046875, 0.0, 0.375>\n\nthe normalized dst value became 0.99999994 (due to precision problems\nin the floating point divide of rgb by alpha).  This caused the color\nburn equation to fail the dst >= 1.0 comparison.  The blue channel would\nthen fall through to the dst < 1.0 && src >= 0 comparison, which was\ntrue, since src.b == 0.  This produced a factor of 0.0 instead of 1.0.\n\nThis is an inherent numerical instability in the color burn and dodge\nequations - depending on the precision of alpha scaling, the value can\nbe either 0.0 or 1.0.  Technically, GLSL floating point division doesn't\neven guarantee that 0.372549027 / 0.372549027 = 1.0.  So arguably, the\nCTS should allow either value.  I've filed a bug at Khronos for further\ndiscussion (linked below).\n\nIn the meantime, this patch improves the precision of alpha scaling by\nreplacing the division with (rgb == alpha ? 1.0 : rgb / alpha).  We may\nnot need this long term, but for now, it fixes the following CTS tests:\n\nES31-CTS.blend_equation_advanced.blend_specific.GL_COLORBURN_KHR\nES31-CTS.blend_equation_advanced.blend_all.GL_COLORBURN_KHR_all_qualifier\n\nCc: currojerez@riseup.net\nCc: mesa-stable@lists.freedesktop.org\nBugzilla: https://cvs.khronos.org/bugzilla/show_bug.cgi?id=16042\nSigned-off-by: Kenneth Graunke <kenneth@whitecape.org>\nReviewed-by: Eduardo Lima Mitev <elima@igalia.com>\nReviewed-by: Francisco Jerez <currojerez@riseup.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/e6aeeace6953a7007d98082e3f44bff40a44106d"},{"sha":"cab3d4673922e2beb1f86f59c6aef17f55da3ac2","author":{"name":"Jason Ekstrand","email":"8d9da6ec6f8b6f23bae58d38147b2bea1269b4d6@intel.com"},"message":"i965: Fix make check after 66fcfa6894ab6\n\nCommit 66fcfa6894ab6 changed the vec4 version of offset() to have 3\nparameters instead of 2 but the vec4_cmod_propagation test was never\nupdated.\n\nSigned-off-by: Jason Ekstrand <jason@jlekstrand.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/cab3d4673922e2beb1f86f59c6aef17f55da3ac2"},{"sha":"e24dc4316487eeaa6ee8aa5c709546d814e96f03","author":{"name":"Marek Olšák","email":"4faf1447f276b79d547990f08bc69bf0766bdee3@amd.com"},"message":"radeonsi: set VGT_GS_ONCHIP_CNTL on CIK and later\n\nReviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>\nCc: 11.2 12.0 13.0  <mesa-stable@lists.freedesktop.org>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/e24dc4316487eeaa6ee8aa5c709546d814e96f03"},{"sha":"4bf45a6079b5cc6b0360b637c0c7baa456b8257d","author":{"name":"Marek Olšák","email":"4faf1447f276b79d547990f08bc69bf0766bdee3@amd.com"},"message":"radeonsi: fix behavior of GLSL findLSB(0)\n\n12.0 and older need the same fix but elsewhere.\n\nCc: 13.0 <mesa-stable@lists.freedesktop.org>\nReviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/4bf45a6079b5cc6b0360b637c0c7baa456b8257d"},{"sha":"2a4a86862c949055c71637429f6d5f2e725d07d8","author":{"name":"Jason Ekstrand","email":"8d9da6ec6f8b6f23bae58d38147b2bea1269b4d6@intel.com"},"message":"i965/fs/generator: Don't use the address immediate for MOV_INDIRECT\n\nThe address immediate field is only 9 bits and, since the value is in\nbytes, the highest GRF we can point to with it is g15.  This makes it\npretty close to useless for MOV_INDIRECT.  There were already piles of\nrestrictions preventing us from using it prior to Broadwell, so let's get\nrid of the gen8+ code path entirely.\n\nSigned-off-by: Jason Ekstrand <jason@jlekstrand.net>\nBugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97779\nCc: \"12.0 13.0\" <mesa-stable@lists.freedesktop.org>\nReviewed-by: Matt Turner <mattst88@gmail.com>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/2a4a86862c949055c71637429f6d5f2e725d07d8"},{"sha":"e3272865c216933168e6c08766d266a33d0e1497","author":{"name":"Vedran Miletić","email":"0e2cc0e39cd09ff159034d5f86622fe05bcef0c9@miletic.net"},"message":"clover: Pass unquoted compiler arguments to Clang\n\nOpenCL apps can quote arguments they pass to the OpenCL compiler, most\ncommonly include paths containing spaces.\n\nIf the Clang OpenCL compiler was called via a shell, the shell would\nsplit the arguments with respect to to quotes and then remove quotes\nbefore passing the arguments to the compiler. Since we call Clang as a\nlibrary, we have to split the argument with respect to quotes and then\nremove quotes before passing the arguments.\n\nv2: move to tokenize(), remove throwing of CL_INVALID_COMPILER_OPTIONS\n\nv3: simplify parsing logic, use more C++11\n\nv4: restore error throwing, clarify a comment\n\nSigned-off-by: Vedran Miletić <vedran@miletic.net>\nReviewed-by: Francisco Jerez <currojerez@riseup.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/e3272865c216933168e6c08766d266a33d0e1497"},{"sha":"2fba72046da09dd28f54df02794b358773899d13","author":{"name":"Vedran Miletić","email":"0e2cc0e39cd09ff159034d5f86622fe05bcef0c9@miletic.net"},"message":"clover: Introduce CLOVER_EXTRA_*_OPTIONS environment variables\n\nThe options specified in the CLOVER_EXTRA_BUILD_OPTIONS shell\nvariable are appended to the options specified by the OpenCL program\nin the clBuildProgram function call, if any.\nAnalogously, the options specified in the CLOVER_EXTRA_COMPILE_OPTIONS\nand CLOVER_EXTRA_LINK_OPTIONS variables are appended to the options\nspecified in clCompileProgram and clLinkProgram function calls,\nrespectively.\n\nv2:\n * rename to CLOVER_EXTRA_COMPILER_OPTIONS\n * use debug_get_option\n * append to linker options as well\n\nv3: code cleanups\n\nv4: separate CLOVER_EXTRA_LINKER_OPTIONS options\n\nv5:\n * fix documentation typo\n * use CLOVER_EXTRA_COMPILER_OPTIONS in link stage\n\nv6:\n * separate in CLOVER_EXTRA_{BUILD,COMPILE,LINK}_OPTIONS\n * append options in cl{Build,Compile,Link}Program\n\nSigned-off-by: Vedran Miletić <vedran@miletic.net>\nReviewed-by[v1]: Edward O'Callaghan <funfunctor@folklore1984.net>\n\nv7 [Francisco Jerez]: Slight simplification.\n\nReviewed-by: Francisco Jerez <currojerez@riseup.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/2fba72046da09dd28f54df02794b358773899d13"},{"sha":"cb0879985a40bcde1516e5341c5a3e5ea0968b87","author":{"name":"Serge Martin","email":"7ff106d3ac46cb459321361187402d921575d2a2@sigluy.net"},"message":"clover: Implement clGetExtensionFunctionAddressForPlatform.\n\nAdd clGetExtensionFunctionAddressForPlatform (CL 1.2).\n\nReviewed-by: Francisco Jerez <currojerez@riseup.net>","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/cb0879985a40bcde1516e5341c5a3e5ea0968b87"},{"sha":"330482177c45dceb71e5787c1a7e7fc87308c144","author":{"name":"Marek Olšák","email":"4faf1447f276b79d547990f08bc69bf0766bdee3@amd.com"},"message":"ralloc: don't memset ralloc_header, clear it manually\n\ntime GALLIUM_NOOP=1 ./run shaders/private/alien_isolation/ >/dev/null\n\nBefore (2 takes):\n\nreal    0m8.734s    0m8.773s\nuser    0m34.232s   0m34.348s\nsys     0m0.084s    0m0.056s\n\nAfter (2 takes):\n\nreal    0m8.448s    0m8.463s\nuser    0m33.104s   0m33.160s\nsys     0m0.088s    0m0.076s\n\nAverage change in \"real\" time spent: -3.4%\n\ncalloc should only do 2 things compared to malloc:\n- check for overflow of \"n * size\"\n- call memset\nI'm not sure if that explains the difference.\n\nv2: clear \"parent\" and \"next\" in the caller of add_child.\n\nReviewed-by: Edward O'Callaghan <funfunctor@folklore1984.net> (v1)\nTested-by: Edmondo Tommasina <edmondo.tommasina@gmail.com> (v1)\nReviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com> (v1)","distinct":false,"url":"https://api.github.com/repos/Igalia/mesa/commits/330482177c45dceb71e5787c1a7e7fc87308c144"}]}
{"actor":{"display_login":"jasuarez"}}