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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 924F3C433F5 for ; Sat, 23 Apr 2022 17:53:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232450AbiDWR4P (ORCPT ); Sat, 23 Apr 2022 13:56:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46092 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229656AbiDWR4M (ORCPT ); Sat, 23 Apr 2022 13:56:12 -0400 Received: from verein.lst.de (verein.lst.de [213.95.11.211]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9DD5B218B for ; Sat, 23 Apr 2022 10:53:14 -0700 (PDT) Received: by verein.lst.de (Postfix, from userid 2407) id 95F4568D05; Sat, 23 Apr 2022 19:53:09 +0200 (CEST) Date: Sat, 23 Apr 2022 19:53:09 +0200 From: Christoph Hellwig To: Kanchan Joshi Cc: Christoph Hellwig , Kanchan Joshi , Jens Axboe , io-uring@vger.kernel.org, linux-nvme@lists.infradead.org, Pavel Begunkov , Ming Lei , Luis Chamberlain , Pankaj Raghav , Javier =?iso-8859-1?Q?Gonz=E1lez?= , Anuj Gupta Subject: Re: [RFC 5/5] nvme: wire-up support for async-passthru on char-device. Message-ID: <20220423175309.GC29219@lst.de> References: <20220401110310.611869-1-joshi.k@samsung.com> <20220401110310.611869-6-joshi.k@samsung.com> <20220404072016.GD444@lst.de> <20220405060224.GE23698@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: io-uring@vger.kernel.org On Wed, Apr 06, 2022 at 10:50:14AM +0530, Kanchan Joshi wrote: > > In that case we will base the newer version on its top. > But if it saves some cycles for you, and also the travel from nvme to > linux-block tree - I can carry that refactoring as a prep patch in > this series. Your call. FYI, this is what I have so far: http://git.infradead.org/users/hch/misc.git/shortlog/refs/heads/nvme-passthrough-refactor the idea would be to use these lower level helpers for uring, and not really share the higher level function at all. This does create a little extra code, but I think it'll be more modular and better maintainable. Feel free to pull this in if it helps you, otherwise I'll try to find some time to do more than just light testing and will post it.