[Top][All Lists]

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

Re: [PATCH] btree support in xfs driver

From: Bean
Subject: Re: [PATCH] btree support in xfs driver
Date: Fri, 1 Feb 2008 11:31:36 +0800

On Feb 1, 2008 3:57 AM, Marco Gerards <address@hidden> wrote:
> Bean <address@hidden> writes:
> Hi Bean!
> > I add btree support for the xfs driver, and fix a few bugs. Now i' m
> > able to list a directory of over 5000 files, please test it.
> You rock!
> Can you load big files now, that are stored in BTrees.  Did you use
> code from elsewhere?

it should be, grub_xfs_read_block is used by both file and directory.

> > 2008-02-01  Bean  <address@hidden>
> >
> >       * fs/xfs.h (grub_xfs_sblock): New member log2_dirblk.
> >       (grub_xfs_btree_node): New structure.
> >       (grub_xfs_btree_root): New structure.
> >       (grub_xfs_inode): New member nblocks, extsize, nextents and btree.
> members
> >       (GRUB_XFS_EXTENT_OFFSET): Use exts instead of inode->data.extents.
> >       (GRUB_XFS_EXTENT_BLOCK): Likewise.
> >       (GRUB_XFS_EXTENT_SIZE): Likewise.
> >       (grub_xfs_read_block): Support btree format type.
> >       (grub_xfs_iterate_dir): Use regparm(1) attribute in call_hook.
> >       Use directory block as basic unit.
> This won't work on non-i386.  So I think this has to be fixed like
> that other bug, for example with the autoconf check extension you
> proposed.

i take a look at the aclocal.m4, the place where NESTED_FUNC_ATTR is defined:


if test "x$grub_cv_i386_check_nested_functions" = xyes; then
        [__attribute__ ((__regparm__ (2)))],
        [Catch gcc bug])
dnl Unfortunately, the above test does not detect a bug in gcc-4.0.
dnl So use regparm 2 until a better test is found.
        [__attribute__ ((__regparm__ (2)))],
        [Catch gcc bug])

is there a problem here ? i think NESTED_FUNC_ATTR is defined
regardless of the test result.


reply via email to

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