Re: [Nfs-ganesha-devel] Request to merge patches to V2.2-stable branch

2015-06-17 Thread Malahal Naineni
I didn't see GLUSTERFSAL_UP_Thread() returning anything other than NULL/0. I think passing NULL to pthread_join is even cleaner! Regards, Malahal. Malahal Naineni [mala...@us.ibm.com] wrote: Soumya Koduri [skod...@redhat.com] wrote: Hi Kaleb/Malahal, Request you to merge below

Re: [Nfs-ganesha-devel] Problems in /usr/libexec/ganesha/dbus-send.sh and ganesha dbus interface when disabling exports from gluster

2015-06-17 Thread Alessandro De Salvo
Hi Soumya, Il giorno 17/giu/2015, alle ore 19:41, Soumya Koduri skod...@redhat.com ha scritto: On 06/17/2015 10:57 PM, Alessandro De Salvo wrote: Hi, when disabling exports from gluster 3.7.1, by using gluster vol set volume ganesha.enable off, I always get the following error:

[Nfs-ganesha-devel] Request to merge patches to V2.2-stable branch

2015-06-17 Thread Soumya Koduri
Hi Kaleb/Malahal, Request you to merge below FSAL_GLUSTER patches into V2.2-stable branch - 366f71c - FSAL_GLUSTER: Fixed an issue with dereferencing a NULL ponter c4f33d6 - FSAL_GLUSTER : Improvements in acl feature b1df525 - FSAL_GLUSTER: Stop polling upcall events if not supported Thanks,

Re: [Nfs-ganesha-devel] Request to merge patches to V2.2-stable branch

2015-06-17 Thread Malahal Naineni
Soumya Koduri [skod...@redhat.com] wrote: Hi Kaleb/Malahal, Request you to merge below FSAL_GLUSTER patches into V2.2-stable branch - 366f71c - FSAL_GLUSTER: Fixed an issue with dereferencing a NULL ponter I just looked at the patch from your other mail. I have few questions on this patch.

Re: [Nfs-ganesha-devel] Problems in /usr/libexec/ganesha/dbus-send.sh and ganesha dbus interface when disabling exports from gluster

2015-06-17 Thread Malahal Naineni
Alessandro De Salvo [alessandro.desa...@roma1.infn.it] wrote: What’s more worrying is the problem with the dbus. Issuing a DisplayExport before the RemoveExport apparently fixes the problem, so something like this always works: # dbus-send --print-reply --system --dest=org.ganesha.nfsd

Re: [Nfs-ganesha-devel] Problems in /usr/libexec/ganesha/dbus-send.sh and ganesha dbus interface when disabling exports from gluster

2015-06-17 Thread Alessandro De Salvo
to the dbus message which is very odd! Indeed! This is why I was worried :-) I’m attaching the ganesha.log with NIV_FULL_DEBUG, you can see the restart at the end, due to my attempt of RemoveExport, which always gave me the same error as before. Thanks, Alessandro ganesha-20150617

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: Add FSAL helpers for rename access checking

2015-06-17 Thread GerritHub
From Daniel Gryniewicz d...@cohortfs.com: Daniel Gryniewicz has uploaded a new change for review. https://review.gerrithub.io/236840 Change subject: Add FSAL helpers for rename access checking .. Add FSAL helpers for rename

[Nfs-ganesha-devel] Problems in /usr/libexec/ganesha/dbus-send.sh and ganesha dbus interface when disabling exports from gluster

2015-06-17 Thread Alessandro De Salvo
Hi, when disabling exports from gluster 3.7.1, by using gluster vol set volume ganesha.enable off, I always get the following error: Error: Dynamic export addition/deletion failed. Please see log file for details This message is produced by the failure of /usr/libexec/ganesha/dbus-send.sh, and

Re: [Nfs-ganesha-devel] Problems in /usr/libexec/ganesha/dbus-send.sh and ganesha dbus interface when disabling exports from gluster

2015-06-17 Thread Malahal Naineni
I have no knowledge of dbus-send.sh, is it GLUSTER fsal specific? I don't see it in ganesha source code at all. I use ganesha_mgr command to show/delete/remove exports. Can you try it and see if that works? ganesha_mgr remove_export 2 should be the correct call in your case. Regards, Malahal.

Re: [Nfs-ganesha-devel] Problems in /usr/libexec/ganesha/dbus-send.sh and ganesha dbus interface when disabling exports from gluster

2015-06-17 Thread Meghana Madhusudhan
On 06/17/2015 10:57 PM, Alessandro De Salvo wrote: Hi, when disabling exports from gluster 3.7.1, by using gluster vol set volume ganesha.enable off, I always get the following error: Error: Dynamic export addition/deletion failed. Please see log file for details This message is

Re: [Nfs-ganesha-devel] [Gluster-users] Questions on ganesha HA and shared storage size

2015-06-17 Thread Kaleb S. KEITHLEY
Hi, I would actually prefer that we solve the issue with using the unbundled libntirpc. That seems like the better long term solution, and one we have to tackle anyway as we have a hard end date for using the bundled libntirpc in Fedora and EPEL. In the mean time I've reverted the EPEL 7 build

Re: [Nfs-ganesha-devel] [Gluster-users] Questions on ganesha HA and shared storage size

2015-06-17 Thread Niels de Vos
On Wed, Jun 17, 2015 at 08:27:57AM -0400, Kaleb S. KEITHLEY wrote: Hi, I would actually prefer that we solve the issue with using the unbundled libntirpc. That seems like the better long term solution, and one we have to tackle anyway as we have a hard end date for using the bundled

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: 9P/RDMA: init opctx in cleanup_fids

2015-06-17 Thread GerritHub
From Dominique Martinet dominique.marti...@cea.fr: Dominique Martinet has uploaded a new change for review. https://review.gerrithub.io/236752 Change subject: 9P/RDMA: init opctx in cleanup_fids .. 9P/RDMA: init opctx in

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: Error handling: handle FSAL's ENODATA (xattr)

2015-06-17 Thread GerritHub
From Dominique Martinet dominique.marti...@cea.fr: Dominique Martinet has uploaded a new change for review. https://review.gerrithub.io/236763 Change subject: Error handling: handle FSAL's ENODATA (xattr) .. Error handling:

[Nfs-ganesha-devel] Another fsal_grace question

2015-06-17 Thread Malahal Naineni
Hi All, There was a mail Soumya posted a while back on fsal_grace. The original patch that introduced fsal_grace seemed to pass everything to FSAL if fsal_grace was true. Essentially it bypassed all grace checks in ganesha, and passed the requests down to FSAL. Then, fsal_grace was made to be

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: VFS/xattrs: add symlink handling to getextattr_id_by_name

2015-06-17 Thread GerritHub
From Dominique Martinet dominique.marti...@cea.fr: Dominique Martinet has uploaded a new change for review. https://review.gerrithub.io/236761 Change subject: VFS/xattrs: add symlink handling to getextattr_id_by_name ..

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: 9P: statfs: replace NFS magic type by 9P's

2015-06-17 Thread GerritHub
From Dominique Martinet dominique.marti...@cea.fr: Dominique Martinet has uploaded a new change for review. https://review.gerrithub.io/236753 Change subject: 9P: statfs: replace NFS magic type by 9P's .. 9P: statfs: replace

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

2015-06-17 Thread Frank Filz
Hi Krishna, The code doesn't seem to match exactly with V2.1.0 but it does look like nfs3_remove() entered label out_fail. Wondering what the cache_status was at the time of the crash. There were some fixes in V2.2-stable related refcounting, but I am not sure if V2.2-stable fixes your