{"push_id":2417584655,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"035cc7a12dc03ef8b8184f75b880ae41e452b215","before":"604cac9f739264377acb22a657bf745762a94656","commits":[{"sha":"e9a9e859486104ff9326cbc424480e53dc4e81ba","author":{"name":"Lionel Landwerlin","email":"0ec1b4b16fd0d4c2b6a37e9f7c362143c92cc31f@intel.com"},"message":"i965: perf: fix a counter return type on hsw\n\nThe equation code computes a float (percentage) yet the return type\nwas an uint64_t.\n\nSigned-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>","distinct":true,"url":"https://api.github.com/repos/freedesktop/mesa/commits/e9a9e859486104ff9326cbc424480e53dc4e81ba"},{"sha":"035cc7a12dc03ef8b8184f75b880ae41e452b215","author":{"name":"Lionel Landwerlin","email":"0ec1b4b16fd0d4c2b6a37e9f7c362143c92cc31f@intel.com"},"message":"i965: perf: reduce i965 binary size\n\nPerformance metric numbers are calculated the following way :\n\n   - out of the 256 bytes long OA reports, we accumulate the deltas\n     into an array of uint64_t\n\n   - the equations' generated code reads the accumulated uint64_t\n     deltas and normalizes them for a particular platform\n\nOur hardware is such that a number of counters in the OA reports\nalways return the same values (i.e. they're not programmable), and\nthey return the same values even across generations, and as a result a\nnumber of equations are identical in different metric sets across\ndifferent generations.\n\nUp to now we've kept the generated code of the equations separated in\ndifferent files (per generation/GT), and didn't apply any\nfactorization of the common equations. We could have make some\nimprovement by reusing equations within a given metrics file, but we\ncan go even further and reuse across generations (i.e. all files).\n\nThis change changes the code generation to emit a single file in which\nwe reuse equations emitted code based on the hash of equations'\nstrings.\n\nHere are the savings in a meson build :\n\nBefore(.old)/after :\n   $ du -h ./build/src/mesa/drivers/dri/libmesa_dri_drivers.so ./build/src/mesa/drivers/dri/libmesa_dri_drivers.so.old\n   43M\t./build/src/mesa/drivers/dri/libmesa_dri_drivers.so\n   47M\t./build/src/mesa/drivers/dri/libmesa_dri_drivers.so.old\n\n   $ size build/src/mesa/drivers/dri/libmesa_dri_drivers.so build/src/mesa/drivers/dri/libmesa_dri_drivers.so.old\n       text   data          bss\t     dec            hex filename\n   13054002 409424\t 671856\t14135282\t d7aff2\tbuild/src/mesa/drivers/dri/libmesa_dri_drivers.so\n   14550386 409552\t 671856\t15631794\t ee85b2\tbuild/src/mesa/drivers/dri/libmesa_dri_drivers.so.old\n\nAs a side comment here is the size of the drivers if we remove all of\nthe metrics from the build :\n\n   $ du -sh build/src/mesa/drivers/dri/libmesa_dri_drivers.so\n   40M\tbuild/src/mesa/drivers/dri/libmesa_dri_drivers.so\n\nv2: Fix an issue with hashing of counter equations (Lionel)\n    Build system rework (Emil)\n\nSigned-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>\nReviewed-by: Emil Velikov <emil.velikov@collabora.com> (build system part)\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>","distinct":true,"url":"https://api.github.com/repos/freedesktop/mesa/commits/035cc7a12dc03ef8b8184f75b880ae41e452b215"}]}
{"actor":{"display_login":"fdo-mirror"}}