Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Nov 2014 10:25:45 -0500
From:      John Baldwin <jhb@freebsd.org>
To:        freebsd-fs@freebsd.org
Subject:   Re: RFC: patch to make d_fileno 64bits
Message-ID:  <9692214.Bs3rxl0ePH@ralph.baldwin.cx>
In-Reply-To: <539201047.4538834.1416539954794.JavaMail.root@uoguelph.ca>
References:  <539201047.4538834.1416539954794.JavaMail.root@uoguelph.ca>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thursday, November 20, 2014 10:19:14 PM Rick Macklem wrote:
> The attached patch covers the basics of a way to
> convert the d_fileno field of "struct dirent" to
> 64bits. This patch is incomplete and won't even
> build, but I thought I'd post it in case anyone
> wanted to take a look and comment on the approach
> it uses.
> 
> - renames the old/current one "struct dirent32"
> - changes d_fileno to 64bits and adds a 64bit
>   d_off field for the offset of the underlying
>   file system
> - defines a new VOP_READDIR() that will return
>   the new "struct dirent" that is used as the
>   default one for a new getdirentries(2).
> - the old/current getdirentries(2) uses the old
>   VOP_READDIR32() by default.
> 
> For the case of a file system that supports both
> the new and old VOP_READDIR(), they are used by
> the corresponding new and old getdirentries(2)
> syscalls.
> 
> For a file system that only supports one of
> the VOP_READDIR()s, the "struct dirent32"
> is copied to "struct dirent" (or vice versa).
> 
> At this point, all file systems would support
> the old VOP_READDIR() and I think the new
> VOP_READDIR() can easily be added for NFS,
> ZFS. (OpenBSD already has UFS code for
> essentially a new struct dirent and hopefully
> that code could be ported easily, too.)
> 
> Anyhow, any comments on this approach? rick

I think this is already done (along with several other changes) more fully in 
the projects/ino64 branch in svn?

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?9692214.Bs3rxl0ePH>