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 93B1BC77B7F for ; Fri, 19 May 2023 14:22:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1518F900004; Fri, 19 May 2023 10:22:08 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 10200900003; Fri, 19 May 2023 10:22:08 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EE498900004; Fri, 19 May 2023 10:22:07 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id DBC56900003 for ; Fri, 19 May 2023 10:22:07 -0400 (EDT) Received: from smtpin16.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id A34821C6E5C for ; Fri, 19 May 2023 14:22:07 +0000 (UTC) X-FDA: 80807219094.16.396E6AD Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by imf11.hostedemail.com (Postfix) with ESMTP id 6DA7A40017 for ; Fri, 19 May 2023 14:22:03 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=AFt2rU6B; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=n6vjt7WK; dmarc=pass (policy=none) header.from=suse.de; spf=pass (imf11.hostedemail.com: domain of hare@suse.de designates 195.135.220.29 as permitted sender) smtp.mailfrom=hare@suse.de ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1684506124; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=WlcBct/wzp7TvW6xV2C5NZk4K9r7Xb2dIshCQXz8B/o=; b=gDPnO9zYcI0gsCkpMAylHcmGwC00TFGa5Af5CEPIDXLaHQQ91ObwBOHS9WdqWQ6mUNxWIb 6bPE4FOdpEmH8v7P8vCE+peiH1NYgxrXSfGuZd/7kcAO3eTg/xYFMp/OOiamZlRMGM6P9r Y+ZTESSKUbBKHenzNv+VqzExyt8csdc= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=pass header.d=suse.de header.s=susede2_rsa header.b=AFt2rU6B; dkim=pass header.d=suse.de header.s=susede2_ed25519 header.b=n6vjt7WK; dmarc=pass (policy=none) header.from=suse.de; spf=pass (imf11.hostedemail.com: domain of hare@suse.de designates 195.135.220.29 as permitted sender) smtp.mailfrom=hare@suse.de ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1684506124; a=rsa-sha256; cv=none; b=KAZgFwKbWm0xbU86UrA9lqpG8T9QZkZCWvSoYp/s4fsGQ/gG4SSIDK6R2pmPQmLdrXfIwG t8UmGoaJnj3M5cG/d+6hb6tMjLivphbt5FiDweMae0jAIZ5fUGVnWrsUVF4kdVWNHG4kaM pT4Ti+xycGqwWMfk1xRY0xkN1Aj9xRY= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id D51C11FDA9; Fri, 19 May 2023 14:22:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1684506122; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=WlcBct/wzp7TvW6xV2C5NZk4K9r7Xb2dIshCQXz8B/o=; b=AFt2rU6BsN/C4LLgMrh7gKGcpHYIHok4I1/ZlixA6CgBGql3l73YlMa9C48tJmr9NqegsT LN8AClWFtwITzTZAUWgoU01jAS1F2lDDOB4JawywQhzdu2BdTVwBO7ISqWbV6ElVvIXcHy pS+mrAVAokuwV8tW3e7shtOpSuVArgU= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1684506122; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=WlcBct/wzp7TvW6xV2C5NZk4K9r7Xb2dIshCQXz8B/o=; b=n6vjt7WKTMi/w+YlBK3seqHcXPDKocScWzVVUDL+yCxdpf+f0GwrDIg68CO/X1/yI4orBU 4xpJjA0dEHwk/dCw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 23E2213A12; Fri, 19 May 2023 14:22:02 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id zjpzBgqGZ2TWHwAAMHmgww (envelope-from ); Fri, 19 May 2023 14:22:02 +0000 Message-ID: Date: Fri, 19 May 2023 16:22:01 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 Subject: Re: [PATCH 16/17] block: use iomap for writes to block devices Content-Language: en-US To: Christoph Hellwig , Jens Axboe Cc: Miklos Szeredi , "Darrick J. Wong" , Andrew Morton , David Howells , Matthew Wilcox , linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, ceph-devel@vger.kernel.org, linux-ext4@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, cluster-devel@redhat.com, linux-xfs@vger.kernel.org, linux-nfs@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org References: <20230424054926.26927-1-hch@lst.de> <20230424054926.26927-17-hch@lst.de> From: Hannes Reinecke In-Reply-To: <20230424054926.26927-17-hch@lst.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 6DA7A40017 X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: 9aiebtfke8m8eo9qoha4tifrdw5fkagp X-HE-Tag: 1684506123-175803 X-HE-Meta: U2FsdGVkX191t/jK4cXEJzb6SKwAaVFMvmsm5SCDbWhxLzHsgcyVyqK6JSI0NxWeu0qEwFjC1UOabHEa8cjEBiEpRRwh2VC2nDGpBUdQPrUwb7kyWN74dEiJvZQZX5SluihyMb3pI0Bw7OY68XMzPiZ0srM4oNHprunoJsi+G1bEDxZbSXtnlacwpFYIzwB4DU6tzFAkz8dWE+dGCCL1AvM0fRzHl8joPDIPLftlRwFQ5HOWhtRQmAx6SypBrM2HmHvcJwEVWwWbXkIKGVWpIVtyN50ALZRzWmOHyyC/HncQnYlsLX76nTRQp0jg7l7cFqUGPKW7vuxZBhGLeGn5PVpECPuV/Zr8c7gY/1e2UlP4G1lRQYbifzSB7ybJRWSXRQIn/+ruLwRL/F5pxtz34M2ZikSLEMA24QC1p9Vmzyy+kA4ILNYLo1SiSLBaH9PKmvk8NW/Fi80VfxM118rjJ3kvIJsfTWlEMgR6BI/x1rNxyROcIKV3x/fQxZMtdAY4dxfhcnb7eJlHtdX226mLXyk+fPUcw4XCs2Y/OZ+/EiT0mvXezk0pHjEaiWeoBoykTiBzw/NCJPd75QQ84u+3D5tldD3hxKFaQSX+ipMo0QBWxPNLOs3N7nuhr6aSn6/z16Eip3y/W6Jo3zOjJ4HoaFIWbUY9pvZ6S0EctmM9xmoPs0uDymYVUDvU0R2+8qnYlFy8R1nvVG7aLjXbME9Mofjcjvyc5wrJF2hUmSIhxP06GeniC36uVeh1HrlRIuboZr9nYpL2HrgQh7QQnD6M/kNYmXreZwInQZxA8/AdkcCscaTQn85XXyp1GnHC8W0h+ApdbkbwJ38E+53rSVy9Y5Q4V0xyuOz3Bwo2dl2gHC4xhJLSpX7dEGonQIPRWtXFTsC/4IBBLYpeB9YE4h/hQkejIlyrLTmt0UaDoHrxaXD15ENOfoUmXgxKgpRK5b3/4ZA2LaBcm4RK0JXSL1+ nZp2+Xmb zll/Uq4ZDMqGOB//nKLNv1oFhUALdeaabQEs7fzJUxZrV7Vx527FzyRKtzNPARcib13fZD52snvAbYRg60lnP/TaQ33WfcLXeRO1+QzXaoxBn2y0vuvqd0hihN5955pbRQEEdnejPGvud4GtDfwRbc4SUFwm/mvayFh5DsGsbD5XaG0ghe3aOpnd0INIuB2GNm0IbbaekoWwmok33zbESCa4+dXp8/IElbaPE 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 4/24/23 07:49, Christoph Hellwig wrote: > Use iomap in buffer_head compat mode to write to block devices. > > Signed-off-by: Christoph Hellwig > --- > block/Kconfig | 1 + > block/fops.c | 33 +++++++++++++++++++++++++++++---- > 2 files changed, 30 insertions(+), 4 deletions(-) > > diff --git a/block/Kconfig b/block/Kconfig > index 941b2dca70db73..672b08f0096ab4 100644 > --- a/block/Kconfig > +++ b/block/Kconfig > @@ -5,6 +5,7 @@ > menuconfig BLOCK > bool "Enable the block layer" if EXPERT > default y > + select IOMAP > select SBITMAP > help > Provide block layer support for the kernel. > diff --git a/block/fops.c b/block/fops.c > index 318247832a7bcf..7910636f8df33b 100644 > --- a/block/fops.c > +++ b/block/fops.c > @@ -15,6 +15,7 @@ > #include > #include > #include > +#include > #include > #include "blk.h" > > @@ -386,6 +387,27 @@ static ssize_t blkdev_direct_IO(struct kiocb *iocb, struct iov_iter *iter) > return __blkdev_direct_IO(iocb, iter, bio_max_segs(nr_pages)); > } > > +static int blkdev_iomap_begin(struct inode *inode, loff_t offset, loff_t length, > + unsigned int flags, struct iomap *iomap, struct iomap *srcmap) > +{ > + struct block_device *bdev = I_BDEV(inode); > + loff_t isize = i_size_read(inode); > + > + iomap->bdev = bdev; > + iomap->offset = ALIGN_DOWN(offset, bdev_logical_block_size(bdev)); > + if (WARN_ON_ONCE(iomap->offset >= isize)) > + return -EIO; I'm hitting this during booting: [ 5.016324] [ 5.030256] iomap_iter+0x11a/0x350 [ 5.030264] iomap_readahead+0x1eb/0x2c0 [ 5.030272] read_pages+0x5d/0x220 [ 5.030279] page_cache_ra_unbounded+0x131/0x180 [ 5.030284] filemap_get_pages+0xff/0x5a0 [ 5.030292] filemap_read+0xca/0x320 [ 5.030296] ? aa_file_perm+0x126/0x500 [ 5.040216] ? touch_atime+0xc8/0x150 [ 5.040224] blkdev_read_iter+0xb0/0x150 [ 5.040228] vfs_read+0x226/0x2d0 [ 5.040234] ksys_read+0xa5/0xe0 [ 5.040238] do_syscall_64+0x5b/0x80 Maybe we should consider this patch: diff --git a/block/fops.c b/block/fops.c index 524b8a828aad..d202fb663f25 100644 --- a/block/fops.c +++ b/block/fops.c @@ -386,10 +386,13 @@ static int blkdev_iomap_begin(struct inode *inode, loff_t offset, loff_t length, iomap->bdev = bdev; iomap->offset = ALIGN_DOWN(offset, bdev_logical_block_size(bdev)); - if (WARN_ON_ONCE(iomap->offset >= isize)) - return -EIO; - iomap->type = IOMAP_MAPPED; - iomap->addr = iomap->offset; + if (WARN_ON_ONCE(iomap->offset >= isize)) { + iomap->type = IOMAP_HOLE; + iomap->addr = IOMAP_NULL_ADDR; + } else { + iomap->type = IOMAP_MAPPED; + iomap->addr = iomap->offset; + } iomap->length = isize - iomap->offset; if (IS_ENABLED(CONFIG_BUFFER_HEAD)) iomap->flags |= IOMAP_F_BUFFER_HEAD; Other that the the system seems fine. Cheers, Hannes