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: <f826dd28-0213-b7ea-494c-8c7a0ff348ce@amd.com>
Date:   Thu, 30 Mar 2023 12:33:25 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.0
Reply-To: babu.moger@amd.com
Subject: Re: [PATCH v5 3/3] Documentation/x86: Documentation for MON group
 move feature
Content-Language: en-US
To:     Peter Newman <peternewman@google.com>, reinette.chatre@intel.com,
        fenghua.yu@intel.com
Cc:     bp@alien8.de, dave.hansen@linux.intel.com, eranian@google.com,
        gupasani@google.com, hpa@zytor.com, james.morse@arm.com,
        linux-kernel@vger.kernel.org, mingo@redhat.com, skodak@google.com,
        tglx@linutronix.de, tony.luck@intel.com, x86@kernel.org
References: <20230330135558.1019658-1-peternewman@google.com>
 <20230330135558.1019658-4-peternewman@google.com>
From:   "Moger, Babu" <babu.moger@amd.com>
In-Reply-To: <20230330135558.1019658-4-peternewman@google.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?U3RpakgxTkQ4dlpabGhsajZ2V1NHcDZyRjlrcUNMeFFmZlVocTFXYnc4SjIv?=
 =?utf-8?B?TEE1T0RxVXFGSm9rdUxQR3lVYXYvUVRlaHZtbEFDMTNHNnhySUdmTkRLZ1dJ?=
 =?utf-8?B?ZnEvekNpZUFPbGpXWDVzOEtFMCsxVVVDNWUzcTdmRFlYNFplbmk0L0hFUHMy?=
 =?utf-8?B?VXpOcmExdFNDMU41YjBOSHAzVlhaZndCRWU4V1Y2amNGYVgzN01GVXhkbi9I?=
 =?utf-8?B?RVFOdmlBOEY5TTB0YzBJQU9xZVNIQkRWSEZDSUNZY0I3dmE2ZysvV0JqS0ln?=
 =?utf-8?B?MFgrZzRIOHBSWDRvMS9NSkF2SW1oN05xMzdlK1pJZyt5QW1wZ2d6SXA2Q1Mz?=
 =?utf-8?B?YTVaekROMmNsVGJqVy8xTjJBNUJMa2gwK0swMHo4SjFROWdacHRNNXZlWDZn?=
 =?utf-8?B?dmoxL04yVVlyVmN4S3V3cEpwS1IxMVJNOHl3T3lRVkVlVFBnbVpKUnJtZjBj?=
 =?utf-8?B?TG9DSUVzc3AzbTVZNEIzRW5yTXFxeE5WUTZodkZoWlE4L0E5bGpmL3NuMlY3?=
 =?utf-8?B?Z2pjalVHWklvSERqZGJzT0VvbThVNVdkSHFoc1VMREsvWHUxeVlUSkVBaVRs?=
 =?utf-8?B?d0gzOXJ1UmNEZ0htamgydW5HRHlFcWVQSDROZVE3YU5ldW9adnJlUFlmSHFi?=
 =?utf-8?B?SmpUYnY1c1JTaFVZdXAzK3RBN3dJNW9acXMzaTJHRlJUdm1MR1QxVGhsWUJ3?=
 =?utf-8?B?d0NkSjNHalJrekQva2xCUVRPQVBMTVVTOHM4aWs4MlJmMW8rUUE4L2M5RmNT?=
 =?utf-8?B?b3M1VDgwem9mVDZqR29HMGc5eldiUFA2QmpaNUZCekhkYVNQb0hCZFpFRlE0?=
 =?utf-8?B?RklsVmNnb3ZZcmQ5SmRHS1FzQWlXR2VKQ3V5L2NGS0wweElkWldXU3UweVVB?=
 =?utf-8?B?ak9VWGFtaUs2Y2tLeVk4OG5Cd2MwNjlUdGtoTUppSnI4WUJ4UzlNQSt4Z0lR?=
 =?utf-8?B?OG9jeXdZT3FjVldHOUk4TWZoQVpPM3I3M1daU1pRaXlTdjJBMHU2OWVraVRT?=
 =?utf-8?B?bFd0aVlRc1BRTE5oOWU5MGpEWWlNL3RYNHN1NVVQVjQwTERhSHdaTGNUQXF3?=
 =?utf-8?B?dWI2L1I0TzBTM2IzL0ZPbEwzcVVIMlBHUWgvTHdJT2c5cWJNUyt4RU5qSnVy?=
 =?utf-8?B?V2hhSFdJdVc2Ym96ZXowVDVmcnU0RFNyUmlwZXdGb2xMdUlQeGRhZko3Q2hK?=
 =?utf-8?B?MzRXUEpGc1BPZnpwckVPaFh2K3FZMHJhUWsxK1laVTRoU3NLOUlsZFBSMUpK?=
 =?utf-8?B?VDlYTGkzb096Ry9mSGVpRWZKK1UrZzRUN1hCdTNZNVd4bWRYbVVtSUNHazVo?=
 =?utf-8?B?RUhsZ3F1QjdUK3B5cmZnYWFQY2diWjJwMzRMUWpWRXdqbVFaSnFvMzJmNFRH?=
 =?utf-8?B?SGV4Tm9iRDhwajFvRFNxelRESVBtYUgzTzQxSTl0NUYrL3NJbHhOMlRtb3gz?=
 =?utf-8?B?VVNKVWNUL09RTUNPSXE5ME5YcVhoWVN2UWhJb0Q4Nk91c3pxTlQ2MExITVlX?=
 =?utf-8?B?M2NKR2dSdzFjVnRKWmk0aDBqYloveW83S2RnNzUySjNTOXpsRUp3dTJJTGdO?=
 =?utf-8?B?a2JQS0N2b2FUQmU0R3FneWc4WUhDTGoxd3BBbVJJK3NwdER5eURZSEZ4Z0Y4?=
 =?utf-8?B?aG5UbDNraXkrdUtkb0dGWlk3NU5rK3JJaEJGbFJTT1o5aEVCOER3bkhoMG9m?=
 =?utf-8?B?emNJTzAxMTArcnE3cXJqR0lNang5RzhqeTBSTWtLZ1BmTUtSeUpmSXZaQ2tK?=
 =?utf-8?B?YldPb0g2UHhWNlBZcnRxRXc5R2o4aERwZWRkMnhQaXdZODNQMWEwZkYxMXF2?=
 =?utf-8?B?WEl0NVJwVjArNTF5U3FOWWhicnVDeGJlNFpZSnFhay80Z1d4MCtlWnQvL0wy?=
 =?utf-8?B?bGx1NzFMU1dCY1BiK3U1dElqclAvK2tweWR2NzZGWmM0VXlYRWh5N1hJWjRJ?=
 =?utf-8?B?RzVaS0hob2xZWTRSUFF1MWNvMHBWM2J4Vi9NNUZCZzVwc0FWYjF6U1dJNkxs?=
 =?utf-8?B?YW10Sk8xYzd4eFRoaTc4ZmpPemMvLzhrRjZCcGZUbVpOcTNyczFUenNzdGdG?=
 =?utf-8?B?SEZRQzVGdjF1ZWkrM0pVRjVaTW40K3ZKa2tsTVQ0dXBSRzNMeG80ZkxZTDB1?=
 =?utf-8?Q?d7I5j64+Wr1Z6kGbTLeg7r+mP?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2081f77a-ce3a-43a1-427b-08db3144dff9
X-MS-Exchange-CrossTenant-AuthSource: MW3PR12MB4553.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Mar 2023 17:33:29.0126
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: t3TqbC0lsd3oxCUaE5ZfszZ6M2ACgozeRYV/ZOY3pxYmWP2Ly0eJ057MLIseBgrO
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4140
X-Spam-Status: No, score=-0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
        DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,NICE_REPLY_A,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE
        autolearn=unavailable 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 Peter,

On 3/30/23 08:55, Peter Newman wrote:
> Describe new support for moving MON groups to a new parent CTRL_MON
> group and its restrictions.

Sorry for coming in late here. I am planning to test these patches. It
would be helpful to give a simple example to test this feature.
Thanks
Babu

> 
> Signed-off-by: Peter Newman <peternewman@google.com>
> ---
>  Documentation/x86/resctrl.rst | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/x86/resctrl.rst b/Documentation/x86/resctrl.rst
> index 387ccbcb558f..cb05d90111b4 100644
> --- a/Documentation/x86/resctrl.rst
> +++ b/Documentation/x86/resctrl.rst
> @@ -287,6 +287,13 @@ Removing a directory will move all tasks and cpus owned by the group it
>  represents to the parent. Removing one of the created CTRL_MON groups
>  will automatically remove all MON groups below it.
>  
> +Moving MON group directories to a new parent CTRL_MON group is supported
> +for the purpose of changing the resource allocations of a MON group
> +without impacting its monitoring data or assigned tasks. This operation
> +is not allowed for MON groups which monitor CPUs. No other move
> +operation is currently allowed other than simply renaming a CTRL_MON or
> +MON group.
> +
>  All groups contain the following files:
>  
>  "tasks":