Subject: Re: CVS commit: src/sys/fs/tmpfs
To: None <yamt@netbsd.org>
From: Julio M. Merino Vidal <jmmv84@gmail.com>
List: source-changes
Date: 09/13/2005 12:48:56
On 9/12/05, YAMAMOTO Takashi <yamt@netbsd.org> wrote:
>=20
> Module Name: src
> Committed By: yamt
> Date: Mon Sep 12 19:56:58 UTC 2005
>=20
> Modified Files:
> src/sys/fs/tmpfs: tmpfs_pool.c tmpfs_pool.h
>=20
> Log Message:
> - don't keep a reference to a variable on stack.
Thanks for fixing this issue.
Please add a little description of the new tp_name field into tmpfs.9.
> - remove a meaningless assumption about the order of structure members.
This is not enough. There are other places that assume this too, such as
the TMPFS_POOL_{GET,PUT} macros. (The initial idea was to be able to treat
a tmpfs_pool object just like a regular pool, thus assuming that the intern=
al
pool was the first field and that pointers could be "downgraded" to
"pool *" without problems. But I guess this is not a good idea, or, in fac=
t,
needed anywhere.)
--=20
Julio M. Merino Vidal <jmmv84@gmail.com>
http://www.livejournal.com/users/jmmv/
The NetBSD Project - http://www.NetBSD.org/