lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.0903041052340.7995@gandalf.stny.rr.com>
Date:	Wed, 4 Mar 2009 10:53:09 -0500 (EST)
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Peter Zijlstra <peterz@...radead.org>
cc:	Arnaldo Carvalho de Melo <acme@...stprotocols.net>,
	"jens.axboe" <jens.axboe@...cle.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Ingo Molnar <mingo@...e.hu>,
	lkml <linux-kernel@...r.kernel.org>
Subject: Re: [RFC][PATCH] tracing: block events



On Wed, 4 Mar 2009, Peter Zijlstra wrote:

> Subject: tracing: block events
> From: Peter Zijlstra <a.p.zijlstra@...llo.nl>
> Date: Wed Mar 04 15:09:43 CET 2009
> 
> Signed-off-by: Peter Zijlstra <a.p.zijlstra@...llo.nl>

Peter, Thanks!

I'll try it out. I know this is an RFC patch, but it could really due
with a change log ;-)

-- Steve

> ---
>  include/trace/block.h             |   69 --------------
>  include/trace/block_event_types.h |  177 ++++++++++++++++++++++++++++++++++++++
>  include/trace/trace_event_types.h |    1 
>  include/trace/trace_events.h      |    1 
>  4 files changed, 180 insertions(+), 68 deletions(-)
> 
> Index: linux-2.6/include/trace/block.h
> ===================================================================
> --- linux-2.6.orig/include/trace/block.h
> +++ linux-2.6/include/trace/block.h
> @@ -4,73 +4,6 @@
>  #include <linux/blkdev.h>
>  #include <linux/tracepoint.h>
>  
> -DECLARE_TRACE(block_rq_abort,
> -	TPPROTO(struct request_queue *q, struct request *rq),
> -		TPARGS(q, rq));
> -
> -DECLARE_TRACE(block_rq_insert,
> -	TPPROTO(struct request_queue *q, struct request *rq),
> -		TPARGS(q, rq));
> -
> -DECLARE_TRACE(block_rq_issue,
> -	TPPROTO(struct request_queue *q, struct request *rq),
> -		TPARGS(q, rq));
> -
> -DECLARE_TRACE(block_rq_requeue,
> -	TPPROTO(struct request_queue *q, struct request *rq),
> -		TPARGS(q, rq));
> -
> -DECLARE_TRACE(block_rq_complete,
> -	TPPROTO(struct request_queue *q, struct request *rq),
> -		TPARGS(q, rq));
> -
> -DECLARE_TRACE(block_bio_bounce,
> -	TPPROTO(struct request_queue *q, struct bio *bio),
> -		TPARGS(q, bio));
> -
> -DECLARE_TRACE(block_bio_complete,
> -	TPPROTO(struct request_queue *q, struct bio *bio),
> -		TPARGS(q, bio));
> -
> -DECLARE_TRACE(block_bio_backmerge,
> -	TPPROTO(struct request_queue *q, struct bio *bio),
> -		TPARGS(q, bio));
> -
> -DECLARE_TRACE(block_bio_frontmerge,
> -	TPPROTO(struct request_queue *q, struct bio *bio),
> -		TPARGS(q, bio));
> -
> -DECLARE_TRACE(block_bio_queue,
> -	TPPROTO(struct request_queue *q, struct bio *bio),
> -		TPARGS(q, bio));
> -
> -DECLARE_TRACE(block_getrq,
> -	TPPROTO(struct request_queue *q, struct bio *bio, int rw),
> -		TPARGS(q, bio, rw));
> -
> -DECLARE_TRACE(block_sleeprq,
> -	TPPROTO(struct request_queue *q, struct bio *bio, int rw),
> -		TPARGS(q, bio, rw));
> -
> -DECLARE_TRACE(block_plug,
> -	TPPROTO(struct request_queue *q),
> -		TPARGS(q));
> -
> -DECLARE_TRACE(block_unplug_timer,
> -	TPPROTO(struct request_queue *q),
> -		TPARGS(q));
> -
> -DECLARE_TRACE(block_unplug_io,
> -	TPPROTO(struct request_queue *q),
> -		TPARGS(q));
> -
> -DECLARE_TRACE(block_split,
> -	TPPROTO(struct request_queue *q, struct bio *bio, unsigned int pdu),
> -		TPARGS(q, bio, pdu));
> -
> -DECLARE_TRACE(block_remap,
> -	TPPROTO(struct request_queue *q, struct bio *bio, dev_t dev,
> -		sector_t from, sector_t to),
> -		TPARGS(q, bio, dev, from, to));
> +#include <trace/block_event_types.h>
>  
>  #endif
> Index: linux-2.6/include/trace/block_event_types.h
> ===================================================================
> --- /dev/null
> +++ linux-2.6/include/trace/block_event_types.h
> @@ -0,0 +1,177 @@
> +
> +#ifndef TRACE_EVENT_FORMAT
> +# error Do not include this file directly.
> +# error Unless you know what you are doing.
> +#endif
> +
> +#undef TRACE_SYSTEM
> +#define TRACE_SYSTEM block
> +
> +#ifdef CONFIG_BLK_DEV_IO_TRACE
> +#define Q_DEV(q) (q->bt ? q->bt->dev : 0)
> +#else
> +#define Q_DEV(q) 0
> +#endif
> +
> +TRACE_FORMAT(block_rq_abort,
> +	TPPROTO(struct request_queue *q, struct request *rq),
> +	TPARGS(q, rq),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		rq->cmd_flags & __REQ_RW ? 'W' : 'R',
> +		blk_pc_request(rq) ? 0 : rq->hard_sector,
> +		blk_pc_request(rq) ? rq->data_len : rq->hard_nr_sectors << 9)
> +	);
> +
> +TRACE_FORMAT(block_rq_insert,
> +	TPPROTO(struct request_queue *q, struct request *rq),
> +	TPARGS(q, rq),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		rq->cmd_flags & __REQ_RW ? 'W' : 'R',
> +		blk_pc_request(rq) ? 0 : rq->hard_sector,
> +		blk_pc_request(rq) ? rq->data_len : rq->hard_nr_sectors << 9)
> +	);
> +
> +TRACE_FORMAT(block_rq_issue,
> +	TPPROTO(struct request_queue *q, struct request *rq),
> +	TPARGS(q, rq),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		rq->cmd_flags & __REQ_RW ? 'W' : 'R',
> +		blk_pc_request(rq) ? 0 : rq->hard_sector,
> +		blk_pc_request(rq) ? rq->data_len : rq->hard_nr_sectors << 9)
> +	);
> +
> +TRACE_FORMAT(block_rq_requeue,
> +	TPPROTO(struct request_queue *q, struct request *rq),
> +	TPARGS(q, rq),
> +	TPFMT("%d:%d %c %llu + %u [%d]",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		rq->cmd_flags & __REQ_RW ? 'W' : 'R',
> +		blk_pc_request(rq) ? 0 : rq->hard_sector,
> +		blk_pc_request(rq) ? rq->data_len : rq->hard_nr_sectors << 9,
> +		rq->errors)
> +	);
> +
> +TRACE_FORMAT(block_rq_complete,
> +	TPPROTO(struct request_queue *q, struct request *rq),
> +	TPARGS(q, rq),
> +	TPFMT("%d:%d %c %llu + %u [%d]",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		rq->cmd_flags & __REQ_RW ? 'W' : 'R',
> +		blk_pc_request(rq) ? 0 : rq->hard_sector,
> +		blk_pc_request(rq) ? rq->data_len : rq->hard_nr_sectors << 9,
> +		rq->errors)
> +	);
> +
> +TRACE_FORMAT(block_bio_bounce,
> +	TPPROTO(struct request_queue *q, struct bio *bio),
> +	TPARGS(q, bio),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		bio->bi_sector, bio->bi_size)
> +	);
> +
> +TRACE_FORMAT(block_bio_complete,
> +	TPPROTO(struct request_queue *q, struct bio *bio),
> +	TPARGS(q, bio),
> +	TPFMT("%d:%d %c %llu + %u [%d]",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		bio->bi_sector, bio->bi_size,
> +		!bio_flagged(bio, BIO_UPTODATE))
> +	);
> +
> +TRACE_FORMAT(block_bio_backmerge,
> +	TPPROTO(struct request_queue *q, struct bio *bio),
> +	TPARGS(q, bio),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		bio->bi_sector, bio->bi_size)
> +	);
> +
> +TRACE_FORMAT(block_bio_frontmerge,
> +	TPPROTO(struct request_queue *q, struct bio *bio),
> +	TPARGS(q, bio),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		bio->bi_sector, bio->bi_size)
> +	);
> +
> +TRACE_FORMAT(block_bio_queue,
> +	TPPROTO(struct request_queue *q, struct bio *bio),
> +	TPARGS(q, bio),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		bio->bi_sector, bio->bi_size)
> +	);
> +
> +TRACE_FORMAT(block_getrq,
> +	TPPROTO(struct request_queue *q, struct bio *bio, int rw),
> +	TPARGS(q, bio, rw),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		(bio ? (bio->bi_rw & WRITE) : rw) ? 'W' : 'R',
> +		bio ? bio->bi_sector : 0,
> +		bio ? bio->bi_size : 0)
> +	);
> +
> +TRACE_FORMAT(block_sleeprq,
> +	TPPROTO(struct request_queue *q, struct bio *bio, int rw),
> +	TPARGS(q, bio, rw),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		(bio ? (bio->bi_rw & WRITE) : rw) ? 'W' : 'R',
> +		bio ? bio->bi_sector : 0,
> +		bio ? bio->bi_size : 0)
> +	);
> +
> +TRACE_FORMAT(block_plug,
> +	TPPROTO(struct request_queue *q),
> +	TPARGS(q),
> +	TPFMT("%d:%d",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)))
> +	);
> +
> +TRACE_FORMAT(block_unplug_timer,
> +	TPPROTO(struct request_queue *q),
> +	TPARGS(q),
> +	TPFMT("%d:%d",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)))
> +	);
> +
> +TRACE_FORMAT(block_unplug_io,
> +	TPPROTO(struct request_queue *q),
> +	TPARGS(q),
> +	TPFMT("%d:%d",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)))
> +	);
> +
> +TRACE_FORMAT(block_split,
> +	TPPROTO(struct request_queue *q, struct bio *bio, unsigned int pdu),
> +	TPARGS(q, bio, pdu),
> +	TPFMT("%d:%d %c %llu + %u",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		bio->bi_sector, bio->bi_size)
> +	); /* FIXME add pdu info */
> +
> +TRACE_FORMAT(block_remap,
> +	TPPROTO(struct request_queue *q, struct bio *bio, dev_t dev,
> +		sector_t from, sector_t to),
> +	TPARGS(q, bio, dev, from, to),
> +	TPFMT("%d:%d %c %llu + %u -> (%d:%d) %llu",
> +		MAJOR(Q_DEV(q)), MINOR(Q_DEV(q)),
> +		bio->bi_rw & WRITE ? 'W' : 'R',
> +		from, bio->bi_size,
> +		MAJOR(dev), MINOR(dev), to)
> +	);
> +
> +#undef Q_DEV
> +
> +#undef TRACE_SYSTEM
> Index: linux-2.6/include/trace/trace_event_types.h
> ===================================================================
> --- linux-2.6.orig/include/trace/trace_event_types.h
> +++ linux-2.6/include/trace/trace_event_types.h
> @@ -3,3 +3,4 @@
>  #include <trace/sched_event_types.h>
>  #include <trace/irq_event_types.h>
>  #include <trace/lockdep_event_types.h>
> +#include <trace/block_event_types.h>
> Index: linux-2.6/include/trace/trace_events.h
> ===================================================================
> --- linux-2.6.orig/include/trace/trace_events.h
> +++ linux-2.6/include/trace/trace_events.h
> @@ -3,3 +3,4 @@
>  #include <trace/sched.h>
>  #include <trace/irq.h>
>  #include <trace/lockdep.h>
> +#include <trace/block.h>
> 
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ