mirror/freebsd
1
0
Fork 0
mirror of https://git.FreeBSD.org/src.git synced 2024-12-19 10:53:58 +00:00
Code Issues Releases Activity

Use the refcount API to implement reference counts on process argument

Browse Source 
structures rather than using a global mutex to protect the reference
counts.

Tested on:	i386, alpha, sparc64
This commit is contained in:
John Baldwin 2005-09-27 18:03:15 +00:00
parent d6fe50b6b9
commit 55b4a5ae0d
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=150630
2 changed files with 4 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
sys/kern/kern_proc.c
View File

@ -43,6 +43,7 @@ __FBSDID("$FreeBSD$");
#include <sys/malloc.h>
#include <sys/mutex.h>
#include <sys/proc.h>
#include <sys/refcount.h>
#include <sys/sysent.h>
#include <sys/sched.h>
#include <sys/smp.h>
@ -90,7 +91,6 @@ struct proclist allproc;
struct proclist zombproc;
struct sx allproc_lock;
struct sx proctree_lock;
struct mtx pargs_ref_lock;
struct mtx ppeers_lock;
uma_zone_t proc_zone;
uma_zone_t ithread_zone;
@ -109,7 +109,6 @@ procinit()
sx_init(&allproc_lock, "allproc");
sx_init(&proctree_lock, "proctree");
mtx_init(&pargs_ref_lock, "struct pargs.ref", NULL, MTX_DEF);
mtx_init(&ppeers_lock, "p_peers", NULL, MTX_DEF);
LIST_INIT(&allproc);
LIST_INIT(&zombproc);
@ -1090,7 +1089,7 @@ pargs_alloc(int len)
MALLOC(pa, struct pargs *, sizeof(struct pargs) + len, M_PARGS,
M_WAITOK);
pa->ar_ref = 1;
refcount_init(&pa->ar_ref, 1);
pa->ar_length = len;
return (pa);
}
@ -1108,9 +1107,7 @@ pargs_hold(struct pargs *pa)
if (pa == NULL)
return;
PARGS_LOCK(pa);
pa->ar_ref++;
PARGS_UNLOCK(pa);
refcount_acquire(&pa->ar_ref);
}
void
@ -1119,12 +1116,8 @@ pargs_drop(struct pargs *pa)
if (pa == NULL)
return;
PARGS_LOCK(pa);
if (--pa->ar_ref == 0) {
PARGS_UNLOCK(pa);
if (refcount_release(&pa->ar_ref))
pargs_free(pa);
} else
PARGS_UNLOCK(pa);
}
/*

5
sys/sys/proc.h
View File

@ -806,10 +806,6 @@ MALLOC_DECLARE(M_ZOMBIE);
curthread->td_pflags &= ~TDP_NOSLEEPING; \
} while (0)
/* Lock and unlock process arguments. */
#define PARGS_LOCK(p) mtx_lock(&pargs_ref_lock)
#define PARGS_UNLOCK(p) mtx_unlock(&pargs_ref_lock)
#define PIDHASH(pid) (&pidhashtbl[(pid) & pidhash])
extern LIST_HEAD(pidhashhead, proc) *pidhashtbl;
extern u_long pidhash;
@ -820,7 +816,6 @@ extern u_long pgrphash;
extern struct sx allproc_lock;
extern struct sx proctree_lock;
extern struct mtx pargs_ref_lock;
extern struct mtx ppeers_lock;
extern struct ksegrp ksegrp0; /* Primary ksegrp in proc0. */
extern struct proc proc0; /* Process slot for swapper. */