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 EE644C433F5 for ; Sun, 22 May 2022 10:36:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244217AbiEVKgN (ORCPT ); Sun, 22 May 2022 06:36:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55244 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S245004AbiEVKgI (ORCPT ); Sun, 22 May 2022 06:36:08 -0400 Received: from zeniv-ca.linux.org.uk (zeniv-ca.linux.org.uk [IPv6:2607:5300:60:148a::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BE862F52 for ; Sun, 22 May 2022 03:36:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=linux.org.uk; s=zeniv-20220401; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=q1/yon4TH+c7rx3CKASMVKd8QqdWbFKt9RzdUi5iguU=; b=NjtZJR6z0WHt5J5MB8l/XmFd5L zsPQjkhurEAgLIT0hw0kuYEW22bBC2GCzzdZz0vu1aqykxMe8YNbDloDlb52V55uLbXxaghpOuys9 Rj8lGzftK7heY47B9fWk/zwMdYSoch8Rn51SmkD5aly+/vwAd79G8D+SkowcIyd0RUUnkcPPgNSPG 2M/YESUwT8J6sWk8LwLckycbC80jjrL4vvF9lnhGLZXa27/pn1OsS35CZoYCKy8wAzNkBqFfqV6Ta v1ktXEQcs3ngpnkYBdgN0Sezj6pwZ3q7NDH+Cb5dmkzxaLqNMMlyFKpUZLOEfc2vIrlLHkyfRfOFH KV/ooXtA==; Received: from viro by zeniv-ca.linux.org.uk with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1nsiwK-00H9Ut-Lk; Sun, 22 May 2022 10:36:00 +0000 Date: Sun, 22 May 2022 10:36:00 +0000 From: Al Viro To: Matthew Wilcox Cc: Christoph Hellwig , Jens Axboe , linux-fsdevel@vger.kernel.org Subject: Re: [RFC] what to do with IOCB_DSYNC? Message-ID: References: <20210621140956.GA1887@lst.de> <20210621142235.GA2391@lst.de> <20210621143501.GA3789@lst.de> <70b5e4a8-1daa-dc75-af58-9d82a732a6be@kernel.dk> <20220522074508.GB15562@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: Al Viro Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Sun, May 22, 2022 at 11:23:43AM +0100, Matthew Wilcox wrote: > On Sun, May 22, 2022 at 09:45:09AM +0200, Christoph Hellwig wrote: > > On Sat, May 21, 2022 at 04:14:07PM -0600, Jens Axboe wrote: > > > Then we're almost on par, and it looks like we just need to special case > > > iov_iter_advance() for the nr_segs == 1 as well to be on par. This is on > > > top of your patch as well, fwiw. > > > > > > It might make sense to special case the single segment cases, for both > > > setup, iteration, and advancing. With that, I think we'll be where we > > > want to be, and there will be no discernable difference between the iter > > > paths and the old style paths. > > > > A while ago willy posted patches to support a new ITER type for direct > > userspace pointer without iov. It might be worth looking through the > > archives and test that. > > https://lore.kernel.org/linux-fsdevel/Yba+YSF6mkM%2FGYlK@casper.infradead.org/ Direct kernel pointer, surely? And from a quick look, iov_iter_is_kaddr() checks for the wrong value...