* [ptxdist] [PATCH] libkmod: avoid calling syscall() with -1
@ 2013-05-02 14:54 Jan Luebbe
2013-05-02 19:53 ` Jan Luebbe
0 siblings, 1 reply; 4+ messages in thread
From: Jan Luebbe @ 2013-05-02 14:54 UTC (permalink / raw)
To: ptxdist; +Cc: Jan Luebbe
This fixes an illegal instruction error on qemu ARM.
Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
---
...0001-libkmod-Avoid-calling-syscall-with-1.patch | 32 ++++++++++++++++++++
patches/kmod-13/series | 4 +++
2 files changed, 36 insertions(+)
create mode 100644 patches/kmod-13/0001-libkmod-Avoid-calling-syscall-with-1.patch
create mode 100644 patches/kmod-13/series
diff --git a/patches/kmod-13/0001-libkmod-Avoid-calling-syscall-with-1.patch b/patches/kmod-13/0001-libkmod-Avoid-calling-syscall-with-1.patch
new file mode 100644
index 0000000..bfeaf34
--- /dev/null
+++ b/patches/kmod-13/0001-libkmod-Avoid-calling-syscall-with-1.patch
@@ -0,0 +1,32 @@
+From: Jan Luebbe <jlu@pengutronix.de>
+Date: Thu, 2 May 2013 16:40:22 +0200
+Subject: [PATCH] libkmod: Avoid calling syscall() with -1
+
+At least in qemu 1.4.1 for vexpress/arm-cortexa9, this resulted in an
+illegal instruction error. Solve that by returning an error when
+__NR_finit_module is -1.
+---
+ libkmod/missing.h | 9 ++++++++-
+ 1 file changed, 8 insertions(+), 1 deletion(-)
+
+diff --git a/libkmod/missing.h b/libkmod/missing.h
+index edb88b9..ac24a35 100644
+--- a/libkmod/missing.h
++++ b/libkmod/missing.h
+@@ -20,8 +20,15 @@
+ #endif
+
+ #ifndef HAVE_FINIT_MODULE
++#include <errno.h>
++
+ static inline int finit_module(int fd, const char *uargs, int flags)
+ {
+- return syscall(__NR_finit_module, fd, uargs, flags);
++ if (__NR_finit_module == -1) {
++ errno = ENOSYS;
++ return -1;
++ } else {
++ return syscall(__NR_finit_module, fd, uargs, flags);
++ }
+ }
+ #endif
diff --git a/patches/kmod-13/series b/patches/kmod-13/series
new file mode 100644
index 0000000..8f9a8ac
--- /dev/null
+++ b/patches/kmod-13/series
@@ -0,0 +1,4 @@
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-libkmod-Avoid-calling-syscall-with-1.patch
+# 663dcc1cd4b4be9a3934dab348129c99 - git-ptx-patches magic
--
1.7.10.4
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [ptxdist] [PATCH] libkmod: avoid calling syscall() with -1
2013-05-02 14:54 [ptxdist] [PATCH] libkmod: avoid calling syscall() with -1 Jan Luebbe
@ 2013-05-02 19:53 ` Jan Luebbe
2013-05-02 20:05 ` Jan Luebbe
0 siblings, 1 reply; 4+ messages in thread
From: Jan Luebbe @ 2013-05-02 19:53 UTC (permalink / raw)
To: ptxdist
On Thu, May 02, 2013 at 04:54:30PM +0200, Jan Luebbe wrote:
> This fixes an illegal instruction error on qemu ARM.
Please do not apply this yet, there is still some discussion going on upstream:
http://bugs.funtoo.org/browse/FL-534
http://thread.gmane.org/gmane.linux.kernel.modules/1061
Jan
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 4+ messages in thread
* [ptxdist] [PATCH] libkmod: avoid calling syscall() with -1
2013-05-02 19:53 ` Jan Luebbe
@ 2013-05-02 20:05 ` Jan Luebbe
2013-05-03 16:58 ` Michael Olbrich
0 siblings, 1 reply; 4+ messages in thread
From: Jan Luebbe @ 2013-05-02 20:05 UTC (permalink / raw)
To: ptxdist; +Cc: Jan Luebbe
This fixes an illegal instruction error on qemu ARM.
Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
---
It seems this patch has been applied upstream:
http://thread.gmane.org/gmane.linux.kernel.modules/1061/focus=1064
...ssing-Don-t-call-syscall-with-syscallno-1.patch | 40 ++++++++++++++++++++
patches/kmod-13/series | 4 ++
2 files changed, 44 insertions(+)
create mode 100644 patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch
create mode 100644 patches/kmod-13/series
diff --git a/patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch b/patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch
new file mode 100644
index 0000000..a385d63
--- /dev/null
+++ b/patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch
@@ -0,0 +1,40 @@
+From: Lucas De Marchi <lucas.de.marchi@gmail.com>
+Date: Thu, 2 May 2013 13:22:57 -0300
+Subject: [PATCH] missing: Don't call syscall() with syscallno == -1
+
+Reported-by: Jean-Francis Roy <jeanfrancis@funtoo.org>
+Reported-by: Jan Luebbe <jlu@pengutronix.de>
+---
+ libkmod/missing.h | 10 ++++++----
+ 1 file changed, 6 insertions(+), 4 deletions(-)
+
+diff --git a/libkmod/missing.h b/libkmod/missing.h
+index edb88b9..b31af84 100644
+--- a/libkmod/missing.h
++++ b/libkmod/missing.h
+@@ -1,5 +1,6 @@
+ #pragma once
+
++#include <errno.h>
+ #include <unistd.h>
+ #include <sys/syscall.h>
+
+@@ -15,13 +16,14 @@
+ # define MODULE_INIT_IGNORE_VERMAGIC 2
+ #endif
+
+-#ifndef __NR_finit_module
+-# define __NR_finit_module -1
+-#endif
+-
+ #ifndef HAVE_FINIT_MODULE
+ static inline int finit_module(int fd, const char *uargs, int flags)
+ {
++#ifndef __NR_finit_module
++ errno = ENOSYS;
++ return -1;
++#else
+ return syscall(__NR_finit_module, fd, uargs, flags);
++#endif
+ }
+ #endif
diff --git a/patches/kmod-13/series b/patches/kmod-13/series
new file mode 100644
index 0000000..3fc301f
--- /dev/null
+++ b/patches/kmod-13/series
@@ -0,0 +1,4 @@
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-missing-Don-t-call-syscall-with-syscallno-1.patch
+# 0b7413dd7632441bc24225859a7d1866 - git-ptx-patches magic
--
1.7.10.4
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [ptxdist] [PATCH] libkmod: avoid calling syscall() with -1
2013-05-02 20:05 ` Jan Luebbe
@ 2013-05-03 16:58 ` Michael Olbrich
0 siblings, 0 replies; 4+ messages in thread
From: Michael Olbrich @ 2013-05-03 16:58 UTC (permalink / raw)
To: ptxdist
On Thu, May 02, 2013 at 10:05:15PM +0200, Jan Luebbe wrote:
> This fixes an illegal instruction error on qemu ARM.
Thanks, applied.
Michael
> Signed-off-by: Jan Luebbe <jlu@pengutronix.de>
> ---
>
> It seems this patch has been applied upstream:
> http://thread.gmane.org/gmane.linux.kernel.modules/1061/focus=1064
>
> ...ssing-Don-t-call-syscall-with-syscallno-1.patch | 40 ++++++++++++++++++++
> patches/kmod-13/series | 4 ++
> 2 files changed, 44 insertions(+)
> create mode 100644 patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch
> create mode 100644 patches/kmod-13/series
>
> diff --git a/patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch b/patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch
> new file mode 100644
> index 0000000..a385d63
> --- /dev/null
> +++ b/patches/kmod-13/0001-missing-Don-t-call-syscall-with-syscallno-1.patch
> @@ -0,0 +1,40 @@
> +From: Lucas De Marchi <lucas.de.marchi@gmail.com>
> +Date: Thu, 2 May 2013 13:22:57 -0300
> +Subject: [PATCH] missing: Don't call syscall() with syscallno == -1
> +
> +Reported-by: Jean-Francis Roy <jeanfrancis@funtoo.org>
> +Reported-by: Jan Luebbe <jlu@pengutronix.de>
> +---
> + libkmod/missing.h | 10 ++++++----
> + 1 file changed, 6 insertions(+), 4 deletions(-)
> +
> +diff --git a/libkmod/missing.h b/libkmod/missing.h
> +index edb88b9..b31af84 100644
> +--- a/libkmod/missing.h
> ++++ b/libkmod/missing.h
> +@@ -1,5 +1,6 @@
> + #pragma once
> +
> ++#include <errno.h>
> + #include <unistd.h>
> + #include <sys/syscall.h>
> +
> +@@ -15,13 +16,14 @@
> + # define MODULE_INIT_IGNORE_VERMAGIC 2
> + #endif
> +
> +-#ifndef __NR_finit_module
> +-# define __NR_finit_module -1
> +-#endif
> +-
> + #ifndef HAVE_FINIT_MODULE
> + static inline int finit_module(int fd, const char *uargs, int flags)
> + {
> ++#ifndef __NR_finit_module
> ++ errno = ENOSYS;
> ++ return -1;
> ++#else
> + return syscall(__NR_finit_module, fd, uargs, flags);
> ++#endif
> + }
> + #endif
> diff --git a/patches/kmod-13/series b/patches/kmod-13/series
> new file mode 100644
> index 0000000..3fc301f
> --- /dev/null
> +++ b/patches/kmod-13/series
> @@ -0,0 +1,4 @@
> +# generated by git-ptx-patches
> +#tag:base --start-number 1
> +0001-missing-Don-t-call-syscall-with-syscallno-1.patch
> +# 0b7413dd7632441bc24225859a7d1866 - git-ptx-patches magic
> --
> 1.7.10.4
>
>
> --
> ptxdist mailing list
> ptxdist@pengutronix.de
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-05-03 16:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-02 14:54 [ptxdist] [PATCH] libkmod: avoid calling syscall() with -1 Jan Luebbe
2013-05-02 19:53 ` Jan Luebbe
2013-05-02 20:05 ` Jan Luebbe
2013-05-03 16:58 ` Michael Olbrich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox