Changeset bfb1179 in mainline

Timestamp:

2018-12-10T10:50:30Z (6 years ago)

Author:

Jiri Svoboda <jiri@…>

Parents:

dd74b5b

git-author:

Jiri Svoboda <jiri@…> (2018-12-10 07:33:08)

git-committer:

Jiri Svoboda <jiri@…> (2018-12-10 10:50:30)

Message:

No need to compute number of used pages again

In as_area_change_flags we can just use used_space.pages which already
contains the number of used pages.

File:

• kernel/generic/src/mm/as.c (modified) (2 diffs)

kernel/generic/src/mm/as.c

@@ -1390 +1390 @@
         mutex_unlock(&area->sh_info->lock);
 
-        /*
-         * Compute total number of used pages
-         */
-        size_t used_pages = 0;
-
-        used_space_ival_t *ival = used_space_first(&area->used_space);
-        while (ival != NULL) {
-                used_pages += ival->count;
-                ival = used_space_next(ival);
-        }
-
         /* An array for storing frame numbers */
-        uintptr_t *old_frame = malloc(used_pages * sizeof(uintptr_t));
+        uintptr_t *old_frame = malloc(area->used_space.pages *
+            sizeof(uintptr_t));
         if (!old_frame) {
                 mutex_unlock(&area->lock);
@@ -1423 +1413 @@
         size_t frame_idx = 0;
 
-        ival = used_space_first(&area->used_space);
+        used_space_ival_t *ival = used_space_first(&area->used_space);
         while (ival != NULL) {
                 uintptr_t ptr = ival->page;