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 89B71C4708E for ; Thu, 5 Jan 2023 20:21:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0EDA18E0002; Thu, 5 Jan 2023 15:21:56 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 09E5C8E0001; Thu, 5 Jan 2023 15:21:56 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EA77A8E0002; Thu, 5 Jan 2023 15:21:55 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id DA5AE8E0001 for ; Thu, 5 Jan 2023 15:21:55 -0500 (EST) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id A9FC1A0218 for ; Thu, 5 Jan 2023 20:21:55 +0000 (UTC) X-FDA: 80321866590.29.16E555E Received: from sin.source.kernel.org (sin.source.kernel.org [145.40.73.55]) by imf21.hostedemail.com (Postfix) with ESMTP id 1D1121C0015 for ; Thu, 5 Jan 2023 20:21:51 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=dBfuqzFn; spf=pass (imf21.hostedemail.com: domain of kbusch@kernel.org designates 145.40.73.55 as permitted sender) smtp.mailfrom=kbusch@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1672950113; h=from:from: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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=tqJ+cCFgcvB1+eDjbHEQx9vFf0zfbcMQw98A35koAjo=; b=4iL6HY1dBR2pV3yA17BcRwlAZ+dF6TTFpiz5lbHzcTTg2OE8LufR531LBsIFJQg0KoW2NT RuAbsGgkWT1fvKSdn5irU1NKTv/ANdfOAV3NQ5tXS1PVHQuBbwv06nOUZ2+evgOeOvKOTs IuH/VXM5rPHgSRdkp12njEbIikrWMQg= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=dBfuqzFn; spf=pass (imf21.hostedemail.com: domain of kbusch@kernel.org designates 145.40.73.55 as permitted sender) smtp.mailfrom=kbusch@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1672950113; a=rsa-sha256; cv=none; b=VGRwRFJygx6lqoiMXnWYq6j41U9lWFQYCNPco67fSv8HE5CimTlbel9sgaujg8K48Ue0j1 SHAKV5GEwGPM55Ce/iBIlrYCZHh5sBtpEGPf9L85P7TkiWK0oUuE2UpgRJ0OrFdtaDbokm Tp3tHNrmN+z8/qjacLdZoQBNLP+OZDM= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sin.source.kernel.org (Postfix) with ESMTPS id 5FA7DCE1AA1; Thu, 5 Jan 2023 20:21:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id CDB5BC433D2; Thu, 5 Jan 2023 20:21:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1672950106; bh=ilMX5v3ZE/EhwtB5jOfYPI8G8uBMESmE0BRULcNsyjg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=dBfuqzFnZoDzXlpxBLXV6lk805Xi8ih1YVM0W7jkCx6q5Y81tK/AsPId5dUbkeNA/ 17EzsVUkR1c1CfOmrKrngR435LUUyGsNLf2T2GEkSDCQP5S5x87ffjPB3Mm/C4Qg70 pr4sPnSMS+4U/vh+LBiYdUS05IY0xFK519e+bKQ4moC7pCHk5e5uFwyJEdzv9yZ4qG C9KKQqFNGPXLY0ugloNE/S9nB+lT4RJPKjAE3eg/7MbIN0yg8mf6Z2uew1SgJDRuhQ X2I5wuZbfE22/q7UVUCynyMdfhnOCC3WVhmw+80v8jebMPWSUZyACzW42WgtFqRUiF WHiuJvCXfeK2w== Date: Thu, 5 Jan 2023 13:21:43 -0700 From: Keith Busch To: Jason Gunthorpe Cc: Yishai Hadas , linux-kernel@vger.kernel.org, linux-block@vger.kernel.org, linux-mm@kvack.org, axboe@kernel.dk, logang@deltatee.com, hch@lst.de, alex.williamson@redhat.com, leonro@nvidia.com, maorg@nvidia.com Subject: Re: [PATCH] lib/scatterlist: Fix to merge contiguous pages into the last SG properly Message-ID: References: <20230105112339.107969-1-yishaih@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 1D1121C0015 X-Rspam-User: X-Stat-Signature: gmpotc6dtzdwdarhwxrfomxo8b7bmc4k X-HE-Tag: 1672950111-332856 X-HE-Meta: U2FsdGVkX1+jeYJ+Kzg/6iM8Ha/AQYjtWcGAISjTCNYTJNLq+rpP5PiXOG7eNiYuamJ69bffUCCtYPfNZYdVw5yqKg9biK5VG36hgYiwoMKq4AvStzl9+wQY7Mcpi70yWfyjgwn7Me0twJYeDtgd7KnSoldA10XOc7qkyY+Kt0Euouq0MGQBQLSERuVl9aPiiwinHViGXif7dp5qahVTwdLDdVLXbgFjLg0j9/4DchDobhsgoxJBjytHusEELngWAWq+KSfGo/xf2qplYUIUtDB2Z29bBWSMVPB8KXdVNY29YVURjAR5Q4bQ3jJg9MNnkPn720+G23/X8U49vsq+hd7hbsNyHZ1s/ljhGeraSsiIeZJiy0W9UgcYpbJ/zcuaF9raGZhB1ww4HFLcDAAIhe00F7d2iIs3u3BBvHhaIxhmocpKXCZ20XmdXy7v5x8d+JHYWrf32OE41wBvWKRCfwRMeF0qScNEpgcCKQg1OhGMHfnoWr3Zc5HEDSqXrk0MGUVvUELywqKwLp7IEa42/KGevPp++uGayv6Fjs2biDHmDGFh06YMGtg9LIzjviiqCGdR3l0ceESCQghEoEqoFiX3XCsdhKHryDuhn2CECK+zZPfpCuJl8QPPMPoAW4qfGYCZwN/apad85u32dSGD9JxwVuvDlroBToWRjr0HUbjiN+3NV+5RoVO1jAOGXw2QiuC+6nyboFtuzjWL4eSQgsfXT44R+t/5IK9jfVyMYNVJHSBY9LayyopmCkveYA4Q5qYenMkPaTHxQhe/rvjIloLv5VrgUVJvR9HksLHPejNzK7aHwj6Ps5SyrI2SM9uxNR9kdOqltgMR7YJI7jKzkn07V4ugW5plcTZLwsr9sZ44UdR29yeRRCfvHRdmOz2CftV6BdZMzWjtAQffTb83M4rhuXWNH2RaI+Alm6YI4TrURwVhy1UB73eJILmWtcnpI7+sdweNAfn4nophdlg pBA== 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: On Thu, Jan 05, 2023 at 04:06:11PM -0400, Jason Gunthorpe wrote: > On Thu, Jan 05, 2023 at 01:23:39PM +0200, Yishai Hadas wrote: > > When sg_alloc_append_table_from_pages() calls to pages_are_mergeable() > > in its 'sgt_append->prv' flow to check whether it can merge contiguous > > pages into the last SG, it passes the page arguments in the wrong order. > > > > The first parameter should be the next candidate page to be merged to > > the last page and not the opposite. > > > > The current code leads to a corrupted SG which resulted in OOPs and > > unexpected errors when non-contiguous pages are merged wrongly. > > > > Fix to pass the page parameters in the right order. > > > > Fixes: 1567b49d1a40 ("lib/scatterlist: add check when merging zone device pages") > > Signed-off-by: Yishai Hadas > > --- > > lib/scatterlist.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > rdma is pretty much the only user of this API and this bug is causing > bad data corruption, so I'm going to take it to the rdma tree and send > it tomorrow. > > Which raises the question why the original patch was done at all, > nothing ever inputs pgmap pages into this function? This just takes any arbitrary user addresses, right? The user could provide addresses from mmap'ing pci resource files that resolve to pgmap pages.