From: Simon Horman <horms@kernel•org>
To: Yi-De Wu <yi-de.wu@mediatek•com>
Cc: Yingshiuan Pan <yingshiuan.pan@mediatek•com>,
Ze-Yu Wang <ze-yu.wang@mediatek•com>,
Rob Herring <robh@kernel•org>,
Krzysztof Kozlowski <krzk+dt@kernel•org>,
Conor Dooley <conor+dt@kernel•org>,
Jonathan Corbet <corbet@lwn•net>,
Catalin Marinas <catalin.marinas@arm•com>,
Will Deacon <will@kernel•org>,
Richard Cochran <richardcochran@gmail•com>,
Matthias Brugger <matthias.bgg@gmail•com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora•com>,
devicetree@vger•kernel.org, linux-kernel@vger•kernel.org,
linux-doc@vger•kernel.org, linux-arm-kernel@lists•infradead.org,
netdev@vger•kernel.org, linux-mediatek@lists•infradead.org,
David Bradil <dbrazdil@google•com>,
Trilok Soni <quic_tsoni@quicinc•com>,
My Chuang <my.chuang@mediatek•com>,
Shawn Hsiao <shawn.hsiao@mediatek•com>,
PeiLun Suei <peilun.suei@mediatek•com>,
Liju Chen <liju-clr.chen@mediatek•com>,
Willix Yeh <chi-shen.yeh@mediatek•com>,
Kevenny Hsieh <kevenny.hsieh@mediatek•com>
Subject: Re: [PATCH v10 19/21] virt: geniezone: Provide individual VM memory statistics within debugfs
Date: Mon, 15 Apr 2024 17:32:12 +0100 [thread overview]
Message-ID: <20240415163212.GE2320920@kernel.org> (raw)
In-Reply-To: <20240412065718.29105-20-yi-de.wu@mediatek.com>
On Fri, Apr 12, 2024 at 02:57:16PM +0800, Yi-De Wu wrote:
> From: "Jerry Wang" <ze-yu.wang@mediatek•com>
>
> Created a dedicated per-VM debugfs folder under gzvm, providing
> user-level programs with easy access to per-VM memory statistics for
> debugging and profiling purposes. This enables users to effectively
> analyze and optimize the memory usage of individual virtual machines.
>
> Two types of information can be obtained:
>
> `cat /sys/kernel/debug/gzvm/<pid>-<vmid>/protected_hyp_mem` shows memory
> used by the hypervisor and the size of the stage 2 table in bytes.
>
> `cat /sys/kernel/debug/gzvm/<pid>-<vmid>/protected_shared_mem` gives
> memory used by the shared resources of the guest and host in bytes.
>
> For example:
> console:/ # cat /sys/kernel/debug/gzvm/3417-15/protected_hyp_mem
> 180328
> console:/ # cat /sys/kernel/debug/gzvm/3417-15/protected_shared_mem
> 262144
> console:/ #
>
> More stats will be added in the future.
>
> Signed-off-by: Jerry Wang <ze-yu.wang@mediatek•com>
> Signed-off-by: Liju-Clr Chen <liju-clr.chen@mediatek•com>
> Signed-off-by: Yi-De Wu <yi-de.wu@mediatek•com>
...
> diff --git a/drivers/virt/geniezone/gzvm_vm.c b/drivers/virt/geniezone/gzvm_vm.c
...
> @@ -398,6 +409,113 @@ static void setup_vm_demand_paging(struct gzvm *vm)
> }
> }
>
> +static int debugfs_open(struct inode *inode, struct file *file)
> +{
> + file->private_data = inode->i_private;
> + return 0;
> +}
nit: Coccinelle suggests that simple_open() can be used in place
of the debugfs_open() implementation above.
...
> +static const struct file_operations hyp_mem_fops = {
> + .owner = THIS_MODULE,
> + .open = debugfs_open,
> + .read = hyp_mem_read,
> + .llseek = no_llseek,
> +};
> +
> +static const struct file_operations shared_mem_fops = {
> + .owner = THIS_MODULE,
> + .open = debugfs_open,
> + .read = shared_mem_read,
> + .llseek = no_llseek,
> +};
...
next prev parent reply other threads:[~2024-04-15 16:32 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-12 6:56 [PATCH v10 00/21] GenieZone hypervisor drivers Yi-De Wu
2024-04-12 6:56 ` [PATCH v10 01/21] virt: geniezone: enable gzvm-ko in defconfig Yi-De Wu
2024-04-12 6:56 ` [PATCH v10 02/21] docs: geniezone: Introduce GenieZone hypervisor Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 03/21] dt-bindings: hypervisor: Add MediaTek " Yi-De Wu
2024-04-15 17:03 ` Conor Dooley
2024-04-12 6:57 ` [PATCH v10 04/21] virt: geniezone: Add GenieZone hypervisor driver Yi-De Wu
2024-04-15 14:28 ` Simon Horman
2024-04-12 6:57 ` [PATCH v10 05/21] virt: geniezone: Add vm support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 06/21] virt: geniezone: Add set_user_memory_region for vm Yi-De Wu
2024-04-15 14:48 ` Simon Horman
2024-04-12 6:57 ` [PATCH v10 07/21] virt: geniezone: Add vm capability check Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 08/21] virt: geniezone: Optimize performance of protected VM memory Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 09/21] virt: geniezone: Add vcpu support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 10/21] virt: geniezone: Add irqchip support for virtual interrupt injection Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 11/21] virt: geniezone: Add irqfd support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 12/21] virt: geniezone: Add ioeventfd support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 13/21] virt: geniezone: Add memory region support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 14/21] virt: geniezone: Add dtb config support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 15/21] virt: geniezone: Add demand paging support Yi-De Wu
2024-04-15 14:52 ` Simon Horman
2024-04-12 6:57 ` [PATCH v10 16/21] virt: geniezone: Add block-based " Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 17/21] virt: geniezone: Add memory pin/unpin support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 18/21] virt: geniezone: Add memory relinquish support Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 19/21] virt: geniezone: Provide individual VM memory statistics within debugfs Yi-De Wu
2024-04-15 16:32 ` Simon Horman [this message]
2024-04-12 6:57 ` [PATCH v10 20/21] virt: geniezone: Add tracing support for hyp call and vcpu exit_reason Yi-De Wu
2024-04-12 6:57 ` [PATCH v10 21/21] virt: geniezone: Enable PTP for synchronizing time between host and guest VMs Yi-De Wu
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=20240415163212.GE2320920@kernel.org \
--to=horms@kernel$(echo .)org \
--cc=angelogioacchino.delregno@collabora$(echo .)com \
--cc=catalin.marinas@arm$(echo .)com \
--cc=chi-shen.yeh@mediatek$(echo .)com \
--cc=conor+dt@kernel$(echo .)org \
--cc=corbet@lwn$(echo .)net \
--cc=dbrazdil@google$(echo .)com \
--cc=devicetree@vger$(echo .)kernel.org \
--cc=kevenny.hsieh@mediatek$(echo .)com \
--cc=krzk+dt@kernel$(echo .)org \
--cc=liju-clr.chen@mediatek$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
--cc=linux-doc@vger$(echo .)kernel.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-mediatek@lists$(echo .)infradead.org \
--cc=matthias.bgg@gmail$(echo .)com \
--cc=my.chuang@mediatek$(echo .)com \
--cc=netdev@vger$(echo .)kernel.org \
--cc=peilun.suei@mediatek$(echo .)com \
--cc=quic_tsoni@quicinc$(echo .)com \
--cc=richardcochran@gmail$(echo .)com \
--cc=robh@kernel$(echo .)org \
--cc=shawn.hsiao@mediatek$(echo .)com \
--cc=will@kernel$(echo .)org \
--cc=yi-de.wu@mediatek$(echo .)com \
--cc=yingshiuan.pan@mediatek$(echo .)com \
--cc=ze-yu.wang@mediatek$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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