Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1213296 > unrolled thread

RE: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  RE: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support "Liang, Kan" <kan.liang@intel.com> - 2015-08-25 22:20 +0200
    Re: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support Stephane Eranian <eranian@google.com> - 2015-08-28 17:00 +0200
      RE: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support "Liang, Kan" <kan.liang@intel.com> - 2015-08-28 17:10 +0200
        RE: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support "Liang, Kan" <kan.liang@intel.com> - 2015-08-28 17:20 +0200
        Re: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support Stephane Eranian <eranian@google.com> - 2015-08-28 17:20 +0200

#1213296 — RE: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs support

DQo+ID4+ID4NCj4gPj4gSSB1bmRlcnN0YW5kIHRoYXQgdGhlc2UgbWV0cmljcyBhcmUgdXNlZnVs
IGFuZCBuZWVkZWQgaG93ZXZlciBpZiBJDQo+ID4+IGxvb2sgYXQgdGhlIGJyb2FkZXIgcGljdHVy
ZSBJIHNlZSBtYW55IFBNVXMgZG9pbmcgc2ltaWxhciB0aGluZ3Mgb3INCj4gPj4gYXBwZWFyaW5n
IGRpZmZlcmVudCB3aGVuIHRoZXkgYXJlIGFjdHVhbGx5IHZlcnkgY2xvc2UuIEl0IHdvdWxkIGJl
DQo+ID4+IG5pY2UgdG8gaGF2ZSBhIG1vcmUgdW5pZmllZCBhcHByb2FjaC4gWW91IGhhdmUgUkFQ
TCAoY2xpZW50LCBzZXJ2ZXIpDQo+ID4+IHdoaWNoIGFwcGVhcnMgYXMgdGhlIHBvd2VyIFBNVS4g
WW91IGhhdmUgdGhlIFBDVSB1bmNvcmUgb24gc2VydmVycw0KPiA+PiB3aGljaCBhbHNvIHByb3Zp
ZGVzIEMtc3RhdGUgcmVzaWRlbmN5IGluZm8uIFlldCwgYWxsIHRoZXNlIGFwcGVhcg0KPiA+PiBk
aWZmZXJlbnRseSBhbmQgZXhwb3NlIGV2ZW50cyB3aXRoIGRpZmZlcmVudCBuYW1lcy4NCj4gPj4g
SSB0aGluayB3ZSBjb3VsZCBiZW5lZml0IGZyb20gYSBtb3JlIHVuaWZpZSBhcHByb2FjaCBoZXJl
IHN1Y2ggdGhhdA0KPiA+PiB5b3Ugd291bGQgYmUgYWJsZSB0byBkbw0KPiA+Pg0KPiA+PiAkICAg
IHBlcmYgc3RhdCAtYSAtZSBwb3dlci9jNi1yZXNpZGVuY3kvLCBwb3dlci9lbmVyZ3ktcGtnLw0K
PiA+Pg0KPiA+PiBvbiBjbGllbnQgYW5kIHNlcnZlciB3aXRob3V0IGhhdmluZyB0byBjaGFuZ2Ug
dGhlIHBtdSBuYW1lIG9mIHRoZQ0KPiA+PiBldmVudCBuYW1lcy4NCj4gPg0KPiA+IFllcywgSSBh
Z3JlZS4gSSdsbCB0aGluayBhYm91dCBpdC4NCj4gPg0KDQpIaSBTdGVwaGFuZSwNCg0KSSB0aG91
Z2h0IG1vcmUgYWJvdXQgeW91ciBzdWdnZXN0aW9uIHJlZ2FyZGluZyB0byBjcmVhdGUgYSB1bmlm
aWVkDQpwb3dlciBQTVUgZm9yIGFsbCByZWxhdGVkIGV2ZW50cyBpbmNsdWRlIFJBUEwgYW5kIHJl
c2lkZW5jeS4NCkl0IGxvb2tzIHdlIGNhbiBiZW5lZml0IGZyb20gYSBzaW1wbGUgdW5pZmllZCBu
YW1lLCBidXQgaXQgYWxzbw0KYnJpbmdzIHRvbyBtdWNoIGNvbmZ1c2lvbi4NCiAgLSBjc3RhdGUg
cmVzaWRlbmN5IGlzIHRoZSB0aW1lIG9mIHRoZSBjb3JlL3NvY2tldCBpbiBzcGVjaWZpYyBjc3Rh
dGUuDQogICAgIFdoaWxlIFJBUEwgZXZlbnQgaXMgdGhlIHBvd2VyIGNvcmUvc29ja2V0IHdoaWNo
IGNvbnN1bWVkLg0KICAgICBUaGV5IGhhdmUgZGlmZmVyZW50IGNvbmNlcHRzLiANCiAgLSBjc3Rh
dGUgcmVzaWRlbmN5IGluY2x1ZGVzIGJvdGggcGVyLWNvcmUgYW5kIHBlci1zb2NrZXQgZXZlbnRz
Lg0KICAgICBSQVBMIGV2ZW50cyBpcyBvbmx5IHBlci1zb2NrZXQuIFNvIHRoZSBDUFUgbWFzayBp
cyBkaWZmZXJlbnQuDQogICAgIEl0J3MgdmVyeSBjb25mdXNlZCB0aGF0IHRoZSBldmVudHMgaW4g
c2FtZSBQTVUgaGFzIGRpZmZlcmVudCBDUFUgbWFzay4NCg0KU28gSSB0aGluayBpdCBzaG91bGQg
YmUgYmV0dGVyIHRvIHVzZSBkaWZmZXJlbnQgUE1VcyBmb3IgUkFQTCBhbmQgcmVzaWRlbmN5Lg0K
DQpXaGF0IGRvIHlvdSB0aGluaz8NCg0KVGhhbmtzLA0KS2FuDQoNCg0K
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1215385

On Tue, Aug 25, 2015 at 1:15 PM, Liang, Kan <kan.liang@intel.com> wrote:
>
>> >> >
>> >> I understand that these metrics are useful and needed however if I
>> >> look at the broader picture I see many PMUs doing similar things or
>> >> appearing different when they are actually very close. It would be
>> >> nice to have a more unified approach. You have RAPL (client, server)
>> >> which appears as the power PMU. You have the PCU uncore on servers
>> >> which also provides C-state residency info. Yet, all these appear
>> >> differently and expose events with different names.
>> >> I think we could benefit from a more unifie approach here such that
>> >> you would be able to do
>> >>
>> >> $    perf stat -a -e power/c6-residency/, power/energy-pkg/
>> >>
>> >> on client and server without having to change the pmu name of the
>> >> event names.
>> >
>> > Yes, I agree. I'll think about it.
>> >
>
> Hi Stephane,
>
> I thought more about your suggestion regarding to create a unified
> power PMU for all related events include RAPL and residency.
> It looks we can benefit from a simple unified name, but it also
> brings too much confusion.
>   - cstate residency is the time of the core/socket in specific cstate.
>      While RAPL event is the power core/socket which consumed.
>      They have different concepts.
>   - cstate residency includes both per-core and per-socket events.
>      RAPL events is only per-socket. So the CPU mask is different.
>      It's very confused that the events in same PMU has different CPU mask.
>
> So I think it should be better to use different PMUs for RAPL and residency.
>
> What do you think?
>
Well, you are maybe confusing events with PMU. If you look at the core PMU, it
cover many events measuring vastly different aspects of the core. Some events
are per-thread, others are per-core.

Here, I was thinking it would be good to have some power// PMU with many
events covering cstate residency, energy consumption. And yes, some events
would be per-socket, others per-core.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1215389

DQoNCj4gT24gVHVlLCBBdWcgMjUsIDIwMTUgYXQgMToxNSBQTSwgTGlhbmcsIEthbiA8a2FuLmxp
YW5nQGludGVsLmNvbT4gd3JvdGU6DQo+ID4NCj4gPj4gPj4gPg0KPiA+PiA+PiBJIHVuZGVyc3Rh
bmQgdGhhdCB0aGVzZSBtZXRyaWNzIGFyZSB1c2VmdWwgYW5kIG5lZWRlZCBob3dldmVyIGlmIEkN
Cj4gPj4gPj4gbG9vayBhdCB0aGUgYnJvYWRlciBwaWN0dXJlIEkgc2VlIG1hbnkgUE1VcyBkb2lu
ZyBzaW1pbGFyIHRoaW5ncw0KPiA+PiA+PiBvciBhcHBlYXJpbmcgZGlmZmVyZW50IHdoZW4gdGhl
eSBhcmUgYWN0dWFsbHkgdmVyeSBjbG9zZS4gSXQgd291bGQNCj4gPj4gPj4gYmUgbmljZSB0byBo
YXZlIGEgbW9yZSB1bmlmaWVkIGFwcHJvYWNoLiBZb3UgaGF2ZSBSQVBMIChjbGllbnQsDQo+ID4+
ID4+IHNlcnZlcikgd2hpY2ggYXBwZWFycyBhcyB0aGUgcG93ZXIgUE1VLiBZb3UgaGF2ZSB0aGUg
UENVIHVuY29yZQ0KPiBvbg0KPiA+PiA+PiBzZXJ2ZXJzIHdoaWNoIGFsc28gcHJvdmlkZXMgQy1z
dGF0ZSByZXNpZGVuY3kgaW5mby4gWWV0LCBhbGwgdGhlc2UNCj4gPj4gPj4gYXBwZWFyIGRpZmZl
cmVudGx5IGFuZCBleHBvc2UgZXZlbnRzIHdpdGggZGlmZmVyZW50IG5hbWVzLg0KPiA+PiA+PiBJ
IHRoaW5rIHdlIGNvdWxkIGJlbmVmaXQgZnJvbSBhIG1vcmUgdW5pZmllIGFwcHJvYWNoIGhlcmUg
c3VjaA0KPiA+PiA+PiB0aGF0IHlvdSB3b3VsZCBiZSBhYmxlIHRvIGRvDQo+ID4+ID4+DQo+ID4+
ID4+ICQgICAgcGVyZiBzdGF0IC1hIC1lIHBvd2VyL2M2LXJlc2lkZW5jeS8sIHBvd2VyL2VuZXJn
eS1wa2cvDQo+ID4+ID4+DQo+ID4+ID4+IG9uIGNsaWVudCBhbmQgc2VydmVyIHdpdGhvdXQgaGF2
aW5nIHRvIGNoYW5nZSB0aGUgcG11IG5hbWUgb2YgdGhlDQo+ID4+ID4+IGV2ZW50IG5hbWVzLg0K
PiA+PiA+DQo+ID4+ID4gWWVzLCBJIGFncmVlLiBJJ2xsIHRoaW5rIGFib3V0IGl0Lg0KPiA+PiA+
DQo+ID4NCj4gPiBIaSBTdGVwaGFuZSwNCj4gPg0KPiA+IEkgdGhvdWdodCBtb3JlIGFib3V0IHlv
dXIgc3VnZ2VzdGlvbiByZWdhcmRpbmcgdG8gY3JlYXRlIGEgdW5pZmllZA0KPiA+IHBvd2VyIFBN
VSBmb3IgYWxsIHJlbGF0ZWQgZXZlbnRzIGluY2x1ZGUgUkFQTCBhbmQgcmVzaWRlbmN5Lg0KPiA+
IEl0IGxvb2tzIHdlIGNhbiBiZW5lZml0IGZyb20gYSBzaW1wbGUgdW5pZmllZCBuYW1lLCBidXQg
aXQgYWxzbyBicmluZ3MNCj4gPiB0b28gbXVjaCBjb25mdXNpb24uDQo+ID4gICAtIGNzdGF0ZSBy
ZXNpZGVuY3kgaXMgdGhlIHRpbWUgb2YgdGhlIGNvcmUvc29ja2V0IGluIHNwZWNpZmljIGNzdGF0
ZS4NCj4gPiAgICAgIFdoaWxlIFJBUEwgZXZlbnQgaXMgdGhlIHBvd2VyIGNvcmUvc29ja2V0IHdo
aWNoIGNvbnN1bWVkLg0KPiA+ICAgICAgVGhleSBoYXZlIGRpZmZlcmVudCBjb25jZXB0cy4NCj4g
PiAgIC0gY3N0YXRlIHJlc2lkZW5jeSBpbmNsdWRlcyBib3RoIHBlci1jb3JlIGFuZCBwZXItc29j
a2V0IGV2ZW50cy4NCj4gPiAgICAgIFJBUEwgZXZlbnRzIGlzIG9ubHkgcGVyLXNvY2tldC4gU28g
dGhlIENQVSBtYXNrIGlzIGRpZmZlcmVudC4NCj4gPiAgICAgIEl0J3MgdmVyeSBjb25mdXNlZCB0
aGF0IHRoZSBldmVudHMgaW4gc2FtZSBQTVUgaGFzIGRpZmZlcmVudCBDUFUNCj4gbWFzay4NCj4g
Pg0KPiA+IFNvIEkgdGhpbmsgaXQgc2hvdWxkIGJlIGJldHRlciB0byB1c2UgZGlmZmVyZW50IFBN
VXMgZm9yIFJBUEwgYW5kDQo+IHJlc2lkZW5jeS4NCj4gPg0KPiA+IFdoYXQgZG8geW91IHRoaW5r
Pw0KPiA+DQo+IFdlbGwsIHlvdSBhcmUgbWF5YmUgY29uZnVzaW5nIGV2ZW50cyB3aXRoIFBNVS4g
SWYgeW91IGxvb2sgYXQgdGhlIGNvcmUNCj4gUE1VLCBpdCBjb3ZlciBtYW55IGV2ZW50cyBtZWFz
dXJpbmcgdmFzdGx5IGRpZmZlcmVudCBhc3BlY3RzIG9mIHRoZSBjb3JlLg0KPiBTb21lIGV2ZW50
cyBhcmUgcGVyLXRocmVhZCwgb3RoZXJzIGFyZSBwZXItY29yZS4NCj4gDQo+IEhlcmUsIEkgd2Fz
IHRoaW5raW5nIGl0IHdvdWxkIGJlIGdvb2QgdG8gaGF2ZSBzb21lIHBvd2VyLy8gUE1VIHdpdGgN
Cj4gbWFueSBldmVudHMgY292ZXJpbmcgY3N0YXRlIHJlc2lkZW5jeSwgZW5lcmd5IGNvbnN1bXB0
aW9uLiBBbmQgeWVzLA0KPiBzb21lIGV2ZW50cyB3b3VsZCBiZSBwZXItc29ja2V0LCBvdGhlcnMg
cGVyLWNvcmUuDQoNClNvIHlvdSBhZ3JlZSB0byBjcmVhdGUgdHdvIG5ldyBjc3RhdGUgUE1VcyAo
cGVyLWNvcmUgYW5kIHBlci1zb2NrZXQpIHRvDQpjb3ZlciBjc3RhdGUgcmVzaWRlbmN5Pw0KSWYg
c28sIEkgd2lsbCBzdGFydCB0byBpbXBsZW1lbnQgdGhlIFYzIHZlcnNpb24gZm9yIHR3byBuZXcg
UE1Vcy4gDQoNClRoYW5rcywNCkthbg0KDQoNCg==
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1215392

DQoNCj4gT24gRnJpLCBBdWcgMjgsIDIwMTUgYXQgODowMCBBTSwgTGlhbmcsIEthbiA8a2FuLmxp
YW5nQGludGVsLmNvbT4gd3JvdGU6DQo+ID4NCj4gPg0KPiA+PiBPbiBUdWUsIEF1ZyAyNSwgMjAx
NSBhdCAxOjE1IFBNLCBMaWFuZywgS2FuIDxrYW4ubGlhbmdAaW50ZWwuY29tPg0KPiB3cm90ZToN
Cj4gPj4gPg0KPiA+PiA+PiA+PiA+DQo+ID4+ID4+ID4+IEkgdW5kZXJzdGFuZCB0aGF0IHRoZXNl
IG1ldHJpY3MgYXJlIHVzZWZ1bCBhbmQgbmVlZGVkIGhvd2V2ZXINCj4gPj4gPj4gPj4gaWYgSSBs
b29rIGF0IHRoZSBicm9hZGVyIHBpY3R1cmUgSSBzZWUgbWFueSBQTVVzIGRvaW5nIHNpbWlsYXIN
Cj4gPj4gPj4gPj4gdGhpbmdzIG9yIGFwcGVhcmluZyBkaWZmZXJlbnQgd2hlbiB0aGV5IGFyZSBh
Y3R1YWxseSB2ZXJ5DQo+ID4+ID4+ID4+IGNsb3NlLiBJdCB3b3VsZCBiZSBuaWNlIHRvIGhhdmUg
YSBtb3JlIHVuaWZpZWQgYXBwcm9hY2guIFlvdQ0KPiA+PiA+PiA+PiBoYXZlIFJBUEwgKGNsaWVu
dCwNCj4gPj4gPj4gPj4gc2VydmVyKSB3aGljaCBhcHBlYXJzIGFzIHRoZSBwb3dlciBQTVUuIFlv
dSBoYXZlIHRoZSBQQ1UNCj4gdW5jb3JlDQo+ID4+IG9uDQo+ID4+ID4+ID4+IHNlcnZlcnMgd2hp
Y2ggYWxzbyBwcm92aWRlcyBDLXN0YXRlIHJlc2lkZW5jeSBpbmZvLiBZZXQsIGFsbA0KPiA+PiA+
PiA+PiB0aGVzZSBhcHBlYXIgZGlmZmVyZW50bHkgYW5kIGV4cG9zZSBldmVudHMgd2l0aCBkaWZm
ZXJlbnQgbmFtZXMuDQo+ID4+ID4+ID4+IEkgdGhpbmsgd2UgY291bGQgYmVuZWZpdCBmcm9tIGEg
bW9yZSB1bmlmaWUgYXBwcm9hY2ggaGVyZSBzdWNoDQo+ID4+ID4+ID4+IHRoYXQgeW91IHdvdWxk
IGJlIGFibGUgdG8gZG8NCj4gPj4gPj4gPj4NCj4gPj4gPj4gPj4gJCAgICBwZXJmIHN0YXQgLWEg
LWUgcG93ZXIvYzYtcmVzaWRlbmN5LywgcG93ZXIvZW5lcmd5LXBrZy8NCj4gPj4gPj4gPj4NCj4g
Pj4gPj4gPj4gb24gY2xpZW50IGFuZCBzZXJ2ZXIgd2l0aG91dCBoYXZpbmcgdG8gY2hhbmdlIHRo
ZSBwbXUgbmFtZSBvZg0KPiA+PiA+PiA+PiB0aGUgZXZlbnQgbmFtZXMuDQo+ID4+ID4+ID4NCj4g
Pj4gPj4gPiBZZXMsIEkgYWdyZWUuIEknbGwgdGhpbmsgYWJvdXQgaXQuDQo+ID4+ID4+ID4NCj4g
Pj4gPg0KPiA+PiA+IEhpIFN0ZXBoYW5lLA0KPiA+PiA+DQo+ID4+ID4gSSB0aG91Z2h0IG1vcmUg
YWJvdXQgeW91ciBzdWdnZXN0aW9uIHJlZ2FyZGluZyB0byBjcmVhdGUgYSB1bmlmaWVkDQo+ID4+
ID4gcG93ZXIgUE1VIGZvciBhbGwgcmVsYXRlZCBldmVudHMgaW5jbHVkZSBSQVBMIGFuZCByZXNp
ZGVuY3kuDQo+ID4+ID4gSXQgbG9va3Mgd2UgY2FuIGJlbmVmaXQgZnJvbSBhIHNpbXBsZSB1bmlm
aWVkIG5hbWUsIGJ1dCBpdCBhbHNvDQo+ID4+ID4gYnJpbmdzIHRvbyBtdWNoIGNvbmZ1c2lvbi4N
Cj4gPj4gPiAgIC0gY3N0YXRlIHJlc2lkZW5jeSBpcyB0aGUgdGltZSBvZiB0aGUgY29yZS9zb2Nr
ZXQgaW4gc3BlY2lmaWMgY3N0YXRlLg0KPiA+PiA+ICAgICAgV2hpbGUgUkFQTCBldmVudCBpcyB0
aGUgcG93ZXIgY29yZS9zb2NrZXQgd2hpY2ggY29uc3VtZWQuDQo+ID4+ID4gICAgICBUaGV5IGhh
dmUgZGlmZmVyZW50IGNvbmNlcHRzLg0KPiA+PiA+ICAgLSBjc3RhdGUgcmVzaWRlbmN5IGluY2x1
ZGVzIGJvdGggcGVyLWNvcmUgYW5kIHBlci1zb2NrZXQgZXZlbnRzLg0KPiA+PiA+ICAgICAgUkFQ
TCBldmVudHMgaXMgb25seSBwZXItc29ja2V0LiBTbyB0aGUgQ1BVIG1hc2sgaXMgZGlmZmVyZW50
Lg0KPiA+PiA+ICAgICAgSXQncyB2ZXJ5IGNvbmZ1c2VkIHRoYXQgdGhlIGV2ZW50cyBpbiBzYW1l
IFBNVSBoYXMgZGlmZmVyZW50DQo+ID4+ID4gQ1BVDQo+ID4+IG1hc2suDQo+ID4+ID4NCj4gPj4g
PiBTbyBJIHRoaW5rIGl0IHNob3VsZCBiZSBiZXR0ZXIgdG8gdXNlIGRpZmZlcmVudCBQTVVzIGZv
ciBSQVBMIGFuZA0KPiA+PiByZXNpZGVuY3kuDQo+ID4+ID4NCj4gPj4gPiBXaGF0IGRvIHlvdSB0
aGluaz8NCj4gPj4gPg0KPiA+PiBXZWxsLCB5b3UgYXJlIG1heWJlIGNvbmZ1c2luZyBldmVudHMg
d2l0aCBQTVUuIElmIHlvdSBsb29rIGF0IHRoZQ0KPiA+PiBjb3JlIFBNVSwgaXQgY292ZXIgbWFu
eSBldmVudHMgbWVhc3VyaW5nIHZhc3RseSBkaWZmZXJlbnQgYXNwZWN0cyBvZg0KPiB0aGUgY29y
ZS4NCj4gPj4gU29tZSBldmVudHMgYXJlIHBlci10aHJlYWQsIG90aGVycyBhcmUgcGVyLWNvcmUu
DQo+ID4+DQo+ID4+IEhlcmUsIEkgd2FzIHRoaW5raW5nIGl0IHdvdWxkIGJlIGdvb2QgdG8gaGF2
ZSBzb21lIHBvd2VyLy8gUE1VIHdpdGgNCj4gPj4gbWFueSBldmVudHMgY292ZXJpbmcgY3N0YXRl
IHJlc2lkZW5jeSwgZW5lcmd5IGNvbnN1bXB0aW9uLiBBbmQgeWVzLA0KPiA+PiBzb21lIGV2ZW50
cyB3b3VsZCBiZSBwZXItc29ja2V0LCBvdGhlcnMgcGVyLWNvcmUuDQo+ID4NCj4gPiBTbyB5b3Ug
YWdyZWUgdG8gY3JlYXRlIHR3byBuZXcgY3N0YXRlIFBNVXMgKHBlci1jb3JlIGFuZCBwZXItc29j
a2V0KQ0KPiA+IHRvIGNvdmVyIGNzdGF0ZSByZXNpZGVuY3k/DQo+ID4gSWYgc28sIEkgd2lsbCBz
dGFydCB0byBpbXBsZW1lbnQgdGhlIFYzIHZlcnNpb24gZm9yIHR3byBuZXcgUE1Vcy4NCj4gPg0K
PiBJIGRpZCBub3Qgc2F5IHRoYXQuIEluc3RlYWQgSSBzYWlkIHRoZXJlIGlzIHNvbWUgYmVuZWZp
dHMgaW4gaGF2aW5nIGV2ZXJ5dGhpbmcNCj4gdW5kZXIgYSBwb3dlci8vIFBNVSwgaW5jbHVkaW5n
IHBvc3NpYmx5IHBvcnRhYmlsaXR5IHRvIG90aGVyIG5vbiB4ODYNCj4gYXJjaGl0ZWN0dXJlcy4N
Cg0KQnV0IHRoZSBldmVudHMgaW4gcG93ZXIvLyBQTVUgd2lsbCBiZSBwZXItc29ja2V0IG9yIHBl
ci1jb3JlLg0KSG93IHNob3VsZCB3ZSBoYW5kbGUgaXQ/DQpXaGF0IHNob3VsZCB3ZSBzaG93IGlu
IGNwdW1hc2s/DQoNCg0K
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1215398

On Fri, Aug 28, 2015 at 8:00 AM, Liang, Kan <kan.liang@intel.com> wrote:
>
>
>> On Tue, Aug 25, 2015 at 1:15 PM, Liang, Kan <kan.liang@intel.com> wrote:
>> >
>> >> >> >
>> >> >> I understand that these metrics are useful and needed however if I
>> >> >> look at the broader picture I see many PMUs doing similar things
>> >> >> or appearing different when they are actually very close. It would
>> >> >> be nice to have a more unified approach. You have RAPL (client,
>> >> >> server) which appears as the power PMU. You have the PCU uncore
>> on
>> >> >> servers which also provides C-state residency info. Yet, all these
>> >> >> appear differently and expose events with different names.
>> >> >> I think we could benefit from a more unifie approach here such
>> >> >> that you would be able to do
>> >> >>
>> >> >> $    perf stat -a -e power/c6-residency/, power/energy-pkg/
>> >> >>
>> >> >> on client and server without having to change the pmu name of the
>> >> >> event names.
>> >> >
>> >> > Yes, I agree. I'll think about it.
>> >> >
>> >
>> > Hi Stephane,
>> >
>> > I thought more about your suggestion regarding to create a unified
>> > power PMU for all related events include RAPL and residency.
>> > It looks we can benefit from a simple unified name, but it also brings
>> > too much confusion.
>> >   - cstate residency is the time of the core/socket in specific cstate.
>> >      While RAPL event is the power core/socket which consumed.
>> >      They have different concepts.
>> >   - cstate residency includes both per-core and per-socket events.
>> >      RAPL events is only per-socket. So the CPU mask is different.
>> >      It's very confused that the events in same PMU has different CPU
>> mask.
>> >
>> > So I think it should be better to use different PMUs for RAPL and
>> residency.
>> >
>> > What do you think?
>> >
>> Well, you are maybe confusing events with PMU. If you look at the core
>> PMU, it cover many events measuring vastly different aspects of the core.
>> Some events are per-thread, others are per-core.
>>
>> Here, I was thinking it would be good to have some power// PMU with
>> many events covering cstate residency, energy consumption. And yes,
>> some events would be per-socket, others per-core.
>
> So you agree to create two new cstate PMUs (per-core and per-socket) to
> cover cstate residency?
> If so, I will start to implement the V3 version for two new PMUs.
>
I did not say that. Instead I said there is some benefits in having everything
under a power// PMU, including possibly portability to other non x86
architectures.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web