Skip to contentSkip to conversation

Make xend set the console limit

PATCH RESEND In review Dan Smith 2 participants 4 messages Oct 18, 2005 – Oct 19, 2005

lore
Share & tools
lore

Dan Smith

author
#0 lore raw

PATCH RESEND 1/1 Make xend set the console limit

This is a resend of the console limit patch I sent last week. If there is anything wrong with the change, please advise.

tools/examples/xend-config.sxp +3 -0
diff -r 446aa56ca4fe tools/examples/xend-config.sxp
@@ -19,6 +19,9 @@
 # Specifying 'localhost' prevents remote connections.
 # Specifying the empty string '' allows all connections.
 (console-address   'localhost')
+
+# The limit (in kilobytes) on the size of the console buffer
+(console-limit 1024)
 
 ## Use the following if VIF traffic is routed.
 # The script used to start/stop networking for xend.
tools/python/xen/xend/XendDomainInfo.py +1 -1
diff -r 446aa56ca4fe tools/python/xen/xend/XendDomainInfo.py
@@ -588,7 +588,7 @@
         to_store = {
             'domid':              str(self.domid),
             'vm':                 self.vmpath,
-
+            'console/limit':      str(xroot.get_console_limit() * 1024),
             'memory/target':      str(self.info['memory_KiB'])
             }
 
tools/python/xen/xend/XendRoot.py +3 -0
diff -r 446aa56ca4fe tools/python/xen/xend/XendRoot.py
@@ -267,6 +267,9 @@
     def get_dom0_vcpus(self):
         return self.get_config_int('dom0-cpus', self.dom0_vcpus_default)
 
+    def get_console_limit(self):
+        return self.get_config_int('console-limit', 1024)
+
 def instance():
     """Get an instance of XendRoot.
     Use this instead of the constructor.
Signature · 4 lines
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: [email protected]

_______________________________________________
Xen-devel mailing list
[email protected]
http://lists.xensource.com/xen-devel

#1 lore raw
Quoted 2 lines · Dan Smith

This is a resend of the console limit patch I sent last week. If there is anything wrong with the change, please advise.

There's nothing wrong with the patch that I can see -- I've just been short of time. Apologies. It will make it in soon.

Ewan.

author
#2 lore raw

EM> There's nothing wrong with the patch that I can see -- I've just EM> been short of time. Apologies. It will make it in soon.

Not a problem. At one point, someone mentioned a 48-hour resend buffer for patches that might have gotten missed.

Thanks Ewan!

Signature · 4 lines
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: [email protected]
#3 lore raw
Quoted 6 lines · Dan Smith

EM> There's nothing wrong with the patch that I can see -- I've just EM> been short of time. Apologies. It will make it in soon.

Not a problem. At one point, someone mentioned a 48-hour resend buffer for patches that might have gotten missed.

Yes, that's definitely a good idea -- we do forget things sometimes!

Ewan.