[runtime] Further removed code that doubted IMT
[mono.git] / mono / metadata / sgen-pinning.c
index d91692d42e94dee71dd2ce0463341091b44dc0e9..2d7a8ca498c81f01374ad692d400d423fc24e07b 100644 (file)
@@ -37,6 +37,7 @@ void
 sgen_init_pinning (void)
 {
        memset (pin_hash_filter, 0, sizeof (pin_hash_filter));
+       pin_queue.mem_type = INTERNAL_MEM_PIN_QUEUE;
 }
 
 void
@@ -59,50 +60,89 @@ sgen_pin_stage_ptr (void *ptr)
        sgen_pointer_queue_add (&pin_queue, ptr);
 }
 
+gboolean
+sgen_find_optimized_pin_queue_area (void *start, void *end, size_t *first_out, size_t *last_out)
+{
+       size_t first = sgen_pointer_queue_search (&pin_queue, start);
+       size_t last = sgen_pointer_queue_search (&pin_queue, end);
+       SGEN_ASSERT (0, last == pin_queue.next_slot || pin_queue.data [last] >= end, "Pin queue search gone awry");
+       *first_out = first;
+       *last_out = last;
+       return first != last;
+}
+
 void**
-sgen_find_optimized_pin_queue_area (void *start, void *end, size_t *num)
+sgen_pinning_get_entry (size_t index)
 {
-       size_t first, last;
-       first = sgen_pointer_queue_search (&pin_queue, start);
-       last = sgen_pointer_queue_search (&pin_queue, end);
-       *num = last - first;
-       if (first == last)
-               return NULL;
-       return pin_queue.data + first;
+       SGEN_ASSERT (0, index <= pin_queue.next_slot, "Pin queue entry out of range");
+       return &pin_queue.data [index];
 }
 
 void
 sgen_find_section_pin_queue_start_end (GCMemSection *section)
 {
        SGEN_LOG (6, "Pinning from section %p (%p-%p)", section, section->data, section->end_data);
-       section->pin_queue_start = sgen_find_optimized_pin_queue_area (section->data, section->end_data, &section->pin_queue_num_entries);
-       SGEN_LOG (6, "Found %zd pinning addresses in section %p", section->pin_queue_num_entries, section);
+
+       sgen_find_optimized_pin_queue_area (section->data, section->end_data,
+                       &section->pin_queue_first_entry, &section->pin_queue_last_entry);
+
+       SGEN_LOG (6, "Found %zd pinning addresses in section %p",
+                       section->pin_queue_last_entry - section->pin_queue_first_entry, section);
 }
 
 /*This will setup the given section for the while pin queue. */
 void
 sgen_pinning_setup_section (GCMemSection *section)
 {
-       section->pin_queue_start = pin_queue.data;
-       section->pin_queue_num_entries = pin_queue.next_slot;
+       section->pin_queue_first_entry = 0;
+       section->pin_queue_last_entry = pin_queue.next_slot;
 }
 
 void
 sgen_pinning_trim_queue_to_section (GCMemSection *section)
 {
-       pin_queue.next_slot = section->pin_queue_num_entries;
+       SGEN_ASSERT (0, section->pin_queue_first_entry == 0, "Pin queue trimming assumes the whole pin queue is used by the nursery");
+       pin_queue.next_slot = section->pin_queue_last_entry;
 }
 
+/*
+ * This is called when we've run out of memory during a major collection.
+ *
+ * After collecting potential pin entries and sorting the array, this is what it looks like:
+ *
+ * +--------------------+---------------------------------------------+--------------------+
+ * | major heap entries |               nursery entries               | major heap entries |
+ * +--------------------+---------------------------------------------+--------------------+
+ *
+ * Of course there might not be major heap entries before and/or after the nursery entries,
+ * depending on where the major heap sections are in the address space, and whether there
+ * were any potential pointers there.
+ *
+ * When we pin nursery objects, we compact the nursery part of the pin array, which leaves
+ * discarded entries after the ones that actually pointed to nursery objects:
+ *
+ * +--------------------+-----------------+---------------------------+--------------------+
+ * | major heap entries | nursery entries | discarded nursery entries | major heap entries |
+ * +--------------------+-----------------+---------------------------+--------------------+
+ *
+ * When, due to being out of memory, we late pin more objects, the pin array looks like
+ * this:
+ *
+ * +--------------------+-----------------+---------------------------+--------------------+--------------+
+ * | major heap entries | nursery entries | discarded nursery entries | major heap entries | late entries |
+ * +--------------------+-----------------+---------------------------+--------------------+--------------+
+ *
+ * This function gets rid of the discarded nursery entries by nulling them out.  Note that
+ * we can late pin objects not only in the nursery but also in the major heap, which happens
+ * when evacuation fails.
+ */
 void
 sgen_pin_queue_clear_discarded_entries (GCMemSection *section, size_t max_pin_slot)
 {
-       void **start = section->pin_queue_start + section->pin_queue_num_entries;
-       void **end = pin_queue.data + max_pin_slot;
+       void **start = sgen_pinning_get_entry (section->pin_queue_last_entry);
+       void **end = sgen_pinning_get_entry (max_pin_slot);
        void *addr;
 
-       if (!start)
-               return;
-
        for (; start < end; ++start) {
                addr = *start;
                if ((char*)addr < section->data || (char*)addr > section->end_data)
@@ -142,10 +182,8 @@ struct _CementHashEntry {
 };
 
 static CementHashEntry cement_hash [SGEN_CEMENT_HASH_SIZE];
-static CementHashEntry cement_hash_concurrent [SGEN_CEMENT_HASH_SIZE];
 
 static gboolean cement_enabled = TRUE;
-static gboolean cement_concurrent = FALSE;
 
 void
 sgen_cement_init (gboolean enabled)
@@ -156,48 +194,10 @@ sgen_cement_init (gboolean enabled)
 void
 sgen_cement_reset (void)
 {
-       SGEN_ASSERT (1, !cement_concurrent, "Concurrent cementing cannot simply be reset");
-
        memset (cement_hash, 0, sizeof (cement_hash));
        binary_protocol_cement_reset ();
 }
 
-/*
- * The reason we cannot simply reset cementing at the start of a
- * concurrent collection is that the nursery collections running
- * concurrently must keep pinning the cemented objects, because we
- * don't have the global remsets that point to them anymore - if the
- * nursery collector moved the cemented objects, we'd have invalid
- * pointers in the major heap.
- *
- * What we do instead is to reset cementing at the start of concurrent
- * collections in such a way that nursery collections happening during
- * the major collection still pin the formerly cemented objects.  We
- * have a shadow cementing table for that purpose.  The nursery
- * collections still work with the old cementing table, while the
- * major collector builds up a new cementing table, adding global
- * remsets whenever needed like usual.  When the major collector
- * finishes, the old cementing table is replaced by the new one.
- */
-
-void
-sgen_cement_concurrent_start (void)
-{
-       SGEN_ASSERT (1, !cement_concurrent, "Concurrent cementing has already been started");
-       cement_concurrent = TRUE;
-
-       memset (cement_hash_concurrent, 0, sizeof (cement_hash));
-}
-
-void
-sgen_cement_concurrent_finish (void)
-{
-       SGEN_ASSERT (1, cement_concurrent, "Concurrent cementing hasn't been started");
-       cement_concurrent = FALSE;
-
-       memcpy (cement_hash, cement_hash_concurrent, sizeof (cement_hash));
-}
-
 gboolean
 sgen_cement_lookup (char *obj)
 {
@@ -222,20 +222,11 @@ sgen_cement_lookup_or_register (char *obj)
 {
        guint hv;
        int i;
-       CementHashEntry *hash;
-       gboolean concurrent_cementing = sgen_concurrent_collection_in_progress ();
+       CementHashEntry *hash = cement_hash;
 
        if (!cement_enabled)
                return FALSE;
 
-       if (concurrent_cementing)
-               SGEN_ASSERT (5, cement_concurrent, "Cementing wasn't inited with concurrent flag");
-
-       if (concurrent_cementing)
-               hash = cement_hash_concurrent;
-       else
-               hash = cement_hash;
-
        hv = mono_aligned_addr_hash (obj);
        i = SGEN_CEMENT_HASH (hv);
 
@@ -253,6 +244,10 @@ sgen_cement_lookup_or_register (char *obj)
 
        ++hash [i].count;
        if (hash [i].count == SGEN_CEMENT_THRESHOLD) {
+               SGEN_ASSERT (9, sgen_get_current_collection_generation () >= 0, "We can only cement objects when we're in a collection pause.");
+               SGEN_ASSERT (9, SGEN_OBJECT_IS_PINNED (obj), "Can only cement pinned objects");
+               SGEN_CEMENT_OBJECT (obj);
+
                if (G_UNLIKELY (MONO_GC_OBJ_CEMENTED_ENABLED())) {
                        MonoVTable *vt G_GNUC_UNUSED = (MonoVTable*)SGEN_LOAD_VTABLE (obj);
                        MONO_GC_OBJ_CEMENTED ((mword)obj, sgen_safe_object_get_size ((MonoObject*)obj),
@@ -265,20 +260,31 @@ sgen_cement_lookup_or_register (char *obj)
        return FALSE;
 }
 
-void
-sgen_cement_iterate (IterateObjectCallbackFunc callback, void *callback_data)
+static void
+pin_from_hash (CementHashEntry *hash, gboolean has_been_reset)
 {
        int i;
        for (i = 0; i < SGEN_CEMENT_HASH_SIZE; ++i) {
-               if (!cement_hash [i].count)
+               if (!hash [i].count)
                        continue;
 
-               SGEN_ASSERT (5, cement_hash [i].count >= SGEN_CEMENT_THRESHOLD, "Cementing hash inconsistent");
+               if (has_been_reset)
+                       SGEN_ASSERT (5, hash [i].count >= SGEN_CEMENT_THRESHOLD, "Cementing hash inconsistent");
 
-               callback (cement_hash [i].obj, 0, callback_data);
+               sgen_pin_stage_ptr (hash [i].obj);
+               binary_protocol_cement_stage (hash [i].obj);
+               /* FIXME: do pin stats if enabled */
+
+               SGEN_CEMENT_OBJECT (hash [i].obj);
        }
 }
 
+void
+sgen_pin_cemented_objects (void)
+{
+       pin_from_hash (cement_hash, TRUE);
+}
+
 void
 sgen_cement_clear_below_threshold (void)
 {