diff options
| author | kib <kib@FreeBSD.org> | 2011-05-18 22:36:58 +0000 |
|---|---|---|
| committer | kib <kib@FreeBSD.org> | 2011-05-18 22:36:58 +0000 |
| commit | 8407c0a698d82282fb1d9dbf38fafe81aca31583 (patch) | |
| tree | 8e3558bfd943ec6be8318121c6d35bd1e1bb4613 /sys/kern/kern_conf.c | |
| parent | ebacc1d4d8e583e56d47397c314656b2b4be0995 (diff) | |
| download | FreeBSD-src-8407c0a698d82282fb1d9dbf38fafe81aca31583.zip FreeBSD-src-8407c0a698d82282fb1d9dbf38fafe81aca31583.tar.gz | |
The CDP_ACTIVE flag is cleared at the beginning of destroy_devl(),
and destroy_devl() drops dev_mtx. The protection against the race
with dev_rel(), introduced in r163328, should be extended to cover
destroy_devl() calls for the children of the destroyed dev.
Reported and tested by: joerg
MFC after: 1 week
Diffstat (limited to 'sys/kern/kern_conf.c')
| -rw-r--r-- | sys/kern/kern_conf.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/kern/kern_conf.c b/sys/kern/kern_conf.c index b2be5cc..59b876c 100644 --- a/sys/kern/kern_conf.c +++ b/sys/kern/kern_conf.c @@ -981,6 +981,8 @@ destroy_devl(struct cdev *dev) /* Remove name marking */ dev->si_flags &= ~SI_NAMED; + dev->si_refcount++; /* Avoid race with dev_rel() */ + /* If we are a child, remove us from the parents list */ if (dev->si_flags & SI_CHILD) { LIST_REMOVE(dev, si_siblings); @@ -997,7 +999,6 @@ destroy_devl(struct cdev *dev) dev->si_flags &= ~SI_CLONELIST; } - dev->si_refcount++; /* Avoid race with dev_rel() */ csw = dev->si_devsw; dev->si_devsw = NULL; /* already NULL for SI_ALIAS */ while (csw != NULL && csw->d_purge != NULL && dev->si_threadcount) { |
