Move dummy_use to a header and give it a sucky MSVC implementation.
[mono.git] / mono / metadata / sgen-cardtable.c
index 8f44be37ab6eb096fab214c0392b6fbcf56d41e7..af04d1584d1f666f28813d5233fdb2495941da22 100644 (file)
@@ -8,6 +8,7 @@
  *
  * 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
  * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
  */
 
+#include "config.h"
+#ifdef HAVE_SGEN_GC
+
+#include "metadata/sgen-gc.h"
+#include "metadata/sgen-cardtable.h"
+#include "utils/mono-counters.h"
+#include "utils/mono-time.h"
+#include "utils/mono-memory-model.h"
+
 #ifdef SGEN_HAVE_CARDTABLE
 
 //#define CARDTABLE_STATS
 
 #include <unistd.h>
+#ifdef HAVE_SYS_MMAN_H
 #include <sys/mman.h>
+#endif
 #include <sys/types.h>
 
 guint8 *sgen_cardtable;
 
+
+#ifdef HEAVY_STATISTICS
+long long marked_cards;
+long long scanned_cards;
+long long scanned_objects;
+long long 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;
+
+#endif
+static long long major_card_scan_time;
+static long long los_card_scan_time;
+
+static long long last_major_scan_time;
+static long long last_los_scan_time;
+
+/*WARNING: This function returns the number of cards regardless of overflow in case of overlapping cards.*/
 static mword
 cards_in_range (mword address, mword size)
 {
-       mword end = address + size;
+       mword end = address + MAX (1, size) - 1;
        return (end >> CARD_BITS) - (address >> CARD_BITS) + 1;
 }
 
+void
+mono_sgen_card_table_wbarrier_set_field (MonoObject *obj, gpointer field_ptr, MonoObject* value)
+{
+       *(void**)field_ptr = value;
+       if (mono_sgen_ptr_in_nursery (value))
+               sgen_card_table_mark_address ((mword)field_ptr);
+       mono_sgen_dummy_use (value);
+}
+
+void
+mono_sgen_card_table_wbarrier_set_arrayref (MonoArray *arr, gpointer slot_ptr, MonoObject* value)
+{
+       *(void**)slot_ptr = value;
+       if (mono_sgen_ptr_in_nursery (value))
+               sgen_card_table_mark_address ((mword)slot_ptr);
+       mono_sgen_dummy_use (value);    
+}
+
+void
+mono_sgen_card_table_wbarrier_arrayref_copy (gpointer dest_ptr, gpointer src_ptr, int count)
+{
+       gpointer *dest = dest_ptr;
+       gpointer *src = src_ptr;
+
+       /*overlapping that required backward copying*/
+       if (src < dest && (src + count) > dest) {
+               gpointer *start = dest;
+               dest += count - 1;
+               src += count - 1;
+
+               for (; dest >= start; --src, --dest) {
+                       gpointer value = *src;
+                       *dest = value;
+                       if (mono_sgen_ptr_in_nursery (value))
+                               sgen_card_table_mark_address ((mword)dest);
+                       mono_sgen_dummy_use (value);
+               }
+       } else {
+               gpointer *end = dest + count;
+               for (; dest < end; ++src, ++dest) {
+                       gpointer value = *src;
+                       *dest = value;
+                       if (mono_sgen_ptr_in_nursery (value))
+                               sgen_card_table_mark_address ((mword)dest);
+                       mono_sgen_dummy_use (value);
+               }
+       }       
+}
+
+void
+mono_sgen_card_table_wbarrier_value_copy (gpointer dest, gpointer src, int count, MonoClass *klass)
+{
+       size_t element_size = mono_class_value_size (klass, NULL);
+       size_t size = count * element_size;
+
+#ifdef DISABLE_CRITICAL_REGION
+       LOCK_GC;
+#else
+       TLAB_ACCESS_INIT;
+       ENTER_CRITICAL_REGION;
+#endif
+       mono_gc_memmove (dest, src, size);
+       sgen_card_table_mark_range ((mword)dest, size);
+#ifdef DISABLE_CRITICAL_REGION
+       UNLOCK_GC;
+#else
+       EXIT_CRITICAL_REGION;
+#endif
+}
+
+void
+mono_sgen_card_table_wbarrier_object_copy (MonoObject* obj, MonoObject *src)
+{
+       int size;
+       TLAB_ACCESS_INIT;
+
+       size = mono_object_class (obj)->instance_size;
+
+#ifdef DISABLE_CRITICAL_REGION
+       LOCK_GC;
+#else
+       ENTER_CRITICAL_REGION;
+#endif
+       mono_gc_memmove ((char*)obj + sizeof (MonoObject), (char*)src + sizeof (MonoObject),
+                       size - sizeof (MonoObject));
+       sgen_card_table_mark_range ((mword)obj, size);
+#ifdef DISABLE_CRITICAL_REGION
+       UNLOCK_GC;
+#else
+       EXIT_CRITICAL_REGION;
+#endif 
+}
+
+void
+mono_sgen_card_table_wbarrier_generic_nostore (gpointer ptr)
+{
+       sgen_card_table_mark_address ((mword)ptr);      
+}
+
 #ifdef SGEN_HAVE_OVERLAPPING_CARDS
 
 guint8 *sgen_shadow_cardtable;
 
+#define SGEN_SHADOW_CARDTABLE_END (sgen_shadow_cardtable + CARD_COUNT_IN_BYTES)
+#define SGEN_CARDTABLE_END (sgen_cardtable + CARD_COUNT_IN_BYTES)
+
 static gboolean
 sgen_card_table_region_begin_scanning (mword start, mword end)
 {
+       /*XXX this can be improved to work on words and have a single loop induction var */
        while (start <= end) {
                if (sgen_card_table_card_begin_scanning (start))
                        return TRUE;
@@ -70,6 +206,7 @@ sgen_card_table_region_begin_scanning (mword start, mword size)
        guint8 *card = sgen_card_table_get_card_address (start);
        guint8 *end = card + cards_in_range (start, size);
 
+       /*XXX this can be improved to work on words and have a branchless body */
        while (card != end) {
                if (*card++) {
                        res = TRUE;
@@ -93,31 +230,19 @@ sgen_card_table_get_card_data (guint8 *data_dest, mword address, mword cards)
        mword *end = (mword*)(data_dest + cards);
        mword mask = 0;
 
-       for (; dest < end; ++dest) {
+       for (; dest < end; ++dest, ++start) {
                mword v = *start;
                *dest = v;
                mask |= v;
 
 #ifndef SGEN_HAVE_OVERLAPPING_CARDS
-               *start++ = 0;
+               *start = 0;
 #endif
        }
 
        return mask;
 }
 
-static gboolean
-sgen_card_table_address_is_marked (mword address)
-{
-       return *sgen_card_table_get_card_address (address) != 0;
-}
-
-void
-sgen_card_table_mark_address (mword address)
-{
-       *sgen_card_table_get_card_address (address) = 1;
-}
-
 void*
 sgen_card_table_align_pointer (void *ptr)
 {
@@ -127,93 +252,136 @@ sgen_card_table_align_pointer (void *ptr)
 void
 sgen_card_table_mark_range (mword address, mword size)
 {
-       mword end = address + size;
-       do {
-               sgen_card_table_mark_address (address);
-               address += CARD_SIZE_IN_BYTES;
-       } while (address < end);
+       memset (sgen_card_table_get_card_address (address), 1, cards_in_range (address, size));
 }
 
 static gboolean
-sgen_card_table_is_range_marked (guint8 *cards, mword size)
+sgen_card_table_is_range_marked (guint8 *cards, mword address, mword size)
 {
-       mword start = 0;
-       while (start <= size) {
+       guint8 *end = cards + cards_in_range (address, size);
+
+       /*This is safe since this function is only called by code that only passes continuous card blocks*/
+       while (cards != end) {
                if (*cards++)
                        return TRUE;
-               start += CARD_SIZE_IN_BYTES;
        }
        return FALSE;
 
 }
 
-static void
-card_table_init (void)
+void
+sgen_card_table_init (void)
 {
        sgen_cardtable = mono_sgen_alloc_os_memory (CARD_COUNT_IN_BYTES, TRUE);
 
 #ifdef SGEN_HAVE_OVERLAPPING_CARDS
        sgen_shadow_cardtable = mono_sgen_alloc_os_memory (CARD_COUNT_IN_BYTES, TRUE);
 #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);
+#endif
+       mono_counters_register ("cardtable major scan time", MONO_COUNTER_GC | MONO_COUNTER_LONG, &major_card_scan_time);
+       mono_counters_register ("cardtable los scan time", MONO_COUNTER_GC | MONO_COUNTER_LONG, &los_card_scan_time);
 }
 
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+
+static void
+move_cards_to_shadow_table (mword start, mword size)
+{
+       guint8 *from = sgen_card_table_get_card_address (start);
+       guint8 *to = sgen_card_table_get_shadow_card_address (start);
+       size_t bytes = cards_in_range (start, size);
 
-void los_scan_card_table (GrayQueue *queue);
-void los_iterate_live_block_ranges (sgen_cardtable_block_callback callback);
+       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;
 
+               memcpy (to, from, first_chunk);
+               memcpy (sgen_shadow_cardtable, sgen_cardtable, second_chunk);
+       } else {
+               memcpy (to, from, bytes);
+       }
+}
 
 static void
 clear_cards (mword start, mword size)
 {
-       memset (sgen_card_table_get_card_address (start), 0, size >> CARD_BITS);
+       guint8 *addr = sgen_card_table_get_card_address (start);
+       size_t bytes = cards_in_range (start, size);
+
+       if (addr + bytes > SGEN_CARDTABLE_END) {
+               size_t first_chunk = SGEN_CARDTABLE_END - addr;
+
+               memset (addr, 0, first_chunk);
+               memset (sgen_cardtable, 0, bytes - first_chunk);
+       } else {
+               memset (addr, 0, bytes);
+       }
 }
 
-#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+
+#else
 
 static void
-move_cards_to_shadow_table (mword start, mword size)
+clear_cards (mword start, mword size)
 {
-       guint8 *from = sgen_card_table_get_card_address (start);
-       guint8 *to = sgen_card_table_get_shadow_card_address (start);
-       size_t bytes = cards_in_range (start, size);
-       memcpy (to, from, bytes);
+       memset (sgen_card_table_get_card_address (start), 0, cards_in_range (start, size));
 }
 
+
 #endif
 
-static void
-card_table_clear (void)
+void
+sgen_card_table_clear (void)
 {
        /*XXX we could do this in 2 ways. using mincore or iterating over all sections/los objects */
-       if (use_cardtable) {
-               major_collector.iterate_live_block_ranges (clear_cards);
-               los_iterate_live_block_ranges (clear_cards);
-       }
+       sgen_major_collector_iterate_live_block_ranges (clear_cards);
+       mono_sgen_los_iterate_live_block_ranges (clear_cards);
 }
-static void
-scan_from_card_tables (void *start_nursery, void *end_nursery, GrayQueue *queue)
+
+void
+sgen_scan_from_card_tables (void *start_nursery, void *end_nursery, SgenGrayQueue *queue)
 {
-       if (use_cardtable) {
+       SGEN_TV_DECLARE (atv);
+       SGEN_TV_DECLARE (btv);
+
 #ifdef SGEN_HAVE_OVERLAPPING_CARDS
+       /*FIXME we should have a bit on each block/los object telling if the object have marked cards.*/
        /*First we copy*/
-       major_collector.iterate_live_block_ranges (move_cards_to_shadow_table);
-       los_iterate_live_block_ranges (move_cards_to_shadow_table);
+       sgen_major_collector_iterate_live_block_ranges (move_cards_to_shadow_table);
+       mono_sgen_los_iterate_live_block_ranges (move_cards_to_shadow_table);
 
        /*Then we clear*/
-       card_table_clear ();
+       sgen_card_table_clear ();
 #endif
-               major_collector.scan_card_table (queue);
-               los_scan_card_table (queue);
-       }
+       SGEN_TV_GETTIME (atv);
+       sgen_major_collector_scan_card_table (queue);
+       SGEN_TV_GETTIME (btv);
+       last_major_scan_time = SGEN_TV_ELAPSED_MS (atv, btv); 
+       major_card_scan_time += last_major_scan_time;
+       mono_sgen_los_scan_card_table (queue);
+       SGEN_TV_GETTIME (atv);
+       last_los_scan_time = SGEN_TV_ELAPSED_MS (btv, atv);
+       los_card_scan_time += last_los_scan_time;
 }
 
 guint8*
 mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 {
-       if (!use_cardtable)
+       if (!sgen_cardtable)
                return NULL;
 
-       g_assert (sgen_cardtable);
        *shift_bits = CARD_BITS;
 #ifdef SGEN_HAVE_OVERLAPPING_CARDS
        *mask = (gpointer)CARD_MASK;
@@ -224,6 +392,7 @@ mono_gc_get_card_table (int *shift_bits, gpointer *mask)
        return sgen_cardtable;
 }
 
+#if 0
 static void
 collect_faulted_cards (void)
 {
@@ -240,92 +409,210 @@ collect_faulted_cards (void)
        printf ("TOTAL card pages %d faulted %d\n", CARD_PAGES, count);
 }
 
+void
+sgen_card_table_dump_obj_card (char *object, size_t size, void *dummy)
+{
+       guint8 *start = sgen_card_table_get_card_scan_address (object);
+       guint8 *end = start + cards_in_range (object, size);
+       int cnt = 0;
+       printf ("--obj %p %d cards [%p %p]--", object, size, start, end);
+       for (; start < end; ++start) {
+               if (cnt == 0)
+                       printf ("\n\t[%p] ", start);
+               printf ("%x ", *start);
+               ++cnt;
+               if (cnt == 8)
+                       cnt = 0;
+       }
+       printf ("\n");
+}
+#endif
+
+#define MWORD_MASK (sizeof (mword) - 1)
+
+static inline int
+find_card_offset (mword card)
+{
+/*XXX Use assembly as this generates some pretty bad code */
+#if defined(__i386__) && defined(__GNUC__)
+       return  (__builtin_ffs (card) - 1) / 8;
+#elif defined(__x86_64__) && defined(__GNUC__)
+       return (__builtin_ffsll (card) - 1) / 8;
+#elif defined(__s390x__)
+       return (__builtin_ffsll (GUINT64_TO_LE(card)) - 1) / 8;
+#else
+       int i;
+       guint8 *ptr = (guint *) &card;
+       for (i = 0; i < sizeof (mword); ++i) {
+               if (ptr[i])
+                       return i;
+       }
+       return 0;
+#endif
+}
+
+static guint8*
+find_next_card (guint8 *card_data, guint8 *end)
+{
+       mword *cards, *cards_end;
+       mword card;
+
+       while ((((mword)card_data) & MWORD_MASK) && card_data < end) {
+               if (*card_data)
+                       return card_data;
+               ++card_data;
+       }
+
+       if (card_data == end)
+               return end;
+
+       cards = (mword*)card_data;
+       cards_end = (mword*)((mword)end & ~MWORD_MASK);
+       while (cards < cards_end) {
+               card = *cards;
+               if (card)
+                       return (guint8*)cards + find_card_offset (card);
+               ++cards;
+       }
+
+       card_data = (guint8*)cards_end;
+       while (card_data < end) {
+               if (*card_data)
+                       return card_data;
+               ++card_data;
+       }
+
+       return end;
+}
 
 void
-sgen_cardtable_scan_object (char *obj, mword obj_size, guint8 *cards, SgenGrayQueue *queue)
+sgen_cardtable_scan_object (char *obj, mword block_obj_size, guint8 *cards, SgenGrayQueue *queue)
 {
-       MonoVTable *vt = (MonoVTable*)LOAD_VTABLE (obj);
+       MonoVTable *vt = (MonoVTable*)SGEN_LOAD_VTABLE (obj);
        MonoClass *klass = vt->klass;
+       CopyOrMarkObjectFunc copy_func = mono_sgen_get_copy_object ();
+       ScanObjectFunc scan_object_func = mono_sgen_get_minor_scan_object ();
+       ScanVTypeFunc scan_vtype_func = mono_sgen_get_minor_scan_vtype ();
 
-       if (!klass->has_references)
+       HEAVY_STAT (++large_objects);
+
+       if (!SGEN_VTABLE_HAS_REFERENCES (vt))
                return;
 
        if (vt->rank) {
-               guint8 *card_data;
+               guint8 *card_data, *card_base;
+               guint8 *card_data_end;
+               char *obj_start = sgen_card_table_align_pointer (obj);
+               mword obj_size = mono_sgen_par_object_get_size (vt, (MonoObject*)obj);
+               char *obj_end = obj + obj_size;
+               size_t card_count;
+               int extra_idx = 0;
+
                MonoArray *arr = (MonoArray*)obj;
                mword desc = (mword)klass->element_class->gc_descr;
-               char *start = sgen_card_table_align_pointer (obj);
-               char *end = obj + obj_size;
-               int size = mono_array_element_size (klass);
+               int elem_size = mono_array_element_size (klass);
 
-               g_assert (desc);
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+               guint8 *overflow_scan_end = NULL;
+#endif
 
                if (cards)
                        card_data = cards;
                else
                        card_data = sgen_card_table_get_card_scan_address ((mword)obj);
 
+               card_base = card_data;
+               card_count = cards_in_range ((mword)obj, obj_size);
+               card_data_end = card_data + card_count;
 
-               for (; start < end; start += CARD_SIZE_IN_BYTES, ++card_data) {
-                       char *elem, *card_end;
-                       uintptr_t index;
 
-                       if (!*card_data)
-                               continue;
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+               /*Check for overflow and if so, setup to scan in two steps*/
+               if (!cards && card_data_end >= SGEN_SHADOW_CARDTABLE_END) {
+                       overflow_scan_end = sgen_shadow_cardtable + (card_data_end - SGEN_SHADOW_CARDTABLE_END);
+                       card_data_end = SGEN_SHADOW_CARDTABLE_END;
+               }
+
+LOOP_HEAD:
+#endif
+
+               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;
+                       char *start = (char*)(obj_start + idx * CARD_SIZE_IN_BYTES);
+                       char *card_end = start + CARD_SIZE_IN_BYTES;
+                       char *elem;
+
+                       HEAVY_STAT (++los_marked_cards);
 
                        if (!cards)
                                sgen_card_table_prepare_card_for_scanning (card_data);
 
-                       card_end = start + CARD_SIZE_IN_BYTES;
-                       if (end < card_end)
-                               card_end = end;
+                       card_end = MIN (card_end, obj_end);
 
                        if (start <= (char*)arr->vector)
                                index = 0;
                        else
-                               index = ARRAY_OBJ_INDEX (start, obj, size);
+                               index = ARRAY_OBJ_INDEX (start, obj, elem_size);
 
-                       elem = (char*)mono_array_addr_with_size ((MonoArray*)obj, size, index);
+                       elem = (char*)mono_array_addr_with_size ((MonoArray*)obj, elem_size, index);
                        if (klass->element_class->valuetype) {
-                               while (elem < card_end) {
-                                       major_collector.minor_scan_vtype (elem, desc, nursery_start, nursery_next, queue);
-                                       elem += size;
-                               }
+                               for (; elem < card_end; elem += elem_size)
+                                       scan_vtype_func (elem, desc, queue);
                        } else {
-                               while (elem < card_end) {
+                               HEAVY_STAT (++los_array_cards);
+                               for (; elem < card_end; elem += SIZEOF_VOID_P) {
                                        gpointer new, old = *(gpointer*)elem;
-                                       if (old) {
-                                               major_collector.copy_object ((void**)elem, queue);
+                                       if (G_UNLIKELY (mono_sgen_ptr_in_nursery (old))) {
+                                               HEAVY_STAT (++los_array_remsets);
+                                               copy_func ((void**)elem, queue);
                                                new = *(gpointer*)elem;
-                                               if (G_UNLIKELY (ptr_in_nursery (new)))
+                                               if (G_UNLIKELY (mono_sgen_ptr_in_nursery (new)))
                                                        mono_sgen_add_to_global_remset (elem);
                                        }
-                                       elem += size;
                                }
                        }
                }
+
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+               if (overflow_scan_end) {
+                       extra_idx = card_data - card_base;
+                       card_base = card_data = sgen_shadow_cardtable;
+                       card_data_end = overflow_scan_end;
+                       overflow_scan_end = NULL;
+                       goto LOOP_HEAD;
+               }
+#endif
+
        } else {
+               HEAVY_STAT (++bloby_objects);
                if (cards) {
-                       if (sgen_card_table_is_range_marked (cards, obj_size))
-                               major_collector.minor_scan_object (obj, queue);
-               } else if (sgen_card_table_region_begin_scanning ((mword)obj, obj_size)) {
-                       major_collector.minor_scan_object (obj, queue);
+                       if (sgen_card_table_is_range_marked (cards, (mword)obj, block_obj_size))
+                               scan_object_func (obj, queue);
+               } else if (sgen_card_table_region_begin_scanning ((mword)obj, block_obj_size)) {
+                       scan_object_func (obj, queue);
                }
        }
 }
 
 #ifdef CARDTABLE_STATS
 
-static int total_cards, marked_cards, remarked_cards;
+typedef struct {
+       int total, marked, remarked;    
+} card_stats;
+
+static card_stats major_stats, los_stats;
+static card_stats *cur_stats;
 
 static void
 count_marked_cards (mword start, mword size)
 {
        mword end = start + size;
        while (start <= end) {
-               ++total_cards;
+               ++cur_stats->total;
                if (sgen_card_table_address_is_marked (start))
-                       ++marked_cards;
+                       ++cur_stats->marked;
                start += CARD_SIZE_IN_BYTES;
        }
 }
@@ -336,25 +623,33 @@ count_remarked_cards (mword start, mword size)
        mword end = start + size;
        while (start <= end) {
                if (sgen_card_table_address_is_marked (start))
-                       ++remarked_cards;
+                       ++cur_stats->remarked;
                start += CARD_SIZE_IN_BYTES;
        }
 }
 
 #endif
 
-static void
-card_tables_collect_starts (gboolean begin)
+void
+sgen_card_tables_collect_stats (gboolean begin)
 {
 #ifdef CARDTABLE_STATS
        if (begin) {
-               total_cards = marked_cards = remarked_cards = 0;
-               major_collector.iterate_live_block_ranges (count_marked_cards);
-               los_iterate_live_block_ranges (count_marked_cards);
+               memset (&major_stats, 0, sizeof (card_stats));
+               memset (&los_stats, 0, sizeof (card_stats));
+               cur_stats = &major_stats;
+               sgen_major_collector_iterate_live_block_ranges (count_marked_cards);
+               cur_stats = &los_stats;
+               mono_sgen_los_iterate_live_block_ranges (count_marked_cards);
        } else {
-               major_collector.iterate_live_block_ranges (count_marked_cards);
-               los_iterate_live_block_ranges (count_remarked_cards);
-               printf ("cards total %d marked %d remarked %d\n", total_cards, marked_cards, remarked_cards);
+               cur_stats = &major_stats;
+               sgen_major_collector_iterate_live_block_ranges (count_marked_cards);
+               cur_stats = &los_stats;
+               mono_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 %lld los_scan %lld\n", 
+                       major_stats.total, major_stats.marked, major_stats.remarked,
+                       los_stats.total, los_stats.marked, los_stats.remarked,
+                       last_major_scan_time, last_los_scan_time);
        }
 #endif
 }
@@ -373,12 +668,6 @@ sgen_card_table_mark_range (mword address, mword size)
        g_assert_not_reached ();
 }
 
-#define sgen_card_table_address_is_marked(p)   FALSE
-#define scan_from_card_tables(start,end,queue)
-#define card_table_clear()
-#define card_table_init()
-#define card_tables_collect_starts(begin)
-
 guint8*
 mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 {
@@ -386,3 +675,5 @@ mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 }
 
 #endif
+
+#endif /*HAVE_SGEN_GC*/
\ No newline at end of file