{"push_id":2347916361,"size":8,"distinct_size":7,"ref":"refs/heads/master","head":"b378a220e42e04a26cc78e7bb38f1a8b7efcace4","before":"be29e7b4432eb00da7e550fa6253ba8ee453db5b","commits":[{"sha":"830a6def7e5e66674f3d87fa289cb7da0c6d5e44","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Set rootLocaleInitialized in RootLocale.deinit\n\nBefore this commit, RootLocale deinit consisted of a\nfor loop over the locales, where it would delete each\nlocale in turn. Note though that deleteing an object\nruns an 'on' statement to go where the object is stored.\nAfter PR #7193, these 'on' statements had the effect\nof changing the running task count field of the locale\nobject - which in some cases was already deleted.\n\nThis commit sets rootLocaleInitialized=false in this\nloop in order to disable the modifications to the\nrunning task count. It wraps both this change an the\ndelete in an on statement.","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/830a6def7e5e66674f3d87fa289cb7da0c6d5e44"},{"sha":"454b38cca5a5a7108ad9ff1d3519997c5d73aae7","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Ignore default initializers for iterator records and classes\n\nIterator records and classes do not use the default constructor today, so there\nisn't any reason to try and use default initializers.  Once the implementation\nfor constructors is removed, this check should no longer be necessary.","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/454b38cca5a5a7108ad9ff1d3519997c5d73aae7"},{"sha":"e3eb90c852193fcc77dda4330cddcb693b8cf2f2","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Put the main process to sleep while waiting to shutdown\n\nThis PR frees up `chpl_comm_barrier()` for use in module code, and prevents the\nmain process from interfering with tasks running on the same CPU.\n\nPreviously, the main process on non-0 locales would spin-wait on a barrier in\n`chpl_comm_pre_task_exit()`, waiting for locale-0 to tell them to do an orderly\nshutdown. This barrier prevented chpl_comm_barrier from being used in user\napplications, and the spin-waiting hurts performance for tasks running on the\nsame CPU. This alleviates both issues by putting the main process to sleep with\na condition variable, and having locale-0 wake everybody up with a AM that\nsignals the cond var when it's time to shutdown.\n\nThis is motivated by an upcoming scalable barrier PR that wants to use\n`chpl_comm_barrier()` in module code. The scalable barrier will significantly\nimprove ISx performance and scalability. Putting the main process to sleep\nreduces interference, but we probably won't notice this in any tests yet\nbecause the progress-thread currently causes more interference. i.e. tasks\nsharing a core with the progress-thread are already the slowest tasks, so the\nbenefit of parking the main process will only be visible once that issue is\nresolved.","distinct":false,"url":"https://api.github.com/repos/ronawho/chapel/commits/e3eb90c852193fcc77dda4330cddcb693b8cf2f2"},{"sha":"d9b5d88a1ab5ec1940408a310ad50dfd5dedc765","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Use processor atomics for Barrier variables\n\nThe atomics are always manipulated inside an on-stmt back to the locale where\nthe barrier lives, so using processor atomics will be faster.\n\nThis shaves about a second off the empty barrier test under ugni (10s to 9s),\nand slightly helps ISx, but ISx is currently so noisy it's hard to see the\ndifference.","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/d9b5d88a1ab5ec1940408a310ad50dfd5dedc765"},{"sha":"6de597b692beca89b83b130b60e2ff6b27eb384d","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Merge pull request #8524 from mppf/fix-locale-deinit-use-after-free\n\nSet rootLocaleInitialized in RootLocale.deinit\r\n\r\nBefore this commit, RootLocale deinit consisted of a\r\nfor loop over the locales, where it would delete each\r\nlocale in turn. Note though that deleteing an object\r\nruns an 'on' statement to go where the object is stored.\r\nAfter PR #7193, these 'on' statements had the effect\r\nof changing the running task count field of the locale\r\nobject - which in some cases was already deleted.\r\n\r\nThis commit sets rootLocaleInitialized=false in this\r\nloop in order to disable the modifications to the\r\nrunning task count. It wraps both this change and the\r\ndelete in an on statement.\r\n\r\nReviewed by @ronawho - thanks!\r\n\r\n - [x] full std correctness testing \r\n - [x] full numa correctness testing (passed expect for 2 known  in-intent failures)\r\n - [x] full gasnet correctness testing","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/6de597b692beca89b83b130b60e2ff6b27eb384d"},{"sha":"9e5e37d6ea6dccc19c62ac6d3a11bee8d7efe322","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Merge pull request #8528 from lydia-duncan/iteratorRecords\n\nIgnore default initializers for iterator records and classes\r\n[reviewed by @benharsh]\r\n\r\nIterator records and classes do not create a default constructor today, so there\r\nisn't any reason to try and use default initializers. Once the implementation\r\nfor constructors is removed, this check should no longer be necessary.\r\n\r\nTested over classes/initializers with futures, a full paratest, and a full\r\nparatest with --force-initializers.  In addition to passing the first two,\r\nthe latter showed 54 failures resolved and no new errors","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/9e5e37d6ea6dccc19c62ac6d3a11bee8d7efe322"},{"sha":"e84bee8a3fa1817924134231a1e510868aaa7c76","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Merge pull request #8533 from ronawho/allow-comm-barrier-in-module-code\n\nPut the main process to sleep while waiting to shutdown\r\n\r\n[reviewed by @gbtitus]\r\n\r\nThis PR frees up `chpl_comm_barrier()` for use in module code, and prevents the\r\nmain process from interfering with tasks running on the same CPU.\r\n\r\nPreviously, the main process on non-0 locales would spin-wait on a barrier in\r\n`chpl_comm_pre_task_exit()`, waiting for locale-0 to tell them to do an orderly\r\nshutdown. This barrier prevented chpl_comm_barrier from being used in user\r\napplications, and the spin-waiting hurts performance for tasks running on the\r\nsame CPU. This alleviates both issues by putting the main process to sleep with\r\na condition variable, and having locale-0 wake everybody up with a AM that\r\nsignals the cond var when it's time to shutdown.\r\n\r\nThis is motivated by an upcoming scalable barrier PR that wants to use\r\n`chpl_comm_barrier()` in module code. The scalable barrier will significantly\r\nimprove ISx performance and scalability. Putting the main process to sleep\r\nreduces interference, but we probably won't notice this in any tests yet\r\nbecause the progress-thread currently causes more interference. i.e. tasks\r\nsharing a core with the progress-thread are already the slowest tasks, so the\r\nbenefit of parking the main process will only be visible once that issue is\r\nresolved.","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/e84bee8a3fa1817924134231a1e510868aaa7c76"},{"sha":"b378a220e42e04a26cc78e7bb38f1a8b7efcace4","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Merge pull request #8534 from ronawho/force-processor-atomics-barrier\n\nUse processor atomics for Barrier variables\r\n\r\n[reviewed by @daviditen]\r\n\r\nThe atomics are always manipulated inside an on-stmt back to the locale where\r\nthe barrier lives, so using processor atomics will be faster.\r\n\r\nThis shaves about a second off the empty barrier test under ugni (10s to 9s),\r\nand slightly helps ISx, but ISx is currently so noisy it's hard to see the\r\ndifference.","distinct":true,"url":"https://api.github.com/repos/ronawho/chapel/commits/b378a220e42e04a26cc78e7bb38f1a8b7efcace4"}]}
{"actor":{"display_login":"ronawho"}}