Changeset 9fa16b20 in mainline for arch/amd64/src/proc/scheduler.c

Timestamp:

2006-04-17T15:45:38Z (19 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

73e9b49

Parents:

97a7eff

Message:

Fix amd64 and ia32 management of I/O bitmap in before_task_runs_arch()
by treating the iomap array as a bitmap. Make use of the bitmap type
and functions added in previous commit.

File:

• arch/amd64/src/proc/scheduler.c (modified) (4 diffs)

arch/amd64/src/proc/scheduler.c

@@ -37 +37 @@
 #include <print.h>
 #include <arch/pm.h>
+#include <adt/bitmap.h>
 
 /** Perform amd64 specific tasks needed before the new task is run.
@@ -44 +45 @@
 void before_task_runs_arch(void)
 {
-        size_t iomap_size;
+        count_t bits;
         ptr_16_64_t cpugdtr;
         descriptor_t *gdt_p;
@@ -54 +55 @@
         /* First, copy the I/O Permission Bitmap. */
         spinlock_lock(&TASK->lock);
-        iomap_size = TASK->arch.iomap_size;
-        if (iomap_size) {
-                ASSERT(TASK->arch.iomap);
-                memcpy(CPU->arch.tss->iomap, TASK->arch.iomap, iomap_size);
-                CPU->arch.tss->iomap[iomap_size] = 0xff;        /* terminating byte */
+        if ((bits = TASK->arch.iomap.bits)) {
+                bitmap_t iomap;
+        
+                ASSERT(TASK->arch.iomap.map);
+                bitmap_initialize(&iomap, CPU->arch.tss->iomap, TSS_IOMAP_SIZE * 8);
+                bitmap_copy(&iomap, &TASK->arch.iomap, TASK->arch.iomap.bits);
+                /*
+                 * It is safe to set the trailing four bits because of the extra
+                 * convenience byte in TSS_IOMAP_SIZE.
+                 */
+                bitmap_set_range(&iomap, TASK->arch.iomap.bits, 4);
         }
         spinlock_unlock(&TASK->lock);
@@ -65 +72 @@
         gdtr_store(&cpugdtr);
         gdt_p = (descriptor_t *) cpugdtr.base;
-        gdt_tss_setlimit(&gdt_p[TSS_DES], TSS_BASIC_SIZE + iomap_size - 1);
+        gdt_tss_setlimit(&gdt_p[TSS_DES], TSS_BASIC_SIZE + BITS2BYTES(bits) - 1);
         gdtr_load(&cpugdtr);
 }