Re: uidinfo has many race conditions.

2000-06-10 Thread Brian Fundakowski Feldman
On Sat, 10 Jun 2000, Doug Rabson wrote: > > Well if we get an atomic_t it could be done that way, even if we > > fail the race for updating and go beyond our rlimit slightly it's > > much cheaper than a spinlock from my PoV. The only problem is > > that afaik on some archs atomic_t can be quite

Re: uidinfo has many race conditions.

2000-06-10 Thread Doug Rabson
On Sat, 10 Jun 2000, Alfred Perlstein wrote: > * Brian Fundakowski Feldman <[EMAIL PROTECTED]> [000610 09:13] wrote: > > On Fri, 9 Jun 2000, Alfred Perlstein wrote: > > > > > * Alfred Perlstein <[EMAIL PROTECTED]> [000609 16:45] wrote: > > > > hi, > > > > > > > > Is it just me or does the fact

Re: uidinfo has many race conditions.

2000-06-10 Thread Brian Fundakowski Feldman
On Sat, 10 Jun 2000, Alfred Perlstein wrote: > * Brian Fundakowski Feldman <[EMAIL PROTECTED]> [000610 09:13] wrote: > > On Fri, 9 Jun 2000, Alfred Perlstein wrote: > > > > > * Alfred Perlstein <[EMAIL PROTECTED]> [000609 16:45] wrote: > > > > hi, > > > > > > > > Is it just me or does the fact

Re: uidinfo has many race conditions.

2000-06-10 Thread Alfred Perlstein
* Brian Fundakowski Feldman <[EMAIL PROTECTED]> [000610 09:13] wrote: > On Fri, 9 Jun 2000, Alfred Perlstein wrote: > > > * Alfred Perlstein <[EMAIL PROTECTED]> [000609 16:45] wrote: > > > hi, > > > > > > Is it just me or does the fact that uidinfo structures (see > > > kern/kern_proc.c) are all

Re: uidinfo has many race conditions.

2000-06-10 Thread Brian Fundakowski Feldman
On Fri, 9 Jun 2000, Alfred Perlstein wrote: > * Alfred Perlstein <[EMAIL PROTECTED]> [000609 16:45] wrote: > > hi, > > > > Is it just me or does the fact that uidinfo structures (see > > kern/kern_proc.c) are allocated with M_WAITOK after finding them > > fails and then inserted into the uidhash

Re: uidinfo has many race conditions.

2000-06-09 Thread Alfred Perlstein
* Alfred Perlstein <[EMAIL PROTECTED]> [000609 16:45] wrote: > hi, > > Is it just me or does the fact that uidinfo structures (see > kern/kern_proc.c) are allocated with M_WAITOK after finding them > fails and then inserted into the uidhash structure a race condition? > > Index: kern_proc.c > ==

uidinfo has many race conditions.

2000-06-09 Thread Alfred Perlstein
hi, Is it just me or does the fact that uidinfo structures (see kern/kern_proc.c) are allocated with M_WAITOK after finding them fails and then inserted into the uidhash structure a race condition? There's also a problem with sbsize checking because of races going on here, what needs to happen i