From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailout2.samsung.com (mailout2.samsung.com [203.254.224.25]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BBFD723A990 for <io-uring@vger.kernel.org>; Wed, 2 Apr 2025 13:21:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=203.254.224.25 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743600087; cv=none; b=UdkGR/vnji8SQ4WXeryUnTMBo4ovi3VOiXG+o7grdLxaMruCGSgHbtP2P4Q23nOhTXlleUfIvjyDBgeIaEhlHtHzJeywYmSW/ukooBwYP/wrcUb8588DCs7VQiEw3MusWQ0EkNP7RQNZVqf3OLlBOmD93yN/okeP1eZSfh8iwh0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743600087; c=relaxed/simple; bh=E4a4QPkVxLQnQSAJ3gMsfo/vKUKAMQjCjdoWo1bROC0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:From:In-Reply-To: Content-Type:References; b=IImCFwAPr8yK1s6wlwBFjlQR/vQgTyC5RpFFncAPz4S9unRJJLz0A+53uBV9iImr0NMoZmmkzhIYNBKvz4NzIdDMcecISDePk5a/tcIiBYFTpRLmkJxptO3XkxmXKbbtOtFQV+Z1Lg0w84zoZozuKrNDWbXnhZK7AgfFBX1Gh5E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com; spf=pass smtp.mailfrom=samsung.com; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b=Fx4SQlT6; arc=none smtp.client-ip=203.254.224.25 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=samsung.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b="Fx4SQlT6" Received: from epcas5p3.samsung.com (unknown [182.195.41.41]) by mailout2.samsung.com (KnoxPortal) with ESMTP id 20250402132117epoutp02a42261892625b2a447ae737ac74f06e9~ygwhPTBmK0038900389epoutp02D for <io-uring@vger.kernel.org>; Wed, 2 Apr 2025 13:21:17 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.samsung.com 20250402132117epoutp02a42261892625b2a447ae737ac74f06e9~ygwhPTBmK0038900389epoutp02D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1743600077; bh=E4a4QPkVxLQnQSAJ3gMsfo/vKUKAMQjCjdoWo1bROC0=; h=Date:Subject:To:Cc:From:In-Reply-To:References:From; b=Fx4SQlT6xpR1gLFnURpCdtu9hnIM2YWEci4Ovxm4Le9XQcVhqMZ9UZbaE5ENOnBo6 ohf5KjtZ6k0NaFzkVV2vW03aRTv7d8B1eYHoyjrd/aBZJJFwqkdyckEoabhHIkoHNA 91Mcf3Bv/bvp5QG5H3y3l4pC8uPSYc0ySyUFAhBo= Received: from epsnrtp01.localdomain (unknown [182.195.42.153]) by epcas5p3.samsung.com (KnoxPortal) with ESMTPS id 20250402132116epcas5p36172588119bb80373180f5a4c1d05e6b~ygwgzGXiT0421804218epcas5p3B; Wed, 2 Apr 2025 13:21:16 +0000 (GMT) Received: from epsmges5p2new.samsung.com (unknown [182.195.38.179]) by epsnrtp01.localdomain (Postfix) with ESMTP id 4ZSQVg0LzTz6B9m4; Wed, 2 Apr 2025 13:21:15 +0000 (GMT) Received: from epcas5p4.samsung.com ( [182.195.41.42]) by epsmges5p2new.samsung.com (Symantec Messaging Gateway) with SMTP id D5.53.09853.AC93DE76; Wed, 2 Apr 2025 22:21:14 +0900 (KST) Received: from epsmtrp1.samsung.com (unknown [182.195.40.13]) by epcas5p3.samsung.com (KnoxPortal) with ESMTPA id 20250402132114epcas5p3dd1ea9335fb7e7b851820410c29b9265~ygweLXxh10400704007epcas5p3W; Wed, 2 Apr 2025 13:21:14 +0000 (GMT) Received: from epsmgms1p2new.samsung.com (unknown [182.195.42.42]) by epsmtrp1.samsung.com (KnoxPortal) with ESMTP id 20250402132114epsmtrp1365f215a05329a109fd4f445cee78f4f~ygweKqyhm0388903889epsmtrp16; Wed, 2 Apr 2025 13:21:14 +0000 (GMT) X-AuditID: b6c32a4a-048e07000000267d-6c-67ed39ca0049 Received: from epsmtip1.samsung.com ( [182.195.34.30]) by epsmgms1p2new.samsung.com (Symantec Messaging Gateway) with SMTP id 87.5B.08805.9C93DE76; Wed, 2 Apr 2025 22:21:14 +0900 (KST) Received: from [107.122.11.51] (unknown [107.122.11.51]) by epsmtip1.samsung.com (KnoxPortal) with ESMTPA id 20250402132112epsmtip164f0a3332ab9afbd1a6783e64129bfaa~ygwcqGxk82563025630epsmtip1g; Wed, 2 Apr 2025 13:21:12 +0000 (GMT) Message-ID: <8b7085de-cc9b-45dc-b5bf-1800085fd01d@samsung.com> Date: Wed, 2 Apr 2025 18:51:11 +0530 Precedence: bulk X-Mailing-List: io-uring@vger.kernel.org List-Id: <io-uring.vger.kernel.org> List-Subscribe: <mailto:io-uring+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:io-uring+unsubscribe@vger.kernel.org> MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 3/3] nvme/ioctl: move fixed buffer lookup to nvme_uring_cmd_io() To: Keith Busch <kbusch@kernel.org> Cc: Caleb Sander Mateos <csander@purestorage.com>, Jens Axboe <axboe@kernel.dk>, Christoph Hellwig <hch@lst.de>, Sagi Grimberg <sagi@grimberg.me>, Pavel Begunkov <asml.silence@gmail.com>, Chaitanya Kulkarni <kch@nvidia.com>, linux-nvme@lists.infradead.org, io-uring@vger.kernel.org, linux-kernel@vger.kernel.org Content-Language: en-US From: Kanchan Joshi <joshi.k@samsung.com> In-Reply-To: <Z-qoevl5Jaf7WFsQ@kbusch-mbp.dhcp.thefacebook.com> Content-Transfer-Encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrDJsWRmVeSWpSXmKPExsWy7bCmlu4py7fpBtc6bCzmrNrGaLH6bj+b xf+Fc1ktVq4+ymTxrvUci8WkQ9cYLZ5encVkcXnXHDaL+cuesluse/2exYHLY+esu+we5+9t ZPG4fLbUY9OqTjaPzUvqPXbfbGDz6G1+x+ax7WEvu8fnTXIBnFHZNhmpiSmpRQqpecn5KZl5 6bZK3sHxzvGmZgaGuoaWFuZKCnmJuam2Si4+AbpumTlAdyoplCXmlAKFAhKLi5X07WyK8ktL UhUy8otLbJVSC1JyCkwK9IoTc4tL89L18lJLrAwNDIxMgQoTsjO+npnFVrCKq+Ld1ifMDYz7 OboYOTkkBEwklu94yNbFyMUhJLCbUWLz553MEM4nRonDPzdDOd8YJdZdn8EG03K6bR8rRGIv o8Sm469YIJy3jBK/7r1hAqniFbCTeLv9BguIzSKgIjG7bwMzRFxQ4uTMJ2BxUQF5ifu3ZrCD 2MICkRI93RsZQWwRAWWJu/Nngm1gFtjPJLHmfRNYA7OAuMStJ/OBFnBwsAloSlyYXApicgrY S9x+rAVRIS+x/e0csKslBPZwSNz5e4sR4moXiTNT9jBB2MISr45vYYewpSRe9rdB2dkSDx49 YIGwayR2bO5jhbDtJRr+3GAF2cUMtHb9Ln2IXXwSvb+fgF0jIcAr0dEmBFGtKHFv0lOoTnGJ hzOWQNkeEjd7n0GDaiOTxO2/J9kmMCrMQgqVWUienIXknVkImxcwsqxilEwtKM5NTy02LTDK Sy2HR3hyfu4mRnAy1vLawfjwwQe9Q4xMHIyHGCU4mJVEeAu13qYL8aYkVlalFuXHF5XmpBYf YjQFRs9EZinR5HxgPsgriTc0sTQwMTMzM7E0NjNUEudt3tmSLiSQnliSmp2aWpBaBNPHxMEp 1cDE4XZeTGGKzjNGu2VZr+at5/7hKqo9u4ZpoZbJ9hsR8/d/vmzs91a8OHTrAvNN999IGFa/ uFfwVHpb0/lTNj+y9dWE4uYHF5wWXcTz1+EG95QXM23rxU/MrWrpnGNatt2tLzrKysCAY89W vd1b/vVyfTaO8fHorlgZ9Mz0a0t5I4fJeyGGFQHuz+W79rPazbC6WHEkw0vytZOr2Hv19Km7 Z2vGXV0aLsgqxXnrv77jhs1uHO/XX9z5qM1BNZfZ65Cd/Pab8z98+cTMsKvNX/nBspxdKkmf DosWFp2xiJN8vXLZ+lsbchOrko+bbXKKq10xT6jY9pUBR0tA2f3DO6fmTl3Ge6yiKdxxlZnE NhElluKMREMt5qLiRADumLO0TwQAAA== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprBIsWRmVeSWpSXmKPExsWy7bCSnO4py7fpBh+eMVnMWbWN0WL13X42 i/8L57JarFx9lMniXes5FotJh64xWjy9OovJ4vKuOWwW85c9ZbdY9/o9iwOXx85Zd9k9zt/b yOJx+Wypx6ZVnWwem5fUe+y+2cDm0dv8js1j28Nedo/Pm+QCOKO4bFJSczLLUov07RK4Mr6e mcVWsIqr4t3WJ8wNjPs5uhg5OSQETCROt+1j7WLk4hAS2M0o8eLBZyaIhLhE87Uf7BC2sMTK f8/ZIYpeM0qseLwLLMErYCfxdvsNFhCbRUBFYnbfBmaIuKDEyZlPwOKiAvIS92/NAKsXFoiU 6OneyAhiiwgoS9ydPxNsM7PAfiaJz7ubmCA2bGSSeDlvLVgHM9AZt57MB0pwcLAJaEpcmFwK YnIK2EvcfqwFUWEm0bW1ixHClpfY/nYO8wRGoVlIzpiFZNAsJC2zkLQsYGRZxSiZWlCcm55b bFhglJdarlecmFtcmpeul5yfu4kRHHlaWjsY96z6oHeIkYmD8RCjBAezkghvodbbdCHelMTK qtSi/Pii0pzU4kOM0hwsSuK83173pggJpCeWpGanphakFsFkmTg4pRqYmt8mCRfuSDhe3cnC ZPnp5Can/pBjL5Yas+jvYJ9rtrcu/JhlxzMumUMH533/qibtFiprylbZk2my5ee21iVnHd58 XV5RrXrpwr7TYV9YBOXf+vP2TM++EvJqlu1am3MV/GG2gSKal598WfXowP0oTuazc7RNVYLW 11/728Vu+dfn8H3p5uq8Z3X53dLsrw2kfi4VcOtLfqG5V9HU68Kpzy1zk7llv86sORJ48tW5 +5UGaktjmwuijj128d8+L2UKQ+kOb/clJydEd7xXFnupv9JqmtfcSb8WrJOVVF+5mk2leemP m8eUT6zY/XpV+MkTBmcmXDDJYRA2NGPh3mJV/ZWz/67O+7/CNhu2uMzeq8RSnJFoqMVcVJwI AJ2sI+UrAwAA X-CMS-MailID: 20250402132114epcas5p3dd1ea9335fb7e7b851820410c29b9265 X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" CMS-TYPE: 105P DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20250328155548epcas5p2368eb1a59883b133a9baf4ac39d6bac6 References: <20250328154647.2590171-1-csander@purestorage.com> <CGME20250328155548epcas5p2368eb1a59883b133a9baf4ac39d6bac6@epcas5p2.samsung.com> <20250328154647.2590171-4-csander@purestorage.com> <48b9a876-0e3b-4c89-9aa3-b48f502868c3@samsung.com> <Z-qoevl5Jaf7WFsQ@kbusch-mbp.dhcp.thefacebook.com> On 3/31/2025 8:06 PM, Keith Busch wrote: > On Mon, Mar 31, 2025 at 12:16:58PM +0530, Kanchan Joshi wrote: >> On 3/28/2025 9:16 PM, Caleb Sander Mateos wrote: >>> For NVMe passthru operations with fixed buffers, the fixed buffer lookup >>> happens in io_uring_cmd_import_fixed(). But nvme_uring_cmd_io() can >>> return -EAGAIN first from nvme_alloc_user_request() if all tags in the >>> tag set are in use. This ordering difference is observable when using >>> UBLK_U_IO_{,UN}REGISTER_IO_BUF SQEs to modify the fixed buffer table. If >>> the NVMe passthru operation is followed by UBLK_U_IO_UNREGISTER_IO_BUF >>> to unregister the fixed buffer and the NVMe passthru goes async, the >>> fixed buffer lookup will fail because it happens after the unregister. >> while the patch looks fine, I wonder what setup is required to >> trigger/test this. Given that io_uring NVMe passthru is on the char >> device node, and ublk does not take char device as the backing file. >> Care to explain? > Not sure I understand the question. A ublk daemon can use anything it > wants on the backend. Are you just referring to the public ublksrv > implementation? That's not used here, if that's what you mean. got it, I did not think beyond public ublksrv. The userspace block over nvme char-device sounds interesting.