Changes in kernel/arch/amd64/src/ddi/ddi.c [98000fb:e9e5b9ab] in mainline

File:

• kernel/arch/amd64/src/ddi/ddi.c (modified) (5 diffs)

kernel/arch/amd64/src/ddi/ddi.c

@@ -36 +36 @@
 #include <arch/ddi/ddi.h>
 #include <proc/task.h>
-#include <arch/types.h>
+#include <typedefs.h>
 #include <adt/bitmap.h>
 #include <mm/slab.h>
@@ -49 +49 @@
  * Interrupts are disabled and task is locked.
  *
- * @param task Task.
+ * @param task   Task.
  * @param ioaddr Startign I/O space address.
- * @param size Size of the enabled I/O range.
+ * @param size   Size of the enabled I/O range.
  *
  * @return 0 on success or an error code from errno.h.
+ *
  */
 int ddi_iospace_enable_arch(task_t *task, uintptr_t ioaddr, size_t size)
 {
-        size_t bits;
-        
-        bits = ioaddr + size;
+        size_t bits = ioaddr + size;
         if (bits > IO_PORTS)
                 return ENOENT;
         
         if (task->arch.iomap.bits < bits) {
-                bitmap_t oldiomap;
-                uint8_t *newmap;
-                
                 /*
                  * The I/O permission bitmap is too small and needs to be grown.
                  */
                 
-                newmap = (uint8_t *) malloc(BITS2BYTES(bits), FRAME_ATOMIC);
+                uint8_t *newmap = (uint8_t *) malloc(BITS2BYTES(bits), FRAME_ATOMIC);
                 if (!newmap)
                         return ENOMEM;
                 
+                bitmap_t oldiomap;
                 bitmap_initialize(&oldiomap, task->arch.iomap.map,
                     task->arch.iomap.bits);
@@ -115 +112 @@
  *
  * Interrupts must be disabled prior this call.
+ *
  */
 void io_perm_bitmap_install(void)
 {
-        size_t bits;
-        ptr_16_64_t cpugdtr;
-        descriptor_t *gdt_p;
-        tss_descriptor_t *tss_desc;
-        size_t ver;
-        
         /* First, copy the I/O Permission Bitmap. */
-        spinlock_lock(&TASK->lock);
-        ver = TASK->arch.iomapver;
-        if ((bits = TASK->arch.iomap.bits)) {
+        irq_spinlock_lock(&TASK->lock, false);
+        size_t ver = TASK->arch.iomapver;
+        size_t bits = TASK->arch.iomap.bits;
+        if (bits) {
+                ASSERT(TASK->arch.iomap.map);
+                
                 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);
+                bitmap_copy(&iomap, &TASK->arch.iomap, bits);
+                
+                /*
+                 * Set the trailing bits in the last byte of the map to disable
+                 * I/O access.
+                 */
+                bitmap_set_range(&iomap, bits, ALIGN_UP(bits, 8) - bits);
                 /*
                  * It is safe to set the trailing eight bits because of the
                  * extra convenience byte in TSS_IOMAP_SIZE.
                  */
-                bitmap_set_range(&iomap, ALIGN_UP(TASK->arch.iomap.bits, 8), 8);
+                bitmap_set_range(&iomap, ALIGN_UP(bits, 8), 8);
         }
-        spinlock_unlock(&TASK->lock);
+        irq_spinlock_unlock(&TASK->lock, false);
         
         /*
@@ -146 +145 @@
          * Take the extra ending byte will all bits set into account. 
          */
+        ptr_16_64_t cpugdtr;
         gdtr_store(&cpugdtr);
-        gdt_p = (descriptor_t *) cpugdtr.base;
+        
+        descriptor_t *gdt_p = (descriptor_t *) cpugdtr.base;
         gdt_tss_setlimit(&gdt_p[TSS_DES], TSS_BASIC_SIZE + BITS2BYTES(bits));
         gdtr_load(&cpugdtr);
@@ -155 +156 @@
          * type must be changed to describe inactive TSS.
          */
-        tss_desc = (tss_descriptor_t *) &gdt_p[TSS_DES];
+        tss_descriptor_t *tss_desc = (tss_descriptor_t *) &gdt_p[TSS_DES];
         tss_desc->type = AR_TSS;
-        tr_load(gdtselector(TSS_DES));
+        tr_load(GDT_SELECTOR(TSS_DES));
         
         /*