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

[Xen-devel] [PATCH] disable log-throttling in xen_platform after boot



Hi all,
currently log throttling in xen_platform can be disabled only if a
particular key is present on xenstore at boot time.
Obviously this requirement makes it difficult to use.
This patch fixes the issue creating a watch on xenstore on the node

/local/domain/$DOMID/log-throttling

so whenever a developer wants to disable log throttling, he just has to
write 0 to /local/domain/$DOMID/log-throttling at any moment after boot.

Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>

---


diff --git a/hw/xen_platform.c b/hw/xen_platform.c
index 36610a4..d282f8e 100644
--- a/hw/xen_platform.c
+++ b/hw/xen_platform.c
@@ -50,6 +50,17 @@ typedef struct PCIXenPlatformState
   PCIDevice  pci_dev;
 } PCIXenPlatformState;
 
+static void log_throttling(const char *path, void *opaque)
+{
+    int len;
+    char *throttling = xenstore_dom_read(domid, "log-throttling", &len);
+    if (throttling != NULL) {
+        throttling_disabled = !(throttling[0] - '0');
+        free(throttling);
+        fprintf(logfile, "log_throttling %s\n", throttling_disabled ? 
"disabled" : "enabled");
+    }
+}
+
 /* We throttle access to dom0 syslog, to avoid DOS attacks.  This is
    modelled as a token bucket, with one token for every byte of log.
    The bucket size is 128KB (->1024 lines of 128 bytes each) and
@@ -276,7 +287,6 @@ static int platform_fixed_ioport_load(QEMUFile *f, void 
*opaque, int version_id)
 void platform_fixed_ioport_init(void)
 {
     struct stat stbuf;
-    char *throttling;
     int len = 1;
 
     register_savevm("platform_fixed_ioport", 0, 1, platform_fixed_ioport_save,
@@ -288,12 +298,6 @@ void platform_fixed_ioport_init(void)
     register_ioport_read(0x10, 16, 2, platform_fixed_ioport_read2, NULL);
     register_ioport_read(0x10, 16, 1, platform_fixed_ioport_read1, NULL);
 
-    throttling = xenstore_vm_read(domid, "log-throttling", &len);
-    if (throttling != NULL) {
-        throttling_disabled = (throttling[0] - '0');
-        free(throttling);
-    }
-
     platform_fixed_ioport_write1(NULL, 0x10, 0);
 }
 
@@ -477,6 +481,6 @@ void pci_xen_platform_init(PCIBus *bus)
 
     register_savevm("platform", 0, 3, xen_pci_save, xen_pci_load, d);
     printf("Done register platform.\n");
-
+    xenstore_dom_watch(domid, "log-throttling", log_throttling, NULL);
 }
 

_______________________________________________
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®.