mirror of
https://github.com/torvalds/linux
synced 2024-09-06 18:01:05 +00:00
samples/bpf: fix bio latency check with tracepoint
Recently, a new tracepoint for the block layer, specifically the block_io_start/done tracepoints, was introduced in commit5a80bd075f
("block: introduce block_io_start/block_io_done tracepoints"). Previously, the kprobe entry used for this purpose was quite unstable and inherently broke relevant probes [1]. Now that a stable tracepoint is available, this commit replaces the bio latency check with it. One of the changes made during this replacement is the key used for the hash table. Since 'struct request' cannot be used as a hash key, the approach taken follows that which was implemented in bcc/biolatency [2]. (uses dev:sector for the key) [1]: https://github.com/iovisor/bcc/issues/4261 [2]: https://github.com/iovisor/bcc/pull/4691 Fixes:450b7879e3
("block: move blk_account_io_{start,done} to blk-mq.c") Signed-off-by: Daniel T. Lee <danieltimlee@gmail.com> Link: https://lore.kernel.org/r/20230818090119.477441-7-danieltimlee@gmail.com Signed-off-by: Alexei Starovoitov <ast@kernel.org>
This commit is contained in:
parent
11430421b4
commit
92632115fb
|
@ -9,6 +9,12 @@
|
||||||
#include <bpf/bpf_helpers.h>
|
#include <bpf/bpf_helpers.h>
|
||||||
#include <bpf/bpf_tracing.h>
|
#include <bpf/bpf_tracing.h>
|
||||||
|
|
||||||
|
struct start_key {
|
||||||
|
dev_t dev;
|
||||||
|
u32 _pad;
|
||||||
|
sector_t sector;
|
||||||
|
};
|
||||||
|
|
||||||
struct {
|
struct {
|
||||||
__uint(type, BPF_MAP_TYPE_HASH);
|
__uint(type, BPF_MAP_TYPE_HASH);
|
||||||
__type(key, long);
|
__type(key, long);
|
||||||
|
@ -16,16 +22,17 @@ struct {
|
||||||
__uint(max_entries, 4096);
|
__uint(max_entries, 4096);
|
||||||
} my_map SEC(".maps");
|
} my_map SEC(".maps");
|
||||||
|
|
||||||
/* kprobe is NOT a stable ABI. If kernel internals change this bpf+kprobe
|
/* from /sys/kernel/tracing/events/block/block_io_start/format */
|
||||||
* example will no longer be meaningful
|
SEC("tracepoint/block/block_io_start")
|
||||||
*/
|
int bpf_prog1(struct trace_event_raw_block_rq *ctx)
|
||||||
SEC("kprobe/blk_mq_start_request")
|
|
||||||
int bpf_prog1(struct pt_regs *ctx)
|
|
||||||
{
|
{
|
||||||
long rq = PT_REGS_PARM1(ctx);
|
|
||||||
u64 val = bpf_ktime_get_ns();
|
u64 val = bpf_ktime_get_ns();
|
||||||
|
struct start_key key = {
|
||||||
|
.dev = ctx->dev,
|
||||||
|
.sector = ctx->sector
|
||||||
|
};
|
||||||
|
|
||||||
bpf_map_update_elem(&my_map, &rq, &val, BPF_ANY);
|
bpf_map_update_elem(&my_map, &key, &val, BPF_ANY);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -47,21 +54,26 @@ struct {
|
||||||
__uint(max_entries, SLOTS);
|
__uint(max_entries, SLOTS);
|
||||||
} lat_map SEC(".maps");
|
} lat_map SEC(".maps");
|
||||||
|
|
||||||
SEC("kprobe/__blk_account_io_done")
|
/* from /sys/kernel/tracing/events/block/block_io_done/format */
|
||||||
int bpf_prog2(struct pt_regs *ctx)
|
SEC("tracepoint/block/block_io_done")
|
||||||
|
int bpf_prog2(struct trace_event_raw_block_rq *ctx)
|
||||||
{
|
{
|
||||||
long rq = PT_REGS_PARM1(ctx);
|
struct start_key key = {
|
||||||
|
.dev = ctx->dev,
|
||||||
|
.sector = ctx->sector
|
||||||
|
};
|
||||||
|
|
||||||
u64 *value, l, base;
|
u64 *value, l, base;
|
||||||
u32 index;
|
u32 index;
|
||||||
|
|
||||||
value = bpf_map_lookup_elem(&my_map, &rq);
|
value = bpf_map_lookup_elem(&my_map, &key);
|
||||||
if (!value)
|
if (!value)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
u64 cur_time = bpf_ktime_get_ns();
|
u64 cur_time = bpf_ktime_get_ns();
|
||||||
u64 delta = cur_time - *value;
|
u64 delta = cur_time - *value;
|
||||||
|
|
||||||
bpf_map_delete_elem(&my_map, &rq);
|
bpf_map_delete_elem(&my_map, &key);
|
||||||
|
|
||||||
/* the lines below are computing index = log10(delta)*10
|
/* the lines below are computing index = log10(delta)*10
|
||||||
* using integer arithmetic
|
* using integer arithmetic
|
||||||
|
|
Loading…
Reference in a new issue