From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from aserp2130.oracle.com ([141.146.126.79]) by casper.infradead.org with esmtps (Exim 4.89 #1 (Red Hat Linux)) id 1ee409-0004tE-Ok for kexec@lists.infradead.org; Tue, 23 Jan 2018 19:12:59 +0000 From: Eric DeVolder Subject: [PATCH v2 2/2] kexec-tools: Make xc_dlhandle static Date: Tue, 23 Jan 2018 13:12:51 -0600 Message-Id: <1516734771-14795-3-git-send-email-eric.devolder@oracle.com> In-Reply-To: <1516734771-14795-1-git-send-email-eric.devolder@oracle.com> References: <1516734771-14795-1-git-send-email-eric.devolder@oracle.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: kexec@lists.infradead.org, horms@verge.net.au, andrew.cooper3@citrix.com Cc: daniel.kiper@oracle.com, eric.devolder@oracle.com, konrad.wilk@oracle.com This patch is a follow-on to commit 43d3932e "kexec-tools: Perform run-time linking of libxenctrl.so". This patch addresses feedback from Daniel Kiper. This patch implements Daniel's suggestion to make the xc_dlhandle variable static. Signed-off-by: Eric DeVolder --- v1: 23jan2018 - Implemented feedback from Daniel Kiper v2: 23jan2018 - Implemented feedback from Daniel Kiper - Broke patch into two --- kexec/kexec-xen.c | 9 ++++++++- kexec/kexec-xen.h | 9 ++++----- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/kexec/kexec-xen.c b/kexec/kexec-xen.c index 75f8758..960fa16 100644 --- a/kexec/kexec-xen.c +++ b/kexec/kexec-xen.c @@ -15,8 +15,15 @@ #include "crashdump.h" #ifdef CONFIG_LIBXENCTRL_DL -void *xc_dlhandle; +/* The handle from dlopen(), needed by dlsym(), dlclose() */ +static void *xc_dlhandle; xc_hypercall_buffer_t XC__HYPERCALL_BUFFER_NAME(HYPERCALL_BUFFER_NULL); + +void *__xc_dlsym(const char *symbol) +{ + return dlsym(xc_dlhandle, symbol); +} + xc_interface *__xc_interface_open(xentoollog_logger *logger, xentoollog_logger *dombuild_logger, unsigned open_flags) diff --git a/kexec/kexec-xen.h b/kexec/kexec-xen.h index ffb8743..8955334 100644 --- a/kexec/kexec-xen.h +++ b/kexec/kexec-xen.h @@ -6,9 +6,8 @@ #ifdef CONFIG_LIBXENCTRL_DL #include - -/* The handle from dlopen(), needed by dlsym(), dlclose() */ -extern void *xc_dlhandle; +/* Lookup symbols in libxenctrl.so */ +void *__xc_dlsym(const char *symbol); /* Wrappers around xc_interface_open/close() to insert dlopen/dlclose() */ xc_interface *__xc_interface_open(xentoollog_logger *logger, @@ -21,13 +20,13 @@ int __xc_interface_close(xc_interface *xch); ( \ { dtype value; \ typedef dtype (*func_t)(xc_interface *, ...); \ - func_t func = dlsym(xc_dlhandle, #name); \ + func_t func = __xc_dlsym(#name); \ value = func(args); \ value; } \ ) #define __xc_data(dtype, name) \ ( \ - { dtype *value = (dtype *)dlsym(xc_dlhandle, #name); value; } \ + { dtype *value = (dtype *)__xc_dlsym(#name); value; } \ ) /* The wrappers around utilized xenctrl.h functions */ -- 2.7.4 _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec