Received-SPF: pass (google.com: domain of linux-kernel+bounces-190380-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <8a4efd9c-8416-4267-b5cf-b2aca80b03f4@intel.com>
Date: Mon, 27 May 2024 11:48:54 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] cpumask: introduce assign_cpu() macro
To: Yury Norov <yury.norov@gmail.com>
References: <20240524185828.1582494-1-yury.norov@gmail.com>
From: Alexander Lobakin <aleksander.lobakin@intel.com>
Content-Language: en-US
CC: <linux-kernel@vger.kernel.org>, Rasmus Villemoes
	<linux@rasmusvillemoes.dk>
In-Reply-To: <20240524185828.1582494-1-yury.norov@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cExjYWprTDdDdEprejNqaHBpUlFsKzlCZnlIRElXOTdYdzhjY25IMGhnRUx0?=
 =?utf-8?B?WlBBV3JjTGhJMnZaN0FTOWkyR2Y0eEJ3M0xKOTVuTmtmbENkbXdSMWEzUWFj?=
 =?utf-8?B?bk1ENWFvaDdYY1owSDFabXQzS0RDRUpHSEtTMDJ6aEkwdHBiWHdkc2Q0dWZE?=
 =?utf-8?B?QnJkcm92SlVEQitZbkxOalBOY0lya29hRHIxYXVYdEc3bEVFWlp6M2dTQ2Ra?=
 =?utf-8?B?ZHZwTUk5Z25CeGNnbUt2NkEzRi9wR1pOYmMzTUFRTjV0SXZGMFFDRjZybHR2?=
 =?utf-8?B?bk5zT2VQbkJpU1c2c2Y5V3FFSzE1YndHeTlJbHBpV3VLemhPbnVYNWVhS1Iz?=
 =?utf-8?B?Yyt3RDBIQVdkYS9sT3NGSXBtMG5lYjBoZEJUS0dNYU5NOWZkRlFyNXF0eGF4?=
 =?utf-8?B?RHk1b29MRkpqS1F5TG9lWFBKYkhMZUE1UG5VTWNjb3R1NEhwZ2pwZVF3Z1BB?=
 =?utf-8?B?VStwSUdDTTF4RkFtY3NvS0VjUGVySkp3MXhzam1XTG9VbUJMdzhiUURwWmw4?=
 =?utf-8?B?U2FDOFhNMFZrQjRNV0Q1blpzTysybnY0RnZONExKT2d0bWMvT0pZRng5NHV5?=
 =?utf-8?B?UTI4QzZZSDV4Q1dLMi9UYUE2dnVGOG9JOUovcEZNRFFpRE9hVGE2VFl6MEs0?=
 =?utf-8?B?N2RTdW0vVkluMEJNTFE0b3N3djIva1lYbWFmdnlXdHRKYnhnTm16eEcvNWlQ?=
 =?utf-8?B?M1A1N0dreWM1YmZZTjZXZm9LdHhya2lzeE1UN1VlUXgyK0JtS0Vndll4NGdu?=
 =?utf-8?B?UVNUWlFCVzZESTllMzAwa0QvaDIrY2gxMSsxY3VraUx4K3orZUFXMjEvd2g5?=
 =?utf-8?B?THJJS3k3Qis1TXZzck1mZE56VktBMXd2RjM1eWZoYll4VE1lUlN5WXh4eGVZ?=
 =?utf-8?B?b25hMkQreFl3UEwzN2F2TVlWSkc2L2NFTXUxamV3RWk1TmE2bURlaDF5YzYx?=
 =?utf-8?B?WFRDS2gzMFRKTGo0NEVieTE1dmxPLzZQS0NaNG14OTdXU3Rqelh5NHFaTkFl?=
 =?utf-8?B?L3lsVjVxdXNuWmdHaUlWOEd5UGUxSkpGUlpkQnRSWldIU1ZJUUJFQy9JVGda?=
 =?utf-8?B?bWNON0xqM2ZsbWs3TE8wTExXU1hFRmtPbDZ1bVh3L01kbkR2K3BlUG05aEp6?=
 =?utf-8?B?RDJSdEhzWkZsTmp4eG9PTEJBeUNzTmNQdkdZVTg5OXdPZ1NCbmF4STJZZ1lD?=
 =?utf-8?B?K250bWEvRDY5NXM4cFd0V2dkd21LTHdqUjVNN3QxS0xvT3JJYlZBVUJnUDhq?=
 =?utf-8?B?QStEQTdPek9ZeFM0Rld5M1o3NnNBUjlBS01UT29Qb0Z4T1FaZ0lPUzRPWVow?=
 =?utf-8?B?Um1VWHBHczJ3cWlCcnBEN2ZraExFN0pjZ3J2bXkxeG0zWlFhcWs2THBUanpw?=
 =?utf-8?B?Q0RIbEtDKzZaNGoydkJkRERPQmd6dkR4SFpqSG1rL1M3TWFxcWlQMkJHemtN?=
 =?utf-8?B?UUJSM05UbURrWFJZbzBTRTVENWhlMVNRakd2ZjFXRXhEa2h2dmt5UXJCM1dL?=
 =?utf-8?B?cWRmMVhCV1EvT1FQWVVLb0JxRXNUT2hPQXZmSU9xMWNhRXd6R21QZmJ5Nkgw?=
 =?utf-8?B?NFNBNExuZE5LY2ZiWGNMZGVhd2Z0MmRNYzQwTzFGTDg5TzVIeGN4c2hEWUQw?=
 =?utf-8?B?SGpwbmxUOUpJcTNCbVozTHVxbkkwUlJuUlNtcHBqazFmVUJLQ09ZRjZ0dWZD?=
 =?utf-8?B?MnFvNTE3NFVPVkdRdytFRGVDWG9HL0lRYVdITThGMlpsVlY0Vlg3Tks5eGVF?=
 =?utf-8?B?aVlCT3h4eEpnNEJxNTkxMTlOMlMrYVNidGM0eHQ2cXpQMEhMMTgzODVnN25k?=
 =?utf-8?B?bVBtdUdFUFF0M3NwK1AwQUNtRE9NN3orUnRkbG9xQnhWQmtVdDYzREZtWUxK?=
 =?utf-8?B?NE5wdFZMRkZOMk8rNURCTExLRmIxT3hiWGRTdWZpNlN1QjYvbGg1dnd2NkNw?=
 =?utf-8?B?OG1RNmxlM3c4NFVoQUp0eWdvZHVaN2QzY1F6RTFSeXdNc25aWGVxVG9XUXVG?=
 =?utf-8?B?OUlEYjhDZHV4NmFLdGd3OTNSb3pzNkduYjB4RTNVQUtCc2QzNUVEWmM0YWhP?=
 =?utf-8?B?bmtVTHpDNkU1ZDY0VVU4b2tJT0hGanJHSTlsOVZYVU9VbW5lSEpJZW0yQkx1?=
 =?utf-8?B?MEc5R2VCOUtFaWlPbC93dTFMMWNWUmlvTm4xN3F5U3c0WHJSM2FsUmRjWHNI?=
 =?utf-8?B?c0E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 6ae0ebc5-f2a8-41e7-7f65-08dc7e32a21d
X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB8718.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 May 2024 09:51:52.4114
 (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: iI3j6UHhvwevTS2rC30zxWlItf+0qK/1vTibRPR1fxNhIjIh3vNkV2ocX1GMnbofU74hNaBibRD3hBrMNGUv3PRjtiqfpXpC3u7cP2VnksY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB8090
X-OriginatorOrg: intel.com

From: Yury Norov <yury.norov@gmail.com>
Date: Fri, 24 May 2024 11:58:28 -0700

> Now that assign_bit() is a thin macro wrapper around set_bit() and
> clear_bit(), we can use it in cpumask API and drop duplicating
> implementations of set_cpu_xxx() helpers with no additional overhead.
> 
> Bloat-o-meter reports almost 2k less of generated code for allyesconfig,
> mostly in kernel/cpu.c:
> 	add/remove: 2/4 grow/shrink: 3/4 up/down: 498/-2228 (-1730)

Nice save!

> 
> Signed-off-by: Yury Norov <yury.norov@gmail.com>

Reviewed-by: Alexander Lobakin <aleksander.lobakin@intel.com>

Thanks,
Olek