aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlan Cox <alc@FreeBSD.org>2003-06-19 03:38:05 +0000
committerAlan Cox <alc@FreeBSD.org>2003-06-19 03:38:05 +0000
commit82b8b189596f04f8b2560efaef2328d8dcfbc42d (patch)
treeb026e599c582db304008002a3b964142a282a9c5
parent95aada383fae9d44c41fcb8f6f2c31889c812294 (diff)
Notes
-rw-r--r--sys/fs/smbfs/smbfs_io.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/sys/fs/smbfs/smbfs_io.c b/sys/fs/smbfs/smbfs_io.c
index 80efbf883352..fb56649e2510 100644
--- a/sys/fs/smbfs/smbfs_io.c
+++ b/sys/fs/smbfs/smbfs_io.c
@@ -428,10 +428,11 @@ smbfs_getpages(ap)
struct smbmount *smp;
struct smbnode *np;
struct smb_cred scred;
+ vm_object_t object;
vm_page_t *pages, m;
vp = ap->a_vp;
- if (vp->v_object == NULL) {
+ if ((object = vp->v_object) == NULL) {
printf("smbfs_getpages: called with non-merged cache vnode??\n");
return VM_PAGER_ERROR;
}
@@ -455,12 +456,14 @@ smbfs_getpages(ap)
if (m->valid != 0) {
/* handled by vm_fault now */
/* vm_page_zero_invalid(m, TRUE); */
+ VM_OBJECT_LOCK(object);
vm_page_lock_queues();
for (i = 0; i < npages; ++i) {
if (i != reqpage)
vm_page_free(pages[i]);
}
vm_page_unlock_queues();
+ VM_OBJECT_UNLOCK(object);
return 0;
}
@@ -488,6 +491,8 @@ smbfs_getpages(ap)
relpbuf(bp, &smbfs_pbuf_freecnt);
+ if (error)
+ VM_OBJECT_LOCK(object);
if (error && (uio.uio_resid == count)) {
printf("smbfs_getpages: error %d\n",error);
vm_page_lock_queues();
@@ -496,6 +501,7 @@ smbfs_getpages(ap)
vm_page_free(pages[i]);
}
vm_page_unlock_queues();
+ VM_OBJECT_UNLOCK(object);
return VM_PAGER_ERROR;
}
@@ -557,6 +563,8 @@ smbfs_getpages(ap)
}
}
vm_page_unlock_queues();
+ if (error)
+ VM_OBJECT_UNLOCK(object);
return 0;
#endif /* SMBFS_RWGENERIC */
}