- bnc#674063 - drop xenpaging.autostart_delay.patch
avoid python excpetions due to intermediate xenstore and .sxp state caused by change of xenpaging variable from int to str OBS-URL: https://build.opensuse.org/package/show/Virtualization/xen?expand=0&rev=99
This commit is contained in:
James Fehlig
Git OBS Bridge
parent
551c9a62bc
commit
44aa83066c
@ -1,3 +1,10 @@
|
||||
-------------------------------------------------------------------
|
||||
Thu Feb 24 10:17:03 CET 2011 - ohering@suse.de
|
||||
|
||||
- bnc#674063 - drop xenpaging.autostart_delay.patch
|
||||
avoid python excpetions due to intermediate xenstore and .sxp
|
||||
state caused by change of xenpaging variable from int to str
|
||||
|
||||
-------------------------------------------------------------------
|
||||
Fri Feb 11 16:25:51 MST 2011 - carnold@novell.com
|
||||
|
||||
|
||||
5
xen.spec
5
xen.spec
@ -1,5 +1,5 @@
|
||||
#
|
||||
# spec file for package xen (Version 4.0.2_02)
|
||||
# spec file for package xen
|
||||
#
|
||||
# Copyright (c) 2011 SUSE LINUX Products GmbH, Nuernberg, Germany.
|
||||
#
|
||||
@ -302,7 +302,6 @@ Patch10019: xenpaging.MRU_SIZE.patch
|
||||
Patch10020: xenpaging.guest_remove_page.patch
|
||||
Patch10021: xenpaging.mem_event_check_ring-free_requests.patch
|
||||
Patch10022: xenpaging.blacklist.patch
|
||||
Patch10023: xenpaging.autostart_delay.patch
|
||||
Patch10024: xenpaging.page_already_populated.patch
|
||||
Patch10025: xenpaging.notify_policy_only_once.patch
|
||||
Patch10026: xenpaging.num_pages_equal_max_pages.patch
|
||||
@ -827,7 +826,6 @@ tar xfj %{SOURCE2} -C $RPM_BUILD_DIR/%{xen_build_dir}/tools
|
||||
%patch10020 -p1
|
||||
%patch10021 -p1
|
||||
%patch10022 -p1
|
||||
%patch10023 -p1
|
||||
%patch10024 -p1
|
||||
%patch10025 -p1
|
||||
%patch10026 -p1
|
||||
@ -840,7 +838,6 @@ tar xfj %{SOURCE2} -C $RPM_BUILD_DIR/%{xen_build_dir}/tools
|
||||
%patch10033 -p1
|
||||
%patch10040 -p1
|
||||
|
||||
|
||||
%build
|
||||
XEN_EXTRAVERSION=%version-%release
|
||||
XEN_EXTRAVERSION=${XEN_EXTRAVERSION#%{xvers}}
|
||||
|
||||
@ -1,85 +0,0 @@
|
||||
Subject: xenpaging: add dynamic startup delay for xenpaging
|
||||
|
||||
This is a debug helper. Since the xenpaging support is still fragile, run
|
||||
xenpaging at different stages in the bootprocess. Different delays will trigger
|
||||
more bugs. This implementation starts without delay for 5 reboots, then
|
||||
increments the delay by 0.1 seconds It uses xenstore for presistant storage of
|
||||
delay values
|
||||
|
||||
TODO: find the correct place to remove the xenstore directory when the guest is shutdown or crashed
|
||||
|
||||
Signed-off-by: Olaf Hering <olaf@aepfle.de>
|
||||
|
||||
---
|
||||
tools/python/xen/xend/image.py | 33 +++++++++++++++++++++++++++++++++
|
||||
1 file changed, 33 insertions(+)
|
||||
|
||||
Index: xen-4.0.2-testing/tools/python/xen/xend/image.py
|
||||
===================================================================
|
||||
--- xen-4.0.2-testing.orig/tools/python/xen/xend/image.py
|
||||
+++ xen-4.0.2-testing/tools/python/xen/xend/image.py
|
||||
@@ -123,6 +123,19 @@ class ImageHandler:
|
||||
|
||||
self.device_model = vmConfig['platform'].get('device_model')
|
||||
self.xenpaging = vmConfig['platform'].get('xenpaging')
|
||||
+ if self.xenpaging is not "0":
|
||||
+ self.xenpaging_delay = xstransact.Read("/local/domain/0/xenpaging/%s/xenpaging_delay" % self.vm.info['name_label'])
|
||||
+ if self.xenpaging_delay == None:
|
||||
+ log.warn("XXX creating /local/domain/0/xenpaging/%s" % self.vm.info['name_label'])
|
||||
+ xstransact.Mkdir("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'])
|
||||
+ xstransact.Store("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'], ('xenpaging_delay', '0.0'))
|
||||
+ xstransact.Store("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'], ('xenpaging_delay_inc', '0.1'))
|
||||
+ xstransact.Store("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'], ('xenpaging_delay_use', '5'))
|
||||
+ xstransact.Store("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'], ('xenpaging_delay_used', '0'))
|
||||
+ self.xenpaging_delay = float(xstransact.Read("/local/domain/0/xenpaging/%s/xenpaging_delay" % self.vm.info['name_label']))
|
||||
+ self.xenpaging_delay_inc = float(xstransact.Read("/local/domain/0/xenpaging/%s/xenpaging_delay_inc" % self.vm.info['name_label']))
|
||||
+ self.xenpaging_delay_use = int(xstransact.Read("/local/domain/0/xenpaging/%s/xenpaging_delay_use" % self.vm.info['name_label']))
|
||||
+ self.xenpaging_delay_used = int(xstransact.Read("/local/domain/0/xenpaging/%s/xenpaging_delay_used" % self.vm.info['name_label']))
|
||||
|
||||
self.display = vmConfig['platform'].get('display')
|
||||
self.xauthority = vmConfig['platform'].get('xauthority')
|
||||
@@ -399,6 +412,17 @@ class ImageHandler:
|
||||
return
|
||||
if self.xenpaging_pid:
|
||||
return
|
||||
+ if self.xenpaging_delay_used < self.xenpaging_delay_use:
|
||||
+ self.xenpaging_delay_used += 1
|
||||
+ else:
|
||||
+ self.xenpaging_delay_used = 0
|
||||
+ self.xenpaging_delay += self.xenpaging_delay_inc
|
||||
+ log.info("delay_used %s" % self.xenpaging_delay_used)
|
||||
+ log.info("delay_use %s" % self.xenpaging_delay_use)
|
||||
+ log.info("delay %s" % self.xenpaging_delay)
|
||||
+ log.info("delay_inc %s" % self.xenpaging_delay_inc)
|
||||
+ xstransact.Store("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'], ('xenpaging_delay', self.xenpaging_delay))
|
||||
+ xstransact.Store("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'], ('xenpaging_delay_used', self.xenpaging_delay_used))
|
||||
xenpaging_bin = auxbin.pathTo("xenpaging")
|
||||
args = [xenpaging_bin]
|
||||
args = args + ([ "%d" % self.vm.getDomid()])
|
||||
@@ -427,6 +451,9 @@ class ImageHandler:
|
||||
except:
|
||||
log.warn("chdir %s failed" % xenpaging_dir)
|
||||
try:
|
||||
+ if self.xenpaging_delay != 0.0:
|
||||
+ log.info("delaying xenpaging startup %s seconds ..." % self.xenpaging_delay)
|
||||
+ time.sleep(self.xenpaging_delay)
|
||||
log.info("starting %s" % args)
|
||||
os.execve(xenpaging_bin, args, env)
|
||||
except Exception, e:
|
||||
@@ -442,10 +469,16 @@ class ImageHandler:
|
||||
self.xenpaging_pid = xenpaging_pid
|
||||
os.close(null)
|
||||
os.close(logfd)
|
||||
+ if self.xenpaging_delay == 0.0:
|
||||
+ log.warn("waiting for xenpaging ...")
|
||||
+ time.sleep(22)
|
||||
+ log.warn("waiting for xenpaging done.")
|
||||
|
||||
def destroyXenPaging(self):
|
||||
if self.xenpaging == "0":
|
||||
return
|
||||
+ # FIXME find correct place for guest shutdown or crash
|
||||
+ #xstransact.Remove("/local/domain/0/xenpaging/%s" % self.vm.info['name_label'])
|
||||
if self.xenpaging_pid:
|
||||
try:
|
||||
os.kill(self.xenpaging_pid, signal.SIGHUP)
|
||||
Loading…
x
Reference in New Issue
Block a user