Re: [Nfs-ganesha-devel] pre-commit refusals for ANSI formatted strings

2016-11-28 Thread William Allen Simpson
On 11/28/16 1:28 PM, Malahal Naineni wrote: > I think SPLIT_STRING is enabled by default. This supposedly helps some > linux kernel hackers to map a given syslog message with the actual > code line by simply grepping. Debatable as printk could contain string > symbols evading the matches. Enabling

[Nfs-ganesha-devel] Kerberos with virtual IP failovers

2016-11-28 Thread sriram patil
Hi All I have been trying to setup HA feature with VIP failovers for nfs ganesha. I have been able to successfully setup HA for sys authentication flavour. I am facing problem with GSS authentication for HA feature. I have exported my mount point krb5 and have mounted on a different machine. All

Re: [Nfs-ganesha-devel] ganesha-2.4.1 + RGW FSAL + Ceph Jewel

2016-11-28 Thread Karol Mroz
Hi Matt, On Mon, Nov 28, 2016 at 10:40:42PM -0500, Matt W. Benjamin wrote: [...] > > > For the moment, what you should be doing is mounting with -osync > > (Linux) to get consistent results. Can you give that a try? > > > > Mounting with '-o sync' appears to remove (2ii). A failed, simple > > w

Re: [Nfs-ganesha-devel] ganesha-2.4.1 + RGW FSAL + Ceph Jewel

2016-11-28 Thread Matt W. Benjamin
Hi Karol, - "Karol Mroz" wrote: > Hi Matt, > > Thanks for the input. Please see inline. > > On Sun, Nov 27, 2016 at 10:58:34PM -0500, Matt W. Benjamin wrote: > [...] > > > 1. `echo` or `touch` a new file > > > > > > File creation via `echo` or `touch` (createmode=1, offset=0, > length=N)

Re: [Nfs-ganesha-devel] ganesha-2.4.1 + RGW FSAL + Ceph Jewel

2016-11-28 Thread Karol Mroz
Hi Matt, Thanks for the input. Please see inline. On Sun, Nov 27, 2016 at 10:58:34PM -0500, Matt W. Benjamin wrote: [...] > > 1. `echo` or `touch` a new file > > > > File creation via `echo` or `touch` (createmode=1, offset=0, length=N) > > has > > so far been successful in my test environment.

Re: [Nfs-ganesha-devel] pre-commit refusals for ANSI formatted strings

2016-11-28 Thread Malahal Naineni
I think SPLIT_STRING is enabled by default. This supposedly helps some linux kernel hackers to map a given syslog message with the actual code line by simply grepping. Debatable as printk could contain string symbols evading the matches. Enabling this and the 80 character line limit is a mess! Both

[Nfs-ganesha-devel] pre-commit refusals for ANSI formatted strings

2016-11-28 Thread William Allen Simpson
If the printf string is on one line, I get dozens of WARNING: line over 80 characters #206: FILE: src/MainNFSD/nfs_worker_thread.c:1332: +"NFS DISPATCHER: FAILURE: Error while calling svc_sendreply on a new request. rpcxid=%" PRIu32 " socket=%d function:%s client:

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: Fix merge/rebase error with conditional NLM compilation

2016-11-28 Thread GerritHub
>From Daniel Gryniewicz : Daniel Gryniewicz has uploaded a new change for review. ( https://review.gerrithub.io/303991 Change subject: Fix merge/rebase error with conditional NLM compilation .. Fix merge/rebase error with cond

Re: [Nfs-ganesha-devel] ERROR: trailing statements should be on next line

2016-11-28 Thread Daniel Gryniewicz
Merge / rebase error at some point. I'll submit a fix. Daniel On 11/27/2016 01:16 AM, Malahal Naineni wrote: > I am not sure about the exact error that the checkpatch is complaining > here. Maybe it is getting confused with our conditionals! About the > _USE_NLM, it is done incorrectly the way

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: [valgrind] Fix mem-leak in gsh_dbus_register_path

2016-11-28 Thread GerritHub
>From Swen Schillig : Swen Schillig has uploaded a new change for review. https://review.gerrithub.io/303678 Change subject: [valgrind] Fix mem-leak in gsh_dbus_register_path .. [valgrind] Fix mem-leak in gsh_dbus_register_pa

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: FSAL_GLUSTER/commit2: Fix deferencing a NULL pointer

2016-11-28 Thread GerritHub
>From Soumya : Soumya has uploaded a new change for review. ( https://review.gerrithub.io/303947 Change subject: FSAL_GLUSTER/commit2: Fix deferencing a NULL pointer .. FSAL_GLUSTER/commit2: Fix deferencing a NULL pointer All

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: FSAL_RGW: fix segfault on missing free of root handle

2016-11-28 Thread GerritHub
>From Gui Hecheng : Gui Hecheng has uploaded a new change for review. https://review.gerrithub.io/303718 Change subject: FSAL_RGW: fix segfault on missing free of root handle .. FSAL_RGW: fix segfault on missing free of root

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: [valgrind] Fix possible mem-leak in get_gsh_client

2016-11-28 Thread GerritHub
>From Swen Schillig : Swen Schillig has uploaded a new change for review. https://review.gerrithub.io/303763 Change subject: [valgrind] Fix possible mem-leak in get_gsh_client .. [valgrind] Fix possible mem-leak in get_gsh_cl