summaryrefslogtreecommitdiffstats
path: root/sys/kern/kern_thread.c
Commit message (Collapse)AuthorAgeFilesLines
* Completely redo thread states.julian2002-09-111-56/+39
| | | | Reviewed by: davidxu@freebsd.org
* fix braino..julian2002-09-071-1/+1
| | | | was clearing part of wrong thread structure..
* fix misplaced schedlockjulian2002-09-071-1/+1
| | | | Submitted by: davidxu@freebsd.org
* Use UMA as a complex object allocator.julian2002-09-061-42/+46
| | | | | | | | | | | | | | | The process allocator now caches and hands out complete process structures *including substructures* . i.e. it get's the process structure with the first thread (and soon KSE) already allocated and attached, all in one hit. For the average non threaded program (non KSE that is) the allocated thread and its stack remain attached to the process, even when the process is unused and in the process cache. This saves having to allocate and attach it later, effectively bringing us (hopefully) close to the efficiency of pre-KSE systems where these were a single structure. Reviewed by: davidxu@freebsd.org, peter@freebsd.org
* s/SGNL/SIG/davidxu2002-09-051-11/+11
| | | | | | | | | | s/SNGL/SINGLE/ s/SNGLE/SINGLE/ Fix abbreviation for P_STOPPED_* etc flags, in original code they were inconsistent and difficult to distinguish between them. Approved by: julian (mentor)
* In the kernel code, we have the tsleep() call with the PCATCH argument.davidxu2002-09-031-6/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PCATCH means 'if we get a signal, interrupt me!" and tsleep returns either EINTR or ERESTART depending on the circumstances. ERESTART is "special" because it causes the system call to fail, but right as it returns back to userland it tells the trap handler to move %eip back a bit so that userland will immediately re-run the syscall. This is a syscall restart. It only works for things like read() etc where nothing has changed yet. Note that *userland* is tricked into restarting the syscall by the kernel. The kernel doesn't actually do the restart. It is deadly for things like select, poll, nanosleep etc where it might cause the elapsed time to be reset and start again from scratch. So those syscalls do this to prevent userland rerunning the syscall: if (error == ERESTART) error = EINTR; Fake "signals" like SIGTSTP from ^Z etc do not normally invoke userland signal handlers. But, in -current, the PCATCH *is* being triggered and tsleep is returning ERESTART, and the syscall is aborted even though no userland signal handler was run. That is the fault here. We're triggering the PCATCH in cases that we shouldn't. ie: it is being triggered on *any* signal processing, rather than the case where the signal is posted to userland. --- Peter The work of psignal() is a patchwork of special case required by the process debugging and job-control facilities... --- Kirk McKusick "The design and impelementation of the 4.4BSD Operating system" Page 105 in STABLE source, when psignal is posting a STOP signal to sleeping process and the signal action of the process is SIG_DFL, system will directly change the process state from SSLEEP to SSTOP, and when SIGCONT is posted to the stopped process, if it finds that the process is still on sleep queue, the process state will be restored to SSLEEP, and won't wakeup the process. this commit mimics the behaviour in STABLE source tree. Reviewed by: Jon Mini, Tim Robbins, Peter Wemm Approved by: julian@freebsd.org (mentor)
* Fix crack-smoking code that was panicing on the quad xeon:julian2002-08-291-26/+23
| | | | | | | | | | | | - If either of proc or kse are NULL during thread_exit(), then the kernel is going to fault because parts of the function assume they aren't NULL. Instead, just assert they aren't NULL (as well as the kse group) and assume they are in all of the code. It doesn't make sense for them to be NULL here anyways. - Move the PROC_UNLOCK(p) up above clearing td_proc, etc. since otherwise we will panic if the proc's lock is contested. Submitted by: jhb@freebsd.org
* slight cleanup of single-threading code for KSE processesjulian2002-08-221-1/+16
|
* Revert some suspension/sleep/signal code from KSE-IIIjulian2002-08-211-0/+11
| | | | | | | | | We need to rethink a bit of this and it doesn't matter if we break the KSE test program for now as long as non-KSE programs act as expected. Submitted by: David Xu <bsddiy@yahoo.com> (this guy's just asking to get hit with a commit bit..)
* Fix a comment.julian2002-08-011-1/+1
|
* get suspension counting right.julian2002-07-251-0/+4
| | | | | | fix an error message Submitted by: David Xu <bsddiy@yahoo.com>
* fix some style problems and remove a mis-merged assert.julian2002-07-251-30/+12
|
* Add some locking asserts and some commentsjulian2002-07-241-0/+4
|
* When single threading a multithreaded program, awaken thejulian2002-07-241-0/+7
| | | | | | | | | 'single threading thread' when the last other thread suspends. I had this code in there before but it seems to have been accidentally deleted somewhere along the way. This would only affect multithreaded processes. Reviewed by: David Xu <bsddiy@yahoo.com>
* When suspending a thread, update the appropriate (sic) statistic.julian2002-07-241-0/+1
|
* ia64 does not have the same degree of stealth include file nesting,peter2002-07-171-0/+2
| | | | | | so it needs an explicit #include <machine/frame.h> to get 'struct trapframe'. The fact that it needs this at this level is rather bogus but it will not compile without it.
* Pacify gcc on ia64peter2002-07-171-0/+3
|
* Thinking about it I came to the conclusion that the KSE states were incorrectlyjulian2002-07-141-3/+0
| | | | | | | | | | | | | | formulated. The correct states should be: IDLE: On the idle KSE list for that KSEG RUNQ: Linked onto the system run queue. THREAD: Attached to a thread and slaved to whatever state the thread is in. This means that most places where we were adjusting kse state can go away as it is just moving around because the thread is.. The only places we need to adjust the KSE state is in transition to and from the idle and run queues. Reviewed by: jhb@freebsd.org
* Collect all the (now equivalent) pmap_new_proc/pmap_dispose_proc/peter2002-07-071-0/+10
| | | | | | | | | | | | | pmap_swapin_proc/pmap_swapout_proc functions from the MD pmap code and use a single equivalent MI version. There are other cleanups needed still. While here, use the UMA zone hooks to keep a cache of preinitialized proc structures handy, just like the thread system does. This eliminates one dependency on 'struct proc' being persistent even after being freed. There are some comments about things that can be factored out into ctor/dtor functions if it is worth it. For now they are mostly just doing statistics to get a feel of how it is working.
* Make this compile on 64 bit platformspeter2002-07-071-4/+4
|
* - In thread_userret(), remove the Giant locking and unlocking around thearr2002-07-011-2/+0
| | | | | | | call to thread_alloc(). Approved by: julian Reviewed by: jake, jeff
* Add files that are new for KSE.julian2002-06-291-0/+790
OpenPOWER on IntegriCloud