Changeset cd50486 in mainline for kernel/generic/src/sysinfo/stats.c

Timestamp:

2011-02-06T22:03:55Z (14 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

25971d2

Parents:

1110ebd (diff), 960ff451 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge development/ changes

File:

• kernel/generic/src/sysinfo/stats.c (modified) (3 diffs)

kernel/generic/src/sysinfo/stats.c

@@ -160 +160 @@
 static size_t get_task_virtmem(as_t *as)
 {
-        size_t result = 0;
-
         /*
-         * We are holding some spinlocks here and therefore are not allowed to
-         * block. Only attempt to lock the address space and address space area
-         * mutexes conditionally. If it is not possible to lock either object,
-         * allow the statistics to be inexact by skipping the respective object.
-         *
-         * Note that it may be infinitely better to let the address space
-         * management code compute these statistics as it proceeds instead of
-         * having them calculated over and over again here.
+         * We are holding spinlocks here and therefore are not allowed to
+         * block. Only attempt to lock the address space and address space
+         * area mutexes conditionally. If it is not possible to lock either
+         * object, return inexact statistics by skipping the respective object.
          */
-
+        
         if (SYNCH_FAILED(mutex_trylock(&as->lock)))
-                return result * PAGE_SIZE;
+                return 0;
+        
+        size_t pages = 0;
         
         /* Walk the B+ tree and count pages */
-        link_t *cur;
-        for (cur = as->as_area_btree.leaf_head.next;
-            cur != &as->as_area_btree.leaf_head; cur = cur->next) {
-                btree_node_t *node =
-                    list_get_instance(cur, btree_node_t, leaf_link);
-                
-                unsigned int i;
-                for (i = 0; i < node->keys; i++) {
-                        as_area_t *area = node->value[i];
-                        
-                        if (SYNCH_FAILED(mutex_trylock(&area->lock)))
-                                continue;
-                        result += area->pages;
-                        mutex_unlock(&area->lock);
-                }
-        }
-        
-        mutex_unlock(&as->lock);
-        
-        return result * PAGE_SIZE;
-}
-
-/** Get the resident (used) size of a virtual address space
- *
- * @param as Address space.
- *
- * @return Size of the resident (used) virtual address space (bytes).
- *
- */
-static size_t get_task_resmem(as_t *as)
-{
-        size_t result = 0;
-        
-        /*
-         * We are holding some spinlocks here and therefore are not allowed to
-         * block. Only attempt to lock the address space and address space area
-         * mutexes conditionally. If it is not possible to lock either object,
-         * allow the statistics to be inexact by skipping the respective object.
-         *
-         * Note that it may be infinitely better to let the address space
-         * management code compute these statistics as it proceeds instead of
-         * having them calculated over and over again here.
-         */
-        
-        if (SYNCH_FAILED(mutex_trylock(&as->lock)))
-                return result * PAGE_SIZE;
-        
-        /* Walk the B+ tree of AS areas */
         link_t *cur;
         for (cur = as->as_area_btree.leaf_head.next;
@@ -238 +186 @@
                                 continue;
                         
-                        /* Walk the B+ tree of resident pages */
-                        link_t *rcur;
-                        for (rcur = area->used_space.leaf_head.next;
-                            rcur != &area->used_space.leaf_head; rcur = rcur->next) {
-                                btree_node_t *rnode =
-                                    list_get_instance(rcur, btree_node_t, leaf_link);
-                                
-                                unsigned int j;
-                                for (j = 0; j < rnode->keys; j++)
-                                        result += (size_t) rnode->value[i];
-                        }
-                        
+                        pages += area->pages;
                         mutex_unlock(&area->lock);
                 }
@@ -256 +193 @@
         mutex_unlock(&as->lock);
         
-        return result * PAGE_SIZE;
+        return (pages << PAGE_WIDTH);
+}
+
+/** Get the resident (used) size of a virtual address space
+ *
+ * @param as Address space.
+ *
+ * @return Size of the resident (used) virtual address space (bytes).
+ *
+ */
+static size_t get_task_resmem(as_t *as)
+{
+        /*
+         * We are holding spinlocks here and therefore are not allowed to
+         * block. Only attempt to lock the address space and address space
+         * area mutexes conditionally. If it is not possible to lock either
+         * object, return inexact statistics by skipping the respective object.
+         */
+        
+        if (SYNCH_FAILED(mutex_trylock(&as->lock)))
+                return 0;
+        
+        size_t pages = 0;
+        
+        /* Walk the B+ tree and count pages */
+        link_t *cur;
+        for (cur = as->as_area_btree.leaf_head.next;
+            cur != &as->as_area_btree.leaf_head; cur = cur->next) {
+                btree_node_t *node =
+                    list_get_instance(cur, btree_node_t, leaf_link);
+                
+                unsigned int i;
+                for (i = 0; i < node->keys; i++) {
+                        as_area_t *area = node->value[i];
+                        
+                        if (SYNCH_FAILED(mutex_trylock(&area->lock)))
+                                continue;
+                        
+                        pages += area->resident;
+                        mutex_unlock(&area->lock);
+                }
+        }
+        
+        mutex_unlock(&as->lock);
+        
+        return (pages << PAGE_WIDTH);
 }