um: rename set_signals() to um_set_signals()
authorJohannes Berg <johannes.berg@intel.com>
Tue, 31 Aug 2021 07:11:15 +0000 (09:11 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 27 Jan 2022 10:03:44 +0000 (11:03 +0100)
[ Upstream commit bbe33504d4a7fdab9011211e55e262c869b3f6cc ]

Rename set_signals() as there's at least one driver that
uses the same name and can now be built on UM due to PCI
support, and thus we can get symbol conflicts.

Also rename set_signals_trace() to be consistent.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: 68f5d3f3b654 ("um: add PCI over virtio emulation driver")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Signed-off-by: Sasha Levin <sashal@kernel.org>
arch/um/include/asm/irqflags.h
arch/um/include/shared/longjmp.h
arch/um/include/shared/os.h
arch/um/kernel/ksyms.c
arch/um/os-Linux/sigio.c
arch/um/os-Linux/signal.c

index dab5744..1e69ef5 100644 (file)
@@ -3,7 +3,7 @@
 #define __UM_IRQFLAGS_H
 
 extern int signals_enabled;
-int set_signals(int enable);
+int um_set_signals(int enable);
 void block_signals(void);
 void unblock_signals(void);
 
@@ -16,7 +16,7 @@ static inline unsigned long arch_local_save_flags(void)
 #define arch_local_irq_restore arch_local_irq_restore
 static inline void arch_local_irq_restore(unsigned long flags)
 {
-       set_signals(flags);
+       um_set_signals(flags);
 }
 
 #define arch_local_irq_enable arch_local_irq_enable
index bdb2869..8863319 100644 (file)
@@ -18,7 +18,7 @@ extern void longjmp(jmp_buf, int);
        enable = *(volatile int *)&signals_enabled;     \
        n = setjmp(*buf);                               \
        if(n != 0)                                      \
-               set_signals_trace(enable);              \
+               um_set_signals_trace(enable);           \
        n; })
 
 #endif
index 96d4003..03ffbdd 100644 (file)
@@ -238,8 +238,8 @@ extern void send_sigio_to_self(void);
 extern int change_sig(int signal, int on);
 extern void block_signals(void);
 extern void unblock_signals(void);
-extern int set_signals(int enable);
-extern int set_signals_trace(int enable);
+extern int um_set_signals(int enable);
+extern int um_set_signals_trace(int enable);
 extern int os_is_signal_stack(void);
 extern void deliver_alarm(void);
 extern void register_pm_wake_signal(void);
index b1e5634..3a85bde 100644 (file)
@@ -6,7 +6,7 @@
 #include <linux/module.h>
 #include <os.h>
 
-EXPORT_SYMBOL(set_signals);
+EXPORT_SYMBOL(um_set_signals);
 EXPORT_SYMBOL(signals_enabled);
 
 EXPORT_SYMBOL(os_stat_fd);
index 6597ea1..9e71794 100644 (file)
@@ -132,7 +132,7 @@ static void update_thread(void)
        int n;
        char c;
 
-       flags = set_signals_trace(0);
+       flags = um_set_signals_trace(0);
        CATCH_EINTR(n = write(sigio_private[0], &c, sizeof(c)));
        if (n != sizeof(c)) {
                printk(UM_KERN_ERR "update_thread : write failed, err = %d\n",
@@ -147,7 +147,7 @@ static void update_thread(void)
                goto fail;
        }
 
-       set_signals_trace(flags);
+       um_set_signals_trace(flags);
        return;
  fail:
        /* Critical section start */
@@ -161,7 +161,7 @@ static void update_thread(void)
        close(write_sigio_fds[0]);
        close(write_sigio_fds[1]);
        /* Critical section end */
-       set_signals_trace(flags);
+       um_set_signals_trace(flags);
 }
 
 int __add_sigio_fd(int fd)
index 6cf098c..24a403a 100644 (file)
@@ -94,7 +94,7 @@ void sig_handler(int sig, struct siginfo *si, mcontext_t *mc)
 
        sig_handler_common(sig, si, mc);
 
-       set_signals_trace(enabled);
+       um_set_signals_trace(enabled);
 }
 
 static void timer_real_alarm_handler(mcontext_t *mc)
@@ -126,7 +126,7 @@ void timer_alarm_handler(int sig, struct siginfo *unused_si, mcontext_t *mc)
 
        signals_active &= ~SIGALRM_MASK;
 
-       set_signals_trace(enabled);
+       um_set_signals_trace(enabled);
 }
 
 void deliver_alarm(void) {
@@ -348,7 +348,7 @@ void unblock_signals(void)
        }
 }
 
-int set_signals(int enable)
+int um_set_signals(int enable)
 {
        int ret;
        if (signals_enabled == enable)
@@ -362,7 +362,7 @@ int set_signals(int enable)
        return ret;
 }
 
-int set_signals_trace(int enable)
+int um_set_signals_trace(int enable)
 {
        int ret;
        if (signals_enabled == enable)