<div dir="ltr"><div dir="ltr">Sorry for reply late.<div><br></div><div>+ ceilometer<br></div><div><br></div><div>The ceilometer is still using the cmt meter <a href="https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/virt/libvirt/inspector.py#L224">https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/virt/libvirt/inspector.py#L224</a> should we deprecate them?</div><div><br></div><div>And there are some other meter depend on perf feature <a href="https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/virt/libvirt/inspector.py#L218">https://github.com/openstack/ceilometer/blob/master/ceilometer/compute/virt/libvirt/inspector.py#L218</a></div><div><br></div><div>So sounds like we shouldn't remove the whole perf feature.</div><div><br></div><div>Thanks</div><div>Alex</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Kashyap Chamarthy <<a href="mailto:kchamart@redhat.com">kchamart@redhat.com</a>> 于2019年7月4日周四 下午6:37写道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Heya folks,<br>
<br>
While removing some dead code I was wondering if anyone here uses<br>
"non-CMT" (Cache Monitoring Technology) performance events events?  I'm<br>
referring to the events here[0], besides the first three, which are<br>
CMT-related.<br>
<br>
Background<br>
----------<br>
<br>
The Intel CMT events (there are three of them) were deprecated during<br>
the Rocky release, in this[1] commit, and with this rationale:<br>
<br>
    Upstream Linux kernel has deleted[*] the `perf` framework integration<br>
    with Intel CMT (Cache Monitoring Technology; or "CQM" in Linux kernel<br>
    parlance), because the feature was broken by design -- an<br>
    incompatibility between Linux's `perf` infrastructure and Intel CMT<br>
    hardware support.  It was removed in upstream kernel version v4.14; but<br>
    bear in mind that downstream Linux distributions with lower kernel<br>
    versions than 4.14 have backported the said change.<br>
<br>
    Nova supports monitoring of the above mentioned Intel CMT events<br>
    (namely: 'cmt', 'mbm_local', and 'mbm_total') via the configuration<br>
    attribute `[libvirt]/enabled_perf_events`. Given that the underlying<br>
    Linux kernel infrastructure for Intel CMT is removed, we should remove<br>
    support for it in Nova too.  Otherwise enabling them in Nova, and<br>
    updating to a Linux kernel 4.14 (or above) will result in instances<br>
    failing to boot.<br>
<br>
    To that end, deprecate support for the three Intel CMT events in<br>
    "Rocky" release, with the intention to remove support for it in<br>
    the upcoming "Stein" release.  Note that we cannot deprecate /<br>
    remove `enabled_perf_events` config attribute altogether --<br>
    since there are other[+] `perf` events besides Intel CMT.<br>
    Whether anyone is using those other events with Nova is a good<br>
    question to which we don't have an equally good answer for, if<br>
    at all.<br>
<br>
Now we're removing[2] support for CMT events altogether.<br>
<br>
Question<br>
--------<br>
<br>
What I'm wondering now is the answer to the last sentence in the above<br>
quoted commit: "Whether anyone is using those other events with Nova is<br>
a good question to which we don't have an equally good answer for, if at<br>
all".<br>
<br>
If we know that "no one" (as if we can tell for sure) is using them, we<br>
can get rid of more dead code.<br>
<br>
So, any operators using the non-CMT events from here[0]?<br>
<br>
[0] <a href="https://libvirt.org/formatdomain.html#elementsPerf" rel="noreferrer" target="_blank">https://libvirt.org/formatdomain.html#elementsPerf</a><br>
[1] <a href="https://opendev.org/openstack/nova/commit/fc4794acc6" rel="noreferrer" target="_blank">https://opendev.org/openstack/nova/commit/fc4794acc6</a> —libvirt:<br>
    Deprecate support for monitoring Intel CMT `perf` events <br>
[2] <a href="https://review.opendev.org/669129" rel="noreferrer" target="_blank">https://review.opendev.org/669129</a> — libvirt: Remove support for<br>
    Intel CMT `perf` event<br>
<br>
-- <br>
/kashyap<br>
<br>
</blockquote></div></div>