From: James Morse <james.morse@arm.com>
To: Peter Newman <peternewman@google.com>,
Reinette Chatre <reinette.chatre@intel.com>
Cc: Tony Luck <tony.luck@intel.com>,
"Yu, Fenghua" <fenghua.yu@intel.com>,
"Eranian, Stephane" <eranian@google.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
Babu Moger <Babu.Moger@amd.com>,
Gaurang Upasani <gupasani@google.com>
Subject: Re: [RFD] resctrl: reassigning a running container's CTRL_MON group
Date: Tue, 25 Oct 2022 16:56:13 +0100 [thread overview]
Message-ID: <c1ed6c5d-c3fa-fa76-686a-033fe803a74d@arm.com> (raw)
In-Reply-To: <CALPaoChgi9Z8J7sca_YRHveBW9NzB=PPC1Yp01S=TGiJLBxkQw@mail.gmail.com>
Hi Peter,
On 21/10/2022 11:09, Peter Newman wrote:
> On Thu, Oct 20, 2022 at 9:08 PM Reinette Chatre
> <reinette.chatre@intel.com> wrote:
>>
>> If the expectation is that PARTID counts are very high then how about
>> a solution where multiple PARTIDs are associated with the same CTRL_MON group?
>> A CTRL_MON group presents a resource allocation to user space, CLOSIDs/PARTIDs
>> are not exposed. So using multiple PARTIDs for a resource group (all with the
>> same allocation) seems conceptually ok to me. (Please note, I did not do an
>> audit to see if there are any hidden assumption or look into lifting required
>> to support his.)
> I did propose using PARTIDs to back additional mon_groups a few days ago
> on the other sub-thread with James. My understanding was that it would
> be less trouble if the user opted to do this on their own rather than
> the kernel somehow doing this automatically.
>
> https://lore.kernel.org/all/835d769b-3662-7be5-dcdd-804cb1f3999a@arm.com/
> So perhaps we can just arrive at some way to inform the user of the
> difference in resources. We may not even need to be able to precisely
> calculate the number of groups we can create, as the logic for us could
> be a simple as:
>
> 1) If num_closids >= desired job count, just use CTRL_MON groups
> 2) Otherwise, fall back to the proposed mon_group-move approach if
> num_rmids is large enough for the desired job count
> To address the glitchy behavior of moving a PMG to a new PARTID, I found
> that the MPAM spec says explicitly that a PMG is subordinate to a
> PARTID, so I would be fine with James finding a way for the MPAM driver
> to block the rename operation, because it's unable to mix and match
> RMIDs and CLOSIDs the way that RDT can.
I'd like to support moving groups of tasks in a sensible way on MPAM too.
I don't think we should conflate it with 'old counters keep counting' - that should be
exposed as a separate property that influences how user-space sets this stuff up.
Thanks,
James
next prev parent reply other threads:[~2022-10-25 15:56 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-07 10:39 [RFD] resctrl: reassigning a running container's CTRL_MON group Peter Newman
2022-10-07 15:36 ` Reinette Chatre
2022-10-07 15:44 ` Yu, Fenghua
2022-10-07 17:28 ` Tony Luck
2022-10-10 23:35 ` Reinette Chatre
2022-10-12 11:21 ` Peter Newman
2022-10-12 16:55 ` James Morse
2022-10-17 10:15 ` Peter Newman
2022-10-19 13:57 ` James Morse
2022-10-20 10:39 ` Peter Newman
2022-10-21 12:42 ` Peter Newman
2022-10-25 15:55 ` James Morse
2022-10-26 8:52 ` Peter Newman
2022-10-26 21:12 ` Reinette Chatre
2022-10-27 7:56 ` Peter Newman
2022-10-27 17:35 ` Reinette Chatre
2022-11-01 15:23 ` Peter Newman
2022-11-01 15:53 ` Peter Newman
2022-11-01 16:48 ` Reinette Chatre
2022-10-25 15:56 ` James Morse
2022-10-21 20:09 ` Reinette Chatre
2022-10-21 20:22 ` Luck, Tony
2022-10-21 21:34 ` Reinette Chatre
2022-11-03 17:06 ` James Morse
2022-11-08 21:28 ` Reinette Chatre
2022-11-08 21:56 ` Luck, Tony
2022-11-08 23:18 ` Reinette Chatre
2022-11-09 17:58 ` James Morse
2022-11-09 9:50 ` Peter Newman
2022-11-09 19:11 ` Reinette Chatre
2022-11-11 18:38 ` James Morse
2022-11-14 18:02 ` Reinette Chatre
2022-11-16 13:20 ` Peter Newman
2022-11-09 17:59 ` James Morse
2022-11-09 19:12 ` Reinette Chatre
2022-11-11 18:36 ` James Morse
2022-10-12 16:57 ` Yu, Fenghua
2022-10-12 17:23 ` Reinette Chatre
2022-10-14 12:56 ` James Morse
2022-10-19 9:08 ` Peter Newman
2022-10-19 13:20 ` James Morse
2022-10-19 23:54 ` Reinette Chatre
2022-10-20 8:48 ` Peter Newman
2022-10-20 19:08 ` Reinette Chatre
2022-10-21 10:09 ` Peter Newman
2022-10-25 15:56 ` James Morse [this message]
2022-10-25 15:55 ` James Morse
2022-10-26 9:36 ` Peter Newman
2022-11-03 17:06 ` James Morse
2022-11-08 21:25 ` Reinette Chatre
2022-10-07 17:57 ` Moger, Babu
2022-10-11 15:00 ` Stephane Eranian
2022-10-11 14:59 ` Stephane Eranian
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c1ed6c5d-c3fa-fa76-686a-033fe803a74d@arm.com \
--to=james.morse@arm.com \
--cc=Babu.Moger@amd.com \
--cc=eranian@google.com \
--cc=fenghua.yu@intel.com \
--cc=gupasani@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=peternewman@google.com \
--cc=reinette.chatre@intel.com \
--cc=tglx@linutronix.de \
--cc=tony.luck@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox