Update to glusterfs 3.12.3

There is an important performance bug fix specific to NetBSD here,
which disable gfid2path by default. This features causes a huge
amount of different extended attributes to be created, and the
NetBSD implementation does not scale well with it.

In order to recover a server after the feature is disabled, stop
glusterfs daemones, disable extended attributes using extattrctl,
remove ${BRICK_ROOT}/.attribute/system/trusted.gfid2path.*
re-enable extended attributes and restart glusterfs.

4 files changed, 58 insertions, 8 deletions

diff --git a/filesystems/glusterfs/Makefile b/filesystems/glusterfs/Makefile
index 1ee9b2a6218..e851a1dc7e3 100644
--- a/filesystems/glusterfs/Makefile
+++ b/filesystems/glusterfs/Makefile
@@ -1,6 +1,6 @@
-# $NetBSD: Makefile,v 1.79 2017/11/04 03:46:56 manu Exp $
+# $NetBSD: Makefile,v 1.80 2017/11/15 03:31:56 manu Exp $
 
-DISTNAME=	glusterfs-3.12.2
+DISTNAME=	glusterfs-3.12.3
 #PKGREVISION=	1
 CATEGORIES=	filesystems
 MASTER_SITES=	http://bits.gluster.org/pub/gluster/glusterfs/src/
diff --git a/filesystems/glusterfs/PLIST b/filesystems/glusterfs/PLIST
index 16fd2a86d49..a273692dd1d 100644
--- a/filesystems/glusterfs/PLIST
+++ b/filesystems/glusterfs/PLIST
@@ -1,4 +1,4 @@
-@comment $NetBSD: PLIST,v 1.37 2017/11/04 03:46:56 manu Exp $
+@comment $NetBSD: PLIST,v 1.38 2017/11/15 03:31:56 manu Exp $
 ${PYSITELIB}/gluster/__init__.py
 ${PYSITELIB}/gluster/__init__.pyc
 ${PYSITELIB}/gluster/__init__.pyo
diff --git a/filesystems/glusterfs/distinfo b/filesystems/glusterfs/distinfo
index e8b40433097..7befaacff03 100644
--- a/filesystems/glusterfs/distinfo
+++ b/filesystems/glusterfs/distinfo
@@ -1,6 +1,7 @@
-$NetBSD: distinfo,v 1.62 2017/11/04 03:46:56 manu Exp $
+$NetBSD: distinfo,v 1.63 2017/11/15 03:31:56 manu Exp $
 
-SHA1 (glusterfs-3.12.2.tar.gz) = 561c9bf5aa8fb767dc51fc20a7849c8888a2e5cd
-RMD160 (glusterfs-3.12.2.tar.gz) = d354c48ddd7097008f8e44b2c4c9ff9c2c935aa5
-SHA512 (glusterfs-3.12.2.tar.gz) = 2bcb3e9356964ea8557ea4291cf6b641122c58a3beb59c00de0acb6772b05b22c0610db4f2bde2e506e15233cfa610db415c6622933e04600ddee3cc55b01166
-Size (glusterfs-3.12.2.tar.gz) = 9404275 bytes
+SHA1 (glusterfs-3.12.3.tar.gz) = ea423aaa1b59fa8f83d01779ac13b2a00e03d2b1
+RMD160 (glusterfs-3.12.3.tar.gz) = 83a41e4006d79237c24d31eef2881fce281179ac
+SHA512 (glusterfs-3.12.3.tar.gz) = fdc16373b50b7b9443e981b76c3456a7a48d63abdccb9557d0eccfed54ccacfae8a88b8fab0ddea9b321a55d25e646e7102f4ceaa6f280c7ad807ffb90ca8ab3
+Size (glusterfs-3.12.3.tar.gz) = 7393120 bytes
+SHA1 (patch-19778) = c46bb27f78c2cd266d639f41795939d745f434c9
diff --git a/filesystems/glusterfs/patches/patch-19778 b/filesystems/glusterfs/patches/patch-19778
new file mode 100644
index 00000000000..e4ff008a4dc
--- /dev/null
+++ b/filesystems/glusterfs/patches/patch-19778
@@ -0,0 +1,49 @@
+$NetBSD: patch-19778,v 1.1 2017/11/15 03:31:56 manu Exp $
+
+From 3bfb4e454eb639f9d1a370dc056eb5b2ef38e484 Mon Sep 17 00:00:00 2001
+From: Emmanuel Dreyfus <manu@netbsd.org>
+Date: Wed, 15 Nov 2017 04:04:39 +0100
+Subject: [PATCH] Disable gfid2path by default on NetBSD
+
+NetBSD storage of extended attributes for UFS1 badly scales when
+the list of extended attributes names rises. gfid2path can add as
+many extended attributes names as we have files, hence we keep it
+disabled for performance sake.
+
+> Change-Id: Id77b5f5ceb4d5eba1b3362b4b9fc693450ffbc2b
+> Signed-off-by: Emmanuel Dreyfus <manu@netbsd.org>
+> BUG: 1129939
+
+Change-Id: I5232a0bd8cb30fa42124f49b1054153f9c31f108
+Signed-off-by: Emmanuel Dreyfus <manu@netbsd.org>
+BUG: 1513258
+---
+ xlators/storage/posix/src/posix.c | 11 +++++++++++
+ 1 file changed, 11 insertions(+)
+
+diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c
+index d858878d3..577515263 100644
+--- xlators/storage/posix/src/posix.c
++++ xlators/storage/posix/src/posix.c
+@@ -7967,7 +7967,18 @@ struct volume_options options[] = {
+         },
+         { .key = {"gfid2path"},
+           .type = GF_OPTION_TYPE_BOOL,
++#ifdef __NetBSD__
++          /* 
++	   * NetBSD storage of extended attributes for UFS1 badly 
++	   * scales when the list of extended attributes names rises.
++	   * This option can add as many extended attributes names
++	   * as we have files, hence we keep it disabled for performance
++	   * sake.
++	   */
++          .default_value = "off",
++#else
+           .default_value = "on",
++#endif
+           .description = "Enable logging metadata for gfid to path conversion"
+         },
+         { .key = {"gfid2path-separator"},
+-- 
+2.11.0
+