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
>
next prev 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.