https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2021-03-19T07:40:45ZCeph Ceph - Fix #49896: qa: verify the benefits of mempool cacheline optimizationhttps://tracker.ceph.com/issues/49896?journal_id=1880722021-03-19T07:40:45ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Related to</strong> <i><a class="issue tracker-1 status-3 priority-5 priority-high3 closed" href="/issues/49781">Bug #49781</a>: unittest_mempool.check_shard_select failed</i> added</li></ul> Ceph - Fix #49896: qa: verify the benefits of mempool cacheline optimizationhttps://tracker.ceph.com/issues/49896?journal_id=1880742021-03-19T07:45:52ZLoïc Dacharyloic@dachary.org
<ul><li><strong>File</strong> <a href="/attachments/download/5429/0001-qa-verify-the-benefits-of-mempool-cacheline-optimiza.patch">0001-qa-verify-the-benefits-of-mempool-cacheline-optimiza.patch</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/5429/0001-qa-verify-the-benefits-of-mempool-cacheline-optimiza.patch">View</a> added</li></ul><p>Here is how it could be done (please let me know if there is a better way):</p>
<ul>
<li>Write a test program (ceph_test_c2c) that can run in two modes: with and without the optimization</li>
<li>Add a teuthology workunit in standalone part of the rados suite to do the following:
<ul>
<li>Run ceph_test_c2c --no-optimization</li>
<li>Run perf c2c & save the data</li>
<li>Run ceph_test c2c --with-optimization</li>
<li>Run perf c2c & save the data</li>
<li>Verify the optimized version performs at least X% better</li>
</ul></li>
</ul>
<p>The attached patch shows the skeleton of this test.</p> Ceph - Fix #49896: qa: verify the benefits of mempool cacheline optimizationhttps://tracker.ceph.com/issues/49896?journal_id=1880752021-03-19T07:46:30ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/188075/diff?detail_id=194639">diff</a>)</li></ul> Ceph - Fix #49896: qa: verify the benefits of mempool cacheline optimizationhttps://tracker.ceph.com/issues/49896?journal_id=1932132021-04-26T07:24:25ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Pull request ID</strong> set to <i>41014</i></li></ul> Ceph - Fix #49896: qa: verify the benefits of mempool cacheline optimizationhttps://tracker.ceph.com/issues/49896?journal_id=1932142021-04-26T07:28:13ZLoïc Dacharyloic@dachary.org
<ul></ul><p>@smithfarm, would you be so kind as to update the pull request description to reference Fixes: <a class="external" href="https://tracker.ceph.com/issues/49896">https://tracker.ceph.com/issues/49896</a> instead of Refs: <a class="external" href="https://tracker.ceph.com/issues/49781">https://tracker.ceph.com/issues/49781</a> ? My mistake :-(</p>
<p>I also re-pushed the commit in <a class="external" href="https://lab.fedeproxy.eu/ceph/ceph/-/tree/wip-mempool-cacheline-49781/">https://lab.fedeproxy.eu/ceph/ceph/-/tree/wip-mempool-cacheline-49781/</a> and it would be great if you could also push it in the pull request so the commit message is in sync.</p> Ceph - Fix #49896: qa: verify the benefits of mempool cacheline optimizationhttps://tracker.ceph.com/issues/49896?journal_id=1935632021-04-30T11:38:30ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li></ul>