* fix vbd rd_sects stat
@ 2021-07-09 8:09 Richard Kojedzinszky
2021-07-09 8:59 ` Andrew Cooper
0 siblings, 1 reply; 6+ messages in thread
From: Richard Kojedzinszky @ 2021-07-09 8:09 UTC (permalink / raw)
To: xen-devel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: 0001-xenstat_linux.c-fix-populating-vbd.rd_sect.patch --]
[-- Type: text/x-diff; name=0001-xenstat_linux.c-fix-populating-vbd.rd_sect.patch, Size: 1130 bytes --]
From d09d27e40cfbdd2dbf5aefaa40a4f46ccb268056 Mon Sep 17 00:00:00 2001
Message-Id: <d09d27e40cfbdd2dbf5aefaa40a4f46ccb268056.1625818025.git.richard@kojedz.in>
From: Richard Kojedzinszky <richard@kojedz.in>
Date: Fri, 9 Jul 2021 10:06:45 +0200
Subject: [PATCH] xenstat_linux.c: fix populating vbd.rd_sect
---
tools/libs/stat/xenstat_linux.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/libs/stat/xenstat_linux.c b/tools/libs/stat/xenstat_linux.c
index 875a0617ad..8939cf3bcd 100644
--- a/tools/libs/stat/xenstat_linux.c
+++ b/tools/libs/stat/xenstat_linux.c
@@ -475,7 +475,7 @@ int xenstat_collect_vbds(xenstat_node * node)
(read_attributes_vbd(dp->d_name, "statistics/wr_req", buf, 256)<=0) ||
((ret = sscanf(buf, "%llu", &vbd.wr_reqs)) != 1) ||
(read_attributes_vbd(dp->d_name, "statistics/rd_sect", buf, 256)<=0) ||
- ((ret = sscanf(buf, "%llu", &vbd.wr_sects)) != 1) ||
+ ((ret = sscanf(buf, "%llu", &vbd.rd_sect)) != 1) ||
(read_attributes_vbd(dp->d_name, "statistics/wr_sect", buf, 256)<=0) ||
((ret = sscanf(buf, "%llu", &vbd.wr_sects)) != 1))
{
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: fix vbd rd_sects stat
2021-07-09 8:09 fix vbd rd_sects stat Richard Kojedzinszky
@ 2021-07-09 8:59 ` Andrew Cooper
2021-07-09 9:32 ` Richard Kojedzinszky
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Cooper @ 2021-07-09 8:59 UTC (permalink / raw)
To: Richard Kojedzinszky, xen-devel
On 09/07/2021 09:09, Richard Kojedzinszky wrote:
Thankyou for the patch. That is quite an embarrassing bug.
However, for copyright reasons, we require all submissions to contain a
Signed-off-by line, as per the Developer's Certificate of Origin,
https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#Signed-off-by
Given how trivial this patch is, feel free to just reply to this email
with confirmation and I can sort it out on commit.
However, for future patches, please include directly. (Also, if you are
able to use git-send-email, and the get_maintainers.pl script, that
would be even better.)
~Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: fix vbd rd_sects stat
2021-07-09 8:59 ` Andrew Cooper
@ 2021-07-09 9:32 ` Richard Kojedzinszky
2021-07-09 11:30 ` Andrew Cooper
0 siblings, 1 reply; 6+ messages in thread
From: Richard Kojedzinszky @ 2021-07-09 9:32 UTC (permalink / raw)
To: Andrew Cooper, xen-devel
[-- Attachment #1: Type: text/plain, Size: 898 bytes --]
Then please add signed-by headers. :)
Thanks
2021. július 9. 10:59:54 CEST, Andrew Cooper <andrew.cooper3@citrix.com> írta:
>On 09/07/2021 09:09, Richard Kojedzinszky wrote:
>
>Thankyou for the patch. That is quite an embarrassing bug.
>
>However, for copyright reasons, we require all submissions to contain a
>Signed-off-by line, as per the Developer's Certificate of Origin,
>https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#Signed-off-by
>
>Given how trivial this patch is, feel free to just reply to this email
>with confirmation and I can sort it out on commit.
>
>However, for future patches, please include directly. (Also, if you
>are
>able to use git-send-email, and the get_maintainers.pl script, that
>would be even better.)
>
>~Andrew
--
Ezt a levelet az Android készülékemről küldtem K-9 Mail-el. Elnézést a tömörségemért.
[-- Attachment #2: Type: text/html, Size: 1274 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: fix vbd rd_sects stat
2021-07-09 9:32 ` Richard Kojedzinszky
@ 2021-07-09 11:30 ` Andrew Cooper
2021-07-09 11:37 ` Richard Kojedzinszky
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Cooper @ 2021-07-09 11:30 UTC (permalink / raw)
To: Richard Kojedzinszky, xen-devel
On 09/07/2021 10:32, Richard Kojedzinszky wrote:
> Then please add signed-by headers. :)
Ok. I have fixed up and committed the patch.
https://xenbits.xen.org/gitweb/?p=xen.git;a=commitdiff;h=89d57f291e37b4769ab26db919eba46548f2e13e
This will want queueing for backporting
~Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: fix vbd rd_sects stat
2021-07-09 11:30 ` Andrew Cooper
@ 2021-07-09 11:37 ` Richard Kojedzinszky
2021-07-09 12:18 ` xenstat-exporter: [Was Re: fix vbd rd_sects stat] Andrew Cooper
0 siblings, 1 reply; 6+ messages in thread
From: Richard Kojedzinszky @ 2021-07-09 11:37 UTC (permalink / raw)
To: Andrew Cooper; +Cc: xen-devel
Dear Andrew,
Many thanks.
Meanwhile I would like to promote our xenstat-exporter, which we use for
monitoring our vps instances.
https://github.com/euronetzrt/xenstat-exporter
We've spotted the bug during browsing the metrics.
Regards,
Richard
2021-07-09 13:30 időpontban Andrew Cooper ezt írta:
> On 09/07/2021 10:32, Richard Kojedzinszky wrote:
>> Then please add signed-by headers. :)
>
> Ok. I have fixed up and committed the patch.
>
> https://xenbits.xen.org/gitweb/?p=xen.git;a=commitdiff;h=89d57f291e37b4769ab26db919eba46548f2e13e
>
> This will want queueing for backporting
>
> ~Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
* xenstat-exporter: [Was Re: fix vbd rd_sects stat]
2021-07-09 11:37 ` Richard Kojedzinszky
@ 2021-07-09 12:18 ` Andrew Cooper
0 siblings, 0 replies; 6+ messages in thread
From: Andrew Cooper @ 2021-07-09 12:18 UTC (permalink / raw)
To: Richard Kojedzinszky; +Cc: xen-devel, community.manager
On 09/07/2021 12:37, Richard Kojedzinszky wrote:
> Dear Andrew,
>
> Many thanks.
>
> Meanwhile I would like to promote our xenstat-exporter, which we use
> for monitoring our vps instances.
>
> https://github.com/euronetzrt/xenstat-exporter
>
> We've spotted the bug during browsing the metrics.
Hello.
We're always interested to hear what people are doing in the Xen
ecosystem. CC-ing the Community Manager.
~Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-07-09 12:19 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-09 8:09 fix vbd rd_sects stat Richard Kojedzinszky
2021-07-09 8:59 ` Andrew Cooper
2021-07-09 9:32 ` Richard Kojedzinszky
2021-07-09 11:30 ` Andrew Cooper
2021-07-09 11:37 ` Richard Kojedzinszky
2021-07-09 12:18 ` xenstat-exporter: [Was Re: fix vbd rd_sects stat] Andrew Cooper
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).