diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-09-06 07:31:48 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-09-14 07:02:28 +0200 |
commit | 694e946dbd64c94343aeb289edd80a60759f7b26 (patch) | |
tree | dadfd23f4d37c623b10dca5aff23b0c84f694de7 /cpukit/posix/src | |
parent | posix: Ignore pshared for semaphores (diff) | |
download | rtems-694e946dbd64c94343aeb289edd80a60759f7b26.tar.bz2 |
libio: Remove special-case reference count
The top-level IO library structures should contain no special-case data.
Update #2859.
Diffstat (limited to 'cpukit/posix/src')
-rw-r--r-- | cpukit/posix/src/mmap.c | 24 | ||||
-rw-r--r-- | cpukit/posix/src/munmap.c | 29 |
2 files changed, 19 insertions, 34 deletions
diff --git a/cpukit/posix/src/mmap.c b/cpukit/posix/src/mmap.c index b5af180d3d..9d9c0634ff 100644 --- a/cpukit/posix/src/mmap.c +++ b/cpukit/posix/src/mmap.c @@ -48,6 +48,7 @@ void *mmap( bool map_anonymous; bool map_shared; bool map_private; + bool is_shared_shm; int err; map_fixed = (flags & MAP_FIXED) == MAP_FIXED; @@ -194,7 +195,6 @@ void *mmap( memset( mapping, 0, sizeof( mmap_mapping )); mapping->len = len; mapping->flags = flags; - mapping->iop = iop; if ( !map_anonymous ) { /* @@ -206,19 +206,19 @@ void *mmap( if ( S_ISREG( sb.st_mode ) || S_ISBLK( sb.st_mode ) || S_ISCHR( sb.st_mode ) || S_ISFIFO( sb.st_mode ) || S_ISSOCK( sb.st_mode ) ) { - mapping->is_shared_shm = false; + is_shared_shm = false; } else { - mapping->is_shared_shm = true; + is_shared_shm = true; } } else { - mapping->is_shared_shm = false; + is_shared_shm = false; } if ( map_fixed ) { mapping->addr = addr; } else if ( map_private ) { /* private mappings of shared memory do not need special treatment. */ - mapping->is_shared_shm = false; + is_shared_shm = false; posix_memalign( &mapping->addr, PAGE_SIZE, len ); if ( !mapping->addr ) { free( mapping ); @@ -228,7 +228,7 @@ void *mmap( } /* MAP_FIXED is not supported for shared memory objects with MAP_SHARED. */ - if ( map_fixed && mapping->is_shared_shm ) { + if ( map_fixed && is_shared_shm ) { free( mapping ); errno = ENOTSUP; return MAP_FAILED; @@ -280,6 +280,11 @@ void *mmap( memset( mapping->addr, 0, len ); } } else if ( map_shared ) { + if ( is_shared_shm ) { + /* FIXME: This use of implementation details is a hack. */ + mapping->shm = iop_to_shm( iop ); + } + err = (*iop->pathinfo.handlers->mmap_h)( iop, &mapping->addr, len, prot, off ); if ( err != 0 ) { @@ -289,13 +294,6 @@ void *mmap( } } - if ( iop != NULL ) { - /* add an extra reference to the file associated with fildes that - * is not removed by a subsequent close(). This reference shall be removed - * when there are no more mappings to the file. */ - rtems_libio_increment_mapping_refcnt(iop); - } - rtems_chain_append_unprotected( &mmap_mappings, &mapping->node ); mmap_mappings_lock_release( ); diff --git a/cpukit/posix/src/munmap.c b/cpukit/posix/src/munmap.c index fb9bb872e0..5348be7a51 100644 --- a/cpukit/posix/src/munmap.c +++ b/cpukit/posix/src/munmap.c @@ -16,23 +16,14 @@ #include <stdlib.h> #include <sys/mman.h> -#include <rtems/posix/mmanimpl.h> +#include <rtems/posix/mmanimpl.h> #include <rtems/posix/shmimpl.h> -static void shm_munmap( rtems_libio_t *iop ) -{ - POSIX_Shm_Control *shm = iop_to_shm( iop ); - - /* decrement mmap's shm reference_count and maybe delete the object */ - POSIX_Shm_Attempt_delete(shm); -} - int munmap(void *addr, size_t len) { - mmap_mapping *mapping; - rtems_chain_node *node; - uint32_t refcnt; - + mmap_mapping *mapping; + rtems_chain_node *node; + /* * Clear errno. */ @@ -60,17 +51,13 @@ int munmap(void *addr, size_t len) if ( ( addr >= mapping->addr ) && ( addr < ( mapping->addr + mapping->len )) ) { rtems_chain_extract_unprotected( node ); + /* FIXME: generally need a way to clean-up the backing object, but * currently it only matters for MAP_SHARED shm objects. */ - if ( mapping->is_shared_shm == true ) { - shm_munmap(mapping->iop); - } - if ( mapping->iop != NULL ) { - refcnt = rtems_libio_decrement_mapping_refcnt(mapping->iop); - if ( refcnt == 0 ) { - rtems_libio_check_deferred_free(mapping->iop); - } + if ( mapping->shm != NULL ) { + POSIX_Shm_Attempt_delete(mapping->shm); } + /* only free the mapping address for non-fixed mapping */ if (( mapping->flags & MAP_FIXED ) != MAP_FIXED ) { /* only free the mapping address for non-shared mapping, because we |