Page MenuHomeFreeBSD
Files F102761419

D46118.diff
No OneTemporary
Actions

D46118.diff
View Options

diff --git a/sys/cam/cam_periph.c b/sys/cam/cam_periph.c
--- a/sys/cam/cam_periph.c
+++ b/sys/cam/cam_periph.c
@@ -928,16 +928,6 @@
}
}
- /*
- * This keeps the kernel stack of current thread from getting
- * swapped. In low-memory situations where the kernel stack might
- * otherwise get swapped out, this holds it and allows the thread
- * to make progress and release the kernel mapped pages sooner.
- *
- * XXX KDM should I use P_NOSWAP instead?
- */
- PHOLD(curproc);
-
for (i = 0; i < numbufs; i++) {
/* Save the user's data address. */
mapinfo->orig[i] = *data_ptrs[i];
@@ -1005,7 +995,6 @@
free(*data_ptrs[i], M_CAMPERIPH);
*data_ptrs[i] = mapinfo->orig[i];
}
- PRELE(curproc);
return(EACCES);
}
@@ -1116,9 +1105,6 @@
*data_ptrs[i] = mapinfo->orig[i];
}
- /* allow ourselves to be swapped once again */
- PRELE(curproc);
-
return (error);
}
diff --git a/sys/compat/linuxkpi/common/src/linux_schedule.c b/sys/compat/linuxkpi/common/src/linux_schedule.c
--- a/sys/compat/linuxkpi/common/src/linux_schedule.c
+++ b/sys/compat/linuxkpi/common/src/linux_schedule.c
@@ -268,11 +268,6 @@
task = current;
- /*
- * Our wait queue entry is on the stack - make sure it doesn't
- * get swapped out while we sleep.
- */
- PHOLD(task->task_thread->td_proc);
sleepq_lock(task);
if (atomic_read(&task->state) != TASK_WAKING) {
ret = linux_add_to_sleepqueue(task, task, "wevent", timeout,
@@ -281,7 +276,6 @@
sleepq_release(task);
ret = 0;
}
- PRELE(task->task_thread->td_proc);
if (lock != NULL)
spin_lock_irq(lock);
diff --git a/sys/dev/nvme/nvme_ctrlr.c b/sys/dev/nvme/nvme_ctrlr.c
--- a/sys/dev/nvme/nvme_ctrlr.c
+++ b/sys/dev/nvme/nvme_ctrlr.c
@@ -1259,11 +1259,6 @@
return EIO;
}
if (is_user_buffer) {
- /*
- * Ensure the user buffer is wired for the duration of
- * this pass-through command.
- */
- PHOLD(curproc);
buf = uma_zalloc(pbuf_zone, M_WAITOK);
buf->b_iocmd = pt->is_read ? BIO_READ : BIO_WRITE;
if (vmapbuf(buf, pt->buf, pt->len, 1) < 0) {
@@ -1309,7 +1304,6 @@
vunmapbuf(buf);
err:
uma_zfree(pbuf_zone, buf);
- PRELE(curproc);
}
return (ret);
@@ -1356,11 +1350,6 @@
if ((npc->opcode & 0x3) == 0 || (npc->opcode & 0x3) == 3)
return (EINVAL);
if (is_user) {
- /*
- * Ensure the user buffer is wired for the duration of
- * this pass-through command.
- */
- PHOLD(curproc);
buf = uma_zalloc(pbuf_zone, M_WAITOK);
buf->b_iocmd = npc->opcode & 1 ? BIO_WRITE : BIO_READ;
if (vmapbuf(buf, (void *)(uintptr_t)npc->addr,
@@ -1408,7 +1397,6 @@
vunmapbuf(buf);
err:
uma_zfree(pbuf_zone, buf);
- PRELE(curproc);
}
return (ret);
diff --git a/sys/kern/kern_physio.c b/sys/kern/kern_physio.c
--- a/sys/kern/kern_physio.c
+++ b/sys/kern/kern_physio.c
@@ -87,12 +87,6 @@
return (EFBIG);
}
- /*
- * Keep the process UPAGES from being swapped. Processes swapped
- * out while holding pbufs, used by swapper, may lead to deadlock.
- */
- PHOLD(curproc);
-
bp = g_alloc_bio();
if (uio->uio_segflg != UIO_USERSPACE) {
pbuf = NULL;
@@ -209,6 +203,5 @@
else if (pages)
free(pages, M_DEVBUF);
g_destroy_bio(bp);
- PRELE(curproc);
return (error);
}
diff --git a/sys/kern/kern_procctl.c b/sys/kern/kern_procctl.c
--- a/sys/kern/kern_procctl.c
+++ b/sys/kern/kern_procctl.c
@@ -573,17 +573,7 @@
w.rk = rk;
w.error = &error;
TASK_INIT(&w.t, 0, reap_kill_proc_work, &w);
-
- /*
- * Prevent swapout, since w, ksi, and possibly rk, are
- * allocated on the stack. We sleep in
- * reap_kill_subtree_once() waiting for task to
- * complete single-threading.
- */
- PHOLD(td->td_proc);
-
reap_kill_subtree(td, p, reaper, &w);
- PRELE(td->td_proc);
crfree(w.cr);
}
PROC_LOCK(p);
diff --git a/sys/kern/subr_firmware.c b/sys/kern/subr_firmware.c
--- a/sys/kern/subr_firmware.c
+++ b/sys/kern/subr_firmware.c
@@ -419,9 +419,7 @@
fwli.flags = flags;
TASK_INIT(&fwload_task, 0, loadimage, (void *)&fwli);
taskqueue_enqueue(firmware_tq, &fwload_task);
- PHOLD(curproc);
msleep((void *)&fwli, &firmware_mtx, 0, "fwload", 0);
- PRELE(curproc);
}
/*
* After attempting to load the module, see if the image is registered.
diff --git a/sys/ufs/ffs/ffs_rawread.c b/sys/ufs/ffs/ffs_rawread.c
--- a/sys/ufs/ffs/ffs_rawread.c
+++ b/sys/ufs/ffs/ffs_rawread.c
@@ -265,11 +265,6 @@
resid = uio->uio_resid;
offset = uio->uio_offset;
- /*
- * keep the process from being swapped
- */
- PHOLD(td->td_proc);
-
error = 0;
nerror = 0;
@@ -389,7 +384,6 @@
if (error == 0)
error = nerror;
- PRELE(td->td_proc);
uio->uio_iov->iov_base = udata;
uio->uio_resid = resid;
uio->uio_offset = offset;
diff --git a/sys/ufs/ffs/ffs_softdep.c b/sys/ufs/ffs/ffs_softdep.c
--- a/sys/ufs/ffs/ffs_softdep.c
+++ b/sys/ufs/ffs/ffs_softdep.c
@@ -1904,7 +1904,6 @@
*/
if (curthread->td_pflags & TDP_COWINPROGRESS)
return (-1);
- PHOLD(curproc); /* Don't let the stack go away. */
ump = VFSTOUFS(mp);
LOCK_OWNED(ump);
matchcnt = 0;
@@ -1977,7 +1976,6 @@
ump->softdep_worklist_tail =
(struct worklist *)sentinel.wk_list.le_prev;
LIST_REMOVE(&sentinel, wk_list);
- PRELE(curproc);
return (matchcnt);
}
@@ -10230,7 +10228,6 @@
return;
marker.wk_type = D_LAST + 1; /* Not a normal workitem */
- PHOLD(curproc); /* Don't swap out kernel stack */
ACQUIRE_LOCK(ump);
/*
* Do any necessary pre-I/O processing.
@@ -10315,7 +10312,6 @@
}
}
FREE_LOCK(ump);
- PRELE(curproc); /* Allow swapout of kernel stack */
}
/*

File Metadata

Mime Type
text/plain
Expires
Sun, Nov 17, 7:37 PM (21 h, 53 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
14683812
Default Alt Text
D46118.diff (5 KB)

Event Timeline