From: ludovic.courtes@laas.fr (Ludovic Courtès)
Cc: guile-devel@gnu.org
Subject: Re: [PATCH] GC code cleanup #2
Date: Thu, 09 Feb 2006 10:16:46 +0100 [thread overview]
Message-ID: <87slqs28ox.fsf@laas.fr> (raw)
In-Reply-To: <873bk3ncb4.fsf_-_@laas.fr> (Ludovic Courtès's message of "Wed, 04 Jan 2006 18:16:31 +0100")
Hi folks,
ludovic.courtes@laas.fr (Ludovic Courtès) writes:
> 2006-01-04 Ludovic Courtès <ludovic.courtes@laas.fr>
>
> * libguile/gc-segment.c (scm_i_sweep_some_cards): Take a SWEEP_STATS
> argument. Don't refer to SCM_GC_CELLS_COLLECTED and
> SCM_CELLS_ALLOCATED. If SEG->FIRST_TIME, let CELLS_COLLECTED as zero.
> Take into account SEG->SPAN when computing CELLS_SWEPT.
> (scm_i_sweep_segment): Take one more argument, similarly.
> (scm_i_sweep_all_segments): Likewise.
> (scm_i_sweep_some_segments): Likewise.
> (scm_i_adjust_min_yield): Change the way MIN_CELLS is computed: do not
> refer to SCM_GC_CELLS_COLLECTED.
>
> * libguile/gc-freelist.c (scm_i_adjust_min_yield): Take one more
> argument, an `scm_i_sweep_statistics' object.
> Change the way DELTA is collected: don't take into account
> SCM_GC_CELLS_COLLECTED_1, only SWEEP_STATS.COLLECTED.
>
> * libguile/gc-malloc.c (scm_realloc): Pass an extra argument
> to `scm_i_sweep_all_segments ()'.
>
> * libguile/gc.c (gc_start_stats): Updated accordingly.
> (gc_end_stats): Take an additional SWEEP_STATS argument.
> Decrement SCM_CELLS_ALLOCATED after calls to `scm_i_sweep_* ()'.
> (scm_gc_for_newcell): Updated callers of `scm_i_sweep_*'.
> Decrement SCM_CELLS_ALLOCATED.
> (scm_i_gc): Likewise.
>
> * libguile/private-gc.h (scm_i_sweep_*): Updated function
> prototypes accordingly.
> (scm_t_sweep_statistics): New type.
> (scm_i_sweep_statistics_init): New macro.
> (scm_i_sweep_statistics_sum): New macro
Please, can somebody review this patch?
Thanks,
Ludovic.
_______________________________________________
Guile-devel mailing list
Guile-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/guile-devel
next prev parent reply other threads:[~2006-02-09 9:16 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-12-20 16:15 [PATCH] GC code cleanup Ludovic Courtès
2005-12-23 11:23 ` Han-Wen Nienhuys
2006-01-04 17:16 ` [PATCH] GC code cleanup #2 Ludovic Courtès
2006-01-30 9:14 ` Ludovic Courtès
2006-02-09 9:16 ` Ludovic Courtès [this message]
2006-02-09 22:38 ` Han-Wen Nienhuys
2006-02-10 8:35 ` Ludovic Courtès
2006-02-10 12:44 ` Han-Wen Nienhuys
2006-10-11 20:04 ` [PATCH] GC code cleanup Kevin Ryde
2006-10-16 7:59 ` Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87slqs28ox.fsf@laas.fr \
--to=ludovic.courtes@laas.fr \
--cc=guile-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).