On Mon, Oct 09, 2006 at 09:16:58AM -0700, Chuck Silvers wrote:
quoted text
> 
> I'd rather rearrange uvm_aio_aiodone() and genfs_getpages() to separate out
> the code that's more or less duplicated between them.  then the new function
> (which would be a uvm thing and not a genfs thing) would be the single place
> to put pagecache I/O completion hooks like this.
> 

It took me a while to get to do this and a while more to straighten
out the mess I made.  The attached diff works for me with LOCKDEBUG
and DIAGNOSTIC on a multiprocessor machine.  I put the veriexec
page checking in there and this too seemed to work fine - detecting
and killing a modified binary on a nfs share.

Unfortunately, it seems like my original problem still exists though -
I think that the fileassoc code needs to use pools because the
veriexec check is done with locks held so I get panics of the
"vmem_alloc with simple_lock held" kind if fileassoc is not using
pools for it's working storage.

Apologies for the horrible diff - cvs has decided that I just renamed
the uvm_aio_aiodone function and then wrote a new one.

-- 
Brett Lymn