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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 9 Aug 2021 23:47:08 -0700 From: "Darrick J. Wong" <djwong@...nel.org> To: Kari Argillander <kari.argillander@...il.com> Cc: Konstantin Komarov <almaz.alexandrovich@...agon-software.com>, linux-fsdevel@...r.kernel.org, viro@...iv.linux.org.uk, linux-kernel@...r.kernel.org, pali@...nel.org, dsterba@...e.cz, aaptel@...e.com, willy@...radead.org, rdunlap@...radead.org, joe@...ches.com, mark@...mstone.com, nborisov@...e.com, linux-ntfs-dev@...ts.sourceforge.net, anton@...era.com, dan.carpenter@...cle.com, hch@....de, ebiggers@...nel.org, andy.lavr@...il.com, oleksandr@...alenko.name Subject: Re: [PATCH v27 00/10] NTFS read-write driver GPL implementation by Paragon Software On Tue, Aug 10, 2021 at 08:46:37AM +0300, Kari Argillander wrote: > On Thu, Jul 29, 2021 at 04:49:33PM +0300, Konstantin Komarov wrote: > > This patch adds NTFS Read-Write driver to fs/ntfs3. > > > > Having decades of expertise in commercial file systems development and huge > > test coverage, we at Paragon Software GmbH want to make our contribution to > > the Open Source Community by providing implementation of NTFS Read-Write > > driver for the Linux Kernel. > > > > This is fully functional NTFS Read-Write driver. Current version works with > > NTFS(including v3.1) and normal/compressed/sparse files and supports journal replaying. > > > > We plan to support this version after the codebase once merged, and add new > > features and fix bugs. For example, full journaling support over JBD will be > > added in later updates. > > I'm not expert but I have try to review this series best I can and have > not found any major mistakes which prevents merging. Yeah there are > couple bugs but because this is not going to replace NTFS driver just > yet then IMO it is best that merge will happend sooner so development > fot others get easier. I will also try to review future patches (from > Paragon and others), test patches and make contribution at my own for this > driver. So please use > > Reviewed by: Kari Argillander <kari.argillander@...il.com> Nit: there's supposed to be a dash between 'Reviewed' and 'by'. That said, thanks for putting your name out there! :) --D
Powered by blists - more mailing lists