From: Alexei Starovoitov <alexei.starovoitov@gmail.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Florian Weimer <fw@deneb.enyo.de>,
Peter Zijlstra <peterz@infradead.org>,
Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
linux-kernel <linux-kernel@vger.kernel.org>,
Matt Mullins <mmullins@mmlx.us>, Ingo Molnar <mingo@redhat.com>,
Alexei Starovoitov <ast@kernel.org>,
Daniel Borkmann <daniel@iogearbox.net>,
Dmitry Vyukov <dvyukov@google.com>,
Martin KaFai Lau <kafai@fb.com>, Song Liu <songliubraving@fb.com>,
Yonghong Song <yhs@fb.com>, Andrii Nakryiko <andriin@fb.com>,
John Fastabend <john.fastabend@gmail.com>,
KP Singh <kpsingh@chromium.org>, netdev <netdev@vger.kernel.org>,
bpf <bpf@vger.kernel.org>, Kees Cook <keescook@chromium.org>,
Josh Poimboeuf <jpoimboe@redhat.com>,
linux-toolchains@vger.kernel.org
Subject: Re: violating function pointer signature
Date: Wed, 18 Nov 2020 11:46:02 -0800 [thread overview]
Message-ID: <CAADnVQLv2pnw1x66Y_cYmdBg=sF+7s31VVoEmSerDnbuR0pU_g@mail.gmail.com> (raw)
In-Reply-To: <20201118092228.4f6e5930@gandalf.local.home>
On Wed, Nov 18, 2020 at 6:22 AM Steven Rostedt <rostedt@goodmis.org> wrote:
>
> Thus, all functions will be non-variadic in these cases.
That's not the only case where it will blow up.
Try this on sparc:
struct foo {
int a;
};
struct foo foo_struct(void) {
struct foo f = {};
return f;
}
int foo_int(void) {
return 0;
}
or this link:
https://godbolt.org/z/EdM47b
Notice:
jmp %i7+12
The function that returns small struct will jump to a different
instruction in the caller.
I think none of the tracepoints return structs and void foo(void) is
fine on x86.
Just pointing out that it's more than just variadic.
next prev parent reply other threads:[~2020-11-18 19:46 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-16 22:51 [PATCH] tracepoint: Do not fail unregistering a probe due to memory allocation Steven Rostedt
2020-11-16 23:16 ` Steven Rostedt
2020-11-17 19:15 ` Mathieu Desnoyers
2020-11-17 19:21 ` Steven Rostedt
2020-11-17 19:47 ` Mathieu Desnoyers
2020-11-17 20:34 ` Steven Rostedt
2020-11-17 20:58 ` Steven Rostedt
2020-11-17 21:22 ` Mathieu Desnoyers
2020-11-17 22:16 ` Steven Rostedt
2020-11-17 23:08 ` Mathieu Desnoyers
2020-11-18 1:11 ` Steven Rostedt
2020-11-17 21:08 ` Mathieu Desnoyers
2020-11-18 13:21 ` violating function pointer signature Peter Zijlstra
2020-11-18 13:59 ` Florian Weimer
2020-11-18 14:12 ` Peter Zijlstra
2020-11-18 14:18 ` Florian Weimer
2020-11-18 14:34 ` [PATCH v3] tracepoint: Do not fail unregistering a probe due to memory allocation Steven Rostedt
2020-11-24 5:59 ` Matt Mullins
2020-11-18 14:22 ` violating function pointer signature Steven Rostedt
2020-11-18 19:46 ` Alexei Starovoitov [this message]
2020-11-18 20:02 ` Steven Rostedt
2020-11-18 14:02 ` Steven Rostedt
2020-11-18 16:01 ` Mathieu Desnoyers
2020-11-18 16:19 ` David Laight
2020-11-18 16:50 ` Nick Desaulniers
2020-11-18 17:17 ` Steven Rostedt
2020-11-18 18:12 ` Segher Boessenkool
2020-11-18 18:31 ` Florian Weimer
2020-11-18 18:55 ` Segher Boessenkool
2020-11-18 18:58 ` Steven Rostedt
2020-11-18 18:59 ` Steven Rostedt
2020-11-18 19:11 ` Segher Boessenkool
2020-11-18 19:33 ` Steven Rostedt
2020-11-18 19:48 ` Segher Boessenkool
2020-11-18 20:44 ` Steven Rostedt
2020-11-19 8:21 ` Peter Zijlstra
2020-11-19 8:36 ` Peter Zijlstra
2020-11-19 14:37 ` Segher Boessenkool
2020-11-19 14:59 ` Steven Rostedt
2020-11-19 16:35 ` Segher Boessenkool
2020-11-19 17:42 ` David Laight
2020-11-19 19:27 ` Segher Boessenkool
2020-11-19 17:04 ` Alexei Starovoitov
2020-11-19 17:30 ` Steven Rostedt
2020-11-20 1:31 ` Nick Desaulniers
2020-11-17 21:33 ` [PATCH] tracepoint: Do not fail unregistering a probe due to memory allocation Kees Cook
2020-11-17 22:19 ` Steven Rostedt
2020-11-17 23:12 ` Mathieu Desnoyers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAADnVQLv2pnw1x66Y_cYmdBg=sF+7s31VVoEmSerDnbuR0pU_g@mail.gmail.com' \
--to=alexei.starovoitov@gmail.com \
--cc=andriin@fb.com \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=dvyukov@google.com \
--cc=fw@deneb.enyo.de \
--cc=john.fastabend@gmail.com \
--cc=jpoimboe@redhat.com \
--cc=kafai@fb.com \
--cc=keescook@chromium.org \
--cc=kpsingh@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-toolchains@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=mingo@redhat.com \
--cc=mmullins@mmlx.us \
--cc=netdev@vger.kernel.org \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=songliubraving@fb.com \
--cc=yhs@fb.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.