This is not a must-have fix for pnfs functionality. pnfs mode can be
enabled and it will work without the libgfapi fix.
The fix is only important from a performance optimization standpoint
alone (esp for large files).
Also note that currently for GlusterFS, we only support the
PNFS-FILE-LAYOUT with stripe-count as 1. And though both configurations
(i) single MDS with multiple DS'es as well as the (ii) all-symmetric
architecture of every ganesha head behaving as both MDS|DS are supported
and work right now, the latter is not functionally complete as we will
need to add an upcall support for Layout recalls.
There is additional thought needed once EC support is completed,
sharding work is in and once the FLEX-FILE-LAYOUT IETF draft is complete.
Anand
On 03/23/2015 03:11 PM, Niels de Vos wrote:
On Mon, Mar 23, 2015 at 12:49:56PM +0530, Anand Subramanian wrote:
FYI.
GlusterFS vols can now be accessed via NFSv4.1 pNFS protocol (mount -t nfs
-o minorversion=1 ...) from nfs-ganesha 2.2-rc5 onwards.
Note: one fix is to go into libgfapi to fix up using anonymous fd's in
ds_write/make_ds_handle() (Avati's sugeestion that really helps here).
Once Jiffin or myself get that fix in, a good large file performance can be
seen with pNFS vs V4.
I could not find the needed change for libgfapi. Could you post the link
to the review?
Thanks,
Niels
All thanks and credit to Jiffin for his terrific effort in coding things up
quickly and for fixing bugs.
Anand
-------- Forwarded Message --------
Subject: Change in ffilz/nfs-ganesha[next]: pNFS code drop enablement and
checkpatch warnings fixed
Date: Sat, 21 Mar 2015 01:04:30 +0100
From: GerritHub <support@xxxxxxxxxxxxxxx>
Reply-To: ffilzlnx@xxxxxxxxxxxxxx
To: Anand Subramanian <anands@xxxxxxxxxx>
CC: onnfrhvruutnzhnaq.-gftf@xxxxxxxxxxxxxxx
From Frank Filz <ffilzlnx@xxxxxxxxxxxxxx>:
Frank Filz has submitted this change and it was merged.
Change subject: pNFS code drop enablement and checkpatch warnings fixed
......................................................................
pNFS code drop enablement and checkpatch warnings fixed
Change-Id: Ia8c58dd6d6326f692681f76b96f29c630db21a92
Signed-off-by: Anand Subramanian <anands@xxxxxxxxxx>
---
A src/FSAL/FSAL_GLUSTER/ds.c
M src/FSAL/FSAL_GLUSTER/export.c
M src/FSAL/FSAL_GLUSTER/gluster_internal.h
M src/FSAL/FSAL_GLUSTER/handle.c
M src/FSAL/FSAL_GLUSTER/main.c
A src/FSAL/FSAL_GLUSTER/mds.c
6 files changed, 993 insertions(+), 0 deletions(-)
--
To view, visit https://review.gerrithub.io/221683
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia8c58dd6d6326f692681f76b96f29c630db21a92
Gerrit-PatchSet: 1
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Owner: Anand Subramanian <anands@xxxxxxxxxx>
Gerrit-Reviewer: Frank Filz <ffilzlnx@xxxxxxxxxxxxxx>
Gerrit-Reviewer: onnfrhvruutnzhnaq.-gftf@xxxxxxxxxxxxxxx
_______________________________________________
Gluster-users mailing list
Gluster-users@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-users
_______________________________________________
Gluster-users mailing list
Gluster-users@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-users
