All of lore.kernel.org
 help / color / mirror / Atom feed
From: Glauber de Oliveira Costa <gcosta@redhat.com>
To: Keir Fraser <keir@xensource.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH/RFC] memory_map + set_memmap_limit hypercall/domctl
Date: Wed, 29 Nov 2006 10:40:15 -0200	[thread overview]
Message-ID: <20061129124015.GC23434@redhat.com> (raw)
In-Reply-To: <C190E817.4E0E%keir@xensource.com>

> The fallbacks in XENMEM_memory_map are pointless. Any stable Xen version
> that implements this hypercall will be matched with a corresponding new
> enough tool stack that sets memmap_limit. I think we should continue to
> ENOSYS if memmap_limit has not in fact been specified.
Done.
 
> The addition of 8MB slack is policy that doesn't belong in Xen. Stick it in
> the top-level caller (XendDomainInfo). In fact it probably belongs in the
> ImnageHandler in image.py, as it's more easily overridden on a per-arch and
> per-image-type basis there.

ImageHandler does not seem to be aware of any distinction between
current memory and the maximum memory a guest can get. Adding it seems
much more work than it requires, so I put it in XenDomainInfo. Is that
okay? 

Patch follows soon.


-- 
Glauber de Oliveira Costa
Red Hat Inc.
"Free as in Freedom"

      reply	other threads:[~2006-11-29 12:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-27 18:56 [PATCH/RFC] memory_map + set_memmap_limit hypercall/domctl Glauber de Oliveira Costa
2006-11-27 19:08 ` Keir Fraser
2006-11-29 12:40   ` Glauber de Oliveira Costa [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=20061129124015.GC23434@redhat.com \
    --to=gcosta@redhat.com \
    --cc=keir@xensource.com \
    --cc=xen-devel@lists.xensource.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.