Received-SPF: pass (google.com: domain of linux-kernel+bounces-71231-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <a293460f-6a40-427f-b5d2-2e701d1af229@amd.com>
Date: Mon, 19 Feb 2024 12:41:20 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] drm/tests/drm_buddy: avoid 64-bit calculation
Content-Language: en-US
To: Arnd Bergmann <arnd@arndb.de>, Randy Dunlap <rdunlap@infradead.org>,
 Arnd Bergmann <arnd@kernel.org>,
 Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
 Maxime Ripard <mripard@kernel.org>, Thomas Zimmermann <tzimmermann@suse.de>
Cc: Dave Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
 Arunpravin Paneer Selvam <Arunpravin.PaneerSelvam@amd.com>,
 David Gow <davidgow@google.com>, =?UTF-8?Q?Ma=C3=ADra_Canal?=
 <mcanal@igalia.com>, Matthew Auld <matthew.auld@intel.com>,
 dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
References: <20240216202442.2493031-1-arnd@kernel.org>
 <745e156e-c3ec-427f-98ad-bfc7d3cfd846@infradead.org>
 <4cdc5b58-11c1-490d-8c3b-6352d8f1b8cb@amd.com>
 <fbd0426c-fdd2-4b7f-a13e-072ed5f973de@app.fastmail.com>
From: =?UTF-8?Q?Christian_K=C3=B6nig?= <christian.koenig@amd.com>
In-Reply-To: <fbd0426c-fdd2-4b7f-a13e-072ed5f973de@app.fastmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?L3ZyTk5EeXR4TmxNaWRWaW1pQVFjUUFjT245SEt0VjJ0a25qVG9FZGREVzRa?=
 =?utf-8?B?U0FWSE9jWGFmYWNUaWJ0UGtkQTVkMlNCNFdyakNBYnhrSy8wQU9MNVkwMXla?=
 =?utf-8?B?V0RPSUVvWnZHQlRPZ3F3dnFCUGtMVFZ6RGNKbk81SlhwTWhpUUdwQmJyVlJY?=
 =?utf-8?B?RmhoQ2lieUVaU1hXUU1nNXZHZFZNSnJXb0NKSlE4a0tSbHYxOG9ib2tmb2Rt?=
 =?utf-8?B?QXNZQ20yK3ZtZEFvWXI3eW9PSDl2eHhVZzd2bDVVV09hMlZkV2FQVWNJdHN4?=
 =?utf-8?B?WGQvM2FENVIydWc3TndmZUgvankzSjJWL2s2M0pEeWZhUVZoaVpKeW5CRThL?=
 =?utf-8?B?VXVvT1Q3d3c4RVUzNVBIN2M3Yk41R29VTEhUTVFZWlU1Si9saUQ0SjJyYVhr?=
 =?utf-8?B?YjN6alkwSzMvUEFVTXRwZ1E3WWd3bWpaWHkydVBxZ0JJOU5kQnVreWl3RlVw?=
 =?utf-8?B?akMyUmdZS0NFc0d0d09LMi90bmFKWkpoakpDYzNQYTcxNU5NYmR1WFdtdE4v?=
 =?utf-8?B?K1R6dzBXYmVURFhhY2Nkc0JtaUdxRTg0MDJ5THJYbk9ob0lNWGFiU3JDUzhu?=
 =?utf-8?B?c1Q1WkFvZUhobzk5M1RmYUpZS0NYWjZ2SFFNQWd0ZGNpQTFJWjdpMlNDeFdo?=
 =?utf-8?B?RmEvN3cwbGpYaU1yZE51UmEwY1h2SjQ1TDY5d1BFaDFsWExXaHdVUGVBZldl?=
 =?utf-8?B?NkRkYjF2Njd5Y0xuZlRaWU5BUTZna052VFNvSzc5aGE5aWE0VlpmaHdOSEhK?=
 =?utf-8?B?SDJ3b2ZENE9ESjBlMGVEUTJmd1oydDFUbU1udUlMRFhBNVFOUmlhUldJTlBM?=
 =?utf-8?B?d3JTNWdXbkdjTFA2czZqZ1YzdEhvTTFMazM4VmpSdVA2dWJNUENkMWFaVm94?=
 =?utf-8?B?cDkyOWgrU3lML0JhMWRkU2lTSGpqSDZTanpFL3pTRFdNL1Q1MTdwQUEzdmdh?=
 =?utf-8?B?RzFvOTMyNVVHb1MrcTVYUDFXRXVBMFlrUnl1RW9SWU12ZGN5bk1YN0pWU0Mx?=
 =?utf-8?B?a2pYV2lIeTRPb1hiL0JpNitwVXBzZUV5NUNEd0FjZ1N5aTVhbjQ2OVB1MkUr?=
 =?utf-8?B?Q3YzWHVFVkROZjFDVTR5cTBqd0pMYXA3RnpKQU5Ha2JZckRhMmRWNUJEQTdQ?=
 =?utf-8?B?YjZ1QjJxdGJrWEY2cUV0OW1aSkNVejIyVFpubCtPUGNQWklQMVh5d2tGNVJR?=
 =?utf-8?B?UFlMZldYQ3JVb2FZeWlvNGx0c2tDTjQzRzcyZUZFRlJ0Z0tMbHh1ZlVoN2FO?=
 =?utf-8?B?K2dwajZKZkVxeGFBeVVzWm5vYml3K29MMW5zamtnVXJsK1dLN09yOWNQTll3?=
 =?utf-8?B?Q3FweGZ1NEJFUUp5eEkwZ3Yva2s5Y09HT1VqbjhPRHVFTUlQbmJGNU1UUDl1?=
 =?utf-8?B?NFZVVHRZYVFhRG1QaTZVUVk3UmFaVk5VSHdEaEFXMUlpSkFOemxqdFQ1aEdV?=
 =?utf-8?B?dXhJTm9MUkRXUDhHRjVZelF1UTU0c21QQk1qT28rcW1iTVVpc0hSbllwQUE0?=
 =?utf-8?B?UHI0MG1VdU0zeFJjT3RaK3Vjb2tPVmt6bm84ejAweUJVaW92a0xLdFJTdTdo?=
 =?utf-8?B?dUVRbVhiUjM1MDNiZUNLbDU2RVMwRlhFUVM0K0UzS1ZISFlqS2tiaFRmVWN3?=
 =?utf-8?B?ZnZNaEY4M0RQckhLQ0NVRWd1ODQzTXNuZ0t0SkkxZEovMjN5VlVqSk9rUW9l?=
 =?utf-8?B?UXRPZWlaNXlKMVZJL0hVakU2ZGlaTnBQKzl1NVljZEZZTG15UkJNS3NFOG4x?=
 =?utf-8?B?Zjk3RjVqYU5YWjJBQ2RFbkdsQTd2cmlualZrUlBtTFBuQTZYWEdKSjI2ZjZI?=
 =?utf-8?B?SWVjYmhSUlRldEROV2R3STRFb3BYeDNlc01CdWpBd055L2hNREF3Qk1ES29o?=
 =?utf-8?B?V3lwcktpVXdaSnRiUjdONHl4N09nV3pad0dSVU9JckR1MlVoTGM5TnpDYVNt?=
 =?utf-8?B?S0hEY3JyQStHbi9JK0Q5QzlUc0NSVnIwbjRyRE52OEdNWHdCRGF0VmdiYmpp?=
 =?utf-8?B?SGw2TTFabHFZeVNldzgyMHlLSEJXNW5CaFFwK2VaNDc0SkJod1I1NDV6ejU1?=
 =?utf-8?B?cFMxOUhQcWtsSzVJR2gzeWNVTGthcy9aOVpXVktBcXRiSnFYN1cxWDVBbnIr?=
 =?utf-8?Q?T7yXT/oNIJpfFVcfFR6O265AS?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1f261ee6-0847-40ae-80a2-08dc313fb4b3
X-MS-Exchange-CrossTenant-AuthSource: PH7PR12MB5685.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Feb 2024 11:41:26.5296
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: E90u0laMipFVJGm3TsX3rddHCXXxNwGMHwa0kGUPie3SJox9n2VTbStN/THLjDg9
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB7272

Am 19.02.24 um 12:29 schrieb Arnd Bergmann:
> On Mon, Feb 19, 2024, at 12:22, Christian König wrote:
>> Am 17.02.24 um 02:31 schrieb Randy Dunlap:
>>> On 2/16/24 12:24, Arnd Bergmann wrote:
>>>> From: Arnd Bergmann <arnd@arndb.de>
>>>>
>>>> The newly added drm_test_buddy_alloc_contiguous() test fails to link on
>>>> 32-bit targets because of inadvertent 64-bit calculations:
>>>>
>>>> ERROR: modpost: "__aeabi_uldivmod" [drivers/gpu/drm/tests/drm_buddy_test.ko] undefined!
>>>> ERROR: modpost: "__aeabi_ldivmod" [drivers/gpu/drm/tests/drm_buddy_test.ko] undefined!
>>>>
>>>> >From what I can tell, the numbers cannot possibly overflow a 32-bit size,
>>>> so use different types for these.
>>>>
>>>> I noticed that the function has another possible flaw in that is mixes
>>>> what it calls pages with 4KB units. This is a big confusing at best,
>>>> or possibly broken when built on machines with larger pages.
>>>>
>>>> Fixes: a64056bb5a32 ("drm/tests/drm_buddy: add alloc_contiguous test")
>>>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>>> Tested-by: Randy Dunlap <rdunlap@infradead.org>
>> I've just pushed a similar patch Mathew came up a bit earlier to
>> drm-misc-fixes.
>>
>> Sorry for the noise, I have to catch up on picking up patches for
>> misc-fixes and misc-next.
> Ok, thanks.
>
> Have you looked at how this code works for larger values of PAGE_SIZE?
> Is there any need to change other things or will this work with the
> hardcoded 4KB chunks?

I haven't looked into the details, but I've pointed out before that 
using PAGE_SIZE in the buddy or its test cases would be incorrect.

Background is that the buddy allocator is for devices and those work 
independent of the CPU PAGE_SIZE. So it can be that on a CPU with 64k 
pages the buddy still needs to work with 4k.

Could be that this is work, but could as well be that this is completely 
broken. Arun and Mathew needs to answer this, I haven't tested it nor 
reviewed the code.

Regards,
Christian.

>
>       Arnd