From nobody Fri Apr 19 17:16:17 2024 Delivered-To: importer2@patchew.org Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer2=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1652438663145863.8242592352188; Fri, 13 May 2022 03:44:23 -0700 (PDT) Received: from localhost ([::1]:37616 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1npSmT-0005qy-Ik for importer2@patchew.org; Fri, 13 May 2022 06:44:21 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33610) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1npSiO-0008LP-3f for qemu-devel@nongnu.org; Fri, 13 May 2022 06:40:08 -0400 Received: from [106.39.185.58] (port=49822 helo=smtp.cecloud.com) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1npSiK-00053d-GV for qemu-devel@nongnu.org; Fri, 13 May 2022 06:40:07 -0400 Received: from localhost (localhost [127.0.0.1]) by smtp.cecloud.com (Postfix) with ESMTP id 644B13000019C; Fri, 13 May 2022 18:39:49 +0800 (CST) Received: from localhost.localdomain (unknown [111.48.58.11]) by smtp.cecloud.com (postfix) whith ESMTP id P4032146T281472375845232S1652438388150339_; Fri, 13 May 2022 18:39:49 +0800 (CST) X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-ANTISPAM-LEVEL: 2 X-ABS-CHECKED: 0 X-IP-DOMAINF: 1 X-UNIQUE-TAG: <578e40f8d92d203647f92d0469089895> X-RL-SENDER: luzhipeng@cestc.cn X-SENDER: luzhipeng@cestc.cn X-LOGIN-NAME: luzhipeng@cestc.cn X-FST-TO: qemu-devel@nongnu.org X-RCPT-COUNT: 7 X-SENDER-IP: 111.48.58.11 X-ATTACHMENT-NUM: 0 X-System-Flag: 0 From: luzhipeng To: qemu-devel Cc: Michael Roth , Konstantin Kostiuk , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , =?UTF-8?q?Daniel=20P=20=2E=20Berrang=C3=A9?= , Michal Privoznik , luzhipeng Subject: [PATCH v2] qga: add guest-get-diskstats command for Linux guests Date: Fri, 13 May 2022 18:39:24 +0800 Message-Id: <20220513103924.1332-1-luzhipeng@cestc.cn> X-Mailer: git-send-email 2.34.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Host-Lookup-Failed: Reverse DNS lookup failed for 106.39.185.58 (failed) Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer2=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=106.39.185.58; envelope-from=luzhipeng@cestc.cn; helo=smtp.cecloud.com X-Spam_score_int: 22 X-Spam_score: 2.2 X-Spam_bar: ++ X-Spam_report: (2.2 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_SBL_CSS=3.335, RDNS_NONE=0.793, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer2=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZM-MESSAGEID: 1652438664757100001 Content-Type: text/plain; charset="utf-8" Add a new 'guest-get-diskstats' command for report disk io statistics for Linux guests. This can be usefull for getting io flow or handling IO fault, no need to enter guests. Signed-off-by: luzhipeng --- Changes v1->v2: v1:https://patchew.org/QEMU/20220512011930.214-1-luzhipeng@cestc.cn/ qga/commands-posix.c | 97 ++++++++++++++++++++++++++++++++++++++++++++ qga/commands-win32.c | 6 +++ qga/qapi-schema.json | 86 +++++++++++++++++++++++++++++++++++++++ 3 files changed, 189 insertions(+) diff --git a/qga/commands-posix.c b/qga/commands-posix.c index 69f209af87..7a16d84e3a 100644 --- a/qga/commands-posix.c +++ b/qga/commands-posix.c @@ -2783,6 +2783,96 @@ GuestMemoryBlockInfo *qmp_guest_get_memory_block_inf= o(Error **errp) return info; } =20 +#define MAX_NAME_LEN 128 +static GuestDiskStatsInfoList *guest_get_diskstats(Error **errp) +{ +#ifdef CONFIG_LINUX + GuestDiskStatsInfoList *head =3D NULL, **tail =3D &head; + const char *diskstats =3D "/proc/diskstats"; + FILE *fp; + size_t n; + char *line =3D NULL; + char dev_name[MAX_NAME_LEN]; + int i; + unsigned int ios_pgr, tot_ticks, rq_ticks, wr_ticks, dc_ticks, fl_tick= s; + unsigned long rd_ios, rd_merges_or_rd_sec, rd_ticks_or_wr_sec, wr_ios; + unsigned long wr_merges, rd_sec_or_wr_ios, wr_sec; + unsigned long dc_ios, dc_merges, dc_sec, fl_ios; + unsigned int major, minor; + + fp =3D fopen(diskstats, "r"); + if (fp =3D=3D NULL) { + error_setg_errno(errp, errno, "open(\"%s\")", diskstats); + return NULL; + } + while (getline(&line, &n, fp) !=3D -1) { + GuestDiskStatsInfo *diskstatinfo; + GuestDiskStats *diskstat; + i =3D sscanf(line, "%u %u %s %lu %lu %lu" + "%lu %lu %lu %lu %u %u %u %u" + "%lu %lu %lu %u %lu %u", + &major, &minor, dev_name, + &rd_ios, &rd_merges_or_rd_sec, &rd_sec_or_wr_ios, + &rd_ticks_or_wr_sec, &wr_ios, &wr_merges, &wr_sec, + &wr_ticks, &ios_pgr, &tot_ticks, &rq_ticks, + &dc_ios, &dc_merges, &dc_sec, &dc_ticks, + &fl_ios, &fl_ticks); + + diskstatinfo =3D g_new0(GuestDiskStatsInfo, 1); + diskstat =3D g_new0(GuestDiskStats, 1); + if (i < 7) { + continue; + } + diskstatinfo->name =3D g_strdup(dev_name); + diskstatinfo->major =3D major; + diskstatinfo->minor =3D minor; + if (i =3D=3D 7) { + diskstat->read_ios =3D rd_ios; + diskstat->read_sectors =3D rd_merges_or_rd_sec; + diskstat->write_ios =3D rd_sec_or_wr_ios; + diskstat->write_sectors =3D rd_ticks_or_wr_sec; + } + if (i >=3D 14) { + diskstat->read_ios =3D rd_ios; + diskstat->read_sectors =3D rd_sec_or_wr_ios; + diskstat->read_merges =3D rd_merges_or_rd_sec; + diskstat->read_ticks =3D rd_ticks_or_wr_sec; + diskstat->write_ios =3D wr_ios; + diskstat->write_sectors =3D wr_sec; + diskstat->write_merges =3D wr_merges; + diskstat->write_ticks =3D wr_ticks; + diskstat->ios_pgr =3D ios_pgr; + diskstat->total_ticks =3D tot_ticks; + diskstat->weight_ticks =3D rq_ticks; + } + if (i >=3D 18) { + diskstat->discard_ios =3D dc_ios; + diskstat->discard_merges =3D dc_merges; + diskstat->discard_sectors =3D dc_sec; + diskstat->discard_ticks =3D dc_ticks; + } + if (i >=3D 20) { + diskstat->flush_ios =3D fl_ios; + diskstat->flush_ticks =3D fl_ticks; + } + + diskstatinfo->stats =3D diskstat; + QAPI_LIST_APPEND(tail, diskstatinfo); + } + g_free(line); + fclose(fp); + return head; +#else + g_debug("disk stats reporting available only for Linux"); + return NULL; +#endif +} + +GuestDiskStatsInfoList *qmp_guest_get_diskstats(Error **errp) +{ + return guest_get_diskstats(errp); +} + #else /* defined(__linux__) */ =20 void qmp_guest_suspend_disk(Error **errp) @@ -3131,6 +3221,13 @@ GuestDiskInfoList *qmp_guest_get_disks(Error **errp) return NULL; } =20 +GuestDiskStatsInfoList *qmp_guest_get_diskstats(Error **errp) +{ + error_setg(errp, QERR_UNSUPPORTED); + return NULL; +} + + #endif /* CONFIG_FSFREEZE */ =20 #if !defined(CONFIG_FSTRIM) diff --git a/qga/commands-win32.c b/qga/commands-win32.c index d56b5fd2a7..dcdeb76a68 100644 --- a/qga/commands-win32.c +++ b/qga/commands-win32.c @@ -2532,3 +2532,9 @@ char *qga_get_host_name(Error **errp) =20 return g_utf16_to_utf8(tmp, size, NULL, NULL, NULL); } + +GuestDiskStatsInfoList *qmp_guest_get_diskstats(Error **errp) +{ + error_setg(errp, QERR_UNSUPPORTED); + return NULL; +} diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json index 4d8e506c9e..94aad4f2ae 100644 --- a/qga/qapi-schema.json +++ b/qga/qapi-schema.json @@ -1490,3 +1490,89 @@ { 'command': 'guest-ssh-remove-authorized-keys', 'data': { 'username': 'str', 'keys': ['str'] }, 'if': 'CONFIG_POSIX' } + +## +# @GuestDiskStats: +# +# @read-sectors: sectors read +# +# @write-sectors: sectors written +# +# @discard-sectors: sectors discarded +# +# @read-ios: reads completed successfully +# +# @read-merges: Number of read requests merged +# +# @write-ios: writes completed +# +# @write-merges: Number of write requests merged +# +# @discard-ios: Number of discards completed successfully +# +# @discard-merges: NUmber of discard requests merged +# +# @flush-ios: Number of flush requests completed successfully +# +# @read-ticks: time spent reading(ms) +# +# @write-ticks: time spent writing(ms) +# +# @discard-ticks: time spent discarding(ms) +# +# @flush-ticks: time spent flushing(ms) +# +# @ios-pgr: Number of I/Os currently in flight +# +# @total-ticks: time spent doing I/Os (ms) +# +# @weight-ticks: weighted time spent doing I/Os since the last update of t= his field(ms) +# +# Since: 7.1 +## +{ 'struct': 'GuestDiskStats', + 'data': {'read-sectors': 'uint64', + 'write-sectors': 'uint64', + 'discard-sectors': 'uint64', + 'read-ios': 'uint64', + 'read-merges': 'uint64', + 'write-ios': 'uint64', + 'write-merges': 'uint64', + 'discard-ios': 'uint64', + 'discard-merges': 'uint64', + 'flush-ios': 'uint64', + 'read-ticks': 'uint64', + 'write-ticks': 'uint64', + 'discard-ticks': 'uint64', + 'flush-ticks': 'uint64', + 'ios-pgr': 'uint64', + 'total-ticks': 'uint64', + 'weight-ticks': 'uint64' + } } + +## +# @GuestDiskStatsInfo: +# +# @name disk name +# +# @major major of disk +# +# @minor minor of disk +## +{ 'struct': 'GuestDiskStatsInfo', + 'data': {'name': 'str', + 'major': 'uint64', + 'minor': 'uint64', + 'stats': 'GuestDiskStats' } } + +## +# @guest-get-diskstats: +# +# Retrieve information about disk stats. +# Returns: List of disk stats of guest. +# +# Since: 7.1 +## +{ 'command': 'guest-get-diskstats', + 'returns': ['GuestDiskStatsInfo'] +} --=20 2.31.1