Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1610392

Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's initial state to number of existing RAM pages"

From "Jan Beulich" <JBeulich@suse.com>
Newsgroups linux.kernel
Subject Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's initial state to number of existing RAM pages"
Date 2017-03-28 10:10 +0200
Message-ID <tpUmu-4GO-23@gated-at.bofh.it> (permalink)
References (3 earlier) <toEMN-7q1-15@gated-at.bofh.it> <toEWt-7xX-5@gated-at.bofh.it> <toIQp-1P9-1@gated-at.bofh.it> <tpIY2-4JX-15@gated-at.bofh.it> <tpOAq-iy-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

>>> On 28.03.17 at 03:57, <boris.ostrovsky@oracle.com> wrote:
> I think there is indeed a disconnect between target memory (provided by 
> the toolstack) and current memory (i.e actual pages available to the guest).
> 
> For example
> 
> [    0.000000] BIOS-e820: [mem 0x000000000009e000-0x000000000009ffff] 
> reserved
> [    0.000000] BIOS-e820: [mem 0x00000000000e0000-0x00000000000fffff] 
> reserved
> 
> are missed in target calculation. The hvmloader marks them as RESERVED 
> (in build_e820_table()) but target value is not aware of this action.
> 
> And then the same problem repeats when kernel removes 
> 0x000a0000-0x000fffff chunk.

But this is all in-guest behavior, i.e. nothing an entity outside the
guest (tool stack or hypervisor) should need to be aware of. That
said, there is still room for improvement in the tools I think:
Regions which architecturally aren't RAM (namely the
0xa0000-0xfffff range) would probably better not be accounted
for as RAM as far as ballooning is concerned. In the hypervisor,
otoh, all memory assigned to the guest (i.e. including such backing
ROMs) needs to be accounted.

Jan

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's initial  state to number of existing RAM pages" Dan Streetman <dan.streetman@canonical.com> - 2017-03-22 22:20 +0100
  Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-03-23 03:20 +0100
    Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Juergen Gross <jgross@suse.com> - 2017-03-23 09:00 +0100
      Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Dan Streetman <dan.streetman@canonical.com> - 2017-03-24 21:40 +0100
    Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Dan Streetman <dan.streetman@canonical.com> - 2017-03-24 21:40 +0100
      Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> - 2017-03-24 22:20 +0100
        Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Dan Streetman <dan.streetman@canonical.com> - 2017-03-24 22:30 +0100
          Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-03-25 02:40 +0100
            Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Dan Streetman <dan.streetman@canonical.com> - 2017-03-27 22:00 +0200
              Re: maybe revert commit c275a57f5ec3 "xen/balloon: Set balloon's  initial state to number of existing RAM pages" Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-03-28 04:00 +0200
                Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon:  Set balloon's initial state to number of existing RAM pages" "Jan Beulich" <JBeulich@suse.com> - 2017-03-28 10:10 +0200
                Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set  balloon's initial state to number of existing RAM pages" Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-03-28 16:40 +0200
                Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon:  Set balloon's initial state to number of existing RAM pages" "Jan Beulich" <JBeulich@suse.com> - 2017-03-28 17:10 +0200
                Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set  balloon's initial state to number of existing RAM pages" Juergen Gross <jgross@suse.com> - 2017-03-28 17:40 +0200
                Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set  balloon's initial state to number of existing RAM pages" Boris Ostrovsky <boris.ostrovsky@oracle.com> - 2017-03-28 18:40 +0200
                Re: [Xen-devel] maybe revert commit c275a57f5ec3 "xen/balloon: Set  balloon's initial state to number of existing RAM pages" Juergen Gross <jgross@suse.com> - 2017-03-29 06:40 +0200

csiph-web