Merge pull request #1063 from esdrubal/bug18482
[mono.git] / mono / metadata / sgen-cardtable.c
index 0744e4c95ca08de86fb4d483b6760c3420fd90a8..aab3bcb6153332ff59565ab941b449aea71b0fdf 100644 (file)
 #include "metadata/sgen-cardtable.h"
 #include "metadata/sgen-memory-governor.h"
 #include "metadata/sgen-protocol.h"
+#include "metadata/sgen-layout-stats.h"
 #include "utils/mono-counters.h"
 #include "utils/mono-time.h"
 #include "utils/mono-memory-model.h"
 
-#ifdef SGEN_HAVE_CARDTABLE
-
 //#define CARDTABLE_STATS
 
 #ifdef HAVE_UNISTD_H
 #endif
 #include <sys/types.h>
 
+#define ARRAY_OBJ_INDEX(ptr,array,elem_size) (((char*)(ptr) - ((char*)(array) + G_STRUCT_OFFSET (MonoArray, vector))) / (elem_size))
+
 guint8 *sgen_cardtable;
 
+static gboolean need_mod_union;
 
 #ifdef HEAVY_STATISTICS
 long long marked_cards;
@@ -83,7 +85,7 @@ static void
 sgen_card_table_wbarrier_set_field (MonoObject *obj, gpointer field_ptr, MonoObject* value)
 {
        *(void**)field_ptr = value;
-       if (sgen_ptr_in_nursery (value))
+       if (need_mod_union || sgen_ptr_in_nursery (value))
                sgen_card_table_mark_address ((mword)field_ptr);
        sgen_dummy_use (value);
 }
@@ -92,7 +94,7 @@ static void
 sgen_card_table_wbarrier_set_arrayref (MonoArray *arr, gpointer slot_ptr, MonoObject* value)
 {
        *(void**)slot_ptr = value;
-       if (sgen_ptr_in_nursery (value))
+       if (need_mod_union || sgen_ptr_in_nursery (value))
                sgen_card_table_mark_address ((mword)slot_ptr);
        sgen_dummy_use (value); 
 }
@@ -112,7 +114,7 @@ sgen_card_table_wbarrier_arrayref_copy (gpointer dest_ptr, gpointer src_ptr, int
                for (; dest >= start; --src, --dest) {
                        gpointer value = *src;
                        *dest = value;
-                       if (sgen_ptr_in_nursery (value))
+                       if (need_mod_union || sgen_ptr_in_nursery (value))
                                sgen_card_table_mark_address ((mword)dest);
                        sgen_dummy_use (value);
                }
@@ -121,7 +123,7 @@ sgen_card_table_wbarrier_arrayref_copy (gpointer dest_ptr, gpointer src_ptr, int
                for (; dest < end; ++src, ++dest) {
                        gpointer value = *src;
                        *dest = value;
-                       if (sgen_ptr_in_nursery (value))
+                       if (need_mod_union || sgen_ptr_in_nursery (value))
                                sgen_card_table_mark_address ((mword)dest);
                        sgen_dummy_use (value);
                }
@@ -140,7 +142,7 @@ sgen_card_table_wbarrier_value_copy (gpointer dest, gpointer src, int count, Mon
        TLAB_ACCESS_INIT;
        ENTER_CRITICAL_REGION;
 #endif
-       mono_gc_memmove (dest, src, size);
+       mono_gc_memmove_atomic (dest, src, size);
        sgen_card_table_mark_range ((mword)dest, size);
 #ifdef DISABLE_CRITICAL_REGION
        UNLOCK_GC;
@@ -160,7 +162,7 @@ sgen_card_table_wbarrier_object_copy (MonoObject* obj, MonoObject *src)
        TLAB_ACCESS_INIT;
        ENTER_CRITICAL_REGION;
 #endif
-       mono_gc_memmove ((char*)obj + sizeof (MonoObject), (char*)src + sizeof (MonoObject),
+       mono_gc_memmove_aligned ((char*)obj + sizeof (MonoObject), (char*)src + sizeof (MonoObject),
                        size - sizeof (MonoObject));
        sgen_card_table_mark_range ((mword)obj, size);
 #ifdef DISABLE_CRITICAL_REGION
@@ -238,7 +240,7 @@ sgen_card_table_get_card_data (guint8 *data_dest, mword address, mword cards)
 #endif
        }
 
-       return mask;
+       return mask != 0;
 }
 
 void*
@@ -279,6 +281,83 @@ sgen_card_table_find_address (char *addr)
        return sgen_card_table_address_is_marked ((mword)addr);
 }
 
+static gboolean
+sgen_card_table_find_address_with_cards (char *cards_start, guint8 *cards, char *addr)
+{
+       cards_start = sgen_card_table_align_pointer (cards_start);
+       return cards [(addr - cards_start) >> CARD_BITS];
+}
+
+static void
+update_mod_union (guint8 *dest, gboolean init, guint8 *start_card, size_t num_cards)
+{
+       if (init) {
+               memcpy (dest, start_card, num_cards);
+       } else {
+               int i;
+               for (i = 0; i < num_cards; ++i)
+                       dest [i] |= start_card [i];
+       }
+}
+
+static guint8*
+alloc_mod_union (size_t num_cards)
+{
+       return sgen_alloc_internal_dynamic (num_cards, INTERNAL_MEM_CARDTABLE_MOD_UNION, TRUE);
+}
+
+guint8*
+sgen_card_table_update_mod_union_from_cards (guint8 *dest, guint8 *start_card, size_t num_cards)
+{
+       gboolean init = dest == NULL;
+
+       if (init)
+               dest = alloc_mod_union (num_cards);
+
+       update_mod_union (dest, init, start_card, num_cards);
+
+       return dest;
+}
+
+guint8*
+sgen_card_table_update_mod_union (guint8 *dest, char *obj, mword obj_size, size_t *out_num_cards)
+{
+       guint8 *start_card = sgen_card_table_get_card_address ((mword)obj);
+#ifndef SGEN_HAVE_OVERLAPPING_CARDS
+       guint8 *end_card = sgen_card_table_get_card_address ((mword)obj + obj_size - 1) + 1;
+#endif
+       size_t num_cards;
+       guint8 *result = NULL;
+
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+       size_t rest;
+
+       rest = num_cards = cards_in_range ((mword) obj, obj_size);
+
+       while (start_card + rest > SGEN_CARDTABLE_END) {
+               size_t count = SGEN_CARDTABLE_END - start_card;
+               dest = sgen_card_table_update_mod_union_from_cards (dest, start_card, count);
+               if (!result)
+                       result = dest;
+               dest += count;
+               rest -= count;
+               start_card = sgen_cardtable;
+       }
+       num_cards = rest;
+#else
+       num_cards = end_card - start_card;
+#endif
+
+       dest = sgen_card_table_update_mod_union_from_cards (dest, start_card, num_cards);
+       if (!result)
+               result = dest;
+
+       if (out_num_cards)
+               *out_num_cards = num_cards;
+
+       return result;
+}
+
 #ifdef SGEN_HAVE_OVERLAPPING_CARDS
 
 static void
@@ -288,7 +367,9 @@ move_cards_to_shadow_table (mword start, mword size)
        guint8 *to = sgen_card_table_get_shadow_card_address (start);
        size_t bytes = cards_in_range (start, size);
 
-       if (to + bytes > SGEN_SHADOW_CARDTABLE_END) {
+       if (bytes >= CARD_COUNT_IN_BYTES) {
+               memcpy (sgen_shadow_cardtable, sgen_cardtable, CARD_COUNT_IN_BYTES);
+       } else if (to + bytes > SGEN_SHADOW_CARDTABLE_END) {
                size_t first_chunk = SGEN_SHADOW_CARDTABLE_END - to;
                size_t second_chunk = MIN (CARD_COUNT_IN_BYTES, bytes) - first_chunk;
 
@@ -305,7 +386,9 @@ clear_cards (mword start, mword size)
        guint8 *addr = sgen_card_table_get_card_address (start);
        size_t bytes = cards_in_range (start, size);
 
-       if (addr + bytes > SGEN_CARDTABLE_END) {
+       if (bytes >= CARD_COUNT_IN_BYTES) {
+               memset (sgen_cardtable, 0, CARD_COUNT_IN_BYTES);
+       } else if (addr + bytes > SGEN_CARDTABLE_END) {
                size_t first_chunk = SGEN_CARDTABLE_END - addr;
 
                memset (addr, 0, first_chunk);
@@ -492,8 +575,10 @@ sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, gboo
 
        HEAVY_STAT (++large_objects);
 
-       if (!SGEN_VTABLE_HAS_REFERENCES (vt))
+       if (!SGEN_VTABLE_HAS_REFERENCES (vt)) {
+               sgen_object_layout_scanned_bitmap (0);
                return;
+       }
 
        if (vt->rank) {
                guint8 *card_data, *card_base;
@@ -502,7 +587,7 @@ sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, gboo
                mword obj_size = sgen_par_object_get_size (vt, (MonoObject*)obj);
                char *obj_end = obj + obj_size;
                size_t card_count;
-               int extra_idx = 0;
+               size_t extra_idx = 0;
 
                MonoArray *arr = (MonoArray*)obj;
                mword desc = (mword)klass->element_class->gc_descr;
@@ -512,6 +597,13 @@ sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, gboo
                guint8 *overflow_scan_end = NULL;
 #endif
 
+#ifdef SGEN_OBJECT_LAYOUT_STATISTICS
+               if (klass->element_class->valuetype)
+                       sgen_object_layout_scanned_vtype_array ();
+               else
+                       sgen_object_layout_scanned_ref_array ();
+#endif
+
                if (cards)
                        card_data = cards;
                else
@@ -534,8 +626,8 @@ LOOP_HEAD:
 
                card_data = find_next_card (card_data, card_data_end);
                for (; card_data < card_data_end; card_data = find_next_card (card_data + 1, card_data_end)) {
-                       int index;
-                       int idx = (card_data - card_base) + extra_idx;
+                       size_t index;
+                       size_t idx = (card_data - card_base) + extra_idx;
                        char *start = (char*)(obj_start + idx * CARD_SIZE_IN_BYTES);
                        char *card_end = start + CARD_SIZE_IN_BYTES;
                        char *first_elem, *elem;
@@ -552,12 +644,12 @@ LOOP_HEAD:
                        else
                                index = ARRAY_OBJ_INDEX (start, obj, elem_size);
 
-                       elem = first_elem = (char*)mono_array_addr_with_size ((MonoArray*)obj, elem_size, index);
+                       elem = first_elem = (char*)mono_array_addr_with_size_fast ((MonoArray*)obj, elem_size, index);
                        if (klass->element_class->valuetype) {
                                ScanVTypeFunc scan_vtype_func = sgen_get_current_object_ops ()->scan_vtype;
 
                                for (; elem < card_end; elem += elem_size)
-                                       scan_vtype_func (elem, desc, queue);
+                                       scan_vtype_func (elem, desc, queue BINARY_PROTOCOL_ARG (elem_size));
                        } else {
                                CopyOrMarkObjectFunc copy_func = sgen_get_current_object_ops ()->copy_or_mark_object;
 
@@ -569,7 +661,7 @@ LOOP_HEAD:
                                                copy_func ((void**)elem, queue);
                                                new = *(gpointer*)elem;
                                                if (G_UNLIKELY (sgen_ptr_in_nursery (new)))
-                                                       sgen_add_to_global_remset (elem, new, mod_union);
+                                                       sgen_add_to_global_remset (elem, new);
                                        }
                                }
                        }
@@ -658,7 +750,7 @@ sgen_card_tables_collect_stats (gboolean begin)
                printf ("cards major (t %d m %d g %d r %d)  los (t %d m %d g %d r %d) major_scan %.2fms los_scan %.2fms\n", 
                        major_stats.total, major_stats.marked, major_stats.gc_marked, major_stats.remarked,
                        los_stats.total, los_stats.marked, los_stats.gc_marked, los_stats.remarked,
-                       last_major_scan_time / 1000.0, last_los_scan_time / 1000.0);
+                       last_major_scan_time / 10000.0f, last_los_scan_time / 10000.0f);
        }
 #endif
 }
@@ -684,8 +776,8 @@ sgen_card_table_init (SgenRemeberedSet *remset)
        mono_counters_register ("cardtable large objects", MONO_COUNTER_GC | MONO_COUNTER_LONG, &large_objects);
        mono_counters_register ("cardtable bloby objects", MONO_COUNTER_GC | MONO_COUNTER_LONG, &bloby_objects);
 #endif
-       mono_counters_register ("cardtable major scan time", MONO_COUNTER_GC | MONO_COUNTER_TIME_INTERVAL, &major_card_scan_time);
-       mono_counters_register ("cardtable los scan time", MONO_COUNTER_GC | MONO_COUNTER_TIME_INTERVAL, &los_card_scan_time);
+       mono_counters_register ("cardtable major scan time", MONO_COUNTER_GC | MONO_COUNTER_LONG | MONO_COUNTER_TIME, &major_card_scan_time);
+       mono_counters_register ("cardtable los scan time", MONO_COUNTER_GC | MONO_COUNTER_LONG | MONO_COUNTER_TIME, &los_card_scan_time);
 
 
        remset->wbarrier_set_field = sgen_card_table_wbarrier_set_field;
@@ -702,28 +794,9 @@ sgen_card_table_init (SgenRemeberedSet *remset)
        remset->prepare_for_major_collection = sgen_card_table_prepare_for_major_collection;
 
        remset->find_address = sgen_card_table_find_address;
-}
-
-#else
+       remset->find_address_with_cards = sgen_card_table_find_address_with_cards;
 
-void
-sgen_card_table_mark_address (mword address)
-{
-       g_assert_not_reached ();
-}
-
-void
-sgen_card_table_mark_range (mword address, mword size)
-{
-       g_assert_not_reached ();
+       need_mod_union = sgen_get_major_collector ()->is_concurrent;
 }
 
-guint8*
-mono_gc_get_card_table (int *shift_bits, gpointer *mask)
-{
-       return NULL;
-}
-
-#endif
-
 #endif /*HAVE_SGEN_GC*/