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 48FE4C6FA82 for ; Mon, 12 Sep 2022 19:28:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id D54458D0001; Mon, 12 Sep 2022 15:28:07 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id CDC806B0075; Mon, 12 Sep 2022 15:28:07 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B7EDC8D0001; Mon, 12 Sep 2022 15:28:07 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id AA2A96B0074 for ; Mon, 12 Sep 2022 15:28:07 -0400 (EDT) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 87EBC1A1162 for ; Mon, 12 Sep 2022 19:28:07 +0000 (UTC) X-FDA: 79904419014.04.B1DC34E Received: from mx0a-00082601.pphosted.com (mx0a-00082601.pphosted.com [67.231.145.42]) by imf08.hostedemail.com (Postfix) with ESMTP id 2D58D16009F for ; Mon, 12 Sep 2022 19:28:06 +0000 (UTC) Received: from pps.filterd (m0148461.ppops.net [127.0.0.1]) by mx0a-00082601.pphosted.com (8.17.1.5/8.17.1.5) with ESMTP id 28CGgk3p023956 for ; Mon, 12 Sep 2022 12:28:06 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fb.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=facebook; bh=OMABLFRhm1RQqGMWOQ0WhDpDVmP34BVIsdLo+W65GBU=; b=HWZ44GeiIhtR+qqhvxxFIMCgYb+sJSbwxeEOqDupLf/Jkluf/5/QcOCqDSI0GzvHcxYw 3wnI5pGodOL57Fz+RJj7DeCJ/pcUWhVo/66YZRrasbwoKwaHO0ed9w0x6kyiUjkyx8Nd 1pWLVXxUCjDF4Wh9wohjbqB60TtgjG43L34= Received: from maileast.thefacebook.com ([163.114.130.16]) by mx0a-00082601.pphosted.com (PPS) with ESMTPS id 3jgqpv4n60-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Mon, 12 Sep 2022 12:28:06 -0700 Received: from twshared20183.05.prn5.facebook.com (2620:10d:c0a8:1b::d) by mail.thefacebook.com (2620:10d:c0a8:83::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Mon, 12 Sep 2022 12:28:04 -0700 Received: by dev1180.prn1.facebook.com (Postfix, from userid 425415) id 04CC9208522E; Mon, 12 Sep 2022 12:27:54 -0700 (PDT) From: Stefan Roesch To: , , , CC: , , , Subject: [PATCH v3 04/12] btrfs: add the ability to use NO_FLUSH for data reservations Date: Mon, 12 Sep 2022 12:27:44 -0700 Message-ID: <20220912192752.3785061-5-shr@fb.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20220912192752.3785061-1-shr@fb.com> References: <20220912192752.3785061-1-shr@fb.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-FB-Internal: Safe Content-Type: text/plain X-Proofpoint-GUID: OQ7qQe0jLrzPySpGAEB55bZrLspxmX5a X-Proofpoint-ORIG-GUID: OQ7qQe0jLrzPySpGAEB55bZrLspxmX5a X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.528,FMLib:17.11.122.1 definitions=2022-09-12_12,2022-09-12_02,2022-06-22_01 ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1663010887; 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=OMABLFRhm1RQqGMWOQ0WhDpDVmP34BVIsdLo+W65GBU=; b=ON4UJQ/JZ16N5QrTabw5aM8ACC3AgEZk5oQHha1rg5cYSOoZ0ZS7Vg+1cq8leKVk5oEF6f fwh70aQ8z4qhxvbODbWQNzF5OGvdtuTEizYtxR4dRWO3rxEe9waOVO+S/cp1dQDGMtANJy mMbfAmyixGBqTq3ZXMZthcUTEiBLMw8= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=pass header.d=fb.com header.s=facebook header.b=HWZ44Gei; spf=pass (imf08.hostedemail.com: domain of "prvs=025416ff2e=shr@fb.com" designates 67.231.145.42 as permitted sender) smtp.mailfrom="prvs=025416ff2e=shr@fb.com"; dmarc=pass (policy=reject) header.from=fb.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1663010887; a=rsa-sha256; cv=none; b=wfFdQp+I0ZqsC2xJoMwxHa6GDavuQJoY1yy1hjh+2gops9EjecBi/ZlnExynYEJZ6ephjC aPb5hvChhbsQ7ysXize+yyfVgVoqbI7nga2bDjfrBjEslpm4DFuV9QVoDUBZXyTNTr/13p lpdyqHtVTdW6AAveL7F1Lm4wSYysMC4= X-Rspam-User: X-Stat-Signature: 7g586w3yb95pk6wa5gjncs797jpdr91d X-Rspamd-Queue-Id: 2D58D16009F Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=fb.com header.s=facebook header.b=HWZ44Gei; spf=pass (imf08.hostedemail.com: domain of "prvs=025416ff2e=shr@fb.com" designates 67.231.145.42 as permitted sender) smtp.mailfrom="prvs=025416ff2e=shr@fb.com"; dmarc=pass (policy=reject) header.from=fb.com X-Rspamd-Server: rspam04 X-HE-Tag: 1663010886-753473 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: From: Josef Bacik In order to accommodate NOWAIT IOCB's we need to be able to do NO_FLUSH data reservations, so plumb this through the delalloc reservation system. Signed-off-by: Josef Bacik Signed-off-by: Stefan Roesch --- fs/btrfs/block-group.c | 2 +- fs/btrfs/delalloc-space.c | 13 ++++++++++--- fs/btrfs/delalloc-space.h | 3 ++- fs/btrfs/file.c | 2 +- fs/btrfs/inode.c | 4 ++-- fs/btrfs/space-info.c | 3 ++- 6 files changed, 18 insertions(+), 9 deletions(-) diff --git a/fs/btrfs/block-group.c b/fs/btrfs/block-group.c index e0375ba9d0fe..9df51245ba93 100644 --- a/fs/btrfs/block-group.c +++ b/fs/btrfs/block-group.c @@ -2869,7 +2869,7 @@ static int cache_save_setup(struct btrfs_block_grou= p *block_group, cache_size *=3D fs_info->sectorsize; =20 ret =3D btrfs_check_data_free_space(BTRFS_I(inode), &data_reserved, 0, - cache_size); + cache_size, false); if (ret) goto out_put; =20 diff --git a/fs/btrfs/delalloc-space.c b/fs/btrfs/delalloc-space.c index 1e8f17ff829e..118b2e20b2e1 100644 --- a/fs/btrfs/delalloc-space.c +++ b/fs/btrfs/delalloc-space.c @@ -127,9 +127,11 @@ int btrfs_alloc_data_chunk_ondemand(struct btrfs_ino= de *inode, u64 bytes) } =20 int btrfs_check_data_free_space(struct btrfs_inode *inode, - struct extent_changeset **reserved, u64 start, u64 len) + struct extent_changeset **reserved, u64 start, + u64 len, bool noflush) { struct btrfs_fs_info *fs_info =3D inode->root->fs_info; + enum btrfs_reserve_flush_enum flush =3D BTRFS_RESERVE_FLUSH_DATA; int ret; =20 /* align the range */ @@ -137,7 +139,12 @@ int btrfs_check_data_free_space(struct btrfs_inode *= inode, round_down(start, fs_info->sectorsize); start =3D round_down(start, fs_info->sectorsize); =20 - ret =3D btrfs_alloc_data_chunk_ondemand(inode, len); + if (noflush) + flush =3D BTRFS_RESERVE_NO_FLUSH; + else if (btrfs_is_free_space_inode(inode)) + flush =3D BTRFS_RESERVE_FLUSH_FREE_SPACE_INODE; + + ret =3D btrfs_reserve_data_bytes(fs_info, len, flush); if (ret < 0) return ret; =20 @@ -454,7 +461,7 @@ int btrfs_delalloc_reserve_space(struct btrfs_inode *= inode, { int ret; =20 - ret =3D btrfs_check_data_free_space(inode, reserved, start, len); + ret =3D btrfs_check_data_free_space(inode, reserved, start, len, false)= ; if (ret < 0) return ret; ret =3D btrfs_delalloc_reserve_metadata(inode, len, len, false); diff --git a/fs/btrfs/delalloc-space.h b/fs/btrfs/delalloc-space.h index 28bf5c3ef430..e07d46043455 100644 --- a/fs/btrfs/delalloc-space.h +++ b/fs/btrfs/delalloc-space.h @@ -7,7 +7,8 @@ struct extent_changeset; =20 int btrfs_alloc_data_chunk_ondemand(struct btrfs_inode *inode, u64 bytes= ); int btrfs_check_data_free_space(struct btrfs_inode *inode, - struct extent_changeset **reserved, u64 start, u64 len); + struct extent_changeset **reserved, u64 start, u64 len, + bool noflush); void btrfs_free_reserved_data_space(struct btrfs_inode *inode, struct extent_changeset *reserved, u64 start, u64 len); void btrfs_delalloc_release_space(struct btrfs_inode *inode, diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c index f4aa198f0f87..0f257205c63d 100644 --- a/fs/btrfs/file.c +++ b/fs/btrfs/file.c @@ -1664,7 +1664,7 @@ static noinline ssize_t btrfs_buffered_write(struct= kiocb *iocb, extent_changeset_release(data_reserved); ret =3D btrfs_check_data_free_space(BTRFS_I(inode), &data_reserved, pos, - write_bytes); + write_bytes, false); if (ret < 0) { /* * If we don't have to COW at the offset, reserve diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index e0fe2c4b721f..52b3abb4c57c 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -4881,7 +4881,7 @@ int btrfs_truncate_block(struct btrfs_inode *inode,= loff_t from, loff_t len, block_end =3D block_start + blocksize - 1; =20 ret =3D btrfs_check_data_free_space(inode, &data_reserved, block_start, - blocksize); + blocksize, false); if (ret < 0) { if (btrfs_check_nocow_lock(inode, block_start, &write_bytes) > 0) { /* For nocow case, no need to reserve data space */ @@ -7766,7 +7766,7 @@ static int btrfs_dio_iomap_begin(struct inode *inod= e, loff_t start, if (write && !(flags & IOMAP_NOWAIT)) { ret =3D btrfs_check_data_free_space(BTRFS_I(inode), &dio_data->data_reserved, - start, data_alloc_len); + start, data_alloc_len, false); if (!ret) dio_data->data_space_reserved =3D true; else if (ret && !(BTRFS_I(inode)->flags & diff --git a/fs/btrfs/space-info.c b/fs/btrfs/space-info.c index 435559ba94fa..a9d4bd374462 100644 --- a/fs/btrfs/space-info.c +++ b/fs/btrfs/space-info.c @@ -1737,7 +1737,8 @@ int btrfs_reserve_data_bytes(struct btrfs_fs_info *= fs_info, u64 bytes, int ret; =20 ASSERT(flush =3D=3D BTRFS_RESERVE_FLUSH_DATA || - flush =3D=3D BTRFS_RESERVE_FLUSH_FREE_SPACE_INODE); + flush =3D=3D BTRFS_RESERVE_FLUSH_FREE_SPACE_INODE || + flush =3D=3D BTRFS_RESERVE_NO_FLUSH); ASSERT(!current->journal_info || flush !=3D BTRFS_RESERVE_FLUSH_DATA); =20 ret =3D __reserve_bytes(fs_info, data_sinfo, bytes, flush); --=20 2.30.2