i don't have the hardware to test, but im pretty confident this is
ok.
ok?
Index: pccbb.c
===================================================================
RCS file: /cvs/src/sys/dev/pci/pccbb.c,v
retrieving revision 1.97
diff -u -p -r1.97 pccbb.c
--- pccbb.c 8 Sep 2017 05:36:52 -0000 1.97
+++ pccbb.c 10 Jul 2018 04:09:38 -0000
@@ -959,50 +959,16 @@ pccbbintr_function(struct pccbb_softc *s
{
int retval = 0, val;
struct pccbb_intrhand_list *pil;
- int s, splchanged;
+ int s;
for (pil = sc->sc_pil; pil != NULL; pil = pil->pil_next) {
- /*
- * XXX priority change. gross. I use if-else
- * sentences instead of switch-case sentences in order
- * to avoid duplicate case value error. More than one
- * IPL_XXX may use the same value. It depends on the
- * implementation.
- */
- splchanged = 1;
-#if 0
- if (pil->pil_level == IPL_SERIAL) {
- s = splserial();
- } else if (pil->pil_level == IPL_HIGH) {
-#endif
- if (pil->pil_level == IPL_HIGH) {
- s = splhigh();
- } else if (pil->pil_level == IPL_CLOCK) {
- s = splclock();
- } else if (pil->pil_level == IPL_AUDIO) {
- s = splaudio();
- } else if (pil->pil_level == IPL_VM) {
- s = splvm();
- } else if (pil->pil_level == IPL_TTY) {
- s = spltty();
-#if 0
- } else if (pil->pil_level == IPL_SOFTSERIAL) {
- s = splsoftserial();
-#endif
- } else if (pil->pil_level == IPL_NET) {
- s = splnet();
- } else {
- splchanged = 0;
- /* XXX: ih lower than IPL_BIO runs w/ IPL_BIO. */
- }
+ s = splraise(pil->pil_level);
val = (*pil->pil_func)(pil->pil_arg);
if (val != 0)
pil->pil_count.ec_count++;
- if (splchanged != 0) {
- splx(s);
- }
+ splx(s);
if (retval == 0 || val != 0)
retval = val;