On Fri, Feb 27, 2015 at 03:28:50PM +0100, Patrick Matthäi wrote: > Am 19.02.2015 um 23:00 schrieb Adam Borowski: > >The sysctl(2) syscall isn't used by glusterfs on Linux on runtime (it would > >spam syslog if called), so it's safe to remove its header. The attached > >patch does this. > > are you also able to reproduce this with 3.6.2-1 from experimental?
Yes, although the patch I provided didn't apply. Here's a new one, for the experimental version. Besides making it fit in the new context, I also changed the condition to defined(CTL_KERN) to match the block where sysctl is actually used. Per the comment, /* DARWIN and *BSD */, it's equivalent to the old version. -- // If you believe in so-called "intellectual property", please immediately // cease using counterfeit alphabets. Instead, contact the nearest temple // of Amon, whose priests will provide you with scribal services for all // your writing needs, for Reasonable and Non-Discriminatory prices.
--- glusterfs-3.6.2/xlators/mount/fuse/src/fuse-helpers.c~ 2015-01-21 13:10:24.000000000 +0100 +++ glusterfs-3.6.2/xlators/mount/fuse/src/fuse-helpers.c 2015-02-28 00:51:14.653040406 +0100 @@ -17,7 +17,7 @@ #include <sys/types.h> #include <libutil.h> #include <sys/user.h> -#else +#elif defined(CTL_KERN) #include <sys/sysctl.h> #endif
