public inbox for linux-hyperv@vger.kernel.org
 help / color / mirror / Atom feed
From: Alexander Atanasov <alexander.atanasov@virtuozzo.com>
To: "Michael Kelley (LINUX)" <mikelley@microsoft.com>,
	Wei Liu <wei.liu@kernel.org>
Cc: KY Srinivasan <kys@microsoft.com>,
	Haiyang Zhang <haiyangz@microsoft.com>,
	Stephen Hemminger <sthemmin@microsoft.com>,
	Dexuan Cui <decui@microsoft.com>,
	"kernel@openvz.org" <kernel@openvz.org>,
	"linux-hyperv@vger.kernel.org" <linux-hyperv@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 1/1] Create debugfs file with hyper-v balloon usage information
Date: Wed, 7 Sep 2022 11:15:28 +0300	[thread overview]
Message-ID: <e3f418a9-97c4-0e8c-4446-b85fefd2cad8@virtuozzo.com> (raw)
In-Reply-To: <BYAPR21MB1688312AED65BEE290CF6C52D77D9@BYAPR21MB1688.namprd21.prod.outlook.com>

Hello,

On 3.09.22 7:37, Michael Kelley (LINUX) wrote:
> 
> Alexander -- I finally caught up on the long discussion of balloon
> driver reporting that occurred over much of August.  I think your
> original plan had been for each of the balloon driver to report
> useful information in debugfs.  But as a result of the discussion,
> it looks like virtio-balloon will be putting the information in
> /proc/meminfo.  If that's the case, it seems like we should
> drop these changes to the Hyper-V balloon driver,  and have
> the Hyper-V balloon driver take the same approach as
> virtio-balloon.
> 
> These Hyper-V balloon driver changes have already gone
> into 6.0-rc1.  If we're going to drop them, we should do
> the revert before 6.0 is done.
> 
> Thoughts?


There is a lot of information that would not go into /proc/meminfo.
vmware have only one file with all, virtio have separate file for 
features to expose. HV doesn't have other files so i think file is still 
useful. Only the memory in kB will go into the /proc/meminfo - 
pages/page sizes would not. But i am not sure how to handle the 
situation since i guess it will take time for the MM guys to decide - 
there is currently only one Ack on the series.



-- 
Regards,
Alexander Atanasov


  parent reply	other threads:[~2022-09-07  8:15 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-05  9:44 [PATCH v1 1/1] Create debugfs file with hyper-v balloon usage information Alexander Atanasov
2022-07-05 17:12 ` Michael Kelley (LINUX)
2022-07-05 17:31   ` Denis V. Lunev
2022-07-08 16:24   ` Alexander Atanasov
2022-07-11 18:18   ` [PATCH v3 " Alexander Atanasov
2022-07-12 16:24     ` Michael Kelley (LINUX)
2022-07-13 15:19       ` Wei Liu
2022-09-03  4:37         ` Michael Kelley (LINUX)
2022-09-05  8:59           ` Denis V. Lunev
2022-09-05 16:20           ` Wei Liu
2022-09-07  8:15           ` Alexander Atanasov [this message]
2022-07-06 18:02 ` [PATCH v1 " Wei Liu
2022-07-06 18:15   ` [PATCH v2 " Alexander Atanasov
2022-07-08 15:39     ` Michael Kelley (LINUX)

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=e3f418a9-97c4-0e8c-4446-b85fefd2cad8@virtuozzo.com \
    --to=alexander.atanasov@virtuozzo.com \
    --cc=decui@microsoft.com \
    --cc=haiyangz@microsoft.com \
    --cc=kernel@openvz.org \
    --cc=kys@microsoft.com \
    --cc=linux-hyperv@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mikelley@microsoft.com \
    --cc=sthemmin@microsoft.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