UFS part (not related to mtime) 2009-03-15 Vladimir Serbinenko UFS improvements * fs/ufs.c (GRUB_UFS_BSD_DIRENT): new definition (struct grub_ufs_dirent): added fields for non-BSD dirents (grub_ufs_get_file_block): fixed double indirect handling (grub_ufs_find_file): support for non-BSD dirents (grub_ufs_dir): support for non-BSD dirents Robert Millan wrote: > On Sun, Mar 01, 2009 at 05:25:10PM +0100, phcoder wrote: >> Hello all. It seems that gcc has trouble with -m32 when structure is >> passed as argument. So I replaced that part by a pointer. Also I made >> some improvements to ufs code to support solaris branch of ufs. I tested >> it also with freebsd and netbsd's branch and it works fine on it too. >> As my 3 FS patches: mtime, FAT and UFS are interdependent I submit a >> patch with all 3 features. If it's really necessary I can split them but >> it requires a lot of unnecessary work > > Please do. It is definitely confusing to review patches that merge > unrelated things. > > Also, please don't include the changelog entry in your patch, since those > break too easily. Just paste it at the top of your mail. > -- Regards Vladimir 'phcoder' Serbinenko