From 5ee3f305e81fd2e1953d8a74530326f728a33ce8 Mon Sep 17 00:00:00 2001 From: Mark Wielaard Date: Fri, 18 Sep 2009 16:13:24 +0200 Subject: Always mark .probes section as writable. Allocated section needs to be writable when creating pic shared objects because we store relocatable addresses in them. We used to make this read only for non-pic executables, but the new semaphore support relies on having a writable .probes section to put the enabled variables in. * includes/sys/sdt.h (ALLOCSEC): Define unconditionally as "aw". --- includes/sys/sdt.h | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'includes') diff --git a/includes/sys/sdt.h b/includes/sys/sdt.h index 104973df..7affade6 100644 --- a/includes/sys/sdt.h +++ b/includes/sys/sdt.h @@ -18,12 +18,10 @@ #endif /* Allocated section needs to be writable when creating pic shared objects - because we store relocatable addresses in them. */ -#ifdef __PIC__ + because we store relocatable addresses in them. We used to make this + read only for non-pic executables, but the new semaphore support relies + on having a writable .probes section to put the enabled variables in. */ #define ALLOCSEC "\"aw\"" -#else -#define ALLOCSEC "\"a\"" -#endif /* An allocated section .probes that holds the probe names and addrs. */ #define STAP_PROBE_DATA_(probe,guard,arg) \ -- cgit