From: Reinette Chatre <reinette.chatre@intel.com>
To: Punit Agrawal <punit.agrawal@oss.qualcomm.com>
Cc: James Morse <james.morse@arm.com>,
Ben Horgan <ben.horgan@arm.com>, <amitsinght@marvell.com>,
<baisheng.gao@unisoc.com>, <baolin.wang@linux.alibaba.com>,
<bobo.shaobowang@huawei.com>, <carl@os.amperecomputing.com>,
<catalin.marinas@arm.com>, <dakr@kernel.org>,
<dave.martin@arm.com>, <david@redhat.com>,
<dfustini@baylibre.com>, <fenghuay@nvidia.com>,
<gregkh@linuxfoundation.org>, <gshan@redhat.com>,
<guohanjun@huawei.com>, <jeremy.linton@arm.com>,
<jonathan.cameron@huawei.com>, <kobak@nvidia.com>,
<lcherian@marvell.com>, <lenb@kernel.org>,
<linux-acpi@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <lpieralisi@kernel.org>,
<peternewman@google.com>, <quic_jiles@quicinc.com>,
<rafael@kernel.org>, <robh@kernel.org>, <rohit.mathew@arm.com>,
<scott@os.amperecomputing.com>, <sdonthineni@nvidia.com>,
<sudeep.holla@arm.com>, <tan.shaopeng@fujitsu.com>,
<will@kernel.org>, <xhao@linux.alibaba.com>
Subject: Re: [PATCH v6 00/34] arm_mpam: Add basic mpam driver
Date: Mon, 8 Dec 2025 10:21:29 -0800 [thread overview]
Message-ID: <f5095a4c-4617-441f-95af-9255cefdcf98@intel.com> (raw)
In-Reply-To: <87tsy0kgwy.fsf@stealth>
Hi Punit,
On 12/8/25 9:43 AM, Punit Agrawal wrote:
> Hi Reinette,
>
> Reinette Chatre <reinette.chatre@intel.com> writes:
>
>> Hi Punit,
>>
>> On 12/5/25 5:08 AM, Punit Agrawal wrote:
>>> James Morse <james.morse@arm.com> writes:
>>>> On 03/12/2025 11:21, Punit Agrawal wrote:
>>>>> Ben Horgan <ben.horgan@arm.com> writes:
>>>>>> On 11/24/25 15:21, Punit Agrawal wrote:
>>
>>>>>>> The platform has MSCs attached
>>>>>>> to shared L2 caches which are being skipped during later stages of
>>>>>>> initialisation. IIUC, the L2 MSCs' limitations stems from the
>>>>>>> assumptions in the resctrl interface.
>>>>>>
>>>>>> What in particualar is being skipped?
>>>>
>>>>> The registration of the discovered MSCs with resctrl and subsequent
>>>>> exposing it to the user.
>>>>
>>>> resctrl's 'L2' support is limited to the CPOR bitmap.
>>>> If you have controls, there is no resctrl 'event' that can exposed them.
>>>> (the problem being they all have 'L3' in the name!)
>>>
>>>>>>> I was wondering if there are any patches available to relax these
>>>>>>> limitations?
>>>
>>>> Knowing which property it is will help - but some of these things are checked
>>>> to match resctrl's ABI. They can't necessarily be relaxed without breaking
>>>> user-space.
>>>
>>> This platform has portion, capacity and priority partitioning, as well
>>> as memory bandwidth and cache storage monitoring. The MPAM code seems to
>>> correctly parse the properties.
>>>
>>> But as you point out, the resctrl 'L2' support doesn't have anything
>>> other than CPOR bitmap yet. Have you looked at what's needed to extend
>>> resctrl to support some of the others?
>>
>> Please see [1] for a summary of ideas on how resctrl can support multiple
>> controls. Nothing is final, please do add your ideas and requirements.
>>
>> There are a couple of opens not included in the summary. If interested
>> [2] is where a lot of discussion happened that prompted the need for
>> a summary.
>
> Thanks a lot for the pointers.
>
> Let me catch up with what's already discussed and raise any points that
> are not clear / need refining.
Please do. There are already a few known open issues. I plan to raise awareness
of these discussions (and the (so far known) sticky points still left to resolve) during
the resctrl session at LPC:
https://lpc.events/event/19/contributions/2093/
I am attending LPC remotely though so I hope increased awareness will result in more traffic
to [1] to help us get this right.
Reinette
>
>>
>> Reinette
>>
>> [1] https://lore.kernel.org/lkml/aPtfMFfLV1l%2FRB0L@e133380.arm.com/
>> [2] https://lore.kernel.org/lkml/aNFliMZTTUiXyZzd@e133380.arm.com/
prev parent reply other threads:[~2025-12-08 18:21 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-19 12:22 [PATCH v6 00/34] arm_mpam: Add basic mpam driver Ben Horgan
2025-11-19 12:22 ` [PATCH v6 01/34] ACPI / PPTT: Add a helper to fill a cpumask from a processor container Ben Horgan
2025-11-19 17:08 ` Jeremy Linton
2025-11-19 12:22 ` [PATCH v6 02/34] ACPI / PPTT: Stop acpi_count_levels() expecting callers to clear levels Ben Horgan
2025-11-19 17:08 ` Jeremy Linton
2025-11-19 12:22 ` [PATCH v6 03/34] ACPI / PPTT: Add acpi_pptt_cache_v1_full to use pptt cache as one structure Ben Horgan
2025-11-19 17:08 ` Jeremy Linton
2025-11-19 12:22 ` [PATCH v6 04/34] ACPI / PPTT: Find cache level by cache-id Ben Horgan
2025-11-19 17:09 ` Jeremy Linton
2025-11-19 12:22 ` [PATCH v6 05/34] ACPI / PPTT: Add a helper to fill a cpumask from a cache_id Ben Horgan
2025-11-19 17:09 ` Jeremy Linton
2025-11-19 12:22 ` [PATCH v6 06/34] arm64: kconfig: Add Kconfig entry for MPAM Ben Horgan
2025-11-19 12:22 ` [PATCH v6 07/34] platform: Define platform_device_put cleanup handler Ben Horgan
2025-11-19 12:22 ` [PATCH v6 08/34] ACPI: Define acpi_put_table cleanup handler and acpi_get_table_pointer() helper Ben Horgan
2025-11-19 12:22 ` [PATCH v6 09/34] ACPI / MPAM: Parse the MPAM table Ben Horgan
2025-11-19 12:22 ` [PATCH v6 10/34] arm_mpam: Add probe/remove for mpam msc driver and kbuild boiler plate Ben Horgan
2025-11-19 12:22 ` [PATCH v6 11/34] arm_mpam: Add the class and component structures for firmware described ris Ben Horgan
2025-11-19 12:22 ` [PATCH v6 12/34] arm_mpam: Add MPAM MSC register layout definitions Ben Horgan
2025-11-19 12:22 ` [PATCH v6 13/34] arm_mpam: Add cpuhp callbacks to probe MSC hardware Ben Horgan
2025-11-19 12:22 ` [PATCH v6 14/34] arm_mpam: Probe hardware to find the supported partid/pmg values Ben Horgan
2025-11-19 12:22 ` [PATCH v6 15/34] arm_mpam: Add helpers for managing the locking around the mon_sel registers Ben Horgan
2025-11-19 12:22 ` [PATCH v6 16/34] arm_mpam: Probe the hardware features resctrl supports Ben Horgan
2025-11-19 12:22 ` [PATCH v6 17/34] arm_mpam: Merge supported features during mpam_enable() into mpam_class Ben Horgan
2025-11-19 12:22 ` [PATCH v6 18/34] arm_mpam: Reset MSC controls from cpuhp callbacks Ben Horgan
2025-11-19 12:22 ` [PATCH v6 19/34] arm_mpam: Add a helper to touch an MSC from any CPU Ben Horgan
2025-11-19 12:22 ` [PATCH v6 20/34] arm_mpam: Extend reset logic to allow devices to be reset any time Ben Horgan
2025-11-19 12:22 ` [PATCH v6 21/34] arm_mpam: Register and enable IRQs Ben Horgan
2025-11-19 12:22 ` [PATCH v6 22/34] arm_mpam: Use a static key to indicate when mpam is enabled Ben Horgan
2025-11-19 12:22 ` [PATCH v6 23/34] arm_mpam: Allow configuration to be applied and restored during cpu online Ben Horgan
2025-11-19 12:22 ` [PATCH v6 24/34] arm_mpam: Probe and reset the rest of the features Ben Horgan
2025-11-19 12:22 ` [PATCH v6 25/34] arm_mpam: Add helpers to allocate monitors Ben Horgan
2025-11-19 12:22 ` [PATCH v6 26/34] arm_mpam: Add mpam_msmon_read() to read monitor value Ben Horgan
2025-11-19 12:22 ` [PATCH v6 27/34] arm_mpam: Track bandwidth counter state for power management Ben Horgan
2025-11-19 12:22 ` [PATCH v6 28/34] arm_mpam: Consider overflow in bandwidth counter state Ben Horgan
2025-11-19 12:22 ` [PATCH v6 29/34] arm_mpam: Probe for long/lwd mbwu counters Ben Horgan
2025-11-19 12:23 ` [PATCH v6 30/34] arm_mpam: Use long MBWU counters if supported Ben Horgan
2025-11-19 12:23 ` [PATCH v6 31/34] arm_mpam: Add helper to reset saved mbwu state Ben Horgan
2025-11-19 12:23 ` [PATCH v6 32/34] arm_mpam: Add kunit test for bitmap reset Ben Horgan
2025-11-19 12:23 ` [PATCH v6 33/34] arm_mpam: Add kunit tests for props_mismatch() Ben Horgan
2025-11-19 12:23 ` [PATCH v6 34/34] MAINTAINERS: new entry for MPAM Driver Ben Horgan
2025-11-19 19:14 ` [PATCH v6 00/34] arm_mpam: Add basic mpam driver Catalin Marinas
2025-11-24 15:21 ` Punit Agrawal
2025-11-27 14:25 ` Ben Horgan
2025-12-03 11:21 ` Punit Agrawal
2025-12-03 17:34 ` James Morse
2025-12-05 13:08 ` Punit Agrawal
2025-12-05 19:11 ` Reinette Chatre
2025-12-08 17:43 ` Punit Agrawal
2025-12-08 18:21 ` Reinette Chatre [this message]
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=f5095a4c-4617-441f-95af-9255cefdcf98@intel.com \
--to=reinette.chatre@intel.com \
--cc=amitsinght@marvell.com \
--cc=baisheng.gao@unisoc.com \
--cc=baolin.wang@linux.alibaba.com \
--cc=ben.horgan@arm.com \
--cc=bobo.shaobowang@huawei.com \
--cc=carl@os.amperecomputing.com \
--cc=catalin.marinas@arm.com \
--cc=dakr@kernel.org \
--cc=dave.martin@arm.com \
--cc=david@redhat.com \
--cc=dfustini@baylibre.com \
--cc=fenghuay@nvidia.com \
--cc=gregkh@linuxfoundation.org \
--cc=gshan@redhat.com \
--cc=guohanjun@huawei.com \
--cc=james.morse@arm.com \
--cc=jeremy.linton@arm.com \
--cc=jonathan.cameron@huawei.com \
--cc=kobak@nvidia.com \
--cc=lcherian@marvell.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=peternewman@google.com \
--cc=punit.agrawal@oss.qualcomm.com \
--cc=quic_jiles@quicinc.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=rohit.mathew@arm.com \
--cc=scott@os.amperecomputing.com \
--cc=sdonthineni@nvidia.com \
--cc=sudeep.holla@arm.com \
--cc=tan.shaopeng@fujitsu.com \
--cc=will@kernel.org \
--cc=xhao@linux.alibaba.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;
as well as URLs for NNTP newsgroup(s).