Niels de Vos
2015-Mar-23 09:41 UTC
[Gluster-users] Fwd: Change in ffilz/nfs-ganesha[next]: pNFS code drop enablement and checkpatch warnings fixed
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
Humble Devassy Chirammal
2015-Mar-23 11:31 UTC
[Gluster-users] Fwd: Change in ffilz/nfs-ganesha[next]: pNFS code drop enablement and checkpatch warnings fixed
Isnt this one http://review.gluster.org/#/c/9971 ? --Humble On Mon, Mar 23, 2015 at 3:11 PM, Niels de Vos <ndevos at redhat.com> 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 > > _______________________________________________ > Gluster-users mailing list > Gluster-users at gluster.org > http://www.gluster.org/mailman/listinfo/gluster-users >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://www.gluster.org/pipermail/gluster-users/attachments/20150323/842d0736/attachment.html>
Anand Subramanian
2015-Mar-24 05:03 UTC
[Gluster-users] Fwd: Change in ffilz/nfs-ganesha[next]: pNFS code drop enablement and checkpatch warnings fixed
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