Changeset fbcdeb8 in mainline for abi/include


Ignore:
Timestamp:
2011-12-19T17:30:39Z (14 years ago)
Author:
Martin Decky <martin@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
58f6229
Parents:
24cf31f1
Message:

Remove the two-phase way of creating virtual memory areas (first asking for a mappable address and then mapping it) which was prone to race conditions when two or more calls to as_get_mappable_page() and as_area_create() were interleaved. This for example caused the e1k driver to randomly fail.

The memory area related syscalls and IPC calls have all been altered to accept a special value (void *) -1, representing a demand to atomically search for a mappable address space "hole" and map to it.

Individual changes:

  • IPC_M_SHARE_OUT: the destination address space area is supplied by the kernel, the callee only specifies the lower bound

(the address is returned to the callee via a pointer in an IPC reply argument)

  • IPC_M_SHARE_IN: the destination address space ares is supplied by the kernel, the callee only specifies the lower bound

(the address is returned to the caller as usual via an IPC argument)

  • SYS_AS_GET_UNMAPPED_AREA was removed
  • dummy implementations of SYS_PHYSMEM_UNMAP and SYS_IOSPACE_DISABLE were added for the sake of symmetry (they do nothing yet)
  • SYS_PHYSMEM_MAP and SYS_DMAMEM_MAP were altered to accept (void *) -1 as address space area base and a lower bound
  • kernel as_area_create() and as_area_share() were altered to accept (void *) -1 as address space area base and a lower bound
  • uspace libraries and programs were altered to reflect the new API
Location:
abi/include
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • abi/include/ipc/methods.h

    r24cf31f1 rfbcdeb8  
    127127 *
    128128 * on answer, the recipient must set:
    129  * - ARG1 - dst as_area base adress
     129 *
     130 * - ARG1 - dst as_area lower bound
     131 * - ARG2 - dst as_area base adress pointer
     132 *          (filled automatically by the kernel)
    130133 *
    131134 */
    … …  
    133136
    134137/** Receive as_area over IPC.
    135  * - ARG1 - destination as_area base address
    136  * - ARG2 - destination as_area size
    137  * - ARG3 - user defined argument
     138 * - ARG1 - destination as_area size
     139 * - ARG2 - user defined argument
    138140 *
    139141 * on answer, the recipient must set:
    … …  
    141143 * - ARG1 - source as_area base address
    142144 * - ARG2 - flags that will be used for sharing
     145 * - ARG3 - dst as_area lower bound
     146 * - ARG4 - dst as_area base address (filled automatically by kernel)
    143147 *
    144148 */

  • abi/include/syscall.h

    r24cf31f1 rfbcdeb8  
    6060        SYS_AS_AREA_CHANGE_FLAGS,
    6161        SYS_AS_AREA_DESTROY,
    62         SYS_AS_GET_UNMAPPED_AREA,
    6362       
    6463        SYS_PAGE_FIND_MAPPING,
    … …  
    8584        SYS_DEVICE_ASSIGN_DEVNO,
    8685        SYS_PHYSMEM_MAP,
     86        SYS_PHYSMEM_UNMAP,
    8787        SYS_DMAMEM_MAP,
    8888        SYS_DMAMEM_UNMAP,
    8989        SYS_IOSPACE_ENABLE,
     90        SYS_IOSPACE_DISABLE,
    9091        SYS_IRQ_REGISTER,
    9192        SYS_IRQ_UNREGISTER,
Note: See TracChangeset for help on using the changeset viewer.