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: <9d80f21c-ee30-9891-5da2-a1113ceffd21@intel.com>
Date:   Fri, 1 Sep 2023 15:14:38 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.11.0
Subject: Re: [PATCH v8 2/8] x86/resctrl: Simplify rftype flag definitions
Content-Language: en-US
To:     Babu Moger <babu.moger@amd.com>, <corbet@lwn.net>,
        <reinette.chatre@intel.com>, <tglx@linutronix.de>,
        <mingo@redhat.com>, <bp@alien8.de>
CC:     <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: <20230821233048.434531-1-babu.moger@amd.com>
 <20230821233048.434531-3-babu.moger@amd.com>
From:   Fenghua Yu <fenghua.yu@intel.com>
In-Reply-To: <20230821233048.434531-3-babu.moger@amd.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cmR5Z3dUUlZwOXI0NUNrd2Z0TytlTEd0QjdZVWtkM2VFZmlCaFZ4ZXBHUkJp?=
 =?utf-8?B?TVNLblVORVVBVUtZZU45YURwZ1graGpTUUdNamh0MHhQZHJ4NjIwdDZQZ1Rj?=
 =?utf-8?B?V3ZMTXFuMGVmNDVpNXFDUHZGSGY2ZHpjZTlMOFpqOHFRbDFHU2I2V2xqaTh6?=
 =?utf-8?B?aTI4b3hub0Z0dWNTOWF6RjRqMy9pcXIxMlR5dEptdS9xUXgyMFNYQ1RuQXVo?=
 =?utf-8?B?SGZnQ28vaW5BNTBMTWxZNTZNNjVpSGRUdGVyT3hiZ1VBaGJyc1k3M2tVQ3c0?=
 =?utf-8?B?UGYvZjVSWmUyRWIrV1lzcGtLT1B1QjZWM0JydXFOSjl4VkJ2NzJBd3ZPU2N0?=
 =?utf-8?B?RVBKQXE4NElnK1FPbTdXQ1BsOUVTcDdtZ05GaWMrOHZoY25aOTdQZDMwRDhJ?=
 =?utf-8?B?TmJCMzdtTklQNEpYdGgzaWtXY2lTNWJESCs5MElRSzJ4M3FRVWZ5Y045eUdN?=
 =?utf-8?B?M3B6K2pJUXk1WVFIT1BnVDIzZU5qd2ljZGhyeW1ZOFNmTFg0TVdodkltMnR0?=
 =?utf-8?B?bkd4ZHhCNEwwV2NFMU52L0pXV1RmYWlLMmhkMldUV3JlWU9DajV2S01Rc3hP?=
 =?utf-8?B?OEcrL3VQY3drRjF2MnE2QjdnbXhjc0ZSMnk1UWJ3WnA4bSs2ZVVwZHFNUlRR?=
 =?utf-8?B?ZEIwVk1vODBVYXVtR2w0dkNLRDJucFpRdzE0UTUrbzZOZXkvRVlMU0tmWHJB?=
 =?utf-8?B?a1hWSE5URjc4cGlxNVRPYW1nT1c4bTUxTW1tSFZXVWphV3c1YzR0UyszMnpQ?=
 =?utf-8?B?YmgycThTU29TZEQxU0o0LzBJNG5yUEVZcjF3YkdQd0JEejAxK0tteWpuV2pG?=
 =?utf-8?B?MUlGSVF6bUt4aS94ZjV2ZG5BS3ZkOEVPWFRaZWcyYWdYbW80ZGdFQTkwUjZ5?=
 =?utf-8?B?WFQvTVYzU3kzQi9jTDhnOFoxd25YZE5ZY2c3cUowb3pqRDFZUDZselpBSFdw?=
 =?utf-8?B?UWpUUUFUM2RZUTN0dFBYdkZqalNqamxDU3FlN2tzWGFtcDJ1L1kvalQ3ejZ6?=
 =?utf-8?B?b1hXcVd1Qnp3YjYxS053MUY5WXowVlh2VVNKVXp1cUNuMm4zMXU2YjkwZWk0?=
 =?utf-8?B?eDVWeGxoemI2YnZYMVI1a1V2Y2JvUndLWkI5OHdNWmo2L1g0MFk3Mm5USmli?=
 =?utf-8?B?SXZvd0d2R0FuaHFINGRWakN0ZEM0S1ZqbGJMazVqVFdCenFpcStLTFdlNUkv?=
 =?utf-8?B?aHRLQzhpSnlFaTVUbmZkOTdEQ2VzY2UzeFVJVFRWUVhiVjVnNnBPZnZGbFB1?=
 =?utf-8?B?a0NEajlzQ1plY0xEWmY1TUpDOXZVeURGUnczVC83ckZGWUhsQk9OcGs1Sjhv?=
 =?utf-8?B?SUNIaHNqZ21Od01PcTBXQTVwVFlmbHJ0R2xEcE9vOHNySlVuZjBFWTRqNEZv?=
 =?utf-8?B?TUp5eG14ZkVPR3FsNFovSUlzVnhUdDVkOExYQ3hrS05EbUZRamRwL3IzMjhh?=
 =?utf-8?B?ZnpDWHp3WFlaRVhCUGVBSFFNQ0p1UzgxZFh0VlVVdkxPQWV0cmc5Q3hEWDhR?=
 =?utf-8?B?M2J1dGpzMWtDN1Vma1ZCQm1mQWpDSUJvUEY0Y2JpYTgzODRWeTZZUTFYL2Vh?=
 =?utf-8?B?VWdkY1pZWHdQY2NQd2JSanpYMTF4Z2VFNjFnNEdOUEhkRVh3WG81K0ZNYmhS?=
 =?utf-8?B?Y1lMY3lSMXVoY0NRckRlRkpZaTFPaUtpWE80aG9CcUg1eER0SzExREdCR094?=
 =?utf-8?B?NXFiSkZTZm1QTHR6dUtjSHR2aWh5Sm43UXNlWUlsYnE5U1dNUHBadDl0c2lO?=
 =?utf-8?B?WDhlVEpGYWFUK1RKVFdPMmpETWtUaWFtSUpUVTJQcnptRXVwSFp1eFlkS3FO?=
 =?utf-8?B?SEIzTk9oWEdHbGxsd2NTY3JOTEtwQzdlbW9EMDhBT0VtTVZNSmI0bGxoN0JN?=
 =?utf-8?B?NUNXa3RPbjRUNldNWGZ2N09UVE9zbzljYy9BWmZDZXdiZ0VUWllxclNSOHp2?=
 =?utf-8?B?Q2NydTZRSkxuOVl6U1FKa3J5NmtGNHc5UERBZStTcDlsQVBGU2lrZDRObnda?=
 =?utf-8?B?SEJpZ3hieitHT1FIZEx5UWtNdlhmd3N2TS84MlRWdU43OXFuVGV5Q1FsNkE5?=
 =?utf-8?B?bEowNGlaUHk2QjZkOUxWdXlBNjRseWNTU3dCNExWeUpVTm1OZVNlWGdpZlJK?=
 =?utf-8?Q?m1f6cfeRRoGEEcOHUVjdmROln?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 5500f92b-7c72-4972-384a-08dbab38d1b5
X-MS-Exchange-CrossTenant-AuthSource: IA1PR11MB6097.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Sep 2023 22:14:33.0109
 (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: Vh336vO0dSh4xKF3bqGQFuMHIqbYPz8hDmlamNKZzw7TpBpgMWt0e4azYnQafaxdu3ByByIX/QaZXeoQlBoiIg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR11MB6981
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.6 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_BLOCKED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,
        SPF_HELO_NONE,SPF_NONE 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


On 8/21/23 16:30, Babu Moger wrote:
> The rftype flags are bitmaps used for adding files under resctrl
> filesystem. Some of these bitmaps have one extra level of indirection
> which is not necessary.
> 
> Make them all direct definition to be consistent and easier to read.
> 
> Reviewed-by: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>
> Tested-by: Shaopeng Tan <tan.shaopeng@jp.fujitsu.com>
> Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
> Signed-off-by: Babu Moger <babu.moger@amd.com>

Reviewed-by: Fenghua Yu <fenghua.yu@intel.com>

Thanks.

-Fenghua