summaryrefslogtreecommitdiffstats
path: root/sys/kern/sys_generic.c
Commit message (Collapse)AuthorAgeFilesLines
...
* select() DKI is now in <sys/selinfo.h>.wollman2001-01-091-0/+1
|
* Only call bwillwrite() for vnodes. Do not penalize devices or pipes.dillon2000-12-071-1/+2
|
* Add necessary bwillwrite() in writev() entry point.dillon2000-12-061-0/+1
| | | | | | Deal with excessive dirty buffers when msync() syncs non-contiguous dirty buffers by checking for the case in UFS *before* checking for clusterability.
* only call bwillwrite() to stall on IO when dealing with VNODEs otherwisealfred2000-11-301-1/+2
| | | | we will stall on non-disk IO for things like fifos and sockets
* Protect p_wchan with sched_lock in selwakeup().jlemon2000-11-211-0/+2
|
* This patchset fixes a large number of file descriptor race conditions.dillon2000-11-181-26/+60
| | | | | | | | | | | | Pre-rfork code assumed inherent locking of a process's file descriptor array. However, with the advent of rfork() the file descriptor table could be shared between processes. This patch closes over a dozen serious race conditions related to one thread manipulating the table (e.g. closing or dup()ing a descriptor) while another is blocked in an open(), close(), fcntl(), read(), write(), etc... PR: kern/11629 Discussed with: Alexander Viro <viro@math.psu.edu>
* Fix a warning that has been annoying me for some time:peter2000-07-281-1/+1
| | | | | | | "kern/sys_generic.c:358: warning: cast discards qualifiers from pointer target type" The idea for using the uintptr_t intermediate cast for de-constifying a pointer was hinted at by bde some time ago.
* Distinguish between whether ktraceing was enabled before an IOgreen2000-07-271-2/+6
| | | | | | operation or after it. If the ktrace operation was enabled while the process was blocked doing IO, the race would allow it to pass down invalid (uninitialized) data and panic later down the call stack.
* For infinite timeouts, set both the tv_sec and tv_usec fields to zero injhb2000-07-131-2/+6
| | | | | | poll() and select(). Noticed by: Wesley Morgan <morganw@chemicals.tacorp.com>
* Fix a very obscure bug in select() and poll() where the timeout wouldjhb2000-07-121-2/+2
| | | | | | never expire if poll() or select() was called before the system had been in multiuser for 1 second. This was caused by only checking to see if tv_sec was zero rather than checking both tv_sec and tv_usec.
* Remove two micro-pessimizations I made. Bruce is teaching me well :)green2000-07-071-2/+2
| | | | | KTRPOINT(p, KTR_GENIO) is more uncommon than error == 0, so it should be first in the && statement.
* Modify ktrace's general I/O tracing, ktrgenio(), to use a struct uio *green2000-07-021-15/+36
| | | | | | | | | | | | | instead of a struct iovec * array and int len. Get rid of stupidly trying to allocate all of the memory and copyin()ing the entire iovec[], and instead just do the proper VOP_WRITE() in ktrwrite() using a copy of the struct uio that the syscall originally used. This solves the DoS which could easily be performed; to work around the DoS, one could also remove "options KTRACE" from the kernel. This is a very strong MFC candidate for 4.1. Found by: art@OpenBSD.org
* unstatic getfp() so that other subsystems can use it.alfred2000-06-121-2/+1
| | | | | | make sendfile() use it. Approved by: dg
* Some ioctl routines assume that the ioctl buffer is aligned, but adillon2000-05-091-3/+6
| | | | | char[] declaration makes no such guarentee. A union is used to force alignment of the char buffer.
* Fix select(2) for the Alpha. (!!) It was never returning true forpeter2000-02-201-6/+8
| | | | | | | | | | | | | fd's in the range of 32-63, 96-127 etc. The first problem was the FD_*() macros were shifting a 32 bit integer "1" left by more than 32 bits. The same problem happened in selscan(). ffs() also takes an int argument and causes failure. For cases where int == long (ie: the usual case for x86, but not always as gcc can have long being a 64 bit quantity) ffs() could be used. Reported by: Marian Stagarescu <marian@bile.skycache.com> Reviewed by: dfr, gallatin (sys/types.h only) Approved by: jkh
* Add aio_waitcomplete(). Make aio work correctly for socket descriptors.jasone2000-01-141-0/+2
| | | | | | | | Make gratuitous style(9) fixes (me, not the submitter) to make the aio code more readable. PR: kern/12053 Submitted by: Chris Sedore <cmsedore@maxwell.syr.edu>
* Export the nselcoll counter via the kern.nselcoll sysctl so we can seepeter2000-01-051-1/+3
| | | | | | just how bad it gets in various situations. Reminded by: adrian
* Missed the second argument of fdrop().green1999-10-141-1/+1
| | | | Submitted by: jhay
* Fix a race condition with shared fd tables and writev(). It'sgreen1999-10-141-2/+7
| | | | | still not safe to consider file table sharing secure. Submitted by: Ville-Pertti Keinonen <will@iki.fi>
* Trim unused options (or #ifdef for undoc options).peter1999-10-111-1/+0
| | | | Submitted by: phk
* This is what was "fdfix2.patch," a fix for fd sharing. It's prettygreen1999-09-191-11/+10
| | | | | | | | | | | | | | | | | far-reaching in fd-land, so you'll want to consult the code for changes. The biggest change is that now, you don't use fp->f_ops->fo_foo(fp, bar) but instead fo_foo(fp, bar), which increments and decrements the fp refcount upon entry and exit. Two new calls, fhold() and fdrop(), are provided. Each does what it seems like it should, and if fdrop() brings the refcount to zero, the fd is freed as well. Thanks to peter ("to hell with it, it looks ok to me.") for his review. Thanks to msmith for keeping me from putting locks everywhere :) Reviewed by: peter
* $Id$ -> $FreeBSD$peter1999-08-281-1/+1
|
* Add standard padding argument to pread and pwrite syscall. That should make themdt1999-04-041-129/+89
| | | | | | | | | NetBSD compatible. Add parameter to fo_read and fo_write. (The only flag FOF_OFFSET mean that the offset is set in the struct uio). Factor out some common code from read/pread/write/pwrite syscalls.
* Added pread and pwrite. These functions are defined by the X/Openalc1999-03-271-1/+132
| | | | | | Threads Extension. (Note: We use the same syscall numbers as NetBSD.) Submitted by: John Plevyak <jplevyak@inktomi.com>
* Removed a bogus cast to c_caddr_t. This is part of terminatingbde1999-01-291-2/+2
| | | | | | | | | | | | | | | | c_caddr_t with extreme prejudice. Here the point of the original cast to caddr_t was to break the warning about the const mismatch between write(2)'s `const void *buf' and `struct uio's `char *iov_base' (previous bitrot gave a gratuitous dependency on caddr_t being char *). Compiling with -Wcast-qual made the cast a full no-op. This change has no effect on the warning for discarding `const' on assignment to iov_base. The warning should not be fixed by splitting `struct iovec' into a non-const version for read() and a const version for write(), since correct const poisoning would affect all pointers to i/o addresses. Const'ness should probably be forgotten by not declaring it in syscalls.master.
* Fix warnings in preparation for adding -Wall -Wcast-qual to thedillon1999-01-271-2/+2
| | | | kernel compile
* poll(2) sets POLLNVAL for descriptors passed in that are less thanjkh1998-12-101-2/+4
| | | | | | | | 0. This makes it difficult to do efficient manipulation of the struct pollfd since you can't leave a slot empty. PR: 8599 Submitted-by: Marc Slemko <marcs@znep.com>
* Installed the second patch attached to kern/7899 with some changes suggestedtruckman1998-11-111-32/+1
| | | | | | | | | | | | | | | | by bde, a few other tweaks to get the patch to apply cleanly again and some improvements to the comments. This change closes some fairly minor security holes associated with F_SETOWN, fixes a few bugs, and removes some limitations that F_SETOWN had on tty devices. For more details, see the description on the PR. Because this patch increases the size of the proc and pgrp structures, it is necessary to re-install the includes and recompile libkvm, the vinum lkm, fstat, gcore, gdb, ipfilter, ps, top, and w. PR: kern/7899 Reviewed by: bde, elvind
* Fixed bogotification of pseudocode for syscall args by rev.1.53 ofbde1998-09-051-5/+5
| | | | syscalls.master.
* Change various syscalls to use size_t arguments instead of u_int.dfr1998-08-241-10/+12
| | | | | | | | | | Add some overflow checks to read/write (from bde). Change all modifications to vm_page::flags, vm_page::busy, vm_object::flags and vm_object::paging_in_progress to use operations which are not interruptable. Reviewed by: Bruce Evans <bde@zeta.org.au>
* 64bit fixes: use u_long not int for ioctl command.dfr1998-06-101-2/+3
|
* s/nanoruntime/nanouptime/gphk1998-05-171-5/+5
| | | | | | s/microruntime/microuptime/g Reviewed by: bde
* Remove unused atv.tv_usec = 0; from select/poll codeache1998-04-051-7/+3
|
* Time changes mark 2:phk1998-04-041-21/+37
| | | | | | | | | | | | | | | | | * Figure out UTC relative to boottime. Four new functions provide time relative to boottime. * move "runtime" into struct proc. This helps fix the calcru() problem in SMP. * kill mono_time. * add timespec{add|sub|cmp} macros to time.h. (XXX: These may change!) * nanosleep, select & poll takes long sleeps one day at a time Reviewed by: bde Tested by: ache and others
* Try to fix poll & select after I broke them.phk1998-04-021-11/+9
|
* Eradicate the variable "time" from the kernel, using various measures.phk1998-03-301-16/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "time" wasn't a atomic variable, so splfoo() protection were needed around any access to it, unless you just wanted the seconds part. Most uses of time.tv_sec now uses the new variable time_second instead. gettime() changed to getmicrotime(0. Remove a couple of unneeded splfoo() protections, the new getmicrotime() is atomic, (until Bruce sets a breakpoint in it). A couple of places needed random data, so use read_random() instead of mucking about with time which isn't random. Add a new nfs_curusec() function. Mark a couple of bogosities involving the now disappeard time variable. Update ffs_update() to avoid the weird "== &time" checks, by fixing the one remaining call that passwd &time as args. Change profiling in ncr.c to use ticks instead of time. Resolution is the same. Add new function "tvtohz()" to avoid the bogus "splfoo(), add time, call hzto() which subtracts time" sequences. Reviewed by: bde
* Fixed some style bugs in the poll() code.bde1997-11-231-18/+8
| | | | | Removed dead code to "Avoid inadvertently sleeping forever". hzto() never returns 0.
* Move the "retval" (3rd) parameter from all syscall functions and putphk1997-11-061-34/+25
| | | | | | | | | | | | it in struct proc instead. This fixes a boatload of compiler warning, and removes a lot of cruft from the sources. I have not removed the /*ARGSUSED*/, they will require some looking at. libkvm, ps and other userland struct proc frobbing programs will need recompiled.
* Last major round (Unless Bruce thinks of somthing :-) of malloc changes.phk1997-10-121-3/+4
| | | | | | | | Distribute all but the most fundamental malloc types. This time I also remembered the trick to making things static: Put "static" in front of them. A couple of finer points by: bde
* Distribute and statizice a lot of the malloc M_* types.phk1997-10-111-1/+4
| | | | Substantial input from: bde
* Implement poll(2). This is mostly taken from the NetBSD implementationpeter1997-09-141-6/+158
| | | | | | (from some time ago) but with a few tweaks along the way. Obtained from: NetBSD
* Removed unused #includes.bde1997-09-021-5/+1
|
* Modifications to existing files to support the initial AIO/LIO anddyson1997-06-161-1/+5
| | | | kernel based threading support.
* Don't include <sys/ioctl.h> in the kernel. Stage 4: includebde1997-03-241-2/+3
| | | | | | | | <sys/ttycom.h> and sometimes <sys/filio.h> instead of <sys/ioctl.h> in miscellaneous files. Most of these files have nothing to do with ttys but need to include <sys/ttycom.h> to get the definitions of TIOC[SG]PGRP which are (ab)used to convert F[SG]ETOWN fcntls into ioctls.
* Don't #include <sys/fcntl.h> in <sys/file.h> if KERNEL is defined.bde1997-03-231-1/+2
| | | | | Fixed everything that depended on getting fcntl.h stuff from the wrong place. Most things don't depend on file.h stuff at all.
* Removed `volatile' from declaration of `time', and removed the resultingbde1997-03-221-2/+2
| | | | | | | | null casts. `time' is nonvolatile for accesses within a region locked by splclock()/splx(). Accesses outside such a region are invalid, and splx() must have the side effect of potentially changing all global variables (since there are hundreds of sort of volatile variables like `time'), so declaring `time' as volatile didn't have any real benefits.
* Back out part 1 of the MCFH that changed $Id$ to $FreeBSD$. We are notpeter1997-02-221-1/+1
| | | | ready for it yet.
* Improved select():bde1997-02-201-36/+51
| | | | | | | | | | | | | | | - avoid malloc() if the number of fds is small. - pack the bits better so that `small' is quite large. - don't waste time generating zero bits for null fd_set pointers or scanning these bits. Possibly improved select(): - free malloc()ed storage before returning. This is simpler and I think huge select()s aren't worth optimizing since they are rare, relative gain would be small and there would be tiny costs for all selects(). Reviewed by: ache (first version by him too)
* Make the long-awaited change from $Id$ to $FreeBSD$jkh1997-01-141-1/+1
| | | | | | | | This will make a number of things easier in the future, as well as (finally!) avoiding the Id-smashing problem which has plagued developers for so long. Boy, I'm glad we're not using sup anymore. This update would have been insane otherwise.
* Fix a minor style error in my code.smpatel1996-08-201-2/+2
|
OpenPOWER on IntegriCloud