mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Gao Xiang <hsiangkao@linux.alibaba.com>
To: linux-erofs@lists.ozlabs.org
Cc: LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 2/4] erofs: support unencoded inodes for fileio
Date: Wed, 4 Sep 2024 14:56:07 +0800	[thread overview]
Message-ID: <443a883e-7565-43ed-9def-77e0d666c454@linux.alibaba.com> (raw)
In-Reply-To: <20240830032840.3783206-2-hsiangkao@linux.alibaba.com>



On 2024/8/30 11:28, Gao Xiang wrote:
> Since EROFS only needs to handle read requests in simple contexts,
> Just directly use vfs_iocb_iter_read() for data I/Os.
> 
> Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>

Unmapped extent could actually split, already fixed as below:

diff --git a/fs/erofs/fileio.c b/fs/erofs/fileio.c
index 598b865ae25f..7f82238047e6 100644
--- a/fs/erofs/fileio.c
+++ b/fs/erofs/fileio.c
@@ -127,6 +127,7 @@ static int erofs_fileio_scan_folio(struct erofs_fileio *io, struct folio *folio)
  			erofs_put_metabuf(&buf);
  		} else if (!(map->m_flags & EROFS_MAP_MAPPED)) {
  			folio_zero_segment(folio, cur, cur + len);
+			attached = 0;
  		} else {
  			if (io->rq && (map->m_pa + ofs != io->dev.m_pa ||
  				       map->m_deviceid != io->dev.m_deviceid)) {

  parent reply	other threads:[~2024-09-04  6:56 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-30  3:28 [PATCH v2 1/4] erofs: add file-backed mount support Gao Xiang
2024-08-30  3:28 ` [PATCH v2 2/4] erofs: support unencoded inodes for fileio Gao Xiang
2024-08-31  0:19   ` Sandeep Dhavale
2024-08-31  2:54     ` Gao Xiang
2024-08-31  4:25   ` Sandeep Dhavale
2024-08-31  7:54     ` Gao Xiang
2024-09-01 20:47       ` Sandeep Dhavale
2024-09-04  6:56   ` Gao Xiang [this message]
2024-09-05  9:01   ` Chao Yu
2024-09-05  9:13     ` Gao Xiang
2024-08-30  3:28 ` [PATCH v2 3/4] erofs: support compressed " Gao Xiang
2024-09-01 20:47   ` Sandeep Dhavale
2024-09-05  9:01   ` Chao Yu
2024-09-05  9:14     ` Gao Xiang
2024-09-05  9:46   ` Chao Yu
2024-08-30  3:28 ` [PATCH v2 4/4] erofs: mark experimental fscache backend deprecated Gao Xiang
2024-09-01 20:48   ` Sandeep Dhavale
2024-09-05  9:03   ` Chao Yu
2024-08-31  0:30 ` [PATCH v2 1/4] erofs: add file-backed mount support Sandeep Dhavale
2024-08-31  3:00   ` Gao Xiang
2024-09-01 20:46 ` Sandeep Dhavale
2024-09-05  8:22 ` Chao Yu
2024-09-05  9:30 ` [PATCH v3 2/4] erofs: support unencoded inodes for fileio Gao Xiang
2024-09-05  9:44   ` Chao Yu
2024-09-24  9:21 ` [PATCH v2 1/4] erofs: add file-backed mount support Geert Uytterhoeven
2024-09-24  9:29   ` Gao Xiang
2024-09-30 14:18   ` Jan Kara
2024-09-30 14:22     ` Geert Uytterhoeven
2024-10-02  6:12     ` Christian Brauner
2024-10-02 11:25       ` Gao Xiang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=443a883e-7565-43ed-9def-77e0d666c454@linux.alibaba.com \
    --to=hsiangkao@linux.alibaba.com \
    --cc=linux-erofs@lists.ozlabs.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®