From: Alexander Atanasov <alexander.atanasov@virtuozzo.com>
To: Muchun Song <muchun.song@linux.dev>
Cc: Jonathan Corbet <corbet@lwn.net>,
Andrew Morton <akpm@linux-foundation.org>,
kernel@openvz.org, David Hildenbrand <david@redhat.com>,
Wei Liu <wei.liu@kernel.org>, Nadav Amit <namit@vmware.com>,
linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
linux-doc@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [PATCH v1 1/2] Enable balloon drivers to report inflated memory
Date: Wed, 10 Aug 2022 08:14:06 +0300 [thread overview]
Message-ID: <9653d74f-ffff-6a25-8c99-5565bcab4d5c@virtuozzo.com> (raw)
In-Reply-To: <5EFFFB23-9F73-4F07-B6FF-3BD05976D427@linux.dev>
Hello,
On 10.08.22 6:05, Muchun Song wrote:
>
>
>> On Aug 9, 2022, at 17:49, Alexander Atanasov <alexander.atanasov@virtuozzo.com> wrote:
>>
>> Display reported in /proc/meminfo as:
>
> Hi,
>
> I am not sure if this is a right place (meminfo) to put this statistic in since
> it is the accounting from a specific driver. IIUC, this driver is only installed
> in a VM, then this accounting will always be zero if this driver is not installed.
> Is this possible to put it in a driver-specific sysfs file (maybe it is better)?
> Just some thoughts from me.
Yes, it is only used if run under hypervisor but it is under a config
option for that reason.
There are several balloon drivers that will use it, not only one driver
- virtio, VMWare, HyperV, XEN and possibly others. I made one as an
example. Initially i worked on a patches for debugfs but discussion lead
to put it into a centralized place.
--
Regards,
Alexander Atanasov
prev parent reply other threads:[~2022-08-10 5:14 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <7bfac48d-2e50-641b-6523-662ea4df0240@virtuozzo.com>
2022-08-09 9:49 ` Alexander Atanasov
2022-08-09 10:32 ` Michael S. Tsirkin
2022-08-10 5:54 ` Alexander Atanasov
2022-08-10 6:05 ` Michael S. Tsirkin
2022-08-10 7:50 ` Alexander Atanasov
2022-08-10 9:16 ` Michael S. Tsirkin
2022-08-10 3:05 ` Muchun Song
2022-08-10 5:14 ` Alexander Atanasov [this message]
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=9653d74f-ffff-6a25-8c99-5565bcab4d5c@virtuozzo.com \
--to=alexander.atanasov@virtuozzo.com \
--cc=akpm@linux-foundation.org \
--cc=corbet@lwn.net \
--cc=david@redhat.com \
--cc=kernel@openvz.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=muchun.song@linux.dev \
--cc=namit@vmware.com \
--cc=wei.liu@kernel.org \
/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