Move dummy_use to a header and give it a sucky MSVC implementation.
[mono.git] / mono / metadata / sgen-cardtable.c
index d8015ad2445cbc7a25fdd922975bb50b1c46fdac..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>
 
-#define CARD_COUNT_BITS (32 - 9)
-#define CARD_COUNT_IN_BYTES (1 << CARD_COUNT_BITS)
+guint8 *sgen_cardtable;
 
 
-static guint8 *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;
 
-guint8*
-sgen_card_table_get_card_address (mword address)
+#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)
 {
-       return cardtable + (address >> CARD_BITS);
+       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
-sgen_card_table_mark_address (mword address)
+mono_sgen_card_table_wbarrier_set_arrayref (MonoArray *arr, gpointer slot_ptr, MonoObject* value)
 {
-       *sgen_card_table_get_card_address (address) = 1;
+       *(void**)slot_ptr = value;
+       if (mono_sgen_ptr_in_nursery (value))
+               sgen_card_table_mark_address ((mword)slot_ptr);
+       mono_sgen_dummy_use (value);    
 }
 
-static gboolean
-sgen_card_table_address_is_marked (mword address)
+void
+mono_sgen_card_table_wbarrier_arrayref_copy (gpointer dest_ptr, gpointer src_ptr, int count)
 {
-       return *sgen_card_table_get_card_address (address) != 0;
+       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*
-sgen_card_table_align_pointer (void *ptr)
+void
+mono_sgen_card_table_wbarrier_value_copy (gpointer dest, gpointer src, int count, MonoClass *klass)
 {
-       return (void*)((mword)ptr & ~(CARD_SIZE_IN_BYTES - 1));
+       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
-sgen_card_table_reset_region (mword start, mword end)
+mono_sgen_card_table_wbarrier_object_copy (MonoObject* obj, MonoObject *src)
 {
-       memset (sgen_card_table_get_card_address (start), 0, (end - start) >> CARD_BITS);
+       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
-sgen_card_table_mark_range (mword address, mword size)
+mono_sgen_card_table_wbarrier_generic_nostore (gpointer ptr)
 {
-       mword end = address + size;
-       do {
-               sgen_card_table_mark_address (address);
-               address += CARD_SIZE_IN_BYTES;
-       } while (address < end);
+       sgen_card_table_mark_address ((mword)ptr);      
 }
 
-gboolean
-sgen_card_table_is_region_marked (mword start, mword end)
+#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_address_is_marked (start))
+               if (sgen_card_table_card_begin_scanning (start))
                        return TRUE;
                start += CARD_SIZE_IN_BYTES;
        }
        return FALSE;
 }
 
-static void
-card_table_init (void)
+#else
+
+static gboolean
+sgen_card_table_region_begin_scanning (mword start, mword size)
+{
+       gboolean res = FALSE;
+       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;
+                       break;
+               }
+       }
+
+       memset (sgen_card_table_get_card_address (start), 0, size >> CARD_BITS);
+
+       return res;
+}
+
+#endif
+
+/*FIXME this assumes that major blocks are multiple of 4K which is pretty reasonable */
+gboolean
+sgen_card_table_get_card_data (guint8 *data_dest, mword address, mword cards)
+{
+       mword *start = (mword*)sgen_card_table_get_card_scan_address (address);
+       mword *dest = (mword*)data_dest;
+       mword *end = (mword*)(data_dest + cards);
+       mword mask = 0;
+
+       for (; dest < end; ++dest, ++start) {
+               mword v = *start;
+               *dest = v;
+               mask |= v;
+
+#ifndef SGEN_HAVE_OVERLAPPING_CARDS
+               *start = 0;
+#endif
+       }
+
+       return mask;
+}
+
+void*
+sgen_card_table_align_pointer (void *ptr)
+{
+       return (void*)((mword)ptr & ~(CARD_SIZE_IN_BYTES - 1));
+}
+
+void
+sgen_card_table_mark_range (mword address, mword size)
+{
+       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 address, mword size)
 {
-       cardtable = mono_sgen_alloc_os_memory (CARD_COUNT_IN_BYTES, TRUE);
+       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;
+       }
+       return FALSE;
+
 }
 
+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);
+}
 
-void los_scan_card_table (GrayQueue *queue);
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
 
 static void
-scan_from_card_tables (void *start_nursery, void *end_nursery, GrayQueue *queue)
+move_cards_to_shadow_table (mword start, mword size)
 {
-       if (use_cardtable) {
-               major.scan_card_table (queue);
-               los_scan_card_table (queue);
+       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);
+
+       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
-card_table_clear (void)
+clear_cards (mword start, mword size)
 {
-       /*XXX we could do this in 2 ways. using mincore or iterating over all sections/los objects */
-       if (use_cardtable) {
-               major.clear_card_table ();
-               los_clear_card_table ();
+       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);
        }
 }
 
+
+#else
+
+static void
+clear_cards (mword start, mword size)
+{
+       memset (sgen_card_table_get_card_address (start), 0, cards_in_range (start, size));
+}
+
+
+#endif
+
+void
+sgen_card_table_clear (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);
+       mono_sgen_los_iterate_live_block_ranges (clear_cards);
+}
+
+void
+sgen_scan_from_card_tables (void *start_nursery, void *end_nursery, SgenGrayQueue *queue)
+{
+       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*/
+       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*/
+       sgen_card_table_clear ();
+#endif
+       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)
+mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 {
-       if (!use_cardtable)
+       if (!sgen_cardtable)
                return NULL;
 
-       g_assert (cardtable);
        *shift_bits = CARD_BITS;
+#ifdef SGEN_HAVE_OVERLAPPING_CARDS
+       *mask = (gpointer)CARD_MASK;
+#else
+       *mask = NULL;
+#endif
 
-       return cardtable;
+       return sgen_cardtable;
 }
 
+#if 0
 static void
 collect_faulted_cards (void)
 {
 #define CARD_PAGES (CARD_COUNT_IN_BYTES / 4096)
        int i, count = 0;
        unsigned char faulted [CARD_PAGES] = { 0 };
-       mincore (cardtable, CARD_COUNT_IN_BYTES, faulted);
+       mincore (sgen_cardtable, CARD_COUNT_IN_BYTES, faulted);
 
        for (i = 0; i < CARD_PAGES; ++i) {
                if (faulted [i])
@@ -150,6 +409,251 @@ 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 block_obj_size, guint8 *cards, SgenGrayQueue *queue)
+{
+       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 ();
+
+       HEAVY_STAT (++large_objects);
+
+       if (!SGEN_VTABLE_HAS_REFERENCES (vt))
+               return;
+
+       if (vt->rank) {
+               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;
+               int elem_size = mono_array_element_size (klass);
+
+#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;
+
+
+#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 = MIN (card_end, obj_end);
+
+                       if (start <= (char*)arr->vector)
+                               index = 0;
+                       else
+                               index = ARRAY_OBJ_INDEX (start, obj, elem_size);
+
+                       elem = (char*)mono_array_addr_with_size ((MonoArray*)obj, elem_size, index);
+                       if (klass->element_class->valuetype) {
+                               for (; elem < card_end; elem += elem_size)
+                                       scan_vtype_func (elem, desc, queue);
+                       } else {
+                               HEAVY_STAT (++los_array_cards);
+                               for (; elem < card_end; elem += SIZEOF_VOID_P) {
+                                       gpointer new, old = *(gpointer*)elem;
+                                       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 (mono_sgen_ptr_in_nursery (new)))
+                                                       mono_sgen_add_to_global_remset (elem);
+                                       }
+                               }
+                       }
+               }
+
+#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, (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
+
+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) {
+               ++cur_stats->total;
+               if (sgen_card_table_address_is_marked (start))
+                       ++cur_stats->marked;
+               start += CARD_SIZE_IN_BYTES;
+       }
+}
+
+static void
+count_remarked_cards (mword start, mword size)
+{
+       mword end = start + size;
+       while (start <= end) {
+               if (sgen_card_table_address_is_marked (start))
+                       ++cur_stats->remarked;
+               start += CARD_SIZE_IN_BYTES;
+       }
+}
+
+#endif
+
+void
+sgen_card_tables_collect_stats (gboolean begin)
+{
+#ifdef CARDTABLE_STATS
+       if (begin) {
+               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 {
+               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
+}
+
 #else
 
 void
@@ -164,15 +668,12 @@ 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()
-
 guint8*
-mono_gc_get_card_table (int *shift_bits)
+mono_gc_get_card_table (int *shift_bits, gpointer *mask)
 {
        return NULL;
 }
 
 #endif
+
+#endif /*HAVE_SGEN_GC*/
\ No newline at end of file