lkml.org 
[lkml]   [2003]   [Jan]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [PATCH] Deprecated exec_usermodehelper, enhance call_usermodehelper
Date
In message <Pine.LNX.4.44.0301051952450.3087-100000@home.transmeta.com> you wri
te:
>
> On Mon, 6 Jan 2003, Rusty Russell wrote:
> >
> > Linus, please apply.
>
> Nope, I really don't want to deprecate any more interfaces while my build
> is still so noisy about the _existing_ deprecated stuff.

OK. I'll work with the various authors to actually remove all 3 users
in the tree, then submit a patch to rip it out.

> The noisiness of the current build is quite distracting, and likely makes
> people just ignore potentially valid warnings simply because there are too
> many of them-

Damn. I guess you don't want this patch then?

Rusty.
--
Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

Name: Deprecate cli/sti/restore_flags etc.
Author: Rusty Russell
Status: Tested on 2.5.54

D: These functions have long been deprecated: they don't exist on SMP.
D: Mark them deprecated.

diff -urNp --exclude TAGS -X /home/rusty/current-dontdiff --minimal linux-2.5-bk/include/linux/interrupt.h working-2.5-bk-clisti/include/linux/interrupt.h
--- linux-2.5-bk/include/linux/interrupt.h Thu Jan 2 12:36:08 2003
+++ working-2.5-bk-clisti/include/linux/interrupt.h Mon Jan 6 14:41:25 2003
@@ -5,6 +5,7 @@
#include <linux/config.h>
#include <linux/linkage.h>
#include <linux/bitops.h>
+#include <linux/compiler.h>
#include <asm/atomic.h>
#include <asm/hardirq.h>
#include <asm/ptrace.h>
@@ -28,12 +29,29 @@ extern void free_irq(unsigned int, void
/*
* Temporary defines for UP kernels, until all code gets fixed.
*/
-#if !CONFIG_SMP
-# define cli() local_irq_disable()
-# define sti() local_irq_enable()
-# define save_flags(x) local_save_flags(x)
-# define restore_flags(x) local_irq_restore(x)
-# define save_and_cli(x) local_irq_save(x)
+#ifndef CONFIG_SMP
+static inline void __deprecated cli(void)
+{
+ local_irq_disable();
+}
+static inline void __deprecated sti(void)
+{
+ local_irq_enable();
+}
+static inline void __deprecated deprecated_save_flags(unsigned long *flags)
+{
+ local_save_flags(*flags);
+}
+static inline void __deprecated restore_flags(unsigned long flags)
+{
+ local_irq_restore(flags);
+}
+static inline void __deprecated deprecated_save_and_cli(unsigned long *flags)
+{
+ local_irq_save(*flags);
+}
+# define save_flags(x) deprecated_save_flags(&(x))
+# define save_and_cli(x) deprecated_save_and_cli(&(x))
#endif


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 13:32    [W:1.747 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site