All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/5] tracing/blktrace: move the tracing file to kernel/trace
@ 2009-02-07 19:46 Frederic Weisbecker
  2009-02-08 12:16 ` Arnaldo Carvalho de Melo
  0 siblings, 1 reply; 2+ messages in thread
From: Frederic Weisbecker @ 2009-02-07 19:46 UTC (permalink / raw)
  To: Steven Rostedt, Ingo Molnar; +Cc: Arnaldo Carvalho de Melo, LKML

Impact: cleanup

Move blktrace.c to kernel/trace, also move its config entry.

Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
---
 block/Kconfig                      |   24 ------------------------
 block/Makefile                     |    1 -
 kernel/trace/Kconfig               |   23 +++++++++++++++++++++++
 kernel/trace/Makefile              |    1 +
 {block => kernel/trace}/blktrace.c |    2 +-
 5 files changed, 25 insertions(+), 26 deletions(-)
 rename {block => kernel/trace}/blktrace.c (99%)

diff --git a/block/Kconfig b/block/Kconfig
index 7cdaa1d..e7d1278 100644
--- a/block/Kconfig
+++ b/block/Kconfig
@@ -44,30 +44,6 @@ config LBD
 
 	  If unsure, say N.
 
-config BLK_DEV_IO_TRACE
-	bool "Support for tracing block io actions"
-	depends on SYSFS
-	select RELAY
-	select DEBUG_FS
-	select TRACEPOINTS
-	select TRACING
-	select STACKTRACE
-	help
-	  Say Y here if you want to be able to trace the block layer actions
-	  on a given queue. Tracing allows you to see any traffic happening
-	  on a block device queue. For more information (and the userspace
-	  support tools needed), fetch the blktrace tools from:
-
-	  git://git.kernel.dk/blktrace.git
-
-	  Tracing also is possible using the ftrace interface, e.g.:
-
-	    echo 1 > /sys/block/sda/sda1/trace/enable
-	    echo blk > /sys/kernel/debug/tracing/current_tracer
-	    cat /sys/kernel/debug/tracing/trace_pipe
-
-	  If unsure, say N.
-
 config BLK_DEV_BSG
 	bool "Block layer SG support v4 (EXPERIMENTAL)"
 	depends on EXPERIMENTAL
diff --git a/block/Makefile b/block/Makefile
index bfe7304..e9fa4dd 100644
--- a/block/Makefile
+++ b/block/Makefile
@@ -13,6 +13,5 @@ obj-$(CONFIG_IOSCHED_AS)	+= as-iosched.o
 obj-$(CONFIG_IOSCHED_DEADLINE)	+= deadline-iosched.o
 obj-$(CONFIG_IOSCHED_CFQ)	+= cfq-iosched.o
 
-obj-$(CONFIG_BLK_DEV_IO_TRACE)	+= blktrace.o
 obj-$(CONFIG_BLOCK_COMPAT)	+= compat_ioctl.o
 obj-$(CONFIG_BLK_DEV_INTEGRITY)	+= blk-integrity.o
diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig
index 28f2644..0e920e2 100644
--- a/kernel/trace/Kconfig
+++ b/kernel/trace/Kconfig
@@ -294,6 +294,29 @@ config WORKQUEUE_TRACER
           For example it can help a developer to decide whether he should
           choose a per cpu workqueue instead of a singlethreaded one.
 
+config BLK_DEV_IO_TRACE
+	bool "Support for tracing block io actions"
+	depends on SYSFS
+	select RELAY
+	select DEBUG_FS
+	select TRACEPOINTS
+	select TRACING
+	select STACKTRACE
+	help
+	  Say Y here if you want to be able to trace the block layer actions
+	  on a given queue. Tracing allows you to see any traffic happening
+	  on a block device queue. For more information (and the userspace
+	  support tools needed), fetch the blktrace tools from:
+
+	  git://git.kernel.dk/blktrace.git
+
+	  Tracing also is possible using the ftrace interface, e.g.:
+
+	    echo 1 > /sys/block/sda/sda1/trace/enable
+	    echo blk > /sys/kernel/debug/tracing/current_tracer
+	    cat /sys/kernel/debug/tracing/trace_pipe
+
+	  If unsure, say N.
 
 config DYNAMIC_FTRACE
 	bool "enable/disable ftrace tracepoints dynamically"
diff --git a/kernel/trace/Makefile b/kernel/trace/Makefile
index f76d48f..627090b 100644
--- a/kernel/trace/Makefile
+++ b/kernel/trace/Makefile
@@ -37,5 +37,6 @@ obj-$(CONFIG_HW_BRANCH_TRACER) += trace_hw_branches.o
 obj-$(CONFIG_POWER_TRACER) += trace_power.o
 obj-$(CONFIG_KMEMTRACE) += kmemtrace.o
 obj-$(CONFIG_WORKQUEUE_TRACER) += trace_workqueue.o
+obj-$(CONFIG_BLK_DEV_IO_TRACE)	+= blktrace.o
 
 libftrace-y := ftrace.o
diff --git a/block/blktrace.c b/kernel/trace/blktrace.c
similarity index 99%
rename from block/blktrace.c
rename to kernel/trace/blktrace.c
index ca6d320..3b91da0 100644
--- a/block/blktrace.c
+++ b/kernel/trace/blktrace.c
@@ -25,7 +25,7 @@
 #include <linux/time.h>
 #include <trace/block.h>
 #include <linux/uaccess.h>
-#include <../kernel/trace/trace_output.h>
+#include "trace_output.h"
 
 static unsigned int blktrace_seq __read_mostly = 1;
 
-- 
1.6.1



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/5] tracing/blktrace: move the tracing file to kernel/trace
  2009-02-07 19:46 [PATCH 1/5] tracing/blktrace: move the tracing file to kernel/trace Frederic Weisbecker
@ 2009-02-08 12:16 ` Arnaldo Carvalho de Melo
  0 siblings, 0 replies; 2+ messages in thread
From: Arnaldo Carvalho de Melo @ 2009-02-08 12:16 UTC (permalink / raw)
  To: Jens Axboe
  Cc: Frederic Weisbecker, Steven Rostedt, Ingo Molnar,
	Arnaldo Carvalho de Melo, LKML

Jens,

	Are you ok with that? If you are, Frederic, please add my:

Acked-by: Arnaldo Carvalho de Melo <acme@redhat.com>

- Arnaldo

Em Sat, Feb 07, 2009 at 08:46:45PM +0100, Frederic Weisbecker escreveu:
> Impact: cleanup
> 
> Move blktrace.c to kernel/trace, also move its config entry.
> 
> Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
> Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
> ---
>  block/Kconfig                      |   24 ------------------------
>  block/Makefile                     |    1 -
>  kernel/trace/Kconfig               |   23 +++++++++++++++++++++++
>  kernel/trace/Makefile              |    1 +
>  {block => kernel/trace}/blktrace.c |    2 +-
>  5 files changed, 25 insertions(+), 26 deletions(-)
>  rename {block => kernel/trace}/blktrace.c (99%)
> 
> diff --git a/block/Kconfig b/block/Kconfig
> index 7cdaa1d..e7d1278 100644
> --- a/block/Kconfig
> +++ b/block/Kconfig
> @@ -44,30 +44,6 @@ config LBD
>  
>  	  If unsure, say N.
>  
> -config BLK_DEV_IO_TRACE
> -	bool "Support for tracing block io actions"
> -	depends on SYSFS
> -	select RELAY
> -	select DEBUG_FS
> -	select TRACEPOINTS
> -	select TRACING
> -	select STACKTRACE
> -	help
> -	  Say Y here if you want to be able to trace the block layer actions
> -	  on a given queue. Tracing allows you to see any traffic happening
> -	  on a block device queue. For more information (and the userspace
> -	  support tools needed), fetch the blktrace tools from:
> -
> -	  git://git.kernel.dk/blktrace.git
> -
> -	  Tracing also is possible using the ftrace interface, e.g.:
> -
> -	    echo 1 > /sys/block/sda/sda1/trace/enable
> -	    echo blk > /sys/kernel/debug/tracing/current_tracer
> -	    cat /sys/kernel/debug/tracing/trace_pipe
> -
> -	  If unsure, say N.
> -
>  config BLK_DEV_BSG
>  	bool "Block layer SG support v4 (EXPERIMENTAL)"
>  	depends on EXPERIMENTAL
> diff --git a/block/Makefile b/block/Makefile
> index bfe7304..e9fa4dd 100644
> --- a/block/Makefile
> +++ b/block/Makefile
> @@ -13,6 +13,5 @@ obj-$(CONFIG_IOSCHED_AS)	+= as-iosched.o
>  obj-$(CONFIG_IOSCHED_DEADLINE)	+= deadline-iosched.o
>  obj-$(CONFIG_IOSCHED_CFQ)	+= cfq-iosched.o
>  
> -obj-$(CONFIG_BLK_DEV_IO_TRACE)	+= blktrace.o
>  obj-$(CONFIG_BLOCK_COMPAT)	+= compat_ioctl.o
>  obj-$(CONFIG_BLK_DEV_INTEGRITY)	+= blk-integrity.o
> diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig
> index 28f2644..0e920e2 100644
> --- a/kernel/trace/Kconfig
> +++ b/kernel/trace/Kconfig
> @@ -294,6 +294,29 @@ config WORKQUEUE_TRACER
>            For example it can help a developer to decide whether he should
>            choose a per cpu workqueue instead of a singlethreaded one.
>  
> +config BLK_DEV_IO_TRACE
> +	bool "Support for tracing block io actions"
> +	depends on SYSFS
> +	select RELAY
> +	select DEBUG_FS
> +	select TRACEPOINTS
> +	select TRACING
> +	select STACKTRACE
> +	help
> +	  Say Y here if you want to be able to trace the block layer actions
> +	  on a given queue. Tracing allows you to see any traffic happening
> +	  on a block device queue. For more information (and the userspace
> +	  support tools needed), fetch the blktrace tools from:
> +
> +	  git://git.kernel.dk/blktrace.git
> +
> +	  Tracing also is possible using the ftrace interface, e.g.:
> +
> +	    echo 1 > /sys/block/sda/sda1/trace/enable
> +	    echo blk > /sys/kernel/debug/tracing/current_tracer
> +	    cat /sys/kernel/debug/tracing/trace_pipe
> +
> +	  If unsure, say N.
>  
>  config DYNAMIC_FTRACE
>  	bool "enable/disable ftrace tracepoints dynamically"
> diff --git a/kernel/trace/Makefile b/kernel/trace/Makefile
> index f76d48f..627090b 100644
> --- a/kernel/trace/Makefile
> +++ b/kernel/trace/Makefile
> @@ -37,5 +37,6 @@ obj-$(CONFIG_HW_BRANCH_TRACER) += trace_hw_branches.o
>  obj-$(CONFIG_POWER_TRACER) += trace_power.o
>  obj-$(CONFIG_KMEMTRACE) += kmemtrace.o
>  obj-$(CONFIG_WORKQUEUE_TRACER) += trace_workqueue.o
> +obj-$(CONFIG_BLK_DEV_IO_TRACE)	+= blktrace.o
>  
>  libftrace-y := ftrace.o
> diff --git a/block/blktrace.c b/kernel/trace/blktrace.c
> similarity index 99%
> rename from block/blktrace.c
> rename to kernel/trace/blktrace.c
> index ca6d320..3b91da0 100644
> --- a/block/blktrace.c
> +++ b/kernel/trace/blktrace.c
> @@ -25,7 +25,7 @@
>  #include <linux/time.h>
>  #include <trace/block.h>
>  #include <linux/uaccess.h>
> -#include <../kernel/trace/trace_output.h>
> +#include "trace_output.h"
>  
>  static unsigned int blktrace_seq __read_mostly = 1;
>  
> -- 
> 1.6.1
> 
> 
> --
> 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/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-02-08 12:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-02-07 19:46 [PATCH 1/5] tracing/blktrace: move the tracing file to kernel/trace Frederic Weisbecker
2009-02-08 12:16 ` Arnaldo Carvalho de Melo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.