lkml.org 
[lkml]   [2022]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.16 0337/1039] um: rename set_signals() to um_set_signals()
    Date
    From: Johannes Berg <johannes.berg@intel.com>

    [ 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 | 4 ++--
    arch/um/include/shared/longjmp.h | 2 +-
    arch/um/include/shared/os.h | 4 ++--
    arch/um/kernel/ksyms.c | 2 +-
    arch/um/os-Linux/sigio.c | 6 +++---
    arch/um/os-Linux/signal.c | 8 ++++----
    6 files changed, 13 insertions(+), 13 deletions(-)

    diff --git a/arch/um/include/asm/irqflags.h b/arch/um/include/asm/irqflags.h
    index dab5744e9253d..1e69ef5bc35e0 100644
    --- a/arch/um/include/asm/irqflags.h
    +++ b/arch/um/include/asm/irqflags.h
    @@ -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
    diff --git a/arch/um/include/shared/longjmp.h b/arch/um/include/shared/longjmp.h
    index bdb2869b72b31..8863319039f3d 100644
    --- a/arch/um/include/shared/longjmp.h
    +++ b/arch/um/include/shared/longjmp.h
    @@ -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
    diff --git a/arch/um/include/shared/os.h b/arch/um/include/shared/os.h
    index 96d400387c93e..03ffbdddcc480 100644
    --- a/arch/um/include/shared/os.h
    +++ b/arch/um/include/shared/os.h
    @@ -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);
    diff --git a/arch/um/kernel/ksyms.c b/arch/um/kernel/ksyms.c
    index b1e5634398d09..3a85bde3e1734 100644
    --- a/arch/um/kernel/ksyms.c
    +++ b/arch/um/kernel/ksyms.c
    @@ -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);
    diff --git a/arch/um/os-Linux/sigio.c b/arch/um/os-Linux/sigio.c
    index 6597ea1986ffa..9e71794839e87 100644
    --- a/arch/um/os-Linux/sigio.c
    +++ b/arch/um/os-Linux/sigio.c
    @@ -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)
    diff --git a/arch/um/os-Linux/signal.c b/arch/um/os-Linux/signal.c
    index 6cf098c23a394..24a403a70a020 100644
    --- a/arch/um/os-Linux/signal.c
    +++ b/arch/um/os-Linux/signal.c
    @@ -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)
    --
    2.34.1


    \
     
     \ /
      Last update: 2022-01-25 01:11    [W:2.222 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site