From: Xiubo Li <xiubli@redhat.com>
To: Jeff Layton <jlayton@kernel.org>
Cc: pdonnell@redhat.com, ceph-devel@vger.kernel.org,
Ilya Dryomov <idryomov@gmail.com>
Subject: Re: [PATCH] ceph: send opened files/pinned caps/opened inodes metrics to MDS daemon
Date: Mon, 22 Mar 2021 20:32:42 +0800 [thread overview]
Message-ID: <ee667d67-ddc6-4ddd-46b9-600c77a059ac@redhat.com> (raw)
In-Reply-To: <5c1461d4f7c03f226ed2458f491885cfe9b44841.camel@kernel.org>
On 2021/3/22 19:55, Jeff Layton wrote:
> Oh! I hadn't realized that this patch was still unmerged.
No worry, this patch was waiting the MDS side patch, which was just
merged last week :-)
I almost forgot this patch too before I saw the ceph tracker today.
> Merged into testing branch. I'd have pulled this into testing a while
> ago if I had realized. Let me know if I miss any in the future (or am
> missing any now).
Sure, np.
BRs
Xiubo
> Cheers,
> Jeff
>
> On Mon, 2021-03-22 at 10:30 +0800, Xiubo Li wrote:
>> Hi Jeff,
>>
>> Ping.
>>
>> The MDS side patch[1] have been merged.
>>
>> [1] https://github.com/ceph/ceph/pull/37945
>>
>> Thanks
>>
>>
>> On 2020/11/26 11:47, xiubli@redhat.com wrote:
>>> From: Xiubo Li <xiubli@redhat.com>
>>>
>>> For the old ceph version, if it received this metric message containing
>>> the send opened files/pinned caps/opened inodes metric info, it will
>>> just ignore them.
>>>
>>> URL: https://tracker.ceph.com/issues/46866
>>> Signed-off-by: Xiubo Li <xiubli@redhat.com>
>>> ---
>>> fs/ceph/metric.c | 38 +++++++++++++++++++++++++++++++++++++-
>>> fs/ceph/metric.h | 44 +++++++++++++++++++++++++++++++++++++++++++-
>>> 2 files changed, 80 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/fs/ceph/metric.c b/fs/ceph/metric.c
>>> index 5ec94bd4c1de..306bd599d940 100644
>>> --- a/fs/ceph/metric.c
>>> +++ b/fs/ceph/metric.c
>>> @@ -17,6 +17,9 @@ static bool ceph_mdsc_send_metrics(struct ceph_mds_client *mdsc,
>>> struct ceph_metric_write_latency *write;
>>> struct ceph_metric_metadata_latency *meta;
>>> struct ceph_metric_dlease *dlease;
>>> + struct ceph_opened_files *files;
>>> + struct ceph_pinned_icaps *icaps;
>>> + struct ceph_opened_inodes *inodes;
>>> struct ceph_client_metric *m = &mdsc->metric;
>>> u64 nr_caps = atomic64_read(&m->total_caps);
>>> struct ceph_msg *msg;
>>> @@ -26,7 +29,8 @@ static bool ceph_mdsc_send_metrics(struct ceph_mds_client *mdsc,
>>> s32 len;
>>>
>>>
>>> len = sizeof(*head) + sizeof(*cap) + sizeof(*read) + sizeof(*write)
>>> - + sizeof(*meta) + sizeof(*dlease);
>>> + + sizeof(*meta) + sizeof(*dlease) + sizeof(files) + sizeof(icaps)
>>> + + sizeof(inodes);
>>>
>>>
>>> msg = ceph_msg_new(CEPH_MSG_CLIENT_METRICS, len, GFP_NOFS, true);
>>> if (!msg) {
>>> @@ -95,6 +99,38 @@ static bool ceph_mdsc_send_metrics(struct ceph_mds_client *mdsc,
>>> dlease->total = cpu_to_le64(atomic64_read(&m->total_dentries));
>>> items++;
>>>
>>>
>>> + sum = percpu_counter_sum(&m->total_inodes);
>>> +
>>> + /* encode the opened files metric */
>>> + files = (struct ceph_opened_files *)(dlease + 1);
>>> + files->type = cpu_to_le32(CLIENT_METRIC_TYPE_OPENED_FILES);
>>> + files->ver = 1;
>>> + files->compat = 1;
>>> + files->data_len = cpu_to_le32(sizeof(*files) - 10);
>>> + files->opened_files = cpu_to_le64(atomic64_read(&m->opened_files));
>>> + files->total = cpu_to_le64(sum);
>>> + items++;
>>> +
>>> + /* encode the pinned icaps metric */
>>> + icaps = (struct ceph_pinned_icaps *)(files + 1);
>>> + icaps->type = cpu_to_le32(CLIENT_METRIC_TYPE_PINNED_ICAPS);
>>> + icaps->ver = 1;
>>> + icaps->compat = 1;
>>> + icaps->data_len = cpu_to_le32(sizeof(*icaps) - 10);
>>> + icaps->pinned_icaps = cpu_to_le64(nr_caps);
>>> + icaps->total = cpu_to_le64(sum);
>>> + items++;
>>> +
>>> + /* encode the opened inodes metric */
>>> + inodes = (struct ceph_opened_inodes *)(icaps + 1);
>>> + inodes->type = cpu_to_le32(CLIENT_METRIC_TYPE_OPENED_INODES);
>>> + inodes->ver = 1;
>>> + inodes->compat = 1;
>>> + inodes->data_len = cpu_to_le32(sizeof(*inodes) - 10);
>>> + inodes->opened_inodes = cpu_to_le64(percpu_counter_sum(&m->opened_inodes));
>>> + inodes->total = cpu_to_le64(sum);
>>> + items++;
>>> +
>>> put_unaligned_le32(items, &head->num);
>>> msg->front.iov_len = len;
>>> msg->hdr.version = cpu_to_le16(1);
>>> diff --git a/fs/ceph/metric.h b/fs/ceph/metric.h
>>> index af6038ff39d4..4ceb462135d7 100644
>>> --- a/fs/ceph/metric.h
>>> +++ b/fs/ceph/metric.h
>>> @@ -14,8 +14,11 @@ enum ceph_metric_type {
>>> CLIENT_METRIC_TYPE_WRITE_LATENCY,
>>> CLIENT_METRIC_TYPE_METADATA_LATENCY,
>>> CLIENT_METRIC_TYPE_DENTRY_LEASE,
>>> + CLIENT_METRIC_TYPE_OPENED_FILES,
>>> + CLIENT_METRIC_TYPE_PINNED_ICAPS,
>>> + CLIENT_METRIC_TYPE_OPENED_INODES,
>>>
>>>
>>> - CLIENT_METRIC_TYPE_MAX = CLIENT_METRIC_TYPE_DENTRY_LEASE,
>>> + CLIENT_METRIC_TYPE_MAX = CLIENT_METRIC_TYPE_OPENED_INODES,
>>> };
>>>
>>>
>>> /*
>>> @@ -28,6 +31,9 @@ enum ceph_metric_type {
>>> CLIENT_METRIC_TYPE_WRITE_LATENCY, \
>>> CLIENT_METRIC_TYPE_METADATA_LATENCY, \
>>> CLIENT_METRIC_TYPE_DENTRY_LEASE, \
>>> + CLIENT_METRIC_TYPE_OPENED_FILES, \
>>> + CLIENT_METRIC_TYPE_PINNED_ICAPS, \
>>> + CLIENT_METRIC_TYPE_OPENED_INODES, \
>>> \
>>> CLIENT_METRIC_TYPE_MAX, \
>>> }
>>> @@ -94,6 +100,42 @@ struct ceph_metric_dlease {
>>> __le64 total;
>>> } __packed;
>>>
>>>
>>> +/* metric opened files header */
>>> +struct ceph_opened_files {
>>> + __le32 type; /* ceph metric type */
>>> +
>>> + __u8 ver;
>>> + __u8 compat;
>>> +
>>> + __le32 data_len; /* length of sizeof(opened_files + total) */
>>> + __le64 opened_files;
>>> + __le64 total;
>>> +} __packed;
>>> +
>>> +/* metric pinned i_caps header */
>>> +struct ceph_pinned_icaps {
>>> + __le32 type; /* ceph metric type */
>>> +
>>> + __u8 ver;
>>> + __u8 compat;
>>> +
>>> + __le32 data_len; /* length of sizeof(pinned_icaps + total) */
>>> + __le64 pinned_icaps;
>>> + __le64 total;
>>> +} __packed;
>>> +
>>> +/* metric opened inodes header */
>>> +struct ceph_opened_inodes {
>>> + __le32 type; /* ceph metric type */
>>> +
>>> + __u8 ver;
>>> + __u8 compat;
>>> +
>>> + __le32 data_len; /* length of sizeof(opened_inodes + total) */
>>> + __le64 opened_inodes;
>>> + __le64 total;
>>> +} __packed;
>>> +
>>> struct ceph_metric_head {
>>> __le32 num; /* the number of metrics that will be sent */
>>> } __packed;
>>
next prev parent reply other threads:[~2021-03-22 12:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-26 3:47 [PATCH] ceph: send opened files/pinned caps/opened inodes metrics to MDS daemon xiubli
2021-03-22 2:30 ` Xiubo Li
2021-03-22 11:55 ` Jeff Layton
2021-03-22 12:32 ` Xiubo Li [this message]
2021-03-23 19:51 ` Jeff Layton
2021-03-24 4:54 ` Xiubo Li
2021-03-23 19:52 ` Jeff Layton
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=ee667d67-ddc6-4ddd-46b9-600c77a059ac@redhat.com \
--to=xiubli@redhat.com \
--cc=ceph-devel@vger.kernel.org \
--cc=idryomov@gmail.com \
--cc=jlayton@kernel.org \
--cc=pdonnell@redhat.com \
/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 NNTP newsgroup(s).