d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make vm_map_insert much more intelligent in the MAP_NOFAULT case so

Browse Source 
that map entries are coalesced when appropriate.  Also, conditionalize
some code that is currently not used in vm_map_insert.  This mod
has been added to eliminate unnecessary map entries in buffer map.

Additionally, there were some cases where map coalescing could be done
when it shouldn't.  That problem has been resolved.
This commit is contained in:
John Dyson 1996-12-07 00:03:43 +00:00
parent 97ba1b329f
commit cdc2c29161
2 changed files with 29 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
sys/vm/vm_map.c
View File

@ -61,7 +61,7 @@
* any improvements or extensions that they make and grant Carnegie the
* rights to redistribute these changes.
*
* $Id: vm_map.c,v 1.57 1996/09/14 11:54:55 bde Exp $
* $Id: vm_map.c,v 1.58 1996/11/30 22:41:47 dyson Exp $
*/
/*
@ -581,6 +581,7 @@ vm_map_lookup_entry(map, address, entry)
return (FALSE);
}
#define VM_MAP_INSERT_NULL_OBJECT_ONLY
/*
* vm_map_insert:
*
@ -605,6 +606,10 @@ vm_map_insert(map, object, offset, start, end, prot, max, cow)
vm_map_entry_t temp_entry;
vm_object_t prev_object;
if ((object != NULL) && (cow & MAP_NOFAULT)) {
panic("vm_map_insert: paradoxical MAP_NOFAULT request");
}
/*
* Check that the start and end points are not bogus.
*/
@ -633,7 +638,11 @@ vm_map_insert(map, object, offset, start, end, prot, max, cow)
if ((prev_entry != &map->header) &&
(prev_entry->end == start) &&
#if !defined(VM_MAP_INSERT_NULL_OBJECT_ONLY)
((object == NULL) || (prev_entry->object.vm_object == object)) &&
#else
(object == NULL) &&
#endif
(prev_entry->is_a_map == FALSE) &&
(prev_entry->is_sub_map == FALSE) &&
(prev_entry->inheritance == VM_INHERIT_DEFAULT) &&
@ -646,12 +655,21 @@ vm_map_insert(map, object, offset, start, end, prot, max, cow)
* See if we can avoid creating a new entry by extending one of our
* neighbors.
*/
#if !defined(VM_MAP_INSERT_NULL_OBJECT_ONLY)
if (object == NULL) {
if (vm_object_coalesce(prev_entry->object.vm_object,
#endif
u_char needs_copy = (cow & MAP_COPY_NEEDED) != 0;
u_char copy_on_write = (cow & MAP_COPY_ON_WRITE) != 0;
u_char nofault = (cow & MAP_NOFAULT) != 0;
if ((needs_copy == prev_entry->needs_copy) &&
(copy_on_write == prev_entry->copy_on_write) &&
(nofault == prev_entry->nofault) &&
(nofault || vm_object_coalesce(prev_entry->object.vm_object,
OFF_TO_IDX(prev_entry->offset),
(vm_size_t) (prev_entry->end
- prev_entry->start),
(vm_size_t) (end - prev_entry->end))) {
(vm_size_t) (end - prev_entry->end)))) {
/*
* Coalesced the two objects - can extend the
@ -660,11 +678,15 @@ vm_map_insert(map, object, offset, start, end, prot, max, cow)
*/
map->size += (end - prev_entry->end);
prev_entry->end = end;
prev_object = prev_entry->object.vm_object;
default_pager_convert_to_swapq(prev_object);
if (!nofault) {
prev_object = prev_entry->object.vm_object;
default_pager_convert_to_swapq(prev_object);
}
return (KERN_SUCCESS);
}
#if !defined(VM_MAP_INSERT_NULL_OBJECT_ONLY)
}
#endif
}
/*
* Create a new entry

4
sys/vm/vm_map.h
View File

@ -61,7 +61,7 @@
* any improvements or extensions that they make and grant Carnegie the
* rights to redistribute these changes.
*
* $Id: vm_map.h,v 1.15 1996/07/30 03:08:11 dyson Exp $
* $Id: vm_map.h,v 1.16 1996/11/30 22:41:48 dyson Exp $
*/
/*
@ -104,7 +104,7 @@ struct vm_map_entry {
vm_offset_t end; /* end address */
union vm_map_object object; /* object I point to */
vm_ooffset_t offset; /* offset into object */
boolean_t is_a_map:1, /* Is "object" a map? */
u_char is_a_map:1, /* Is "object" a map? */
is_sub_map:1, /* Is "object" a submap? */
copy_on_write:1, /* is data copy-on-write */
needs_copy:1, /* does object need to be copied */