diff options
author | pjd <pjd@FreeBSD.org> | 2010-10-07 18:16:22 +0000 |
---|---|---|
committer | pjd <pjd@FreeBSD.org> | 2010-10-07 18:16:22 +0000 |
commit | 52123d52726c193dbe67c32b4a0b4f857c2326bb (patch) | |
tree | a5b990d17dd9adea27c69c4f8e7e16ce6c9588e1 /sbin | |
parent | 86ef76003513bdab705bb75c5b1eed24a4cb1a76 (diff) | |
download | FreeBSD-src-52123d52726c193dbe67c32b4a0b4f857c2326bb.zip FreeBSD-src-52123d52726c193dbe67c32b4a0b4f857c2326bb.tar.gz |
Don't close local component on exit as we can hang waiting on g_waitidle.
I'm unable to reproduce the race described in comment anymore and also the
comment is incorrect - localfd represents local component from configuration
file, eg. /dev/da0 and not HAST provider.
Reported by: Mikolaj Golub <to.my.trociny@gmail.com>
MFC after: 1 week
Diffstat (limited to 'sbin')
-rw-r--r-- | sbin/hastd/primary.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/sbin/hastd/primary.c b/sbin/hastd/primary.c index d99bfd7..091c236 100644 --- a/sbin/hastd/primary.c +++ b/sbin/hastd/primary.c @@ -234,12 +234,6 @@ cleanup(struct hast_resource *res) /* Remember errno. */ rerrno = errno; - /* - * Close descriptor to /dev/hast/<name> - * to work-around race in the kernel. - */ - close(res->hr_localfd); - /* Destroy ggate provider if we created one. */ if (res->hr_ggateunit >= 0) { struct g_gate_ctl_destroy ggiod; |