[Gluster-users] Fwd: Change in ffilz/nfs-ganesha[next]: pNFS code drop enablement and checkpatch warnings fixed

[Anand Subramanian]

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 at gerritforge.com>
>> Reply-To: 	ffilzlnx at mindspring.com
>> To: 	Anand Subramanian <anands at redhat.com>
>> CC: 	onnfrhvruutnzhnaq.-gftf at noclue.notk.org
>>
>>
>>
>>  From Frank Filz <ffilzlnx at mindspring.com>:
>>
>> 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 at redhat.com>
>> ---
>> 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 at redhat.com>
>> Gerrit-Reviewer: Frank Filz <ffilzlnx at mindspring.com>
>> Gerrit-Reviewer: onnfrhvruutnzhnaq.-gftf at noclue.notk.org
>>
>>
>>
>> _______________________________________________
>> Gluster-users mailing list
>> Gluster-users at gluster.org
>> http://www.gluster.org/mailman/listinfo/gluster-users