From: Pavel Begunkov <[email protected]>
To: Xiaobing Li <[email protected]>, [email protected]
Cc: [email protected], [email protected],
[email protected], [email protected], [email protected],
[email protected], [email protected],
[email protected]
Subject: Re: [PATCH v7] io_uring: Statistics of the true utilization of sq threads.
Date: Mon, 29 Jan 2024 15:01:43 +0000 [thread overview]
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
On 1/29/24 07:18, Xiaobing Li wrote:
> On 1/18/24 19:34, Jens Axboe wrote:
>>> diff --git a/io_uring/sqpoll.h b/io_uring/sqpoll.h
>>> index 8df37e8c9149..c14c00240443 100644
>>> --- a/io_uring/sqpoll.h
>>> +++ b/io_uring/sqpoll.h
>>> @@ -16,6 +16,7 @@ struct io_sq_data {
>>> pid_t task_pid;
>>> pid_t task_tgid;
>>>
>>> + long long work_time;
>>> unsigned long state;
>>> struct completion exited;
>>> };
>>
>> Probably just make that an u64.
>>
>> As Pavel mentioned, I think we really need to consider if fdinfo is the
>> appropriate API for this. It's fine if you're running stuff directly and
>> you're just curious, but it's a very cumbersome API in general as you
>> need to know the pid of the task holding the ring, the fd of the ring,
>> and then you can get it as a textual description. If this is something
>> that is deemed useful, would it not make more sense to make it
>> programatically available in addition, or even exclusively?
>
> Hi, Jens and Pavel
> sorry for the late reply.
>
> I've tried some other methods, but overall, I haven't found a more suitable
> method than fdinfo.
I wouldn't mind if it's fdinfo only for now, that can be changed later
if needed. I'm more concerned that reading fdinfo and then parsing it
is incompatible with the word performance, which you mentioned in the
context of using 1 vs 2 syscalls to get the stats.
That can be left to be resolved later, however. Let's just be clear
in docs that stats could be 0, which means the feature is not
working/disabled.
Another question I raised in my reply (v6 thread), why it's using
ktime_get(), which same as jiffies but more precise, instead of a
task time?
> If you think it is troublesome to obtain the PID, then I can provide
I missed the context, where do we need to know PIDs?
> a shell script to output the total_time and work_time of all sqpoll threads
> to the terminal, so that we do not have to manually obtain the PID of each
> thread (the script can be placed in tools/ include/io_uring).
>
> eg:
>
> PID WorkTime(us) TotalTime(us) COMMAND
> 9330 1106578 2215321 iou-sqp-9329
> 9454 1510658 1715321 iou-sqp-9453
> 9478 165785 223219 iou-sqp-9477
> 9587 106578 153217 iou-sqp-9586
>
> What do you think of this solution?
--
Pavel Begunkov
next prev parent reply other threads:[~2024-01-29 15:09 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20240118073844epcas5p2f346bc8ef02a2f48eef0020a6ad5165d@epcas5p2.samsung.com>
2024-01-18 7:30 ` [PATCH v7] io_uring: Statistics of the true utilization of sq threads Xiaobing Li
2024-01-18 19:34 ` Jens Axboe
[not found] ` <CGME20240129072655epcas5p35d140dba2234e1658b7aa40770b93314@epcas5p3.samsung.com>
2024-01-29 7:18 ` Xiaobing Li
2024-01-29 14:00 ` Jens Axboe
[not found] ` <CGME20240130070518epcas5p41bdf16f1e864c2590e2f39dac703febb@epcas5p4.samsung.com>
2024-01-30 6:57 ` Xiaobing Li
2024-01-29 15:01 ` Pavel Begunkov [this message]
[not found] ` <CGME20240130055554epcas5p2ad9d28f1e6c256db540ce985c4351461@epcas5p2.samsung.com>
2024-01-30 5:47 ` Xiaobing Li
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 \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
/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