{"push_id":738250346,"size":342,"distinct_size":342,"ref":"refs/heads/master","head":"d69da58e84448188808488ad1c1c0181b5630a74","before":"ea633db65ffa684ea5237b8cb5bd96fbc1a7769a","commits":[{"sha":"759ed0bd03818c912e7f1fa62bafc50ef52ef291","author":{"email":"789b315743a28dd066a4ba1459c35951c291d8a6@gmail.com","name":"Matt Turner"},"message":"i965: Mark constant static data as const.\n\nReviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/759ed0bd03818c912e7f1fa62bafc50ef52ef291"},{"sha":"ab80519b3cd08401dff2d07343064a27f32b33ca","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Add perf debug for when we wait on BOs.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/ab80519b3cd08401dff2d07343064a27f32b33ca"},{"sha":"e4c540f6d09390013a9cb66060a29f236ad7dcfc","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Store reloc pointers as pointers, not offsets.\n\nNow that we don't resize the CL as we build (it's set up at the top by\nvc4_start_draw()), we can store the pointers instead of offsets from\nthe base.  Saves a bit of math in emitting relocs (about 60 bytes of\ncode).","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/e4c540f6d09390013a9cb66060a29f236ad7dcfc"},{"sha":"748bf459b46b44e184ee1d425ce612da61a0800e","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Drop separate cl*_reloc_hindex().\n\nNow that RCL generation is in the kernel, we don't have any other\ncallers.  Oddly, the compiler generates another 8 bytes of code for\nthis, but the simplification is worth it.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/748bf459b46b44e184ee1d425ce612da61a0800e"},{"sha":"a0d3915663fb7cbd3c1a5561450e256e00ecf11b","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Make a helper function for getting the current offset in the CL.\n\nI needed to rewrite this a bit for safety checking in the next commit.\nDespite being a static inline of the same thing that was being done, we\nlose 36 bytes of code for some reason.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/a0d3915663fb7cbd3c1a5561450e256e00ecf11b"},{"sha":"7432017f65174e82a3de7afef3e4e6f60932356c","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Rework cl handling to be friendlier to the compiler.\n\nDrops 680 bytes of code, from avoiding a bunch of extra updates to the\nnext pointer in the struct.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/7432017f65174e82a3de7afef3e4e6f60932356c"},{"sha":"3df78928786134874eafa6f68186c8edbbdd3ae7","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Drop reloc_count tracking for debug asserts on non-debug builds.\n\nCuts another 88 bytes of compiled code.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/3df78928786134874eafa6f68186c8edbbdd3ae7"},{"sha":"1e80c9fab98d7de216937a47f8e231f3beb78403","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Add better debug for register allocation failure.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/1e80c9fab98d7de216937a47f8e231f3beb78403"},{"sha":"cd7dd45bfec9ad68719c5e4e04b66ea4bcc1a2c1","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Fix compiler warnings on release builds.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/cd7dd45bfec9ad68719c5e4e04b66ea4bcc1a2c1"},{"sha":"320089dbd63de3ac1bd3d42ee8cec41837486d8c","author":{"email":"a79006884ba9cf5f4f7fcbb92a15b208780d8c97@intel.com","name":"Jordan Justen"},"message":"i965/cs: Initialize GPGPU Thread Count\n\nThis field should always be set for gen8. In the bdw PRM, Volume 2d:\nCommand Reference: Structures under INTERFACE_DESCRIPTOR_DATA, DWORD\n6, Bits 9:0, Number of Threads in GPGPU Thread Group:\n\n\"This field should not be set to 0 even if the barrier is disabled,\nsince an accurate value is needed for proper pre-emption.\"\n\nIn the HSW PRM, the it doesn't mention that it must always be set, but\nit should not hurt.\n\nReported-by: Kristian Høgsberg <krh@bitplanet.net>\nSigned-off-by: Jordan Justen <jordan.l.justen@intel.com>\nReviewed-by: Ben Widawsky <ben@bwidawsk.net>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/320089dbd63de3ac1bd3d42ee8cec41837486d8c"},{"sha":"9476b11d6edc67403dd7c5aaddbc375400e02425","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Fix some -Wdouble-promotion warnings.\n\nNo code generation changes from this, but it'll be useful to have this\nnext time I go checking -Wdouble-promotion.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/9476b11d6edc67403dd7c5aaddbc375400e02425"},{"sha":"1835ce6e35e6a186c2ba1bdf39b73783a2cb2ad5","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Move uniforms handling to a separate file.\n\nThe rest of vc4_program.c is about compiling, while this is about\nuniform emit at draw time.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/1835ce6e35e6a186c2ba1bdf39b73783a2cb2ad5"},{"sha":"0f4d2b0a2dd3fa39426f2789bf2a8fc939adf001","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Cache texture p0/p1 setup for the sampler view.\n\nIn exchange for a bit of space and computation in CSO setup, we cut\nvc4_uniform.c (draw time) code size by 4.8%.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/0f4d2b0a2dd3fa39426f2789bf2a8fc939adf001"},{"sha":"7124feba1b879deb88dbf2baf600ed42309d9839","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"vc4: Cache the texture p1 for the sampler.\n\nCuts another 12% of vc4_uniforms.o, in exchange for computing it at\nCSO creation time.","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/7124feba1b879deb88dbf2baf600ed42309d9839"},{"sha":"141e1eb29fe80ad341e718147a1277cc3b1b9c11","author":{"email":"3cb4e1df5ec4da2c7c4af7c52cec8cf340a55a10@vmware.com","name":"Brian Paul"},"message":"osmesa: fix OSMesaPixelsStore typo\n\nBugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91337\nCc: 10.6 <mesa-stable@lists.freedesktop.org>\n\nReviewed-by: Emil Velikov <emil.l.velikov@gmail.com>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/141e1eb29fe80ad341e718147a1277cc3b1b9c11"},{"sha":"fbf3aebf1f33fbec559c5b69bdf3b5dec6031612","author":{"email":"789b315743a28dd066a4ba1459c35951c291d8a6@gmail.com","name":"Matt Turner"},"message":"i965: Move BEGIN_BATCH() into same control flow as ADVANCE_BATCH().\n\nBEGIN_BATCH() and ADVANCE_BATCH() will contain \"do {\" and \"} while (0)\"\nrespectively to allow declaring local variables used by intervening\nOUT_BATCH macros. As such, BEGIN_BATCH() and ADVANCE_BATCH() need to be\nin the same control flow.\n\nReviewed-by: Iago Toral Quiroga <itoral@igalia.com>\nReviewed-by: Chris Wilson <chris@chris-wilson.co.uk>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/fbf3aebf1f33fbec559c5b69bdf3b5dec6031612"},{"sha":"09348c12fceba59c22219fe3272260eb8ea6051e","author":{"email":"789b315743a28dd066a4ba1459c35951c291d8a6@gmail.com","name":"Matt Turner"},"message":"i965: Split batch emission from relocation functions.\n\nSo that everything writing to the batch between BEGIN_BATCH() and\nADVANCE_BATCH() goes through OUT_BATCH.\n\nReviewed-by: Chris Wilson <chris@chris-wilson.co.uk>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/09348c12fceba59c22219fe3272260eb8ea6051e"},{"sha":"131573df7aea0b10e97d9d5db0d26d89f8dfef54","author":{"email":"789b315743a28dd066a4ba1459c35951c291d8a6@gmail.com","name":"Matt Turner"},"message":"i965: Add and use USED_BATCH macro.\n\nThe next patch will replace the .used field with an on-demand\ncalculation of batchbuffer usage.\n\nReviewed-by: Chris Wilson <chris@chris-wilson.co.uk>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/131573df7aea0b10e97d9d5db0d26d89f8dfef54"},{"sha":"f11c6f09cf36909ff399353b20195a31cf0f1907","author":{"email":"789b315743a28dd066a4ba1459c35951c291d8a6@gmail.com","name":"Matt Turner"},"message":"i965: Optimize batchbuffer macros.\n\nPreviously OUT_BATCH was just a macro around an inline function which\ndoes\n\n   brw->batch.map[brw->batch.used++] = dword;\n\nWhen making consecutive calls to intel_batchbuffer_emit_dword() the\ncompiler isn't able to recognize that we're writing consecutive memory\nlocations or that it doesn't need to write batch.used back to memory\neach time.\n\nWe can avoid both of these problems by making a local pointer to the\nnext location in the batch in BEGIN_BATCH().\n\nCuts 18k from the .text size.\n\n   text     data      bss      dec      hex  filename\n4946956   195152    26192  5168300   4edcac  i965_dri.so before\n4928956   195152    26192  5150300   4e965c  i965_dri.so after\n\nThis series (including commit c0433948) improves performance of Synmark\nOglBatch7 by 8.01389% +/- 0.63922% (n=83) on Ivybridge.\n\nReviewed-by: Chris Wilson <chris@chris-wilson.co.uk>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/f11c6f09cf36909ff399353b20195a31cf0f1907"},{"sha":"642f289824dc9a07e8209c905badef31b4841ae1","author":{"email":"e2026475996c48f0b97fd37e84129b3912980428@gmail.com","name":"Anuj Phogat"},"message":"i965: Fix 32 bit build warnings in intel_get_yf_ys_bo_size()\n\nAlong with fixing the type of pitch parameter, patch also changes\nthe types of few local variables and function return type.\n\nWarnings fixed are:\nintel_mipmap_tree.c:671:7: warning: passing argument 3 of\n'intel_get_yf_ys_bo_size' from incompatible pointer type\n\nintel_mipmap_tree.c:563:1: note: expected 'uint64_t *' but\nargument is of type 'long unsigned int *'\n\nSigned-off-by: Anuj Phogat <anuj.phogat@gmail.com>\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>","distinct":true,"url":"https://api.github.com/repos/Distrotech/Mesa/commits/642f289824dc9a07e8209c905badef31b4841ae1"}]}