From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9B441D711AC for ; Wed, 20 Nov 2024 15:14:53 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3604B6B0093; Wed, 20 Nov 2024 10:14:53 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 310AC6B0095; Wed, 20 Nov 2024 10:14:53 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 18ACE6B0096; Wed, 20 Nov 2024 10:14:53 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id F00176B0093 for ; Wed, 20 Nov 2024 10:14:52 -0500 (EST) Received: from smtpin18.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 923FD120D99 for ; Wed, 20 Nov 2024 15:14:52 +0000 (UTC) X-FDA: 82806819732.18.8FF0510 Received: from mail-pl1-f179.google.com (mail-pl1-f179.google.com [209.85.214.179]) by imf17.hostedemail.com (Postfix) with ESMTP id A55844000A for ; Wed, 20 Nov 2024 15:14:10 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=b3PeMZOM; spf=pass (imf17.hostedemail.com: domain of groeck7@gmail.com designates 209.85.214.179 as permitted sender) smtp.mailfrom=groeck7@gmail.com; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1732115540; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=Ku1MZJAz5Bzs7SE3oqIhwzzExWney3ixxSDkR7dCYa0=; b=2AGNMAtZo9fxQoztOGQB9v89UdlfXsPruw1NCpCIQsXiWNWcBStqDsF9rJRywTik2bRmJc X/BGCzzgOR+y+ObwdY8BYiSJaNQIcFVAnYUkPL+CccQz2hPGVoOuv/BI/Z+OsJwLRsoVP8 /RrEkC+3wdbblWytH+akFMxI5w+hxFw= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1732115540; a=rsa-sha256; cv=none; b=zuY5zYhcu8EqfiHo5Oy+Zk4H37dS9bcXvTorcrnK/YDd7AX9ySJO8WkSK3BgUfYrB2piMB jUN+nT5IhegL2TVVNxyKapD1kbhswxq6Dx1Pljv3kLwYdl39G83ivh+yEnhnEPNEXEjNym nn4UlNMCtijKQHMJ2E4PVAVeEOu88DA= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=pass header.d=gmail.com header.s=20230601 header.b=b3PeMZOM; spf=pass (imf17.hostedemail.com: domain of groeck7@gmail.com designates 209.85.214.179 as permitted sender) smtp.mailfrom=groeck7@gmail.com; dmarc=none Received: by mail-pl1-f179.google.com with SMTP id d9443c01a7336-21271dc4084so6815175ad.2 for ; Wed, 20 Nov 2024 07:14:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1732115689; x=1732720489; darn=kvack.org; h=content-transfer-encoding:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:sender:from:to:cc:subject:date:message-id:reply-to; bh=Ku1MZJAz5Bzs7SE3oqIhwzzExWney3ixxSDkR7dCYa0=; b=b3PeMZOMG9PZWsdGuFJawOTbL90ml9R9IRP0XydvmmrR4DNlwodlt2afcol92EbyRR S4PmG6NwsTkRqcG1oKRT1zwXUcCxiYe8VgHkBepLHJ83JI/p2Pus4pEvcarHCodveGgZ /kmHjyK4b+JtO+oHLlFr7RsgZFGm780nPxrTBOTNNv9XdlWVZ8/JCfUc6FlXcy47hAMr s9ZfH2Myx31Fp2BWsefV4Cnd6f3Ixs2iYhjAhKouSCbYMze6rqPLQSLLuG7jRwFIXmFU AxfbsLBpDFgGB82Nd6qLTX2ZspbC/V/yh8Ny/rOEblqvGN4ksepCiuJSJTtNUO2Mr+xW qIpQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732115689; x=1732720489; h=content-transfer-encoding:in-reply-to:autocrypt:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:sender:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Ku1MZJAz5Bzs7SE3oqIhwzzExWney3ixxSDkR7dCYa0=; b=dYFZ91M3c0hnMdpU7j5qfMUHNeKFgPk/9doAf+XEqh0hkBOJ9RiZ5BhfJ6tJj6uz4u eXRjvRNrBe9k+xvrHxgREFU0yONil7uEmoret7xcvSzo3V5/4/B0hkyYkoDAKlTsj/wA DuozrbIMT1IXBN0kzBore8Df+ltIvMH/3KceJJT5253tn3uKEHxJNu2jlwTugQnhkpNY rnd5aHdDdwAOYuf5vBAcmOC1yvXDFgY2l/H1hxYJwFC9gMej8U7PMSvog3dczKm6I3j8 g/R2fDr8xQhg76M5bT45D2mcLpAT4M42LPiXw6swcLOFDkLO3ssi3dhKRu4Eu1Uh1vK6 2+2w== X-Gm-Message-State: AOJu0Yxkwx2hAxl3In6/PgLC4GVbMwwcbGR21OfYUyW71SmeQxDGj+Vd 532pjZiDPJbzYv1akyUDdaaq9GmnePc2SIrA0aWOq7/k3Y5vAaiH X-Google-Smtp-Source: AGHT+IFCGRfMUgd6ZOZpLObIzKajPBEboqB3VgqgN5jgyWp1xQuRbA4Qz6jd8S5Y+FZY7HoEPC/l7Q== X-Received: by 2002:a17:902:f64e:b0:20c:a19b:8ddd with SMTP id d9443c01a7336-2126cb3c32emr29237725ad.51.1732115688826; Wed, 20 Nov 2024 07:14:48 -0800 (PST) Received: from ?IPV6:2600:1700:e321:62f0:329c:23ff:fee3:9d7c? ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2125bb02b50sm27753165ad.69.2024.11.20.07.14.46 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 20 Nov 2024 07:14:48 -0800 (PST) Message-ID: Date: Wed, 20 Nov 2024 07:14:46 -0800 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] slab: Fix too strict alignment check in create_cache() To: Vlastimil Babka , Geert Uytterhoeven , Christoph Lameter , Pekka Enberg , David Rientjes , Joonsoo Kim , Andrew Morton , Roman Gushchin , Hyeonggon Yoo <42.hyeyoo@gmail.com>, Jens Axboe , Pavel Begunkov , Mike Rapoport , Christian Brauner , Kees Cook , Jann Horn Cc: linux-mm@kvack.org, io-uring@vger.kernel.org, linux-m68k@vger.kernel.org, linux-kernel@vger.kernel.org References: <80c767a5d5927c099aea5178fbf2c897b459fa90.1732106544.git.geert@linux-m68k.org> <4f70f8d3-4ba5-43dc-af1c-f8e207d27e9f@suse.cz> Content-Language: en-US From: Guenter Roeck Autocrypt: addr=linux@roeck-us.net; keydata= xsFNBE6H1WcBEACu6jIcw5kZ5dGeJ7E7B2uweQR/4FGxH10/H1O1+ApmcQ9i87XdZQiB9cpN RYHA7RCEK2dh6dDccykQk3bC90xXMPg+O3R+C/SkwcnUak1UZaeK/SwQbq/t0tkMzYDRxfJ7 nyFiKxUehbNF3r9qlJgPqONwX5vJy4/GvDHdddSCxV41P/ejsZ8PykxyJs98UWhF54tGRWFl 7i1xvaDB9lN5WTLRKSO7wICuLiSz5WZHXMkyF4d+/O5ll7yz/o/JxK5vO/sduYDIlFTvBZDh gzaEtNf5tQjsjG4io8E0Yq0ViobLkS2RTNZT8ICq/Jmvl0SpbHRvYwa2DhNsK0YjHFQBB0FX IdhdUEzNefcNcYvqigJpdICoP2e4yJSyflHFO4dr0OrdnGLe1Zi/8Xo/2+M1dSSEt196rXaC kwu2KgIgmkRBb3cp2vIBBIIowU8W3qC1+w+RdMUrZxKGWJ3juwcgveJlzMpMZNyM1jobSXZ0 VHGMNJ3MwXlrEFPXaYJgibcg6brM6wGfX/LBvc/haWw4yO24lT5eitm4UBdIy9pKkKmHHh7s jfZJkB5fWKVdoCv/omy6UyH6ykLOPFugl+hVL2Prf8xrXuZe1CMS7ID9Lc8FaL1ROIN/W8Vk BIsJMaWOhks//7d92Uf3EArDlDShwR2+D+AMon8NULuLBHiEUQARAQABzTJHdWVudGVyIFJv ZWNrIChMaW51eCBhY2NvdW50KSA8bGludXhAcm9lY2stdXMubmV0PsLBgQQTAQIAKwIbAwYL CQgHAwIGFQgCCQoLBBYCAwECHgECF4ACGQEFAlVcphcFCRmg06EACgkQyx8mb86fmYFg0RAA nzXJzuPkLJaOmSIzPAqqnutACchT/meCOgMEpS5oLf6xn5ySZkl23OxuhpMZTVX+49c9pvBx hpvl5bCWFu5qC1jC2eWRYU+aZZE4sxMaAGeWenQJsiG9lP8wkfCJP3ockNu0ZXXAXwIbY1O1 c+l11zQkZw89zNgWgKobKzrDMBFOYtAh0pAInZ9TSn7oA4Ctejouo5wUugmk8MrDtUVXmEA9 7f9fgKYSwl/H7dfKKsS1bDOpyJlqhEAH94BHJdK/b1tzwJCFAXFhMlmlbYEk8kWjcxQgDWMu GAthQzSuAyhqyZwFcOlMCNbAcTSQawSo3B9yM9mHJne5RrAbVz4TWLnEaX8gA5xK3uCNCeyI sqYuzA4OzcMwnnTASvzsGZoYHTFP3DQwf2nzxD6yBGCfwNGIYfS0i8YN8XcBgEcDFMWpOQhT Pu3HeztMnF3HXrc0t7e5rDW9zCh3k2PA6D2NV4fews9KDFhLlTfCVzf0PS1dRVVWM+4jVl6l HRIAgWp+2/f8dx5vPc4Ycp4IsZN0l1h9uT7qm1KTwz+sSl1zOqKD/BpfGNZfLRRxrXthvvY8 BltcuZ4+PGFTcRkMytUbMDFMF9Cjd2W9dXD35PEtvj8wnEyzIos8bbgtLrGTv/SYhmPpahJA l8hPhYvmAvpOmusUUyB30StsHIU2LLccUPPOwU0ETofVZwEQALlLbQeBDTDbwQYrj0gbx3bq 7kpKABxN2MqeuqGr02DpS9883d/t7ontxasXoEz2GTioevvRmllJlPQERVxM8gQoNg22twF7 pB/zsrIjxkE9heE4wYfN1AyzT+AxgYN6f8hVQ7Nrc9XgZZe+8IkuW/Nf64KzNJXnSH4u6nJM J2+Dt274YoFcXR1nG76Q259mKwzbCukKbd6piL+VsT/qBrLhZe9Ivbjq5WMdkQKnP7gYKCAi pNVJC4enWfivZsYupMd9qn7Uv/oCZDYoBTdMSBUblaLMwlcjnPpOYK5rfHvC4opxl+P/Vzyz 6WC2TLkPtKvYvXmdsI6rnEI4Uucg0Au/Ulg7aqqKhzGPIbVaL+U0Wk82nz6hz+WP2ggTrY1w ZlPlRt8WM9w6WfLf2j+PuGklj37m+KvaOEfLsF1v464dSpy1tQVHhhp8LFTxh/6RWkRIR2uF I4v3Xu/k5D0LhaZHpQ4C+xKsQxpTGuYh2tnRaRL14YMW1dlI3HfeB2gj7Yc8XdHh9vkpPyuT nY/ZsFbnvBtiw7GchKKri2gDhRb2QNNDyBnQn5mRFw7CyuFclAksOdV/sdpQnYlYcRQWOUGY HhQ5eqTRZjm9z+qQe/T0HQpmiPTqQcIaG/edgKVTUjITfA7AJMKLQHgp04Vylb+G6jocnQQX JqvvP09whbqrABEBAAHCwWUEGAECAA8CGwwFAlVcpi8FCRmg08MACgkQyx8mb86fmYHNRQ/+ J0OZsBYP4leJvQF8lx9zif+v4ZY/6C9tTcUv/KNAE5leyrD4IKbnV4PnbrVhjq861it/zRQW cFpWQszZyWRwNPWUUz7ejmm9lAwPbr8xWT4qMSA43VKQ7ZCeTQJ4TC8kjqtcbw41SjkjrcTG wF52zFO4bOWyovVAPncvV9eGA/vtnd3xEZXQiSt91kBSqK28yjxAqK/c3G6i7IX2rg6pzgqh hiH3/1qM2M/LSuqAv0Rwrt/k+pZXE+B4Ud42hwmMr0TfhNxG+X7YKvjKC+SjPjqp0CaztQ0H nsDLSLElVROxCd9m8CAUuHplgmR3seYCOrT4jriMFBtKNPtj2EE4DNV4s7k0Zy+6iRQ8G8ng QjsSqYJx8iAR8JRB7Gm2rQOMv8lSRdjva++GT0VLXtHULdlzg8VjDnFZ3lfz5PWEOeIMk7Rj trjv82EZtrhLuLjHRCaG50OOm0hwPSk1J64R8O3HjSLdertmw7eyAYOo4RuWJguYMg5DRnBk WkRwrSuCn7UG+qVWZeKEsFKFOkynOs3pVbcbq1pxbhk3TRWCGRU5JolI4ohy/7JV1TVbjiDI HP/aVnm6NC8of26P40Pg8EdAhajZnHHjA7FrJXsy3cyIGqvg9os4rNkUWmrCfLLsZDHD8FnU mDW4+i+XlNFUPUYMrIKi9joBhu18ssf5i5Q= In-Reply-To: <4f70f8d3-4ba5-43dc-af1c-f8e207d27e9f@suse.cz> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: A55844000A X-Stat-Signature: 3h1bqniswou3zanj64p179ywxd3x57u9 X-HE-Tag: 1732115650-156531 X-HE-Meta: U2FsdGVkX1+IyFV9Omx3enzh8hCZJ88zeGvp3aZrD7N0B7GMYnBb88+H7KkzUn/I0BAz5Cwp3n0E2S7Lqce2R3QWu4H2b9ncBw6YT4ziFycD07MS3EUyBqu/yi6b2m217gbd8gqJCXscHdqZlgn46zRBJE54r3W5+PchBWzyKIfrB7NW953DjsxunSjFB3N/+cEQaEzbahTB/LMQg/1nDRP7jaEVAJEE9lIoMJkXXI0RP39SNlu5obbxhPmOMqBCs440NnAkhaf9da+NujiS8L5k+osczS98Py6OkUMgVy40SO6B/R+MkbpXE51W4xdsj4nGIHtKMXDKACbMNvJO2dBTw1qF8uxjUHvNK0qdPsoR2iAIC19603aDTwV9iKIJ4l4fzrwirFZ8Y9Mc7lPR/VEmvSfpd2JXeOFH/30aMimm8t/3H0mn/v+EI0zaC82PHiKfpgcolynJ4IHAs0oIf4pvf5yJ+2/qlKezue2YJZbDCecQfDluy8YKVa8fDYYQO4S2SYkzhA0Pw8MZ3n73Qa69ylJ4aEC7LAZwSlfNkqeh7EQrnPMKMSi4eIjq0eulkez0njHiUlWIWDngJyJnijVukPPtdmqp8cDgFbRfFGQ+R1KtCIzSYf2XcJSoqRB+ZdS5wV5jHhOoziTYhACd5S/iT67XhYleUVZxeq9HoZxzoMnwbkwpiqh0ifyGpS9YqdCu08aRqaU4jCbi7R5soUzuQOfIRUymL3fcCU+juyzWHULmOsk/Tx8QdQhaa4lxptXzQCWYfcEVPTrPxhq5eYlt/2PX3TUh5zxThLByRP/goQPxEMmamf0sutrzyAYg/Yz3SfnMdi98/6gN07yBN5trxXTrB6sJS2OcscdG4lauKWEEeDSAahDeroJ74pbOwchQUwOEA4V/sh5JAwyd3eFS9fkdnCjV3ByWlvQ3vxX64x5zJTCbZt/AV1SbEDvP5+YKrsIEw1nPyCuS7t/ 17QU/wKY p6HVrVEi7HnQ05kJrWaDPwpRM3ye7x+1fd14q+3zc7k33MLrKC+7ngVe8Q2U++cWjzZ5sp7RhwvNlBS7kktwW8qTnssPHvCzjXJky3RObqHFzFfN62OSVw2Tmo1dXz1SNhYGupOmCptiQU12Z5JbbDeX8Ei0WsBgPXjOsnpU7/aQ6AujCdVbzzHfSSecFsB0H2jz10DuBOdoivFI2sMEaG9CNpn175mB6bRnYdhNCtERkSwVbseYlip+qiLx28xUlg0SxHF/JiRyiLdebAyQ5Cq65rrsAtAcYDc0otm/Vk13UVNu8gO6lj0GlqXg4NAuk99Lc1IL3+AvKxdT9pVDMMgZHsSWJKzI1a/1AXve3IvNcFGhwchdiW1ZivNFCpyHpe4fEV4tgguJ5+TxP7V6OBv7clvDT8jKZ+6qcMpBsGXc4fV3/quRMLshify3pkUqOEPOuJsaZ3ea+biN2h8RAIC2ikRJobQI3NNS3ws5nDsru+5+F6AB6Pak2/IJLUWwOiIeIg6YJoVVbJ4NRdi7UFXkSxg3/mhL+Ayedzon7IQhTFUgPLW9LgOw3UfJoQyZOWvJw9DRsHQoJ6AFG9+RChsrge7mSPnlZplZrRW+j7FASH65FhjwOv+OGvYadNxJ4beBy0Jkgoy4J8bM0bY57BeRpbG6jYcnQ45qZOHXBG965xeqZOPrEu0ybHhm63DgyvarhIzNAjO1Rir2NlWfETL3JxhIJ5LRgbfOG48Abvv/CT7k= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 11/20/24 07:03, Vlastimil Babka wrote: > On 11/20/24 13:49, Geert Uytterhoeven wrote: >> On m68k, where the minimum alignment of unsigned long is 2 bytes: >> >> Kernel panic - not syncing: __kmem_cache_create_args: Failed to create slab 'io_kiocb'. Error -22 >> CPU: 0 UID: 0 PID: 1 Comm: swapper Not tainted 6.12.0-atari-03776-g7eaa1f99261a #1783 >> Stack from 0102fe5c: >> 0102fe5c 00514a2b 00514a2b ffffff00 00000001 0051f5ed 00425e78 00514a2b >> 0041eb74 ffffffea 00000310 0051f5ed ffffffea ffffffea 00601f60 00000044 >> 0102ff20 000e7a68 0051ab8e 004383b8 0051f5ed ffffffea 000000b8 00000007 >> 01020c00 00000000 000e77f0 0041e5f0 005f67c0 0051f5ed 000000b6 0102fef4 >> 00000310 0102fef4 00000000 00000016 005f676c 0060a34c 00000010 00000004 >> 00000038 0000009a 01000000 000000b8 005f668e 0102e000 00001372 0102ff88 >> Call Trace: [<00425e78>] dump_stack+0xc/0x10 >> [<0041eb74>] panic+0xd8/0x26c >> [<000e7a68>] __kmem_cache_create_args+0x278/0x2e8 >> [<000e77f0>] __kmem_cache_create_args+0x0/0x2e8 >> [<0041e5f0>] memset+0x0/0x8c >> [<005f67c0>] io_uring_init+0x54/0xd2 >> >> The minimal alignment of an integral type may differ from its size, >> hence is not safe to assume that an arbitrary freeptr_t (which is >> basically an unsigned long) is always aligned to 4 or 8 bytes. >> >> As nothing seems to require the additional alignment, it is safe to fix >> this by relaxing the check to the actual minimum alignment of freeptr_t. >> >> Fixes: aaa736b186239b7d ("io_uring: specify freeptr usage for SLAB_TYPESAFE_BY_RCU io_kiocb cache") >> Fixes: d345bd2e9834e2da ("mm: add kmem_cache_create_rcu()") >> Reported-by: Guenter Roeck >> Closes: https://lore.kernel.org/37c588d4-2c32-4aad-a19e-642961f200d7@roeck-us.net >> Signed-off-by: Geert Uytterhoeven > > Thanks, will add it to slab pull for 6.13. > >> --- >> mm/slab_common.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/mm/slab_common.c b/mm/slab_common.c >> index 893d320599151845..f2f201d865c108bd 100644 >> --- a/mm/slab_common.c >> +++ b/mm/slab_common.c >> @@ -230,7 +230,7 @@ static struct kmem_cache *create_cache(const char *name, >> if (args->use_freeptr_offset && >> (args->freeptr_offset >= object_size || >> !(flags & SLAB_TYPESAFE_BY_RCU) || >> - !IS_ALIGNED(args->freeptr_offset, sizeof(freeptr_t)))) >> + !IS_ALIGNED(args->freeptr_offset, __alignof(freeptr_t)))) > > Seems only bunch of places uses __alignof but many use __alignoff__ and this > also is what seems to be documented? __alignoff__ -> __alignof__ Guenter