|
 |
3604df |
From 9c06a794d358d482f4b8b9e12034b618a57eb0fc Mon Sep 17 00:00:00 2001
|
|
|
3604df |
From: Niels de Vos <ndevos@redhat.com>
|
|
|
3604df |
Date: Fri, 18 Nov 2016 14:46:00 +0100
|
|
|
3604df |
Subject: [PATCH 186/206] gfapi: add glfs_free() to glfs.h
|
|
|
3604df |
|
|
|
3604df |
Commit 4721188a154acd9a0a4c096d8d73e97f3bf1b2a9 introduces glfs_free()
|
|
|
3604df |
but missed adding the function to the header. The symbol is correctly
|
|
|
3604df |
available in the library though. Testcases do not seem to fail when a
|
|
|
3604df |
function is missing for the headers...
|
|
|
3604df |
|
|
|
3604df |
The glusterfs-3.7.16 packages have been released with the missing
|
|
|
3604df |
declaration in the header and symbol-maps. Still, the function is
|
|
|
3604df |
available for applications:
|
|
|
3604df |
|
|
|
3604df |
$ objdump -T usr/lib64/libgfapi.so.0 | grep -w glfs_free
|
|
|
3604df |
0000000000006aa0 g DF .text 0000000000000035 GFAPI_3.7.16 glfs_free
|
|
|
3604df |
|
|
|
3604df |
Cherry picked from commit 5ab0b43ce67d8d131242542a80b6aa6b129ed5ee:
|
|
|
3604df |
> Change-Id: Ia707ee957f090dbfca028192fcc81a83dfdf4ae0
|
|
|
3604df |
> BUG: 1344714
|
|
|
3604df |
> Reported-by: Jiffin Tony Thottan <jthottan@redhat.com>
|
|
|
3604df |
> Signed-off-by: Niels de Vos <ndevos@redhat.com>
|
|
|
3604df |
> Reviewed-on: http://review.gluster.org/15653
|
|
|
3604df |
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
|
|
|
3604df |
> Smoke: Gluster Build System <jenkins@build.gluster.org>
|
|
|
3604df |
> Reviewed-by: jiffin tony Thottan <jthottan@redhat.com>
|
|
|
3604df |
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
|
|
|
3604df |
|
|
|
3604df |
Cherry picked from commit a5bf3c3a586ccf20f5f735968ec9a56d33f62326:
|
|
|
3604df |
> Change-Id: Ia707ee957f090dbfca028192fcc81a83dfdf4ae0
|
|
|
3604df |
> BUG: 1347717
|
|
|
3604df |
> Reported-by: Jiffin Tony Thottan <jthottan@redhat.com>
|
|
|
3604df |
> Signed-off-by: Niels de Vos <ndevos@redhat.com>
|
|
|
3604df |
> Reviewed-on: http://review.gluster.org/15680
|
|
|
3604df |
> Smoke: Gluster Build System <jenkins@build.gluster.org>
|
|
|
3604df |
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
|
|
|
3604df |
> Reviewed-by: jiffin tony Thottan <jthottan@redhat.com>
|
|
|
3604df |
> Reviewed-by: Oleksandr Natalenko <oleksandr@natalenko.name>
|
|
|
3604df |
> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
|
|
|
3604df |
|
|
|
3604df |
Change-Id: Ia707ee957f090dbfca028192fcc81a83dfdf4ae0
|
|
|
3604df |
BUG: 1386635
|
|
|
3604df |
Signed-off-by: Niels de Vos <ndevos@redhat.com>
|
|
|
3604df |
Reviewed-on: https://code.engineering.redhat.com/gerrit/90579
|
|
|
3604df |
Reviewed-by: Soumya Koduri <skoduri@redhat.com>
|
|
|
3604df |
Tested-by: Soumya Koduri <skoduri@redhat.com>
|
|
|
3604df |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
3604df |
---
|
|
|
3604df |
api/src/gfapi.aliases | 1 +
|
|
|
3604df |
api/src/gfapi.map | 1 +
|
|
|
3604df |
api/src/glfs.h | 3 +++
|
|
|
3604df |
3 files changed, 5 insertions(+)
|
|
|
3604df |
|
|
|
3604df |
diff --git a/api/src/gfapi.aliases b/api/src/gfapi.aliases
|
|
|
3604df |
index b37665f..a3eddda 100644
|
|
|
3604df |
--- a/api/src/gfapi.aliases
|
|
|
3604df |
+++ b/api/src/gfapi.aliases
|
|
|
3604df |
@@ -141,6 +141,7 @@ _priv_glfs_process_upcall_event _glfs_process_upcall_event$GFAPI_PRIVATE_3.7.0
|
|
|
3604df |
|
|
|
3604df |
_pub_glfs_h_lookupat _glfs_h_lookupat$GFAPI_3.7.4
|
|
|
3604df |
|
|
|
3604df |
+_pub_glfs_free _glfs_free$GFAPI_3.7.16
|
|
|
3604df |
_pub_glfs_h_poll_upcall _glfs_h_poll_upcall$GFAPI_3.7.16
|
|
|
3604df |
_pub_glfs_upcall_get_fs _glfs_upcall_get_fs$GFAPI_3.7.16
|
|
|
3604df |
_pub_glfs_upcall_get_reason _glfs_upcall_get_reason$GFAPI_3.7.16
|
|
|
3604df |
diff --git a/api/src/gfapi.map b/api/src/gfapi.map
|
|
|
3604df |
index f38ef02..02da0fe 100644
|
|
|
3604df |
--- a/api/src/gfapi.map
|
|
|
3604df |
+++ b/api/src/gfapi.map
|
|
|
3604df |
@@ -174,6 +174,7 @@ GFAPI_3.7.15 {
|
|
|
3604df |
|
|
|
3604df |
GFAPI_3.7.16 {
|
|
|
3604df |
global:
|
|
|
3604df |
+ glfs_free;
|
|
|
3604df |
glfs_upcall_get_fs;
|
|
|
3604df |
glfs_upcall_get_reason;
|
|
|
3604df |
glfs_upcall_inode_get_event;
|
|
|
3604df |
diff --git a/api/src/glfs.h b/api/src/glfs.h
|
|
|
3604df |
index b53af35..8b5782e 100644
|
|
|
3604df |
--- a/api/src/glfs.h
|
|
|
3604df |
+++ b/api/src/glfs.h
|
|
|
3604df |
@@ -769,6 +769,9 @@ int glfs_posix_lock (glfs_fd_t *fd, int cmd, struct flock *flock) __THROW
|
|
|
3604df |
glfs_fd_t *glfs_dup (glfs_fd_t *fd) __THROW
|
|
|
3604df |
GFAPI_PUBLIC(glfs_dup, 3.4.0);
|
|
|
3604df |
|
|
|
3604df |
+void glfs_free (void *ptr) __THROW
|
|
|
3604df |
+ GFAPI_PUBLIC(glfs_free, 3.7.16);
|
|
|
3604df |
+
|
|
|
3604df |
/*
|
|
|
3604df |
* No xdata support for now. Nobody needs this call at all yet except for the
|
|
|
3604df |
* test script, and that doesn't need xdata. Adding dict_t support and a new
|
|
|
3604df |
--
|
|
|
3604df |
2.9.3
|
|
|
3604df |
|