summaryrefslogtreecommitdiffstats
path: root/sys/netgraph
diff options
context:
space:
mode:
authoravg <avg@FreeBSD.org>2011-07-25 09:12:48 +0000
committeravg <avg@FreeBSD.org>2011-07-25 09:12:48 +0000
commit50b05401d3190631fb48430b4c0b48789f259b8d (patch)
treeff97ee9903ea140928fcd9303e0d93873d3c2d2e /sys/netgraph
parentffeefed9fc8fa85d3fcbd19640ba38e51e2ff4da (diff)
downloadFreeBSD-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.c3
-rw-r--r--sys/netgraph/ng_mppc.c3
-rw-r--r--sys/netgraph/ng_parse.c12
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);
OpenPOWER on IntegriCloud