Skip to contentSkip to conversation

export operations rewrite

PATCH In review [email protected] 3 participants 5 messages Sep 14, 2007 – Sep 28, 2007

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

cover export operations rewrite

This patchset is a medium scale rewrite of the export operations interface. The goal is to make the interface less complex, and easier to understand from the filesystem side, aswell as preparing generic support for exporting of 64bit inode numbers.

This touches all nfs exporting filesystems, and I've done testing on all of the filesystems I have here locally (xfs, ext2, ext3, reiserfs, jfs)

Compared to the last version I've fixed the white space issues that checkpatch.pl complained about.

Note that this patch series is against mainline. There will be some xfs changes landing in -mm soon that revamp lots of the code touched here. They should hopefully include the first path in the series so it can be simply dropped, but the xfs conversion will need some smaller updates. I will send this update as soon as the xfs tree updates get pulled into -mm.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

#1 lore raw
Quoted 11 lines · On Fri, Sep 14, 2007 at 01:47:46PM +0200, [email protected] wrote:

This patchset is a medium scale rewrite of the export operations interface. The goal is to make the interface less complex, and easier to understand from the filesystem side, aswell as preparing generic support for exporting of 64bit inode numbers.

This touches all nfs exporting filesystems, and I've done testing on all of the filesystems I have here locally (xfs, ext2, ext3, reiserfs, jfs)

Compared to the last version I've fixed the white space issues that checkpatch.pl complained about.

OK, thanks again. Everything I have now is in

git://linux-nfs.org/~bfields/linux.git for-mm

I'm hoping Neil can take a quick look as well (and make a response to the comment on patch #1 along the way).

Quoted 6 lines

Note that this patch series is against mainline. There will be some xfs changes landing in -mm soon that revamp lots of the code touched here. They should hopefully include the first path in the series so it can be simply dropped, but the xfs conversion will need some smaller updates. I will send this update as soon as the xfs tree updates get pulled into -mm.

OK. Let me know if you need me to do anything when that happens.

--b.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

#2 lore raw
Quoted 2 lines · J. Bruce Fields

I'm hoping Neil can take a quick look as well (and make a response to the comment on patch #1 along the way).

The exportfs_d_alloc one? I have a patch for the done, I just need to run it through my testing setup before sending it out.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs

#3 lore raw
Quoted 6 lines · Christoph Hellwig
Quoted 2 lines · J. Bruce Fields

I'm hoping Neil can take a quick look as well (and make a response to the comment on patch #1 along the way).

The exportfs_d_alloc one? I have a patch for the done, I just need to run it through my testing setup before sending it out.

Oops, I meant #19, not #1--there's still a note addressed to Neil sitting in there.

Typing faster than I was thinking, sorry--b.

Acked-by on the v1 series
#4 lore raw
In reply to [email protected]
Quoted 4 lines · On Friday September 14, [email protected] wrote:

This patchset is a medium scale rewrite of the export operations interface. The goal is to make the interface less complex, and easier to understand from the filesystem side, aswell as preparing generic support for exporting of 64bit inode numbers.

These 19 patches

I didn't do a very thorough review this time, but I did the previous time, and the one problem I found (module dependencies) has been addressed - Thanks.

NeilBrown

Quoted 19 lines

This touches all nfs exporting filesystems, and I've done testing on all of the filesystems I have here locally (xfs, ext2, ext3, reiserfs, jfs)

Compared to the last version I've fixed the white space issues that checkpatch.pl complained about.

Note that this patch series is against mainline. There will be some xfs changes landing in -mm soon that revamp lots of the code touched here. They should hopefully include the first path in the series so it can be simply dropped, but the xfs conversion will need some smaller updates. I will send this update as soon as the xfs tree updates get pulled into -mm.

-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html