linux-kernel - Re: [PATCH v2 2/4] erofs: support unencoded inodes for fileio

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <443a883e-7565-43ed-9def-77e0d666c454@linux.alibaba.com>
Date: Wed, 4 Sep 2024 14:56:07 +0800
From: Gao Xiang <hsiangkao@...ux.alibaba.com>
To: linux-erofs@...ts.ozlabs.org
Cc: LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 2/4] erofs: support unencoded inodes for fileio



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@...ux.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)) {