On 29 Jun 2015, at 18:47, David Holland <dholland-sourcechan...@netbsd.org> wrote:
> On Mon, Jun 29, 2015 at 12:25:49PM -0400, Christos Zoulas wrote: >> Module Name: src >> Committed By: christos >> Date: Mon Jun 29 16:25:49 UTC 2015 >> >> Modified Files: >> src/sys/miscfs/specfs: spec_vnops.c >> >> Log Message: >> Revert previous, and explain why. > > Hrm, shouldn't it be &vp? > > (as it is, it's using the uvm_object lock pointer as the key) Yes, just changed the key to the address of vp->v_specdev which is invariant over the lifetime of the vnode. Coverity is quite cool ... -- J. Hannken-Illjes - hann...@eis.cs.tu-bs.de - TU Braunschweig (Germany)