From:   "Fabio M. De Francesco" <fmdefrancesco@gmail.com>
To:     Al Viro <viro@zeniv.linux.org.uk>, Jan Kara <jack@suse.com>
Cc:     Linus Torvalds <torvalds@linux-foundation.org>,
        linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: Re: [git pull] vfs.git sysv pile
Date:   Wed, 01 Mar 2023 12:20:56 +0100
Message-ID: <13214812.uLZWGnKmhe@suse>
In-Reply-To: <Y/gugbqq858QXJBY@ZenIV>
References: <Y/gugbqq858QXJBY@ZenIV>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="iso-8859-1"
Precedence: bulk

On venerd=EC 24 febbraio 2023 04:26:57 CET Al Viro wrote:
> 	Fabio's "switch to kmap_local_page()" patchset (originally after the
> ext2 counterpart, with a lot of cleaning up done to it; as the matter of
> fact, ext2 side is in need of similar cleanups - calling conventions there
> are bloody awful).

If nobody else is already working on these cleanups in ext2 following your=
=20
suggestion, I'd be happy to work on this by the end of this week. I only ne=
ed=20
a confirmation because I'd hate to duplicate someone else work.

> Plus the equivalents of minix stuff...

I don't know this other filesystem but I could take a look and see whether =
it=20
resembles somehow sysv and ext2 (if so, this work would be pretty simple to=
o,=20
thanks to your kind suggestions when I worked on sysv and ufs).

I'm adding Jan to the Cc list to hear whether he is aware of anybody else=20
working on this changes for ext2. I'm waiting for a reply from you (@Al) or=
=20
Jan to avoid duplication (as said above).

Thanks,

=46abio=20

> The following changes since commit b7bfaa761d760e72a969d116517eaa12e404c2=
62:
>=20
>   Linux 6.2-rc3 (2023-01-08 11:49:43 -0600)
>=20
> are available in the Git repository at:
>=20
>   git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git work.sysv
>=20
> for you to fetch changes up to abb7c742397324f8676c5b622effdce911cd52e3:
>=20
>   sysv: fix handling of delete_entry and set_link failures (2023-01-19
> 23:24:42 -0500)
>=20
> ----------------------------------------------------------------
> Al Viro (1):
>       sysv: fix handling of delete_entry and set_link failures
>=20
> Christoph Hellwig (1):
>       sysv: don't flush page immediately for DIRSYNC directories
>=20
> Fabio M. De Francesco (4):
>       fs/sysv: Use the offset_in_page() helper
>       fs/sysv: Change the signature of dir_get_page()
>       fs/sysv: Use dir_put_page() in sysv_rename()
>       fs/sysv: Replace kmap() with kmap_local_page()
>=20
>  fs/sysv/dir.c   | 154
> ++++++++++++++++++++++++++++++++------------------------ fs/sysv/namei.c =
|=20
> 42 ++++++++--------
>  fs/sysv/sysv.h  |   3 +-
>  3 files changed, 111 insertions(+), 88 deletions(-)