[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] Re: [PATCH] xen-mapcache: Fix rlimit set size.



On 08/01/2011 09:26 PM, Anthony PERARD wrote:
Previously, the address space soft limit was set mcache_max_size. So,
before the mcache_max_size was reached by the mapcache, QEMU was killed
for overuse of the virtual address space.

This patch fix that by setting the soft limit to mcache_max_size + 80MB.
I observed that QEMU use 75MB more than max_mcache_size after several
empirical tests.

Not sure I understand. What are those 75MB?


Alex

Signed-off-by: Anthony PERARD<anthony.perard@xxxxxxxxxx>
---
  xen-mapcache.c |   13 ++++++++++---
  1 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/xen-mapcache.c b/xen-mapcache.c
index 007136a..40212f7 100644
--- a/xen-mapcache.c
+++ b/xen-mapcache.c
@@ -40,6 +40,13 @@
  #endif
  #define MCACHE_BUCKET_SIZE (1UL<<  MCACHE_BUCKET_SHIFT)

+/* This is the size of the virtual address space reserve to QEMU that will not
+ * be use by MapCache.
+ * From empirical tests I observed that qemu use 75MB more than the
+ * max_mcache_size.
+ */
+#define NON_MCACHE_MEMORY_SIZE (80 * 1024 * 1024)
+
  #define mapcache_lock()   ((void)0)
  #define mapcache_unlock() ((void)0)

@@ -93,14 +100,14 @@ void xen_map_cache_init(void)
      mapcache->last_address_index = -1;

      getrlimit(RLIMIT_AS,&rlimit_as);
-    if (rlimit_as.rlim_max<  MCACHE_MAX_SIZE) {
+    if (rlimit_as.rlim_max<  MCACHE_MAX_SIZE + NON_MCACHE_MEMORY_SIZE) {
          rlimit_as.rlim_cur = rlimit_as.rlim_max;
      } else {
-        rlimit_as.rlim_cur = MCACHE_MAX_SIZE;
+        rlimit_as.rlim_cur = MCACHE_MAX_SIZE + NON_MCACHE_MEMORY_SIZE;
      }

      setrlimit(RLIMIT_AS,&rlimit_as);
-    mapcache->max_mcache_size = rlimit_as.rlim_cur;
+    mapcache->max_mcache_size = rlimit_as.rlim_cur - NON_MCACHE_MEMORY_SIZE;

      mapcache->nr_buckets =
          (((mapcache->max_mcache_size>>  XC_PAGE_SHIFT) +


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.