Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Message-ID: <9f6b8aaa-50b1-435b-a525-9a7986f63845@vivo.com>
Date:   Fri, 15 Sep 2023 10:28:39 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [External] Re: [RFC PATCH v2 2/5] mm: Add policy_name to identify
 OOM policies
To:     Chuyi Zhou <zhouchuyi@bytedance.com>,
        Jonathan Corbet <corbet@lwn.net>, hannes@cmpxchg.org,
        mhocko@kernel.org, roman.gushchin@linux.dev, ast@kernel.org,
        daniel@iogearbox.net, andrii@kernel.org, muchun.song@linux.dev
Cc:     bpf@vger.kernel.org, linux-kernel@vger.kernel.org,
        wuyun.abel@bytedance.com, robin.lu@bytedance.com
References: <20230810081319.65668-1-zhouchuyi@bytedance.com>
 <20230810081319.65668-3-zhouchuyi@bytedance.com>
 <87h6p1uz3w.fsf@meer.lwn.net> <5343d12a-630c-4d54-91f1-7a7d08326840@vivo.com>
 <89295904-3afa-4c8f-ccdb-1d78d9ad3024@bytedance.com>
From:   Bixuan Cui <cuibixuan@vivo.com>
In-Reply-To: <89295904-3afa-4c8f-ccdb-1d78d9ad3024@bytedance.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?eTJWMTYyV0pHVExnSnpSQjFzNkNaUGMrSlVobzBSL2pYMUtrbmVFMkRiSWNV?=
 =?utf-8?B?cDJEY2wvWit4M3JQWkJzdjRQTVdkcVFVMVNOSUkvYjhqUnpzQlU2cHhrRzZH?=
 =?utf-8?B?OEFtbWtaSGhncnBHbjlhK0c0TlcyQlM0OHJtdU5DUXp4Q0d2NVMrZ240V09Q?=
 =?utf-8?B?MUthQy9FTXJHZngrcnEyaVBZaENVd0VDaWwrY0tGY0xNUlJRcTlSRmxNelc3?=
 =?utf-8?B?R2JTNFFLSzE5VW82ajlzaXF4RDIwcjlQOURubnpNMjhtenlrOFRPY2tKb2pZ?=
 =?utf-8?B?V1pqRFJmMUQraHhzNXNSczN3TjI5dWlFWEhPU1hmZ3NxeVh0M1Bhajk0REp0?=
 =?utf-8?B?OTB3TFlKQzRDVm9jcGc3ZmdESk4wQ0gyZjdETjNPVkFGL3JRdktSVHZ5QzJs?=
 =?utf-8?B?OVNub1NwVlN6MFRkeGJ1cXdNVGNIa2UyR2FvMlVHVElDa2JvZTJsZ0FRa1NC?=
 =?utf-8?B?K3NDT0s2UHhsbEVDNTdtWFZFZjZUNWpTdzNZS051eE9td21KcGFoOU0vQXVQ?=
 =?utf-8?B?TXpKSWNUMzd6bkVQU0w1VjN2WVljR2hFT0VGTnUyTmxUcXIxcDN0UURlcUhF?=
 =?utf-8?B?bDVoQktyclFkZ3lyRWJtSXNhQ0dObmlteVdZaE54a3FORlRZL1ZrSVM0clla?=
 =?utf-8?B?aEVVNkhGMXZWcitLdUN6YVJTZ2UwcUJET3ltaVc0dEZnWHljRWRZU0YvclY5?=
 =?utf-8?B?L3lVd0NRQWJDZGdEYjE0d1NhVWVQS0pMVFRkQ3p2bThCcnplK2h6M3ozcHpY?=
 =?utf-8?B?bmpMK21hNk9OMHM1Zk9OZ2cxbkRqMzQ3QWU0TXJCanZFbUVaV2JtR2p5VWpx?=
 =?utf-8?B?by9OOWZBYnRLQXVGcjBVczhNOVRaVnE2YThXemNMbmo5OHVWeElqUWxzbDNx?=
 =?utf-8?B?d2FUUTRCSlFaMjlackVOb2FmSDVCSFF4ZTlpb21OQzB0Q254NTMvSTl6QUtZ?=
 =?utf-8?B?VUdJZ0lJNkJBdE5KMXJuWGFOYzNreG9kWXN2MXZYUDV2aXErWk5SckNJalVz?=
 =?utf-8?B?OVpOd2RFaFFCTE16NEJBc1NmcjcrOTEvY3haZmVodmxaeHpQSTMwSCthb1BU?=
 =?utf-8?B?Q2QxZkx3cXRWYkxIMFNCL25UMGc2WnpoYklFWUVwUngwZnp5NE5iVHUxTW5V?=
 =?utf-8?B?TzhaQVM1VkZtMmhuL2RmOU0veHNNY2dPNDhwQTkreUdDeUpiN2Q1TGcwTEp0?=
 =?utf-8?B?SkhiU0hSYzZJSW9hMG4zVUhMUzJMZGpnSG1la0N0VDdJMmpydlJiMXAvd2lv?=
 =?utf-8?B?R3pEd0ljMmI0ZnRFdkJsdFdISjljWklTK3cxMVN1UzI2Z0JZUy81cGtLWUUz?=
 =?utf-8?B?QTFKMXF1c0pTd0YzYlBSQVpBekVHMHVkV01qVVVEZUlVdHh2a2NjNzROMitZ?=
 =?utf-8?B?N3EvZkNMMlFYTHJhdDVyTm5aLzNOUms0RmZiY0JDQ3p6VlNkclFaNjJZSDM5?=
 =?utf-8?B?cHFObHBlbHR0UlAyaWYwTHRaN1phVi9EbVRBQkw4Y1RNUjlkb2YwaC92Wm9E?=
 =?utf-8?B?ZFl5SWJvNTFBQlFtTm9LWklTT2xrWEwwb1U2WWdEZzd3UUtYUEtrdUpsV3Va?=
 =?utf-8?B?c21qUGRxVFNVaXlFZjc3bytHbUFVTklHRURHNE9tUHBsQVcyVEtkRGptOGh0?=
 =?utf-8?B?bVdKeStaZlpzMk5SZy9pSUx6a0FGdmhEVXVWcU9DV3A2NlhqZGxIMzRpYVRO?=
 =?utf-8?B?TVhpajNENmYySnNRb3FZN0gwNmNLcUFwZnJYQTQ5VDJCeVlZUmY4ZHVDYmFu?=
 =?utf-8?B?NkhsNnpGUjcwbmVCVWdUWXU2Rmh0ZEJSZ2dqbzFKKzVyVmYxV3crRG82by85?=
 =?utf-8?B?UUFKWWFLNkxoQit2SkZBZURGYVo0eU41cy9QUmZFQ1JYVUMrMXBvRWlqZWIr?=
 =?utf-8?B?TzdCRjN3OGtGMVRzcHNqb2JZRFdrMmp3WWlGQTN3Z0FTcVNTQ3dRZ1dqekFm?=
 =?utf-8?B?d201bjhWcDVGSWNUT3VoQzRxdFk5VjJjQllMd1RnSVMxZWFBU1phL2Mxa2hR?=
 =?utf-8?B?RnFIMWM1TmFZbjVzUW82bDVXd08rSTRMZlFXb1V6UEFNbnFGaGw1ZitKK1Js?=
 =?utf-8?B?VUpMaFBvN0lKNVBaQWpHYkh2TkY1c3B1eWlzRkFQOHBCZDJpNVdKK2lFVCt0?=
 =?utf-8?Q?8ysxSWJ5yj4NmwRCJPF/NhNui?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 57228ad4-8236-48b9-16ca-08dbb5937d1e
X-MS-Exchange-CrossTenant-AuthSource: TYZPR06MB4045.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Sep 2023 02:28:46.8980
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: kl8l2gvloIaj7tsUg/eBCfDOUxsfRzQoWFWoyR/PiVpharR/XcN2anb26LnKr5tYOX/RNFqcLi9ALPURQ/EKdw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEYPR06MB5468
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Thu, 14 Sep 2023 19:28:59 -0700 (PDT)


在 2023/9/14 20:50, Chuyi Zhou 写道:
>>
>> Delete set_oom_policy_name, it makes no sense for users to set policy 
>> names. ????
>>
> 
> There can be multiple OOM policy in the system at the same time.
> 
> If we need to apply different OOM policies to different memcgs based on 
> different scenarios, we can use this hook(set_oom_policy_name) to set 
> name to identify which policy in invoked at that time.
> 
> Just some thoughts.
Well, I thought the system would only load one OOM policy(set one policy 
name), which would set the prio of all memcgs.

What you mean is that multiple bpf.c may be loaded at the system, and 
each bpf.c sets the different policy for different memcgs?