[ fixed typos, separated __real functions, exported assert_nrt ]

Provide assert_nrt helper that checks if the caller is either not a
shadow thread or is currently running in relaxed mode. If not, SIG_XCPU
is raised.

This service is then used to provide wrappers for glibc functions that
are not realtime-safe but do not always trigger a syscall. Such
functions may therefore be used by RT threads in primary mode for quite
a while without being detected via some Xenomai mode switch. Moreover,
some functions that go through the vsyscall page even to raise an
ordinary syscall may not allow proper stack backtraces, making it harder
to find their callers.

So far we provide wrappers (for use with the --wrap linker switch) for
malloc/free, gettimeofday and clock_gettime. Adding more (if there
are/will be more) is trivial.

Signed-off-by: Jan Kiszka <jan.kis...@siemens.com>
---

 include/rtdk.h            |    2 +
 src/rtdk/Makefile.am      |    6 ++--
 src/rtdk/assert_context.c |   73 +++++++++++++++++++++++++++++++++++++++++++++
 src/rtdk/internal.h       |    9 ++++++
 src/rtdk/wrappers.c       |   45 ++++++++++++++++++++++++++++
 5 files changed, 133 insertions(+), 2 deletions(-)
 create mode 100644 src/rtdk/assert_context.c
 create mode 100644 src/rtdk/wrappers.c

diff --git a/include/rtdk.h b/include/rtdk.h
index fc7525d..981bfda 100644
--- a/include/rtdk.h
+++ b/include/rtdk.h
@@ -58,6 +58,8 @@ void rt_print_cleanup(void);
 void rt_print_auto_init(int enable);
 const char *rt_print_buffer_name(void);
 
+void assert_nrt(void);
+
 #ifdef __cplusplus
 }
 #endif /* __cplusplus */
diff --git a/src/rtdk/Makefile.am b/src/rtdk/Makefile.am
index cb80262..778c697 100644
--- a/src/rtdk/Makefile.am
+++ b/src/rtdk/Makefile.am
@@ -1,10 +1,12 @@
 lib_LTLIBRARIES = librtdk.la
 
-librtdk_la_LDFLAGS = -version-info 0:0:0 -lpthread
+librtdk_la_LDFLAGS = -version-info 0:0:0 -lpthread -lrt
 
 librtdk_la_SOURCES = \
        init.c \
-       rt_print.c
+       rt_print.c \
+       assert_context.c \
+       wrappers.c
 
 librtdk_la_CPPFLAGS = \
        @XENO_USER_CFLAGS@ \
diff --git a/src/rtdk/assert_context.c b/src/rtdk/assert_context.c
new file mode 100644
index 0000000..2176db3
--- /dev/null
+++ b/src/rtdk/assert_context.c
@@ -0,0 +1,73 @@
+/*
+ * Copyright (C) 2008, 2009 Jan Kiszka <jan.kis...@siemens.com>.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA.
+ */
+
+#include <signal.h>
+#include <stdlib.h>
+
+#include "internal.h"
+
+#include <nucleus/thread.h>
+#include <asm-generic/syscall.h>
+#include <asm-generic/bits/current.h>
+
+void assert_nrt(void)
+{
+       xnthread_info_t info;
+       int err;
+
+       if (unlikely(xeno_get_current() != XN_NO_HANDLE &&
+                    !(xeno_get_current_mode() & XNRELAX))) {
+
+               err = XENOMAI_SYSCALL1(__xn_sys_current_info, &info);
+
+               if (err) {
+                       fprintf(stderr, "__xn_sys_current_info failed: %s\n",
+                               strerror(-err));
+                       return;
+               }
+
+               if (info.state & XNTRAPSW)
+                       pthread_kill(pthread_self(), SIGXCPU);
+       }
+}
+
+/* Memory allocation services */
+void *__wrap_malloc(size_t size)
+{
+       assert_nrt();
+       return __real_malloc(size);
+}
+
+void __wrap_free(void *ptr)
+{
+       assert_nrt();
+       __real_free(ptr);
+}
+
+/* vsyscall-based services */
+int __wrap_gettimeofday(struct timeval *tv, struct timezone *tz)
+{
+       assert_nrt();
+       return __real_gettimeofday(tv, tz);
+}
+
+int __wrap_clock_gettime(clockid_t clk_id, struct timespec *tp)
+{
+       assert_nrt();
+       return __real_clock_gettime(clk_id, tp);
+}
diff --git a/src/rtdk/internal.h b/src/rtdk/internal.h
index 69b869f..bd15b2d 100644
--- a/src/rtdk/internal.h
+++ b/src/rtdk/internal.h
@@ -19,6 +19,15 @@
 #ifndef _LIBRTUTILS_INTERNAL_H
 #define _LIBRTUTILS_INTERNAL_H
 
+#include <time.h>
+#include <sys/time.h>
+
 void __rt_print_init(void);
 
+void __real_free(void *ptr);
+void *__real_malloc(size_t size);
+
+int __real_gettimeofday(struct timeval *tv, struct timezone *tz);
+int __real_clock_gettime(clockid_t clk_id, struct timespec *tp);
+
 #endif /* !_LIBRTUTILS_INTERNAL_H */
diff --git a/src/rtdk/wrappers.c b/src/rtdk/wrappers.c
new file mode 100644
index 0000000..e5340c9
--- /dev/null
+++ b/src/rtdk/wrappers.c
@@ -0,0 +1,45 @@
+/*
+ * Copyright (C) 2008, 2009 Jan Kiszka <jan.kis...@siemens.com>.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA.
+ */
+
+#include <stdlib.h>
+
+#include "internal.h"
+
+__attribute__ ((weak))
+void *__real_malloc(size_t size)
+{
+       return malloc(size);
+}
+
+__attribute__ ((weak))
+void __real_free(void *ptr)
+{
+       free(ptr);
+}
+
+__attribute__ ((weak))
+int __real_gettimeofday(struct timeval *tv, struct timezone *tz)
+{
+       return gettimeofday(tv, tz);
+}
+
+__attribute__ ((weak))
+int __real_clock_gettime(clockid_t clk_id, struct timespec *tp)
+{
+       return clock_gettime(clk_id, tp);
+}

_______________________________________________
Xenomai-core mailing list
Xenomai-core@gna.org
https://mail.gna.org/listinfo/xenomai-core

Reply via email to