Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-05 Thread Christian Horn
* Igor Mammedov さんが書きました: > On Tue, 5 Nov 2024 08:11:14 +0100 > Christian Horn wrote: > > > - For reading the metrics in the guest, I was tempted to suggest PCP with > > pmda-denki to cover RAPL, but it's right now just reading /sysfs, not > > MSR's. pmda-lmsensors for further sensors offere

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-05 Thread Christian Horn
Hi all, some thoughts: - I vote for making the metrics as much as possible in the guest available as on the host. Allows cascading, and having in-guest-monitoring working like on bare metal. - As result, really just plain vCPU consumption would be made available in the guest as rapl-core.

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-05 Thread Igor Mammedov
On Tue, 5 Nov 2024 08:11:14 +0100 Christian Horn wrote: > Hi all, > > some thoughts: > > - I vote for making the metrics as much as possible in the guest available > as on the host. Allows cascading, and having in-guest-monitoring working > like on bare metal. > - As result, really just pl

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-04 Thread Igor Mammedov
On Sat, 02 Nov 2024 10:32:17 +0100 "Anthony Harivel" wrote: > Hi Igor, > > Igor Mammedov, Nov 01, 2024 at 16:09: > > On Tue, 22 Oct 2024 16:16:36 +0200 > > "Anthony Harivel" wrote: > > > >> Daniel P. Berrangé, Oct 22, 2024 at 15:15: > >> > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mamm

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-04 Thread Igor Mammedov
On Tue, 22 Oct 2024 15:49:33 +0200 "Anthony Harivel" wrote: > Igor Mammedov, Oct 18, 2024 at 14:25: > > On Wed, 16 Oct 2024 14:56:39 +0200 > > "Anthony Harivel" wrote: > > > >> Hi Igor, > >> > >> Igor Mammedov, Oct 16, 2024 at 13:52: > >> > On Wed, 22 May 2024 17:34:49 +0200 > >> > Anthony

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-02 Thread Anthony Harivel
Hi Igor, Igor Mammedov, Nov 01, 2024 at 16:09: > On Tue, 22 Oct 2024 16:16:36 +0200 > "Anthony Harivel" wrote: > >> Daniel P. Berrangé, Oct 22, 2024 at 15:15: >> > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: >> >> On Fri, 18 Oct 2024 13:59:34 +0100 >> >> Daniel P. Berrangé

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-11-01 Thread Igor Mammedov
On Tue, 22 Oct 2024 16:16:36 +0200 "Anthony Harivel" wrote: > Daniel P. Berrangé, Oct 22, 2024 at 15:15: > > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: > >> On Fri, 18 Oct 2024 13:59:34 +0100 > >> Daniel P. Berrangé wrote: > >> > >> > On Fri, Oct 18, 2024 at 02:25:26PM +

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Igor Mammedov
On Tue, 22 Oct 2024 14:15:44 +0100 Daniel P. Berrangé wrote: > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: > > On Fri, 18 Oct 2024 13:59:34 +0100 > > Daniel P. Berrangé wrote: > > > > > On Fri, Oct 18, 2024 at 02:25:26PM +0200, Igor Mammedov wrote: > > > > On Wed, 16 Oct

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Anthony Harivel
Daniel P. Berrangé, Oct 22, 2024 at 16:29: > On Tue, Oct 22, 2024 at 04:16:36PM +0200, Anthony Harivel wrote: >> Daniel P. Berrangé, Oct 22, 2024 at 15:15: >> > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: >> >> On Fri, 18 Oct 2024 13:59:34 +0100 >> >> Daniel P. Berrangé wrote: >

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Daniel P . Berrangé
On Tue, Oct 22, 2024 at 04:16:36PM +0200, Anthony Harivel wrote: > Daniel P. Berrangé, Oct 22, 2024 at 15:15: > > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: > >> On Fri, 18 Oct 2024 13:59:34 +0100 > >> Daniel P. Berrangé wrote: > >> > >> > On Fri, Oct 18, 2024 at 02:25:26PM +0

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Anthony Harivel
Daniel P. Berrangé, Oct 22, 2024 at 15:15: > On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: >> On Fri, 18 Oct 2024 13:59:34 +0100 >> Daniel P. Berrangé wrote: >> >> > On Fri, Oct 18, 2024 at 02:25:26PM +0200, Igor Mammedov wrote: >> > > On Wed, 16 Oct 2024 14:56:39 +0200 >> > > "A

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Anthony Harivel
Igor Mammedov, Oct 18, 2024 at 14:25: > On Wed, 16 Oct 2024 14:56:39 +0200 > "Anthony Harivel" wrote: > >> Hi Igor, >> >> Igor Mammedov, Oct 16, 2024 at 13:52: >> > On Wed, 22 May 2024 17:34:49 +0200 >> > Anthony Harivel wrote: >> > >> >> Dear maintainers, >> >> >> >> First of all, thank you

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Daniel P . Berrangé
On Tue, Oct 22, 2024 at 02:46:15PM +0200, Igor Mammedov wrote: > On Fri, 18 Oct 2024 13:59:34 +0100 > Daniel P. Berrangé wrote: > > > On Fri, Oct 18, 2024 at 02:25:26PM +0200, Igor Mammedov wrote: > > > On Wed, 16 Oct 2024 14:56:39 +0200 > > > "Anthony Harivel" wrote: > [...] > > > > > > > Thi

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-22 Thread Igor Mammedov
On Fri, 18 Oct 2024 13:59:34 +0100 Daniel P. Berrangé wrote: > On Fri, Oct 18, 2024 at 02:25:26PM +0200, Igor Mammedov wrote: > > On Wed, 16 Oct 2024 14:56:39 +0200 > > "Anthony Harivel" wrote: [...] > > > > This also leads to a question, if we should account for > > not VCPU threads at all. L

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-18 Thread Daniel P . Berrangé
On Fri, Oct 18, 2024 at 02:25:26PM +0200, Igor Mammedov wrote: > On Wed, 16 Oct 2024 14:56:39 +0200 > "Anthony Harivel" wrote: > > > Hi Igor, > > > > Igor Mammedov, Oct 16, 2024 at 13:52: > > > On Wed, 22 May 2024 17:34:49 +0200 > > > Anthony Harivel wrote: > > > > > >> Dear maintainers, > >

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-18 Thread Igor Mammedov
On Wed, 16 Oct 2024 14:56:39 +0200 "Anthony Harivel" wrote: > Hi Igor, > > Igor Mammedov, Oct 16, 2024 at 13:52: > > On Wed, 22 May 2024 17:34:49 +0200 > > Anthony Harivel wrote: > > > >> Dear maintainers, > >> > >> First of all, thank you very much for your review of my patch > >> [1].

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-16 Thread Anthony Harivel
Hi Igor, Igor Mammedov, Oct 16, 2024 at 13:52: > On Wed, 22 May 2024 17:34:49 +0200 > Anthony Harivel wrote: > >> Dear maintainers, >> >> First of all, thank you very much for your review of my patch >> [1]. > > I've tried to play with this feature and have a few questions about it > Thanks f

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-10-16 Thread Igor Mammedov
On Wed, 22 May 2024 17:34:49 +0200 Anthony Harivel wrote: > Dear maintainers, > > First of all, thank you very much for your review of my patch > [1]. I've tried to play with this feature and have a few questions about it 1. trying to start with non accessible or not existent socket

Re: [PATCH v6 0/3] Add support for the RAPL MSRs series

2024-06-26 Thread Anthony Harivel
Just a gentle ping for the above patch series. Anthony Harivel, May 22, 2024 at 17:34: > Dear maintainers, > > First of all, thank you very much for your review of my patch > [1]. > > In this version (v6), I have attempted to address all the problems > addressed by Daniel and Paolo during th

[PATCH v6 0/3] Add support for the RAPL MSRs series

2024-05-22 Thread Anthony Harivel
Dear maintainers, First of all, thank you very much for your review of my patch [1]. In this version (v6), I have attempted to address all the problems addressed by Daniel and Paolo during the last review. However, two open questions remains unanswered that would require the attention of a