Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <479ea81d-024a-5a7e-f045-4de52e97a599@intel.com>
Date:   Thu, 15 Jun 2023 16:28:40 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.11.2
Subject: Re: [PATCH v4 00/24] x86/resctrl: monitored closid+rmid together,
 separate arch/fs locking
Content-Language: en-US
To:     James Morse <james.morse@arm.com>, <x86@kernel.org>,
        <linux-kernel@vger.kernel.org>
CC:     Fenghua Yu <fenghua.yu@intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        H Peter Anvin <hpa@zytor.com>,
        Babu Moger <Babu.Moger@amd.com>,
        <shameerali.kolothum.thodi@huawei.com>,
        D Scott Phillips OS <scott@os.amperecomputing.com>,
        <carl@os.amperecomputing.com>, <lcherian@marvell.com>,
        <bobo.shaobowang@huawei.com>, <tan.shaopeng@fujitsu.com>,
        <xingxin.hx@openanolis.org>, <baolin.wang@linux.alibaba.com>,
        Jamie Iles <quic_jiles@quicinc.com>,
        Xin Hao <xhao@linux.alibaba.com>, <peternewman@google.com>,
        <dfustini@baylibre.com>
References: <20230525180209.19497-1-james.morse@arm.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20230525180209.19497-1-james.morse@arm.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SDR5Kzh5Qk9SZWhJOGJQYnZFMjRlRlZLVitZY3pmWXJPblExQmpIZlB0cU9l?=
 =?utf-8?B?Q05HRFUxVVRyeDNoajN6TTk1ZzNHR3V5eFBxQStlMGxuNHFWZWNCMGZlbTY2?=
 =?utf-8?B?bFE0YVZzSnRnVXdZaU1PQ21KTXlLZm13YmtoSjdDMDYwQW9UQktXS0Q5ekox?=
 =?utf-8?B?RWN5Y3gvNnB4TlJ2MmZWbmQxOHJ4WTJJNE8zdTJPS0RYZHo0b1hDN1VPSFBv?=
 =?utf-8?B?ZXg3K3V0NFdua0JFMmlYMEZHdWpWVllZSHQxUUZpU3ZqRzY4OWJZSm01ZEI0?=
 =?utf-8?B?Ujc4aFBLeDYvR2pQZDhmMUZoWCtrMTJoc3NTd2ZncVRWZ2ZJbFBIVkVYeU9Z?=
 =?utf-8?B?VTYxbGRaTk03VTNlcks2Q2lzdXZ1MFhFWE9FNDI2QlFKM3NndlZQMlgxbWpB?=
 =?utf-8?B?OXorREpYVUhsaXR5K3BIVmNZdVNSS04yV0trdmIyWllDTG03eTMrVzhBT0Qr?=
 =?utf-8?B?MWNtSy9zZnp5RC9LV0RvOHFPc0tuaEx0eWxGYVZveHVURkNqcUVJOHM4TUNk?=
 =?utf-8?B?Q2d0dStYYkwwL1JjZ0NlUXdGQWZZVUdCOENVRzlnREZLdVAvNytnWElWNDg2?=
 =?utf-8?B?RkJCNkh4T0J2NnZRNFZLbUxVVHVMWWc3OFVSL2ZibWJrOElkQ2ZQNHZPRG5X?=
 =?utf-8?B?TjhpMFJjeVo2NUczcXNiTHliaEtkcFdKRm9JSVB0UnJybkRhUGRJZWw5TEZL?=
 =?utf-8?B?QjRSM1FlZi94cUpRd2drTStrYkxzZzg0YVdSQndpbkNvN3d1MysxMEVteSsv?=
 =?utf-8?B?K3JFUFZ4Z2VjVUE2b2tHOUt4VStrc0FISDY2SmQ2Yi9SdG0vTEVJNHFabndC?=
 =?utf-8?B?bStQajNmcENXdlkxTHZwYyt4ZXBlNTVuUXlTSmxEc1Y0aFJLanRBYmVab1pL?=
 =?utf-8?B?SnJGU1dIZjArcXRKOXZ3K2g5dzdoTlpEaWtaZXZjVFpnSEhSL2djRlpiSFpX?=
 =?utf-8?B?QndmUW9VT0JubmxDMTlSZi9Id1FjUWdGa3B0RFpKYWVZcW44SGRsanpGRHVn?=
 =?utf-8?B?bWRMRktrK1FMMzF6dVFhSEdidHNPS21Qd2RkR0FaeUhIdTFub05jWDZVZlhP?=
 =?utf-8?B?M3FoM1pnN2dZMG4vS2s4R2ZGNzdYaDJIZWphN0hpNlhqd3gwUmFPYlBZWlhC?=
 =?utf-8?B?QkVYYTl4YUE3N0pqUVROMng5V2N6cE14dWxkdFllMjMyQWEvc1UxcVBvYzRX?=
 =?utf-8?B?Rm5pd3J2MnplMm0wdE5OMVRPZHBtc2ZCZmhqTjQyMmsrZ0dpRGJRNlVVZ2Rr?=
 =?utf-8?B?czVVZkh0L0U1VTFPVWpneEZ4eTNRcTBDVzZEbzlKS3dYMXhkeWp0ZFN4ZFRO?=
 =?utf-8?B?K09ncldhRnVVQ2hQdzdlcE5BQjdvdE1NdVM1V2xIaTNFMEZkb2ZhaGJCQlU5?=
 =?utf-8?B?ZGZDakxrSXZpYXNCYjBkc3lWOTJXRFBDeWNScjVPOXA2eThGZDM5bW5uMTEr?=
 =?utf-8?B?bnhaL25OY1I2bjRINjVyZFJNTXE2Mm9tSEFiWjF2Rm80OFJMM2o0R3NhV2Q5?=
 =?utf-8?B?ZUdkdTYvNGI3Mis5ekJrb1NvQy9QNVhuWTZOTElMdEFFb1VvbFZsdzQ2cmsx?=
 =?utf-8?B?amhRRldZcm5KVW80alFOOUlTTUgxYWRIVGtybFZKa2dUNnNIV1NvVUhsTGdu?=
 =?utf-8?B?cTdrc1lCWkpKQ3RyYzRkbUFLOXUrY1JZaUVjS3Z0OUlMOUpXUy9Rc3FyaXpu?=
 =?utf-8?B?TUxnSkNTL3JIOUsxdjlWcWlra0MzZE9ZSEdCUndVN3VyNUhmQVFKMkhHN1FE?=
 =?utf-8?B?dFY0d2FueUVVQUFTS3huTlhqSXRPWCs1UDlRQUgvcXFINDc4aGRzYUJIQ2tt?=
 =?utf-8?B?YStpUkJDeVFXUGpOVXY1Vm9KbUxURzVpV25ZZHkrYmxsdlhxcnBzdVdhUEUw?=
 =?utf-8?B?RTNTMU1ZK2Y3cnpaTlhtZDVHVml6bll2MWpmWVZqTTlXVXhrS01pbE9pU21s?=
 =?utf-8?B?bkFBNTNJeHdrTHM1dnRIZVZmUnBQYmh1cnRmL1BEbkN4U29nM2ZrQ1JUT2dL?=
 =?utf-8?B?d0FPYWhUZ2Rzd0hJbWNaL1VaTEliYmt1SXJ2REZva29YSkVlWWNEbVhyVXpG?=
 =?utf-8?B?bkhEVlp0TkV4c0U4WmxwOHJVd0RLcGZKb0hqbElQQ3ZMMGcvSE1pRjVMeHBX?=
 =?utf-8?B?eDRyVU16S0U0TGRxKzJLbksvd2pobEwrNFBkSTNpeXN4QnN2MUt2QytCWTVy?=
 =?utf-8?B?OVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 904d7e1f-b5b8-403c-f98b-08db6df841e8
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jun 2023 23:28:43.0348
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: cUtC6eLLwU8cO1ylhiPweNr2ACCeYAKWhqui8UmoE/oAb5jGgGRACn6CFMaY4gJRNRAZyHlI/hdu3GizWk+XBbeF1z4oiDJzRZs3L469heg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5219
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi James,

On 5/25/2023 11:01 AM, James Morse wrote:
> 
> The second part of this series adds a domain_list_lock to protect writes to the
> domain list, and protects the domain list with RCU - or read_cpus_lock().

read_cpus_lock() -> cpus_read_lock()

> 
> Use of RCU is to allow lockless readers of the domain list, today resctrl only has
> one, rdt_bit_usage_show(). But to get MPAMs monitors working, its very likely

This may need an update to reflect changes since initial version (similar to the
relevant patch).

Reinette