Module Name: src
Committed By: riastradh
Date: Sat Nov 11 19:25:29 UTC 2017
Modified Files:
src/sys/arch/x86/x86: intr.c
Log Message:
KNF NFC
To generate a diff of this commit:
cvs rdiff -u -r1.110 -r1.111 src/sys/arch/x86/x86/intr.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sys/arch/x86/x86/intr.c
diff -u src/sys/arch/x86/x86/intr.c:1.110 src/sys/arch/x86/x86/intr.c:1.111
--- src/sys/arch/x86/x86/intr.c:1.110 Sat Nov 11 17:37:03 2017
+++ src/sys/arch/x86/x86/intr.c Sat Nov 11 19:25:29 2017
@@ -1,4 +1,4 @@
-/* $NetBSD: intr.c,v 1.110 2017/11/11 17:37:03 riastradh Exp $ */
+/* $NetBSD: intr.c,v 1.111 2017/11/11 19:25:29 riastradh Exp $ */
/*-
* Copyright (c) 2007, 2008, 2009 The NetBSD Foundation, Inc.
@@ -133,7 +133,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: intr.c,v 1.110 2017/11/11 17:37:03 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: intr.c,v 1.111 2017/11/11 19:25:29 riastradh Exp $");
#include "opt_intrdebug.h"
#include "opt_multiprocessor.h"
@@ -1210,7 +1210,7 @@ intr_num_handlers(struct intrsource *isp
#else /* XEN */
void *
intr_establish(int legacy_irq, struct pic *pic, int pin,
- int type, int level, int (*handler)(void *) , void *arg,
+ int type, int level, int (*handler)(void *), void *arg,
bool known_mpsafe)
{
@@ -1220,7 +1220,7 @@ intr_establish(int legacy_irq, struct pi
void *
intr_establish_xname(int legacy_irq, struct pic *pic, int pin,
- int type, int level, int (*handler)(void *) , void *arg,
+ int type, int level, int (*handler)(void *), void *arg,
bool known_mpsafe, const char *xname)
{
if (pic->pic_type == PIC_XEN) {
@@ -1228,8 +1228,7 @@ intr_establish_xname(int legacy_irq, str
event_set_handler(pin, handler, arg, level, xname);
- rih = kmem_zalloc(sizeof(struct intrhand),
- cold ? KM_NOSLEEP : KM_SLEEP);
+ rih = kmem_zalloc(sizeof(*rih), cold ? KM_NOSLEEP : KM_SLEEP);
if (rih == NULL) {
printf("%s: can't allocate handler info\n", __func__);
return NULL;
@@ -1244,7 +1243,6 @@ intr_establish_xname(int legacy_irq, str
* All that goes away when we nuke event_set_handler()
* et. al. and unify with x86/intr.c
*/
-
rih->ih_pin = pin; /* port */
rih->ih_fun = handler;
rih->ih_arg = arg;
@@ -1273,8 +1271,9 @@ intr_establish_xname(int legacy_irq, str
#else /* NIOAPIC */
return NULL;
#endif /* NIOAPIC */
- } else
+ } else {
snprintf(evname, sizeof(evname), "irq%d", legacy_irq);
+ }
evtchn = xen_pirq_alloc((intr_handle_t *)&legacy_irq, type);
pih = pirq_establish(legacy_irq & 0xff, evtchn, handler, arg, level,
@@ -1315,7 +1314,7 @@ intr_disestablish(struct intrhand *ih)
} else {
where = xc_unicast(0, intr_disestablish_xcall, ih, NULL, ci);
xc_wait(where);
- }
+ }
if (!msipic_is_msi_pic(isp->is_pic) && intr_num_handlers(isp) < 1) {
intr_free_io_intrsource_direct(isp);
}
@@ -1323,7 +1322,8 @@ intr_disestablish(struct intrhand *ih)
kmem_free(ih, sizeof(*ih));
#else /* XEN */
if (ih->pic_type == PIC_XEN) {
- event_remove_handler(ih->ih_pin, ih->ih_realfun, ih->ih_realarg);
+ event_remove_handler(ih->ih_pin, ih->ih_realfun,
+ ih->ih_realarg);
kmem_free(ih, sizeof(*ih));
return;
}