linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
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/



      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).