call for help on TMPFS

Howard Su howard0su at gmail.com
Sun May 27 02:36:06 UTC 2007


On 5/27/07, Pawel Jakub Dawidek <pjd at freebsd.org> wrote:
> On Sat, May 26, 2007 at 12:06:14AM +0800, Howard Su wrote:
> I think you are missing some cases in lookup where you should add
> SAVENAME flag. I'd start from adding this (or similar) assertions:
>
>         KASSERT((ap->a_cnp->cn_flags & SAVENAME), ("No SAVENAME"));
>
> to the following functions:
>
>         tmpfs_create()
>         tmpfs_remove()
>         tmpfs_mkdir()
>         tmpfs_rmdir()
>         tmpfs_rename() (assert both: fcnp and tcnp)
>         tmpfs_symlink()
>         tmpfs_link()
Thank you very much. You are right. I just fixed this long standing panic.

PS: I noticed that kernel option DIAGNOSTIC will clean ap->a_cnp with
NULL if you don't give SAVENAME. So I enable DIAGNOSTIC instead of
adding asserts.

-- 
-Howard


More information about the freebsd-fs mailing list