[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Endless garbage collection
Simon Munton wrote:
> It's possible to get the garbage collection going in an endless loop by
> creating lots of small files. eg, if I create 25 files, each 0 bytes long,
> on an empty filesystem, garbage collection starts and goes on and on.
> Perhaps the test for too much RAM being used in thread_should_wake() ought
> to take the number of files into account? ie should it be:
> /* If there is too much RAM used by the various structures, GC */
> if (jffs_get_node_inuse() >
> (c->fmc->used_size/c->fmc->max_chunk_size * 5 + 25 + jffs_get_file_count()))
> where jffs_get_file_count() returns the number of files on the filesystem.
You're absolutly right. The only issues are performance of the
jffs_get_file_count(), because it is executed every times an insert_node() is
called. We should keep a running count of how many files exists in the
partition, but I did not do that yet.
Here's my version:
if (jffs_get_node_inuse() > (c->fmc->used_size/c->fmc->max_chunk_size * 5
+ jffs_foreach_file(c, jffs_file_count) * 2 + 50))
I suggest to raise 25 to 50, to avoid silly problems like this.
Colubris Networks (http://www.colubris.com)
To unsubscribe from this list: send the line "unsubscribe jffs-dev" in
the body of a message to firstname.lastname@example.org