lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 10 Feb 2017 14:05:36 -0500 From: Jason Baron <jbaron@...mai.com> To: "Michael S. Tsirkin" <mst@...hat.com>, linux-kernel@...r.kernel.org Cc: Peter Zijlstra <peterz@...radead.org>, Steven Rostedt <rostedt@...dmis.org>, Borislav Petkov <bp@...en8.de>, Andrew Morton <akpm@...ux-foundation.org>, Linus Torvalds <torvalds@...ux-foundation.org>, Thomas Gleixner <tglx@...utronix.de>, Mel Gorman <mgorman@...e.de>, Ingo Molnar <mingo@...nel.org> Subject: Re: [PATCH] jump_label: don't export section tags On 02/10/2017 12:29 PM, Michael S. Tsirkin wrote: > These are supplied by the linker and only used by jump_label.c. > This trivial patch moves them out of jump_label.h. > > Cc: Jason Baron <jbaron@...mai.com> > Cc: Peter Zijlstra <peterz@...radead.org> > Cc: Steven Rostedt <rostedt@...dmis.org> > Cc: Borislav Petkov <bp@...en8.de> > Cc: Andrew Morton <akpm@...ux-foundation.org> > Cc: Linus Torvalds <torvalds@...ux-foundation.org> > Cc: Thomas Gleixner <tglx@...utronix.de> > Cc: Mel Gorman <mgorman@...e.de> > Cc: Ingo Molnar <mingo@...nel.org> > Signed-off-by: Michael S. Tsirkin <mst@...hat.com> > --- > include/linux/jump_label.h | 3 --- > kernel/jump_label.c | 4 ++++ > 2 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/include/linux/jump_label.h b/include/linux/jump_label.h > index 48ed83f..21701da 100644 > --- a/include/linux/jump_label.h > +++ b/include/linux/jump_label.h > @@ -132,9 +132,6 @@ static __always_inline bool static_key_true(struct static_key *key) > return !arch_static_branch(key, true); > } > > -extern const struct jump_entry __start___jump_table[]; > -extern const struct jump_entry __stop___jump_table[]; > - > extern void jump_label_init(void); > extern void jump_label_lock(void); > extern void jump_label_unlock(void); hmmm - looks this was on top of your previous patch? Otherwise, looks good to me. Thanks, -Jason > diff --git a/kernel/jump_label.c b/kernel/jump_label.c > index a9b8cf5..6eaa84c 100644 > --- a/kernel/jump_label.c > +++ b/kernel/jump_label.c > @@ -18,6 +18,10 @@ > > #ifdef HAVE_JUMP_LABEL > > +/* Supplied by a linker script */ > +extern struct jump_entry __start___jump_table[]; > +extern struct jump_entry __stop___jump_table[]; > + > /* mutex to protect coming/going of the the jump_label table */ > static DEFINE_MUTEX(jump_label_mutex); > >
Powered by blists - more mailing lists