Re: kern/38889

[Martin Husemann <martin%duskware.de@localhost>]

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

From: Martin Husemann <martin%duskware.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: kern/38889
Date: Fri, 25 May 2012 11:32:07 +0200

 --bg08WKrSYDhXBjb5
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 We can at least prevent the crash for -6. If I understood yamt
 correctly, this is correct anyway, only needs more changes in genfs.
 
 Should we commit this as a interim step and pull it up?
 
 Martin
 
 --bg08WKrSYDhXBjb5
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename=patch
 
 Index: uvm_vnode.c
 ===================================================================
 RCS file: /cvsroot/src/sys/uvm/uvm_vnode.c,v
 retrieving revision 1.97
 diff -c -u -r1.97 uvm_vnode.c
 --- uvm_vnode.c        6 Sep 2011 16:41:55 -0000       1.97
 +++ uvm_vnode.c        25 May 2012 09:29:47 -0000
 @@ -170,7 +170,8 @@
  
        UVMHIST_LOG(ubchist, "vp %p off 0x%x", vp, (int)offset, 0,0);
  
 -      if ((access_type & VM_PROT_WRITE) == 0 && (flags & PGO_LOCKED) == 0) {
 +      if (vp->v_type == VREG && (access_type & VM_PROT_WRITE) == 0
 +          && (flags & PGO_LOCKED) == 0) {
                vn_ra_allocctx(vp);
                uvm_ra_request(vp->v_ractx, advice, uobj, offset,
                    *npagesp << PAGE_SHIFT);
 
 --bg08WKrSYDhXBjb5--