diff options
author | avg <avg@FreeBSD.org> | 2011-07-25 09:12:48 +0000 |
---|---|---|
committer | avg <avg@FreeBSD.org> | 2011-07-25 09:12:48 +0000 |
commit | 50b05401d3190631fb48430b4c0b48789f259b8d (patch) | |
tree | ff97ee9903ea140928fcd9303e0d93873d3c2d2e /sys/netgraph | |
parent | ffeefed9fc8fa85d3fcbd19640ba38e51e2ff4da (diff) | |
download | FreeBSD-src-50b05401d3190631fb48430b4c0b48789f259b8d.zip FreeBSD-src-50b05401d3190631fb48430b4c0b48789f259b8d.tar.gz |
remove RESTARTABLE_PANICS option
This is done per request/suggestion from John Baldwin
who introduced the option. Trying to resume normal
system operation after a panic is very unpredictable
and dangerous. It will become even more dangerous
when we allow a thread in panic(9) to penetrate all
lock contexts.
I understand that the only purpose of this option was
for testing scenarios potentially resulting in panic.
Suggested by: jhb
Reviewed by: attilio, jhb
X-MFC-After: never
Approved by: re (kib)
Diffstat (limited to 'sys/netgraph')
-rw-r--r-- | sys/netgraph/ng_ether.c | 3 | ||||
-rw-r--r-- | sys/netgraph/ng_mppc.c | 3 | ||||
-rw-r--r-- | sys/netgraph/ng_parse.c | 12 |
3 files changed, 0 insertions, 18 deletions
diff --git a/sys/netgraph/ng_ether.c b/sys/netgraph/ng_ether.c index afedaa3..15e8b2d 100644 --- a/sys/netgraph/ng_ether.c +++ b/sys/netgraph/ng_ether.c @@ -604,9 +604,6 @@ ng_ether_rcvdata(hook_p hook, item_p item) NG_FREE_ITEM(item); panic("%s: weird hook", __func__); -#ifdef RESTARTABLE_PANICS /* so we don't get an error msg in LINT */ - return (0); -#endif } /* diff --git a/sys/netgraph/ng_mppc.c b/sys/netgraph/ng_mppc.c index 75194e8..cb65316 100644 --- a/sys/netgraph/ng_mppc.c +++ b/sys/netgraph/ng_mppc.c @@ -404,9 +404,6 @@ ng_mppc_rcvdata(hook_p hook, item_p item) /* Oops */ panic("%s: unknown hook", __func__); -#ifdef RESTARTABLE_PANICS - return (EINVAL); -#endif } /* diff --git a/sys/netgraph/ng_parse.c b/sys/netgraph/ng_parse.c index 1215df1..fbbefc6 100644 --- a/sys/netgraph/ng_parse.c +++ b/sys/netgraph/ng_parse.c @@ -374,9 +374,6 @@ ng_int8_unparse(const struct ng_parse_type *type, break; default: panic("%s: unknown type", __func__); -#ifdef RESTARTABLE_PANICS - return(0); -#endif } if ((error = ng_parse_append(&cbuf, &cbuflen, fmt, fval)) != 0) return (error); @@ -473,9 +470,6 @@ ng_int16_unparse(const struct ng_parse_type *type, break; default: panic("%s: unknown type", __func__); -#ifdef RESTARTABLE_PANICS - return(0); -#endif } if ((error = ng_parse_append(&cbuf, &cbuflen, fmt, fval)) != 0) return (error); @@ -575,9 +569,6 @@ ng_int32_unparse(const struct ng_parse_type *type, break; default: panic("%s: unknown type", __func__); -#ifdef RESTARTABLE_PANICS - return(0); -#endif } if ((error = ng_parse_append(&cbuf, &cbuflen, fmt, fval)) != 0) return (error); @@ -673,9 +664,6 @@ ng_int64_unparse(const struct ng_parse_type *type, break; default: panic("%s: unknown type", __func__); -#ifdef RESTARTABLE_PANICS - return(0); -#endif } if ((error = ng_parse_append(&cbuf, &cbuflen, fmt, fval)) != 0) return (error); |