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/