lkml.org 
[lkml]   [2012]   [Mar]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 08/16] dynamic_debug: deprecate ddebug_query, suggest dyndbg instead
Date
From: Jim Cromie <jim.cromie@gmail.com>

Deprecate ddebug_query= in feature-removal-schedule.txt. Add a
warning when processing ddebug_query= param to change it to dyndbg=.

Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
---
Documentation/feature-removal-schedule.txt | 9 ++++++++-
lib/dynamic_debug.c | 2 ++
2 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index 0cad480..e926daf 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -2,7 +2,14 @@ The following is a list of files and features that are going to be
removed in the kernel source tree. Every entry should contain what
exactly is going away, why it is happening, and who is going to be doing
the work. When the feature is removed from the kernel, it should also
-be removed from this file.
+be removed from this file. The default deprecation period is 2-3 releases.
+
+---------------------------
+
+What: ddebug_query="query" cmdline param
+When: v3.6
+Why: obsoleted by dyndbg="query" and module.dyndbg="query"
+Who: Jim Cromie <jim.cromie@gmail.com>, Jason Baron <jbaron@redhat.com>

---------------------------

diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
index 32e7c52..a3ab3bd 100644
--- a/lib/dynamic_debug.c
+++ b/lib/dynamic_debug.c
@@ -993,6 +993,8 @@ static int __init dynamic_debug_init(void)

/* ddebug_query boot param got passed -> set it up */
if (ddebug_setup_string[0] != '\0') {
+ pr_warn("ddebug_query param name is deprecated,"
+ " change it to dyndbg\n");
ret = ddebug_exec_queries(ddebug_setup_string);
if (ret < 0)
pr_warn("Invalid ddebug boot param %s",
--
1.7.7.6


\
 
 \ /
  Last update: 2012-03-26 01:29    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site