Commit b2ca74d3 authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'core-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip

Pull core fixes from Ingo Molnar:
 "A handful of objtool updates, plus a documentation addition for
  __ab_c_size()"

* 'core-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
  objtool: Fix whitelist documentation typo
  objtool: Fix function fallthrough detection
  objtool: Don't use ignore flag for fake jumps
  overflow.h: Add comment documenting __ab_c_size()
parents 8c05f3b9 2decec48
...@@ -284,11 +284,15 @@ static inline __must_check size_t array3_size(size_t a, size_t b, size_t c) ...@@ -284,11 +284,15 @@ static inline __must_check size_t array3_size(size_t a, size_t b, size_t c)
return bytes; return bytes;
} }
static inline __must_check size_t __ab_c_size(size_t n, size_t size, size_t c) /*
* Compute a*b+c, returning SIZE_MAX on overflow. Internal helper for
* struct_size() below.
*/
static inline __must_check size_t __ab_c_size(size_t a, size_t b, size_t c)
{ {
size_t bytes; size_t bytes;
if (check_mul_overflow(n, size, &bytes)) if (check_mul_overflow(a, b, &bytes))
return SIZE_MAX; return SIZE_MAX;
if (check_add_overflow(bytes, c, &bytes)) if (check_add_overflow(bytes, c, &bytes))
return SIZE_MAX; return SIZE_MAX;
......
...@@ -306,7 +306,7 @@ ignore it: ...@@ -306,7 +306,7 @@ ignore it:
- To skip validation of a file, add - To skip validation of a file, add
OBJECT_FILES_NON_STANDARD_filename.o := n OBJECT_FILES_NON_STANDARD_filename.o := y
to the Makefile. to the Makefile.
......
...@@ -28,6 +28,8 @@ ...@@ -28,6 +28,8 @@
#include <linux/hashtable.h> #include <linux/hashtable.h>
#include <linux/kernel.h> #include <linux/kernel.h>
#define FAKE_JUMP_OFFSET -1
struct alternative { struct alternative {
struct list_head list; struct list_head list;
struct instruction *insn; struct instruction *insn;
...@@ -568,7 +570,7 @@ static int add_jump_destinations(struct objtool_file *file) ...@@ -568,7 +570,7 @@ static int add_jump_destinations(struct objtool_file *file)
insn->type != INSN_JUMP_UNCONDITIONAL) insn->type != INSN_JUMP_UNCONDITIONAL)
continue; continue;
if (insn->ignore) if (insn->ignore || insn->offset == FAKE_JUMP_OFFSET)
continue; continue;
rela = find_rela_by_dest_range(insn->sec, insn->offset, rela = find_rela_by_dest_range(insn->sec, insn->offset,
...@@ -745,10 +747,10 @@ static int handle_group_alt(struct objtool_file *file, ...@@ -745,10 +747,10 @@ static int handle_group_alt(struct objtool_file *file,
clear_insn_state(&fake_jump->state); clear_insn_state(&fake_jump->state);
fake_jump->sec = special_alt->new_sec; fake_jump->sec = special_alt->new_sec;
fake_jump->offset = -1; fake_jump->offset = FAKE_JUMP_OFFSET;
fake_jump->type = INSN_JUMP_UNCONDITIONAL; fake_jump->type = INSN_JUMP_UNCONDITIONAL;
fake_jump->jump_dest = list_next_entry(last_orig_insn, list); fake_jump->jump_dest = list_next_entry(last_orig_insn, list);
fake_jump->ignore = true; fake_jump->func = orig_insn->func;
} }
if (!special_alt->new_len) { if (!special_alt->new_len) {
...@@ -1957,7 +1959,8 @@ static int validate_branch(struct objtool_file *file, struct instruction *first, ...@@ -1957,7 +1959,8 @@ static int validate_branch(struct objtool_file *file, struct instruction *first,
return 1; return 1;
} }
func = insn->func ? insn->func->pfunc : NULL; if (insn->func)
func = insn->func->pfunc;
if (func && insn->ignore) { if (func && insn->ignore) {
WARN_FUNC("BUG: why am I validating an ignored function?", WARN_FUNC("BUG: why am I validating an ignored function?",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment