unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
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


  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).