linux-btrace.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andreas Gruenbacher <agruenba@redhat.com>
To: linux-btrace@vger.kernel.org
Subject: [PATCH 0/3] Split io tracing
Date: Mon, 06 Apr 2020 14:09:12 +0000	[thread overview]
Message-ID: <20200406140915.109022-1-agruenba@redhat.com> (raw)

Hi,

we recently ran into bio splitting problems in the kernel.  A typical
request would show up in the blktrace output as follows or worse:

  253,2    8       23     0.041305439  4907  Q   R 645838360 + 896 [dd]
  253,2    8       24     0.041307791  4907  X   R 645838360 / 645838872 [dd]
  253,2    8       25     0.041308789  4907  X   R 645838360 / 645838592 [dd]
  253,2    8       26     0.041313578  4907  X   R 645838872 / 645839104 [dd]
  253,2    8       27     0.041319035  4907  X   R 645838592 / 645838848 [dd]
  [...]
  253,2    8       68     0.051036527     0  C   R 645839104 + 152 ( 9731088) [0]

The kernel bug leading to this kind of result (a split into
232-256-24-232-152 instead of 232-256-256-152) has meanwhile been fixed,
but those kinds of traces remain difficult to read, for two reasons:
first, it's hard to match a request to the corresponding completion
because the sector and length of the completion don't match the request.
Second, it's difficult to figure out which pieces a request is getting
split into.

I don't have a good answer to the second problem, but how about at least
addressing the first problem by fixing up the sector and length of the
completion to match the request in blkparse when --track-ios is used, as
in the last of these three patches?  The last line above would turn
into:

  253,2    8       68     0.051036527     0  C   R 645838360 + 896 ( 9731088) [0]

Thanks,
Andreas

Andreas Gruenbacher (3):
  blkparse: Fix device in event tracking error messages
  blkparse: Allow request tracking on non md/dm devices
  blkparse: Fix up the sector and length of split completions

 blkparse.c | 49 +++++++++++++++++++++++++++++++++++++------------
 1 file changed, 37 insertions(+), 12 deletions(-)


base-commit: db4f6340e04716285ea56fe26d76381c3adabe58
-- 
2.25.1

                 reply	other threads:[~2020-04-06 14:09 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200406140915.109022-1-agruenba@redhat.com \
    --to=agruenba@redhat.com \
    --cc=linux-btrace@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).