Re: [Gluster-devel] All builds are failing with BUILD ERROR

2014-06-01 Thread Justin Clift
It _may_ be related to the new regression test cleanup function I'm working on: http://review.gluster.org/#/c/7937/ (possibly cleaned up / removed too much stuff) It also might not be, as it's only supposed to have run on the slaves. + Justin On 01/06/2014, at 9:18 AM, Pranith Kumar

[Gluster-devel] struct dirent in snapview-server.c

2014-06-01 Thread Emmanuel Dreyfus
Hi NetBSD build chokes in master because of struct dirent usage in snapview-server.c:svs_glfs_readdir() Linux and NetBSD struct dirent do not have the same layout, and in fact the whole buffer returned by readdir() has a different layout and is not straightforward to convert. Where does the

Re: [Gluster-devel] struct dirent in snapview-server.c

2014-06-01 Thread Harshavardhana
Emmanuel i sent a patch to disable building snapview on OSX, since it was of no use on 'darwin' . You could see if snapshots and snapview would work on NetBSD (LVM support), in such cases one doesn't have to disable it for NetBSD. d_off might be necessary internally, turning it off might not

Re: [Gluster-devel] struct dirent in snapview-server.c

2014-06-01 Thread Emmanuel Dreyfus
Harshavardhana har...@harshavardhana.net wrote: On the other hand NetBSD has 'sys/compat/linux/*' - can we not leverage it? is it some dependency which is not warranted? It is always possible to translate structures, the question is whether it is useful of not. d_off is the offset of this

Re: [Gluster-devel] struct dirent in snapview-server.c

2014-06-01 Thread Harshavardhana
It is always possible to translate structures, the question is whether it is useful of not. d_off is the offset of this struct dirent within the buffer for the whole directory returned by getdents(2) system call. Since we glusterfs does not use getdents(2) but upper level

Re: [Gluster-devel] All builds are failing with BUILD ERROR

2014-06-01 Thread Vijay Bellur
On 06/01/2014 04:13 PM, Justin Clift wrote: It _may_ be related to the new regression test cleanup function I'm working on: http://review.gluster.org/#/c/7937/ (possibly cleaned up / removed too much stuff) It also might not be, as it's only supposed to have run on the slaves. + Justin

Re: [Gluster-devel] inode lru limit

2014-06-01 Thread Raghavendra G
On Fri, May 30, 2014 at 2:24 PM, Raghavendra Bhat rab...@redhat.com wrote: Hi, Currently the lru-limit of the inode table in brick processes is 16384. There is a option to configure it to some other value. The protocol/server uses inode_lru_limit variable present in its private structure