Changeset a1b9f63 in mainline for kernel/genarch/src/acpi/acpi.c

Timestamp:

2018-08-31T10:32:40Z (6 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

6bf5b8c

Parents:

b1834a01

git-author:

Jakub Jermar <jakub@…> (2018-08-31 09:54:11)

git-committer:

Jakub Jermar <jakub@…> (2018-08-31 10:32:40)

Message:

Add alignment argument to km_map()

km_map() currently always applies alignment requirement equal to the
size of the mapped region. Most of the time, the natural alignment is
unnecessarily strong and especially on 32-bit systems may contribute to
km_map() failures for regions with size in the order of several hundred
megabytes.

This change adds an extra argument to km_map() which allows the caller
to indicate the desired alignment. The old behaviour can be specified
by passing KM_NATURAL_ALIGNMENT as alignment.

This change only adds the alignment argument, but does not change the
alignment requirement anywhere.

File:

• kernel/genarch/src/acpi/acpi.c (modified) (1 diff)

kernel/genarch/src/acpi/acpi.c

@@ -105 +105 @@
         /* Start with mapping the header only. */
         vhdr = (struct acpi_sdt_header *) km_map((uintptr_t) psdt,
-            sizeof(struct acpi_sdt_header), PAGE_READ | PAGE_NOT_CACHEABLE);
+            sizeof(struct acpi_sdt_header), KM_NATURAL_ALIGNMENT,
+            PAGE_READ | PAGE_NOT_CACHEABLE);
 
         /* Now we can map the entire structure. */
         vsdt = (struct acpi_sdt_header *) km_map((uintptr_t) psdt,
-            vhdr->length, PAGE_WRITE | PAGE_NOT_CACHEABLE);
+            vhdr->length, KM_NATURAL_ALIGNMENT,
+            PAGE_WRITE | PAGE_NOT_CACHEABLE);
 
         // TODO: do not leak vtmp