On Thu, Feb 09, 2017 at 11:11:32PM +0200, Michael S. Tsirkin wrote: > Attempts to assign values through these pointers would fail. > Mark them const to make sure the failure is compile time. > > Cc: Jason Baron <[email protected]> > Cc: Peter Zijlstra <[email protected]> > Cc: Steven Rostedt <[email protected]> > Cc: Borislav Petkov <[email protected]> > Cc: Andrew Morton <[email protected]> > Cc: Linus Torvalds <[email protected]> > Cc: Thomas Gleixner <[email protected]> > Cc: Mel Gorman <[email protected]> > Cc: Ingo Molnar <[email protected]> > Signed-off-by: Michael S. Tsirkin <[email protected]>
Hmm this produces a bunch of warnings that I managed to somehow miss. Self-Nack. Pls disregard. > --- > include/linux/jump_label.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/linux/jump_label.h b/include/linux/jump_label.h > index a0547c5..48ed83f 100644 > --- a/include/linux/jump_label.h > +++ b/include/linux/jump_label.h > @@ -132,8 +132,8 @@ static __always_inline bool static_key_true(struct > static_key *key) > return !arch_static_branch(key, true); > } > > -extern struct jump_entry __start___jump_table[]; > -extern struct jump_entry __stop___jump_table[]; > +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); > -- > MST
