Re: kern/47739: tmpfs panic: kernel diagnostic assertion "(node)->tn_spec.tn_dir.tn_readdir_lastp == NULL..."

[Mindaugas Rasiukevicius <rmind%netbsd.org@localhost>]

The following reply was made to PR kern/47739; it has been noted by GNATS.

From: Mindaugas Rasiukevicius <rmind%netbsd.org@localhost>
To: Thomas Klausner <wiz%NetBSD.org@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, kern-bug-people%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
Subject: Re: kern/47739: tmpfs panic: kernel diagnostic assertion
 "(node)->tn_spec.tn_dir.tn_readdir_lastp == NULL..."
Date: Thu, 8 Aug 2013 21:27:34 +0100

 Thomas Klausner <wiz%NetBSD.org@localhost> wrote:
 > The following reply was made to PR kern/47739; it has been noted by GNATS.
 > 
 > From: Thomas Klausner <wiz%NetBSD.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: 
 > Subject: Re: kern/47739: tmpfs panic: kernel diagnostic assertion
 >  "(node)->tn_spec.tn_dir.tn_readdir_lastp == NULL..."
 > Date: Mon, 22 Jul 2013 19:42:17 +0200
 > 
 >  These are highly reproducible.
 >  
 >  Whenever I do a bulk build from scratch, I usually reboot at least
 >  once due to it.
 >  
 >  Can someone please take a look?
 >   Thomas
 
 Most likely this is due to tmpfs_dircookie() truncation here:
 
 http://nxr.netbsd.org/source/xref/src/sys/fs/tmpfs/tmpfs.h#88
 
 It is wrong and there are other PRs because of it.  Thomas, you can test
 this by replacing the function body with the following:
 
        return (off_t)(uintptr_t)de;
 
 This breaks linux32 compat, but we really just need to decide how we want
 to fix it (it would be good to avoid penalising the native code, but better
 to penalise than fail).
 
 -- 
 Mindaugas