Re: [Nfs-ganesha-devel] V2.3-dev-9 Coverity status

2015-06-30 Thread Daniel Gryniewicz
Thanks, Fredric. This is a false positive; even though the switch is on a locally defined enumeration, the actual value comes off the wire, and so could be outside the enumeration. This will be caught by the default clause, and -1 will be returned. Daniel On 06/30/2015 02:35 AM, SAUNIER,

Re: [Nfs-ganesha-devel] V2.3-dev-9 Coverity status

2015-06-30 Thread Malahal Naineni
Just took a quick look, looks like the code combined type and flag into a single enum (with OR operator)! Coverity is probably getting confused the way the enum is used here. Usually accomplished with bitfieds if space is an issue here... Regards, Malahal. Daniel Gryniewicz [d...@linuxbox.com]

Re: [Nfs-ganesha-devel] 'clustered' configuration parameter + use of nodeid

2015-06-30 Thread Soumya Koduri
Hi Frank/Malahal, Re-opening this thread. On 06/24/2015 04:47 AM, Frank Filz wrote: Note that the way Ganesha handles the epoch for clientids is structured in a way that Ganesha doesn't have to be party to the details. There is a command line option that allows and external actor

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: Set core file size to unlimited for ganesha daemon.

2015-06-30 Thread GerritHub
From Malahal mala...@gmail.com: Malahal has uploaded a new change for review. https://review.gerrithub.io/238101 Change subject: Set core file size to unlimited for ganesha daemon. .. Set core file size to unlimited for

Re: [Nfs-ganesha-devel] V2.3-dev-9 Coverity status

2015-06-30 Thread Frank Filz
Thanks, Fredric. This is a false positive; even though the switch is on a locally defined enumeration, the actual value comes off the wire, and so could be outside the enumeration. This will be caught by the default clause, and -1 will be returned. Hmm, I guess I should have looked at the

Re: [Nfs-ganesha-devel] Announce Push of V2.3-dev-10

2015-06-30 Thread William Allen Simpson
On 6/29/15 7:09 PM, Frank Filz wrote: Branch next Tag:V2.3-dev-10 NOTE: This introduces an update to libntirpc. Please update your submodule and rebase any patches you push. Highlights * many RPC changes Thanks, Frank. This stuff needs lots of field testing. There should be a

Re: [Nfs-ganesha-devel] Ganesha crash on a rm

2015-06-30 Thread Omkar Joshi
Frank .. didn't get you... for cache inode I have already uploaded a fix. https://bugzilla.redhat.com/show_bug.cgi?id=1218422 On Tue, Jun 30, 2015 at 3:56 PM, Frank Filz ffilz...@mindspring.com wrote: Oh, I thought someone had a fix… Frank *From:* Omkar Joshi

Re: [Nfs-ganesha-devel] Ganesha crash on a rm

2015-06-30 Thread Malahal Naineni
Frank, I saw a fix/workaround posted at the bugzilla. I don't know if that is a right way to fix it though. Regards, Malahal. Frank Filz [ffilz...@mindspring.com] wrote: Oh, I thought someone had a fix... Frank From: Omkar Joshi [mailto:o...@hedviginc.com] Sent: