[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] a new filessytem module for nilfs2

From: Vladimir 'φ-coder/phcoder' Serbinenko
Subject: Re: [PATCH] a new filessytem module for nilfs2
Date: Wed, 10 Feb 2010 01:31:35 +0100
User-agent: Mozilla-Thunderbird (X11/20091109)

Jiro SEKIBA wrote:
> Hi,
> This is a patch to support nilfs2 filesystem.
> I would like you to review the patch.
> I've checked with revno 2153 and kvm(qemu) environment.
> Also the patch has already an ability to boot the kernel from
> the nilfs partition, I have a following ToDo list to improve the module.
> 1. revision check for future disk layout changes
> 2. CRC check for superblock to ensure the partition is a nilfs2 partition
> 3. CRC check latest log block to ensure valid log 
> 4. search the latest log in case unclean unmount happened
> nilfs2 stands for New Implementation of Log Filesystem version2.
> nilfs2 has been merged into linux main line kernel since 2.6.30.
> Please give me any comments or advice.
> thank you very much in advance

+pkglib_MODULES += fshelp.mod fat.mod ufs1.mod ufs2.mod ext2.mod
nilfs2.mod \
+       ntfs.mod ntfscomp.mod minix.mod hfs.mod jfs.mod iso9660.mod
xfs.mod \

For new modules we prefer to add pkglib_MODULES+= directly before the
module itself
+/* ext2.c - Second Extended filesystem */
+#define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d))
This can go into misc.h
+  /* Search the file.  */
+  while (fpos < grub_le_to_cpu64 (diro->inode.i_size))
+    {
Another leftover
I'm no nilfs2 expert but assuming that it works it's a good patch. Have
you tested it on big-endian system? If no nd you have no access to such
send me testing instructions, I have a powerpc machine (or one can use
Nice job
> ------------------------------------------------------------------------
> ------------------------------------------------------------------------
> _______________________________________________
> Grub-devel mailing list
> address@hidden

Vladimir 'φ-coder/phcoder' Serbinenko

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]