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: <0a2bb59f-769e-42b0-9277-10a4b04a03ed@intel.com>
Date:   Thu, 4 May 2023 12:00:11 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.10.1
Subject: Re: [PATCH v4 3/7] x86/resctrl: Rename rftype flags for consistency
Content-Language: en-US
To:     Babu Moger <babu.moger@amd.com>, <corbet@lwn.net>,
        <tglx@linutronix.de>, <mingo@redhat.com>, <bp@alien8.de>
CC:     <fenghua.yu@intel.com>, <dave.hansen@linux.intel.com>,
        <x86@kernel.org>, <hpa@zytor.com>, <paulmck@kernel.org>,
        <akpm@linux-foundation.org>, <quic_neeraju@quicinc.com>,
        <rdunlap@infradead.org>, <damien.lemoal@opensource.wdc.com>,
        <songmuchun@bytedance.com>, <peterz@infradead.org>,
        <jpoimboe@kernel.org>, <pbonzini@redhat.com>,
        <chang.seok.bae@intel.com>, <pawan.kumar.gupta@linux.intel.com>,
        <jmattson@google.com>, <daniel.sneddon@linux.intel.com>,
        <sandipan.das@amd.com>, <tony.luck@intel.com>,
        <james.morse@arm.com>, <linux-doc@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <bagasdotme@gmail.com>,
        <eranian@google.com>, <christophe.leroy@csgroup.eu>,
        <jarkko@kernel.org>, <adrian.hunter@intel.com>,
        <quic_jiles@quicinc.com>, <peternewman@google.com>
References: <168177435378.1758847.8317743523931859131.stgit@bmoger-ubuntu>
 <168177446947.1758847.11380042804869155387.stgit@bmoger-ubuntu>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <168177446947.1758847.11380042804869155387.stgit@bmoger-ubuntu>
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?RW4rbFlXYnl2Vld3NHdXOUxqcy81QzBUL0pyUzZzdWRRVjloR0VIblE0dGFy?=
 =?utf-8?B?WmxrWGg2RkNtNXpJR3Rabk9la3NLZWlWWW01NkpwUEcyY3d0TUJyV1RmNXhT?=
 =?utf-8?B?dHdnZEl3YVFDeXNrOTRUTldZdVd2YkVPZGpWcWh4bW15NHhMeWREaFpTdVd2?=
 =?utf-8?B?V09tYjBaNVBQazVKMEhlcWR3S0FlblFhME0weC9CSkJhTFNIbVR6aXFZME5u?=
 =?utf-8?B?VjBVRVp2U1BDRWJENkZsWXQ4VUQ2ajVaNEszeitReEpsV0R2RmxrRzI5bG5q?=
 =?utf-8?B?WjhpcU5rbk1wdHdMeWZEZFZ4eTNTU3hTaWxOY0YzdUhUVWxhRTNWM3N0MlNO?=
 =?utf-8?B?MGhxWERPQlhxR2RQTitRSmkrb002bHV3YTFHQmREbTFRVGdwQ2pIRzFKWmtQ?=
 =?utf-8?B?dURnaGZKU1RjV3lhZWV3d0lDblNJRG1FeVZ6RzA1dDM3Ykwwa3AxK2VDaXJF?=
 =?utf-8?B?RmZ1SkZNVm54V3JzajlhTkZhYjEwa2R2RGZ6cDVTUmUyaG9HeGU5ZnU4aDMr?=
 =?utf-8?B?c0xsMkdJaFRWd1lsWmhPUklaNVkvUzBmbm5jYXFURzNJRGMzaEUvVzJpeGh6?=
 =?utf-8?B?RVBiOXFzOTE2bmdwRTEwdkJ0dkJ0OGdyN1pVbGRwL0dOZ3hlbHJlbVg4Rkgr?=
 =?utf-8?B?KzhHSnJrU2wxSTh0SXVCWWZQcXBQUVBSWkhvc1NmS0FNeHhaNDVlWVp1ZzYw?=
 =?utf-8?B?a1FBOUlyVGNHV05WWHhuU0thYkh3T2xreGMxVnllSVdvZzZndGJrZGFHMGVG?=
 =?utf-8?B?TW1KTUhnZzBtaC9UdUU1R01lN0FaWWFxYy84UUR0eWtsQnFBZzM3RlZZWHFO?=
 =?utf-8?B?eG9lTm9Rbm5kcWJXbkp1clRUbUtXU3BPQjhISms0VnQyTnZVTVlGSi8vOTha?=
 =?utf-8?B?Y2dvZlZnbWJZZ2d6TkFaOGowZitaSFpVNnp3alhjNHpGRzB4Mkl3Qm9RUHZD?=
 =?utf-8?B?WXFaZ1F1R2pIWnVMdUVtR2s3SWUrRTMwcGF1cHlGY2JaUzBwTEYySXBkKy9s?=
 =?utf-8?B?R1RCTWFtUmZUZ095ZlFxNElpR3p5SnVOWDFzQUtTRHFHOHNxbUl4dUx6U1c5?=
 =?utf-8?B?bmpGeGNTdFgwaWs5aXUyQ05tTmZRTDEvSTNZRDZ6WFNlU1g0cERvYkJubWIw?=
 =?utf-8?B?MVhhbWFzS3FUY0lqbExhSU5CdVFlUUNHQ25HVk15NFE3eDdvc2JJMkYwWjVB?=
 =?utf-8?B?TFpsK3kxN3dXWGZoMnUzaktJYnRGekhoWS9Yd2M5TVZIRWFoTXpSUS9CWlVZ?=
 =?utf-8?B?MFovaFoyYzIyT0lQVEIwbFRhLzM0VVNyQkFQdGhXcS9md0psQ1FWb1I5QUJz?=
 =?utf-8?B?c2wyQ3hXUkk2S3d0aDV1YytrK3NCeCt1ampubThUTmVMcXVVdGZ1WG9RRG8y?=
 =?utf-8?B?Y2k2QzhaK1NieVptNUl1WVlyeTRVc3BKV0lKaWEvOHliOHRYZGw2bFE0OVZB?=
 =?utf-8?B?QUw5TnhuY1BGK3Zpa2dldldwcEpkSWVVQy8xeXM1SVBDc3hLaC9tWWJxTVIz?=
 =?utf-8?B?Ym9ISHNlV2FnQUJuMzFJM3JFOFlFWGVxb2I5SzdDOEI0WE5GSjBtR3FrUWVO?=
 =?utf-8?B?cGxzbUVxblZtM3BaQ1dzN3psK253UU5VdGo1NVYrK3ZMaWxKZ3dvVXlwdktV?=
 =?utf-8?B?TzRKc0VEQlpoMTAxMUFhZXYvTTNZcE5tc25tSy9UMllkVmxYaWhJSm9xaHF2?=
 =?utf-8?B?aW5rdU1WS25na3FXTzVmL2d4ZGF2ZkJsKzFiSlFJTkVtMlp6aFFtQlZodERB?=
 =?utf-8?B?ZTE2UUZCSE1Uc25BVlBJSUNUR0oxVzdmZHJtbVZPTG84WTB3TExZK0R1R0R3?=
 =?utf-8?B?YTh2T3dQRU9nV01iM0Q3dDgzZ0xmbVZJNXVSeiticjVlNGRIYzRqNE0vOG9h?=
 =?utf-8?B?clR0VjRwOXdlS1ZjdWxKby8zbHJsUDZ3WHRzenpMUGw4alM4TnZqbFhJQzg5?=
 =?utf-8?B?b202MXdyeVBGcGhnajROQ0hQV1pvdzgrMFBIOUxrT2FNeHl6U3RJWU54a2xk?=
 =?utf-8?B?d085WXpCQjBDbnhSaW9ORzZHZktweTI0djZJa2RFS3AzOThjYTJKQlAxaUND?=
 =?utf-8?B?NEUvZ0hBbjdQRDJXamoxc0xCQlpaRVE2T2NLSHVDSGg5UzFETVUwbGR6OFhV?=
 =?utf-8?B?UktDM0R2RXB2YjNENEhDdFVCd3lCbS9waEZmcWVZYlloR21pcE9qMkZscmRh?=
 =?utf-8?B?cHc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 5b8f282f-723c-42e7-63fa-08db4cd1cac9
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2023 19:00:13.8745
 (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: C/hZTbF/ttI6Ujy3D7aijDHnZMpSCdMIPB55z6ubNdKAGkn4MzIljRjMahMlYVy6FExj4821Mqo63RCqX6UGkhYPSv9HOLBWKNUIcOp90aI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR11MB6947
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-8.8 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,
        URIBL_BLOCKED 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 Babu,

On 4/17/2023 4:34 PM, Babu Moger wrote:
> The rftype flags have two different prefixes even though they are used

Could this changelog please start with some context about what the
rftype flags are? Please note that the changelogs are required
to comply to the x86 requirements as documented in:
Documentation/process/maintainer-tip.rst (there is a whole
section on changelogs).

After a context the problem could be made specific with something
like "rftype flags use the RF_ as well as RFTYPE_ prefixes."

Reinette