[Socket] Move IOControl related methods to their region
[mono.git] / mono / metadata / sgen-cardtable.c
index 4fad8b6678afb2e45404160fd6ecdcfc9b653e7d..3b57a1a974a786f1261dc15fe1f7fc4961827180 100644 (file)
@@ -4,30 +4,23 @@
  * Author:
  *     Rodrigo Kumpera (rkumpera@novell.com)
  *
- * SGen is licensed under the terms of the MIT X11 license
- *
  * Copyright 2001-2003 Ximian, Inc
  * Copyright 2003-2010 Novell, Inc.
  * Copyright 2011 Xamarin Inc (http://www.xamarin.com)
- * 
- * Permission is hereby granted, free of charge, to any person obtaining
- * a copy of this software and associated documentation files (the
- * "Software"), to deal in the Software without restriction, including
- * without limitation the rights to use, copy, modify, merge, publish,
- * distribute, sublicense, and/or sell copies of the Software, and to
- * permit persons to whom the Software is furnished to do so, subject to
- * the following conditions:
- * 
- * The above copyright notice and this permission notice shall be
- * included in all copies or substantial portions of the Software.
- * 
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
- * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
- * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
- * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
- * LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
- * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
- * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+ * Copyright (C) 2012 Xamarin Inc
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License 2.0 as published by the Free Software Foundation;
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License 2.0 along with this library; if not, write to the Free
+ * Software Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
 
 #include "config.h"
 #include "metadata/sgen-gc.h"
 #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;
-long long scanned_cards;
-long long scanned_objects;
-long long remarked_cards;
+guint64 marked_cards;
+guint64 scanned_cards;
+guint64 scanned_objects;
+guint64 remarked_cards;
 
-static long long los_marked_cards;
-static long long large_objects;
-static long long bloby_objects;
-static long long los_array_cards;
-static long long los_array_remsets;
+static guint64 los_marked_cards;
+static guint64 large_objects;
+static guint64 bloby_objects;
+static guint64 los_array_cards;
+static guint64 los_array_remsets;
 
 #endif
-static long long major_card_scan_time;
-static long long los_card_scan_time;
+static guint64 major_card_scan_time;
+static guint64 los_card_scan_time;
 
-static long long last_major_scan_time;
-static long long last_los_scan_time;
+static guint64 last_major_scan_time;
+static guint64 last_los_scan_time;
 
 static void sgen_card_tables_collect_stats (gboolean begin);
 
@@ -89,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);
 }
@@ -98,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); 
 }
@@ -117,8 +113,8 @@ 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))
+                       SGEN_UPDATE_REFERENCE_ALLOW_NULL (dest, value);
+                       if (need_mod_union || sgen_ptr_in_nursery (value))
                                sgen_card_table_mark_address ((mword)dest);
                        sgen_dummy_use (value);
                }
@@ -126,8 +122,8 @@ sgen_card_table_wbarrier_arrayref_copy (gpointer dest_ptr, gpointer src_ptr, int
                gpointer *end = dest + count;
                for (; dest < end; ++src, ++dest) {
                        gpointer value = *src;
-                       *dest = value;
-                       if (sgen_ptr_in_nursery (value))
+                       SGEN_UPDATE_REFERENCE_ALLOW_NULL (dest, value);
+                       if (need_mod_union || sgen_ptr_in_nursery (value))
                                sgen_card_table_mark_address ((mword)dest);
                        sgen_dummy_use (value);
                }
@@ -146,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;
@@ -166,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
@@ -190,10 +186,11 @@ guint8 *sgen_shadow_cardtable;
 #define SGEN_CARDTABLE_END (sgen_cardtable + CARD_COUNT_IN_BYTES)
 
 static gboolean
-sgen_card_table_region_begin_scanning (mword start, mword end)
+sgen_card_table_region_begin_scanning (mword start, mword size)
 {
+       mword end = start + size;
        /*XXX this can be improved to work on words and have a single loop induction var */
-       while (start <= end) {
+       while (start < end) {
                if (sgen_card_table_card_begin_scanning (start))
                        return TRUE;
                start += CARD_SIZE_IN_BYTES;
@@ -244,7 +241,7 @@ sgen_card_table_get_card_data (guint8 *data_dest, mword address, mword cards)
 #endif
        }
 
-       return mask;
+       return mask != 0;
 }
 
 void*
@@ -285,6 +282,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
@@ -294,7 +368,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;
 
@@ -311,7 +387,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);
@@ -334,7 +412,7 @@ clear_cards (mword start, mword size)
 #endif
 
 static void
-sgen_card_table_prepare_for_major_collection (void)
+sgen_card_table_clear_cards (void)
 {
        /*XXX we could do this in 2 ways. using mincore or iterating over all sections/los objects */
        sgen_major_collector_iterate_live_block_ranges (clear_cards);
@@ -348,7 +426,7 @@ sgen_card_table_finish_minor_collection (void)
 }
 
 static void
-sgen_card_table_finish_scan_remsets (void *start_nursery, void *end_nursery, SgenGrayQueue *queue)
+sgen_card_table_scan_remsets (SgenGrayQueue *queue)
 {
        SGEN_TV_DECLARE (atv);
        SGEN_TV_DECLARE (btv);
@@ -362,14 +440,14 @@ sgen_card_table_finish_scan_remsets (void *start_nursery, void *end_nursery, Sge
        sgen_los_iterate_live_block_ranges (move_cards_to_shadow_table);
 
        /*Then we clear*/
-       sgen_card_table_prepare_for_major_collection ();
+       sgen_card_table_clear_cards ();
 #endif
        SGEN_TV_GETTIME (atv);
        sgen_major_collector_scan_card_table (queue);
        SGEN_TV_GETTIME (btv);
        last_major_scan_time = SGEN_TV_ELAPSED (atv, btv); 
        major_card_scan_time += last_major_scan_time;
-       sgen_los_scan_card_table (queue);
+       sgen_los_scan_card_table (FALSE, queue);
        SGEN_TV_GETTIME (atv);
        last_los_scan_time = SGEN_TV_ELAPSED (btv, atv);
        los_card_scan_time += last_los_scan_time;
@@ -378,6 +456,9 @@ sgen_card_table_finish_scan_remsets (void *start_nursery, void *end_nursery, Sge
 guint8*
 mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 {
+#ifndef MANAGED_WBARRIER
+       return NULL;
+#else
        if (!sgen_cardtable)
                return NULL;
 
@@ -389,25 +470,16 @@ mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 #endif
 
        return sgen_cardtable;
+#endif
 }
 
-#if 0
-static void
-collect_faulted_cards (void)
+gboolean
+mono_gc_card_table_nursery_check (void)
 {
-#define CARD_PAGES (CARD_COUNT_IN_BYTES / 4096)
-       int i, count = 0;
-       unsigned char faulted [CARD_PAGES] = { 0 };
-       mincore (sgen_cardtable, CARD_COUNT_IN_BYTES, faulted);
-
-       for (i = 0; i < CARD_PAGES; ++i) {
-               if (faulted [i])
-                       ++count;
-       }
-
-       printf ("TOTAL card pages %d faulted %d\n", CARD_PAGES, count);
+       return !major_collector.is_concurrent;
 }
 
+#if 0
 void
 sgen_card_table_dump_obj_card (char *object, size_t size, void *dummy)
 {
@@ -441,7 +513,7 @@ find_card_offset (mword card)
        return (__builtin_ffsll (GUINT64_TO_LE(card)) - 1) / 8;
 #else
        int i;
-       guint8 *ptr = (guint *) &card;
+       guint8 *ptr = (guint8 *) &card;
        for (i = 0; i < sizeof (mword); ++i) {
                if (ptr[i])
                        return i;
@@ -485,15 +557,17 @@ find_next_card (guint8 *card_data, guint8 *end)
 }
 
 void
-sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, SgenGrayQueue *queue)
+sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, gboolean mod_union, SgenGrayQueue *queue)
 {
        MonoVTable *vt = (MonoVTable*)SGEN_LOAD_VTABLE (obj);
        MonoClass *klass = vt->klass;
 
        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 +576,7 @@ sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, Sgen
                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 +586,13 @@ sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, Sgen
                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,11 +615,11 @@ 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 *elem;
+                       char *first_elem, *elem;
 
                        HEAVY_STAT (++los_marked_cards);
 
@@ -552,27 +633,29 @@ LOOP_HEAD:
                        else
                                index = ARRAY_OBJ_INDEX (start, obj, elem_size);
 
-                       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;
 
                                HEAVY_STAT (++los_array_cards);
                                for (; elem < card_end; elem += SIZEOF_VOID_P) {
                                        gpointer new, old = *(gpointer*)elem;
-                                       if (G_UNLIKELY (sgen_ptr_in_nursery (old))) {
+                                       if ((mod_union && old) || G_UNLIKELY (sgen_ptr_in_nursery (old))) {
                                                HEAVY_STAT (++los_array_remsets);
                                                copy_func ((void**)elem, queue);
                                                new = *(gpointer*)elem;
                                                if (G_UNLIKELY (sgen_ptr_in_nursery (new)))
-                                                       sgen_add_to_global_remset (elem);
+                                                       sgen_add_to_global_remset (elem, new);
                                        }
                                }
                        }
+
+                       binary_protocol_card_scan (first_elem, elem - first_elem);
                }
 
 #ifdef SGEN_HAVE_OVERLAPPING_CARDS
@@ -589,17 +672,19 @@ LOOP_HEAD:
                HEAVY_STAT (++bloby_objects);
                if (cards) {
                        if (sgen_card_table_is_range_marked (cards, (mword)obj, block_obj_size))
-                               sgen_get_current_object_ops ()->scan_object (obj, queue);
+                               sgen_get_current_object_ops ()->scan_object (obj, sgen_obj_get_descriptor (obj), queue);
                } else if (sgen_card_table_region_begin_scanning ((mword)obj, block_obj_size)) {
-                       sgen_get_current_object_ops ()->scan_object (obj, queue);
+                       sgen_get_current_object_ops ()->scan_object (obj, sgen_obj_get_descriptor (obj), queue);
                }
+
+               binary_protocol_card_scan (obj, sgen_safe_object_get_size ((MonoObject*)obj));
        }
 }
 
 #ifdef CARDTABLE_STATS
 
 typedef struct {
-       int total, marked, remarked;    
+       int total, marked, remarked, gc_marked; 
 } card_stats;
 
 static card_stats major_stats, los_stats;
@@ -610,9 +695,12 @@ count_marked_cards (mword start, mword size)
 {
        mword end = start + size;
        while (start <= end) {
+               guint8 card = *sgen_card_table_get_card_address (start);
                ++cur_stats->total;
-               if (sgen_card_table_address_is_marked (start))
+               if (card)
                        ++cur_stats->marked;
+               if (card == 2)
+                       ++cur_stats->gc_marked;
                start += CARD_SIZE_IN_BYTES;
        }
 }
@@ -622,8 +710,10 @@ count_remarked_cards (mword start, mword size)
 {
        mword end = start + size;
        while (start <= end) {
-               if (sgen_card_table_address_is_marked (start))
+               if (sgen_card_table_address_is_marked (start)) {
                        ++cur_stats->remarked;
+                       *sgen_card_table_get_card_address (start) = 2;
+               }
                start += CARD_SIZE_IN_BYTES;
        }
 }
@@ -643,19 +733,19 @@ sgen_card_tables_collect_stats (gboolean begin)
                sgen_los_iterate_live_block_ranges (count_marked_cards);
        } else {
                cur_stats = &major_stats;
-               sgen_major_collector_iterate_live_block_ranges (count_marked_cards);
+               sgen_major_collector_iterate_live_block_ranges (count_remarked_cards);
                cur_stats = &los_stats;
                sgen_los_iterate_live_block_ranges (count_remarked_cards);
-               printf ("cards major (t %d m %d r %d)  los (t %d m %d r %d) major_scan %.2fms los_scan %.2fms\n", 
-                       major_stats.total, major_stats.marked, major_stats.remarked,
-                       los_stats.total, los_stats.marked, los_stats.remarked,
-                       last_major_scan_time / 1000.0, last_los_scan_time / 1000.0);
+               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 / 10000.0f, last_los_scan_time / 10000.0f);
        }
 #endif
 }
 
 void
-sgen_card_table_init (SgenRemeberedSet *remset)
+sgen_card_table_init (SgenRememberedSet *remset)
 {
        sgen_cardtable = sgen_alloc_os_memory (CARD_COUNT_IN_BYTES, SGEN_ALLOC_INTERNAL | SGEN_ALLOC_ACTIVATE, "card table");
 
@@ -664,19 +754,19 @@ sgen_card_table_init (SgenRemeberedSet *remset)
 #endif
 
 #ifdef HEAVY_STATISTICS
-       mono_counters_register ("marked cards", MONO_COUNTER_GC | MONO_COUNTER_LONG, &marked_cards);
-       mono_counters_register ("scanned cards", MONO_COUNTER_GC | MONO_COUNTER_LONG, &scanned_cards);
-       mono_counters_register ("remarked cards", MONO_COUNTER_GC | MONO_COUNTER_LONG, &remarked_cards);
-
-       mono_counters_register ("los marked cards", MONO_COUNTER_GC | MONO_COUNTER_LONG, &los_marked_cards);
-       mono_counters_register ("los array cards scanned ", MONO_COUNTER_GC | MONO_COUNTER_LONG, &los_array_cards);
-       mono_counters_register ("los array remsets", MONO_COUNTER_GC | MONO_COUNTER_LONG, &los_array_remsets);
-       mono_counters_register ("cardtable scanned objects", MONO_COUNTER_GC | MONO_COUNTER_LONG, &scanned_objects);
-       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);
+       mono_counters_register ("marked cards", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &marked_cards);
+       mono_counters_register ("scanned cards", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &scanned_cards);
+       mono_counters_register ("remarked cards", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &remarked_cards);
+
+       mono_counters_register ("los marked cards", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &los_marked_cards);
+       mono_counters_register ("los array cards scanned ", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &los_array_cards);
+       mono_counters_register ("los array remsets", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &los_array_remsets);
+       mono_counters_register ("cardtable scanned objects", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &scanned_objects);
+       mono_counters_register ("cardtable large objects", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &large_objects);
+       mono_counters_register ("cardtable bloby objects", MONO_COUNTER_GC | MONO_COUNTER_ULONG, &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_ULONG | MONO_COUNTER_TIME, &major_card_scan_time);
+       mono_counters_register ("cardtable los scan time", MONO_COUNTER_GC | MONO_COUNTER_ULONG | MONO_COUNTER_TIME, &los_card_scan_time);
 
 
        remset->wbarrier_set_field = sgen_card_table_wbarrier_set_field;
@@ -687,34 +777,15 @@ sgen_card_table_init (SgenRemeberedSet *remset)
        remset->wbarrier_generic_nostore = sgen_card_table_wbarrier_generic_nostore;
        remset->record_pointer = sgen_card_table_record_pointer;
 
-       remset->finish_scan_remsets = sgen_card_table_finish_scan_remsets;
+       remset->scan_remsets = sgen_card_table_scan_remsets;
 
        remset->finish_minor_collection = sgen_card_table_finish_minor_collection;
-       remset->prepare_for_major_collection = sgen_card_table_prepare_for_major_collection;
+       remset->clear_cards = sgen_card_table_clear_cards;
 
        remset->find_address = sgen_card_table_find_address;
-}
-
-#else
-
-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 ();
-}
+       remset->find_address_with_cards = sgen_card_table_find_address_with_cards;
 
-guint8*
-mono_gc_get_card_table (int *shift_bits, gpointer *mask)
-{
-       return NULL;
+       need_mod_union = sgen_get_major_collector ()->is_concurrent;
 }
 
-#endif
-
 #endif /*HAVE_SGEN_GC*/