Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp3049868pxj; Mon, 14 Jun 2021 13:11:09 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzOF82iqpdiidjmbmISFlyKJh6u3aQz1W/zdof98QQdM7QfqhTC25X7ZDurnSyEbxsIZiPB X-Received: by 2002:aa7:d14d:: with SMTP id r13mr19348050edo.212.1623701468914; Mon, 14 Jun 2021 13:11:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1623701468; cv=none; d=google.com; s=arc-20160816; b=MgsiMT+On9bm8KEK/vi54N+yaEgtIaCDFi5PvODJggBqRvrhlhNyiODktkC6ncdmG5 v/fnDz8yJnPwsh9pCWoJG3noOjwCAi+gwF9BWUL4QNhp4nLT3ofIzmbLK3bkEH/zCQWq lK1LbkKBUN7fGs43RowiesWhx0LGmu0bOiQAnZqzoLf97Uf7Xkq+X1AvUv1eUjHgYwNX ecMf6DEpFB3xNAoAKLywKMSa2A5pPYjW+upnP9cr+OrZsanjBk/QuyJn4byd2UTWjCCQ 67SQtjDUAl1i0ImuXF1YxfpvG6mauiBr76omwf02odmZarhVo8yY031LHoUFAQxTJ51h kHcg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=wMlSDbP4RnsROvzxjcO3VaHCYDalLysEoTXPZ3g4GKY=; b=y7/0TEYxAid0U26gVpyzwf1hDNn0L9c6gYMr+b443IP31PPGk1TYxTH7f4+sSmh9i4 oNE4MGosArEY/JchuSZPjM6b2T6zaf3bie9kSpExK6fMGNnVwZ01OXzKe2JKpW/bYjTF k0/aGzTslDC0qvWnKzclhBGD8XvVt/CRH+8po4oyYV0BTt2FYll1EGWl7hRPhxpoeQ+T jyllXf+9aLTn4FSQI5QQPxw5AATyagiEMJjVXW9nqOPuV576qapby9QDqJ2MB+KmwPm0 QkZJJeOcXwXw19QpP6McIO7KntcjBAgZp16vtxUcqpZ2A4IaN8e8mvRZNZbFxffYBbds 5SLw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q8si12056203edd.329.2021.06.14.13.10.45; Mon, 14 Jun 2021 13:11:08 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233671AbhFNUL5 (ORCPT + 99 others); Mon, 14 Jun 2021 16:11:57 -0400 Received: from foss.arm.com ([217.140.110.172]:45372 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233352AbhFNUL4 (ORCPT ); Mon, 14 Jun 2021 16:11:56 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id D66AC11D4; Mon, 14 Jun 2021 13:09:52 -0700 (PDT) Received: from merodach.members.linode.com (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id A2F4A3F694; Mon, 14 Jun 2021 13:09:50 -0700 (PDT) From: James Morse To: x86@kernel.org, linux-kernel@vger.kernel.org Cc: Fenghua Yu , Reinette Chatre , Thomas Gleixner , Ingo Molnar , Borislav Petkov , H Peter Anvin , Babu Moger , James Morse , shameerali.kolothum.thodi@huawei.com, Jamie Iles , D Scott Phillips OS , lcherian@marvell.com Subject: [PATCH v4 00/24] x86/resctrl: Merge the CDP resources Date: Mon, 14 Jun 2021 20:09:17 +0000 Message-Id: <20210614200941.12383-1-james.morse@arm.com> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi folks, Changes since v3? Fixed the 'unsigned u32', remove a few spaces and fixed a few spelling mistakes. ---- This series re-folds the resctrl code so the CDP resources (L3CODE et al) behaviour is all contained in the filesystem parts, with a minimum amount of arch specific code. Arm have some CPU support for dividing caches into portions, and applying bandwidth limits at various points in the SoC. The collective term for these features is MPAM: Memory Partitioning and Monitoring. MPAM is similar enough to Intel RDT, that it should use the defacto linux interface: resctrl. This filesystem currently lives under arch/x86, and is tightly coupled to the architecture. Ultimately, my plan is to split the existing resctrl code up to have an arch<->fs abstraction, then move all the bits out to fs/resctrl. From there MPAM can be wired up. x86 might have two resources with cache controls, (L2 and L3) but has extra copies for CDP: L{2,3}{CODE,DATA}, which are marked as enabled if CDP is enabled for the corresponding cache. MPAM has an equivalent feature to CDP, but its a property of the CPU, not the cache. Resctrl needs to have x86's odd/even behaviour, as that its the ABI, but this isn't how the MPAM hardware works. It is entirely possible that an in-kernel user of MPAM would not be using CDP, whereas resctrl is. Pretending L3CODE and L3DATA are entirely separate resources is a neat trick, but doing this is specific to x86. Doing this leaves the arch code in control of various parts of the filesystem ABI: the resources names, and the way the schemata are parsed. Allowing this stuff to vary between architectures is bad for user space. This series collapses the CODE/DATA resources, moving all the user-visible resctrl ABI into what becomes the filesystem code. CDP becomes the type of configuration being applied to a cache. This is done by adding a struct resctrl_schema to the parts of resctrl that will move to fs. This holds the arch-code resource that is in use for this schema, along with other properties like the name, and whether the configuration being applied is CODE/DATA/BOTH. This lets us fold the extra resources out of the arch code so that they don't need to be duplicated if the equivalent feature to CDP is missing, or implemented in a different way. The first two patches split the resource and domain structs to have an arch specific 'hw' portion, and the rest that is visible to resctrl. Future series massage the resctrl code so there are no accesses to 'hw' structures in the parts of resctrl that will move to fs, providing helpers where necessary. This series adds temporary scaffolding, which it removes a few patches later. This is to allow things like the ctrlval arrays and resources to be merged separately, which should make is easier to bisect. These things are marked temporary, and should all be gone by the end of the series. This series is a little rough around the monitors, would a fake struct resctrl_schema for the monitors simplify things, or be a source of bugs? This series is based on v5.12-rc6, and can be retrieved from: git://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git mpam/resctrl_merge_cdp/v4 v3: https://lore.kernel.org/lkml/20210519162424.27654-1-james.morse@arm.com/ v2: https://lore.kernel.org/lkml/20210312175849.8327-1-james.morse@arm.com/ v1: https://lore.kernel.org/lkml/20201030161120.227225-1-james.morse@arm.com/ Parts were previously posted as an RFC here: https://lore.kernel.org/lkml/20200214182947.39194-1-james.morse@arm.com/ James Morse (24): x86/resctrl: Split struct rdt_resource x86/resctrl: Split struct rdt_domain x86/resctrl: Add a separate schema list for resctrl x86/resctrl: Pass the schema in info dir's private pointer x86/resctrl: Label the resources with their configuration type x86/resctrl: Walk the resctrl schema list instead of an arch list x86/resctrl: Store the effective num_closid in the schema x86/resctrl: Add resctrl_arch_get_num_closid() x86/resctrl: Pass the schema to resctrl filesystem functions x86/resctrl: Swizzle rdt_resource and resctrl_schema in pseudo_lock_region x86/resctrl: Move the schemata names into struct resctrl_schema x86/resctrl: Group staged configuration into a separate struct x86/resctrl: Allow different CODE/DATA configurations to be staged x86/resctrl: Rename update_domains() resctrl_arch_update_domains() x86/resctrl: Add a helper to read a closid's configuration x86/resctrl: Add a helper to read/set the CDP configuration x86/resctrl: Pass configuration type to resctrl_arch_get_config() x86/resctrl: Make ctrlval arrays the same size x86/resctrl: Apply offset correction when config is staged x86/resctrl: Calculate the index from the configuration type x86/resctrl: Merge the ctrl_val arrays x86/resctrl: Remove rdt_cdp_peer_get() x86/resctrl: Expand resctrl_arch_update_domains()'s msr_param range x86/resctrl: Merge the CDP resources arch/x86/kernel/cpu/resctrl/core.c | 269 +++++-------- arch/x86/kernel/cpu/resctrl/ctrlmondata.c | 164 +++++--- arch/x86/kernel/cpu/resctrl/internal.h | 234 ++++------- arch/x86/kernel/cpu/resctrl/monitor.c | 44 ++- arch/x86/kernel/cpu/resctrl/pseudo_lock.c | 12 +- arch/x86/kernel/cpu/resctrl/rdtgroup.c | 448 ++++++++++++---------- include/linux/resctrl.h | 181 +++++++++ 7 files changed, 758 insertions(+), 594 deletions(-) -- 2.30.2