All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Mark Williamson <mark.williamson@cl.cam.ac.uk>
Cc: Daniel Stekloff <dsteklof@us.ibm.com>,
	xen-devel@lists.xensource.com, Ewan Mellor <ewan@xensource.com>
Subject: Re: [PATCH] Add Small Memory Warning to XM for create and	mem-set
Date: Fri, 21 Apr 2006 17:41:34 -0500	[thread overview]
Message-ID: <44495F9E.6020704@us.ibm.com> (raw)
In-Reply-To: <200604211629.40691.mark.williamson@cl.cam.ac.uk>

Mark Williamson wrote:
> I really like this idea.  I think it's most important for ballooning, which is 
> a really easy way to get a domain to commit suicide otherwise ;-)
>
> I'd quite like to see it implemented Xend-side, though, with the "warning 
> level" configurable (and probably disablable) in the xend-config.sxp so that 
> users who really do want to create very small domUs can tweak the system not 
> to warn them.
>   

I'm not sure I agree that Xend is the right place to do this.  You would 
have to add another parameter to the either the domain config or to the 
actual create() call that basically had the semantics of "if memory is 
less than X, throw an exception."  That seems like a rather awkward 
addition to the interface and it makes client code harder to write.

I'll attempt to be telepathic here and guess that you suggested this 
because there's no xm config file so there's no easy way to be able to 
tell xm to never give an error when mem is too small?  I think this is a 
good point in time to start thinking about just implementing an xm 
config file.

If you use SafeConfigParser and just grab from a standard location (say 
/etc/xen/xm.conf), you should be able to do it properly (just within xm) 
with very little code.

Thoughts?

Regards,

Anthony Liguori

> Cheers,
> Mark
>
> On Friday 21 April 2006 16:17, Daniel Stekloff wrote:
>   
>> Hi,
>>
>> This patch adds a warning to xm for creating and setting a domain's
>> memory under 32 MBs. Patch adds a "--force-mem" option to "xm create"
>> and "xm mem-set" commands for creating or setting memory less than 32
>> MBs. Patch also updates man page and user manual.
>>
>> All comments welcome.
>>
>> Thanks,
>>
>> Dan
>>     
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>   

  parent reply	other threads:[~2006-04-21 22:41 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-04-21 15:17 [PATCH] Add Small Memory Warning to XM for create and mem-set Daniel Stekloff
2006-04-21 15:28 ` Ewan Mellor
2006-04-21 15:29 ` Mark Williamson
2006-04-21 17:17   ` Daniel Stekloff
2006-04-21 22:41   ` Anthony Liguori [this message]
2006-04-21 22:57     ` Dan Smith
2006-04-24  0:55     ` Ewan Mellor
2006-04-24 23:57     ` Mark Williamson

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=44495F9E.6020704@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --cc=dsteklof@us.ibm.com \
    --cc=ewan@xensource.com \
    --cc=mark.williamson@cl.cam.ac.uk \
    --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.