[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v7 04/14] xen/arm: add Dom0 cache coloring support


  • To: Carlo Nonato <carlo.nonato@xxxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, "Volodymyr Babchuk" <volodymyr_babchuk@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • From: Michal Orzel <michal.orzel@xxxxxxx>
  • Date: Wed, 27 Mar 2024 12:57:32 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=minervasys.tech smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=glkmBnGMe+O51lp+AmzEypqT5ZsbGA5Usg9YV4dFEks=; b=kSl1ts2Nj5HPmmQqEDSxJ9cRak8Ul2K1gWfR31aiCrg306MC14X+9wsSAJiVd8X4nnw3dohKRTBTXz1BoCXqixgkzsS5QYfPMhWjlCJI9opN3ozova47AdVT51MsvSNOoFP2pZeLPJlwFqDbrLY9SJqzVOBZrv4gmFxa88HwF1PkEToPIqUtLqxkOiUhdm2mYFcMUYC59wUh076dXnP7ubtE+Y/ybbc2mumuHqwGPfB8ekEZoIoTABRc7TOmPGLz+ze+D2c02rrii38RMM+NykKs+93NGqekmz3kuiRalmBvjaCj7oIb6NV1tiroF72x5k+nfutfQQARZ040U/UqEg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=AsILYJBxk0YUgwnLWZ7tQ7v5cAwt3uvC15Z0hcj5aTXqdhlLHcfATs0CxdveqDoddXD32GgiN5Xh9PWpewzOojlNafRFYNSGgAl1qL6vrnF9XpDk7pMNAuD87wG96DOv8T2vM0WCdlorC1CPTcRH2rV/O4kjhjyDxBqSsrvEvihHPvRgTRZa4LHnpyK3tU1VmD+FjdgbhSY4qIi3/xecRitR4pNOWgxtrk/nh50VJmRWujrrp3my8Lg6VaJ26Itg/AzU4YSUs/RvgC0r0uHKGV5DRdsYQ9vvMJO/z1ODwABzN8h0MfZ0YA6OylHPE0U7xOoEYJvG/dGLMRy9JuUPog==
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, "Wei Liu" <wl@xxxxxxx>, Marco Solieri <marco.solieri@xxxxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrea Bastoni <andrea.bastoni@xxxxxxxxxxxxxxx>
  • Delivery-date: Wed, 27 Mar 2024 11:57:45 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>


On 27/03/2024 12:39, Carlo Nonato wrote:
> 
> 
> Hi guys,
> 
> On Fri, Mar 22, 2024 at 8:26 AM Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>
>> On 21.03.2024 18:31, Carlo Nonato wrote:
>>> On Thu, Mar 21, 2024 at 4:57 PM Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>>>
>>>> On 21.03.2024 16:04, Carlo Nonato wrote:
>>>>> On Tue, Mar 19, 2024 at 4:30 PM Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>>>>> On 15.03.2024 11:58, Carlo Nonato wrote:
>>>>>>> --- a/docs/misc/xen-command-line.pandoc
>>>>>>> +++ b/docs/misc/xen-command-line.pandoc
>>>>>>> @@ -963,6 +963,15 @@ Controls for the dom0 IOMMU setup.
>>>>>>>
>>>>>>>  Specify a list of IO ports to be excluded from dom0 access.
>>>>>>>
>>>>>>> +### dom0-llc-colors
>>>>>>> +> `= List of [ <integer> | <integer>-<integer> ]`
>>>>>>> +
>>>>>>> +> Default: `All available LLC colors`
>>>>>>> +
>>>>>>> +Specify dom0 LLC color configuration. This option is available only 
>>>>>>> when
>>>>>>> +`CONFIG_LLC_COLORING` is enabled. If the parameter is not set, all 
>>>>>>> available
>>>>>>> +colors are used.
>>>>>>
>>>>>> My reservation towards this being a top-level option remains.
>>>>>
>>>>> How can I turn this into a lower-level option? Moving it into "dom0=" 
>>>>> doesn't
>>>>> seem possible to me. How can I express a list (llc-colors) inside another 
>>>>> list
>>>>> (dom0)? dom0=llc-colors=0-3,12-15,other-param=... How can I stop parsing
>>>>> before reaching other-param?
>>>>
>>>> For example by using a different separator:
>>>>
>>>> dom0=llc-colors=0-3+12-15,other-param=...
>>>
>>> Ok, but that would mean to change the implementation of the parsing function
>>> and to adopt this syntax also in other places, something that I would've
>>> preferred to avoid. Anyway I'll follow your suggestion.
>>
>> Well, this is all used by Arm only for now. You will want to make sure Arm
>> folks are actually okay with this alternative approach.
>>
>> Jan
> 
> Are you Arm maintainers ok with this?
I'm not a fan of this syntax and I find it more difficult to parse compared to 
the usual case, where
every option is clearly separated. That said, I won't oppose to it.

~Michal



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.