ocfs2: remove __mlog_cpu_guess
authorAndrew Morton <akpm@linux-foundation.org>
Wed, 24 Jun 2015 23:54:56 +0000 (16:54 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 25 Jun 2015 00:49:39 +0000 (17:49 -0700)
raw_smp_processor_id() is the means of avoiding the runtime preemptibility
check.

[akpm@linux-foundation.org: fix printk warning]
Cc: Joe Perches <joe@perches.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/cluster/masklog.c

index c2182ba1013cbca159bea346d7769eb4b03843a7..dfe162f5fd4cf3ef23098187ea7bd9823f4b0527 100644 (file)
@@ -64,19 +64,6 @@ static ssize_t mlog_mask_store(u64 mask, const char *buf, size_t count)
        return count;
 }
 
-/*
- * smp_processor_id() "helpfully" screams when called outside preemptible
- * regions in current kernels.  sles doesn't have the variants that don't
- * scream.  just do this instead of trying to guess which we're building
- * against.. *sigh*.
- */
-#define __mlog_cpu_guess                                               \
-({                                                                     \
-       unsigned long _cpu = get_cpu();                                 \
-       put_cpu();                                                      \
-       _cpu;                                                           \
-})
-
 void __mlog_printk(const u64 *mask, const char *func, int line,
                   const char *fmt, ...)
 {
@@ -103,9 +90,9 @@ void __mlog_printk(const u64 *mask, const char *func, int line,
        vaf.fmt = fmt;
        vaf.va = &args;
 
-       printk("%s(%s,%u,%lu):%s:%d %s%pV",
-              level, current->comm, task_pid_nr(current), __mlog_cpu_guess,
-              func, line, prefix, &vaf);
+       printk("%s(%s,%u,%u):%s:%d %s%pV",
+              level, current->comm, task_pid_nr(current),
+              raw_smp_processor_id(), func, line, prefix, &vaf);
 
        va_end(args);
 }