diff options
author | Nadav Amit <namit@vmware.com> | 2019-06-13 02:48:05 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2019-06-23 08:26:26 -0400 |
commit | caa759323c73676b3e48c8d9c86093c88b4aba97 (patch) | |
tree | 15c62c8f6b18b481233ab7edf9cc7cea091dfd71 /arch/alpha/kernel/smp.c | |
parent | a22793c79d6ea0a492ce1a308ec46df52ee9406e (diff) |
smp: Remove smp_call_function() and on_each_cpu() return values
The return value is fixed. Remove it and amend the callers.
[ tglx: Fixup arm/bL_switcher and powerpc/rtas ]
Signed-off-by: Nadav Amit <namit@vmware.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Andy Lutomirski <luto@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Richard Henderson <rth@twiddle.net>
Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru>
Cc: Matt Turner <mattst88@gmail.com>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Link: https://lkml.kernel.org/r/20190613064813.8102-2-namit@vmware.com
Diffstat (limited to 'arch/alpha/kernel/smp.c')
-rw-r--r-- | arch/alpha/kernel/smp.c | 19 |
1 files changed, 5 insertions, 14 deletions
diff --git a/arch/alpha/kernel/smp.c b/arch/alpha/kernel/smp.c index d0dccae53ba9..5f90df30be20 100644 --- a/arch/alpha/kernel/smp.c +++ b/arch/alpha/kernel/smp.c | |||
@@ -614,8 +614,7 @@ void | |||
614 | smp_imb(void) | 614 | smp_imb(void) |
615 | { | 615 | { |
616 | /* Must wait other processors to flush their icache before continue. */ | 616 | /* Must wait other processors to flush their icache before continue. */ |
617 | if (on_each_cpu(ipi_imb, NULL, 1)) | 617 | on_each_cpu(ipi_imb, NULL, 1); |
618 | printk(KERN_CRIT "smp_imb: timed out\n"); | ||
619 | } | 618 | } |
620 | EXPORT_SYMBOL(smp_imb); | 619 | EXPORT_SYMBOL(smp_imb); |
621 | 620 | ||
@@ -630,9 +629,7 @@ flush_tlb_all(void) | |||
630 | { | 629 | { |
631 | /* Although we don't have any data to pass, we do want to | 630 | /* Although we don't have any data to pass, we do want to |
632 | synchronize with the other processors. */ | 631 | synchronize with the other processors. */ |
633 | if (on_each_cpu(ipi_flush_tlb_all, NULL, 1)) { | 632 | on_each_cpu(ipi_flush_tlb_all, NULL, 1); |
634 | printk(KERN_CRIT "flush_tlb_all: timed out\n"); | ||
635 | } | ||
636 | } | 633 | } |
637 | 634 | ||
638 | #define asn_locked() (cpu_data[smp_processor_id()].asn_lock) | 635 | #define asn_locked() (cpu_data[smp_processor_id()].asn_lock) |
@@ -667,9 +664,7 @@ flush_tlb_mm(struct mm_struct *mm) | |||
667 | } | 664 | } |
668 | } | 665 | } |
669 | 666 | ||
670 | if (smp_call_function(ipi_flush_tlb_mm, mm, 1)) { | 667 | smp_call_function(ipi_flush_tlb_mm, mm, 1); |
671 | printk(KERN_CRIT "flush_tlb_mm: timed out\n"); | ||
672 | } | ||
673 | 668 | ||
674 | preempt_enable(); | 669 | preempt_enable(); |
675 | } | 670 | } |
@@ -720,9 +715,7 @@ flush_tlb_page(struct vm_area_struct *vma, unsigned long addr) | |||
720 | data.mm = mm; | 715 | data.mm = mm; |
721 | data.addr = addr; | 716 | data.addr = addr; |
722 | 717 | ||
723 | if (smp_call_function(ipi_flush_tlb_page, &data, 1)) { | 718 | smp_call_function(ipi_flush_tlb_page, &data, 1); |
724 | printk(KERN_CRIT "flush_tlb_page: timed out\n"); | ||
725 | } | ||
726 | 719 | ||
727 | preempt_enable(); | 720 | preempt_enable(); |
728 | } | 721 | } |
@@ -772,9 +765,7 @@ flush_icache_user_range(struct vm_area_struct *vma, struct page *page, | |||
772 | } | 765 | } |
773 | } | 766 | } |
774 | 767 | ||
775 | if (smp_call_function(ipi_flush_icache_page, mm, 1)) { | 768 | smp_call_function(ipi_flush_icache_page, mm, 1); |
776 | printk(KERN_CRIT "flush_icache_page: timed out\n"); | ||
777 | } | ||
778 | 769 | ||
779 | preempt_enable(); | 770 | preempt_enable(); |
780 | } | 771 | } |