Module Name: src Committed By: msaitoh Date: Mon Jun 17 06:38:30 UTC 2019
Modified Files: src/sys/arch/x86/pci: msipic.c pci_intr_machdep.c pcib.c src/sys/arch/x86/x86: idt.c intr.c ioapic.c Log Message: KNF. No functional change. To generate a diff of this commit: cvs rdiff -u -r1.14 -r1.15 src/sys/arch/x86/pci/msipic.c cvs rdiff -u -r1.49 -r1.50 src/sys/arch/x86/pci/pci_intr_machdep.c cvs rdiff -u -r1.17 -r1.18 src/sys/arch/x86/pci/pcib.c cvs rdiff -u -r1.10 -r1.11 src/sys/arch/x86/x86/idt.c cvs rdiff -u -r1.145 -r1.146 src/sys/arch/x86/x86/intr.c cvs rdiff -u -r1.61 -r1.62 src/sys/arch/x86/x86/ioapic.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/pci/msipic.c diff -u src/sys/arch/x86/pci/msipic.c:1.14 src/sys/arch/x86/pci/msipic.c:1.15 --- src/sys/arch/x86/pci/msipic.c:1.14 Mon Jun 17 05:45:46 2019 +++ src/sys/arch/x86/pci/msipic.c Mon Jun 17 06:38:29 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: msipic.c,v 1.14 2019/06/17 05:45:46 msaitoh Exp $ */ +/* $NetBSD: msipic.c,v 1.15 2019/06/17 06:38:29 msaitoh Exp $ */ /* * Copyright (c) 2015 Internet Initiative Japan Inc. @@ -27,7 +27,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: msipic.c,v 1.14 2019/06/17 05:45:46 msaitoh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: msipic.c,v 1.15 2019/06/17 06:38:29 msaitoh Exp $"); #include "opt_intrdebug.h" @@ -203,7 +203,7 @@ msipic_find_msi_pic_locked(int devid) KASSERT(mutex_owned(&msipic_list_lock)); LIST_FOREACH(mpp, &msipic_list, mp_list) { - if(mpp->mp_devid == devid) + if (mpp->mp_devid == devid) return mpp->mp_pic; } return NULL; @@ -645,7 +645,7 @@ msipic_construct_msix_pic(const struct p tbl = pci_conf_read(pc, tag, off + PCI_MSIX_TBLOFFSET); table_offset = tbl & PCI_MSIX_TBLOFFSET_MASK; bir = tbl & PCI_MSIX_PBABIR_MASK; - switch(bir) { + switch (bir) { case 0: bar = PCI_BAR0; break; @@ -665,18 +665,19 @@ msipic_construct_msix_pic(const struct p bar = PCI_BAR5; break; default: - aprint_error("detect an illegal device! The device use reserved BIR values.\n"); + aprint_error("detect an illegal device! " + "The device use reserved BIR values.\n"); msipic_destruct_common_msi_pic(msix_pic); return NULL; } memtype = pci_mapreg_type(pc, tag, bar); - /* - * PCI_MSIX_TABLE_ENTRY_SIZE consists below - * - Vector Control (32bit) - * - Message Data (32bit) - * - Message Upper Address (32bit) - * - Message Lower Address (32bit) - */ + /* + * PCI_MSIX_TABLE_ENTRY_SIZE consists below + * - Vector Control (32bit) + * - Message Data (32bit) + * - Message Upper Address (32bit) + * - Message Lower Address (32bit) + */ table_size = table_nentry * PCI_MSIX_TABLE_ENTRY_SIZE; #if 0 err = pci_mapreg_submap(pa, bar, memtype, BUS_SPACE_MAP_LINEAR, Index: src/sys/arch/x86/pci/pci_intr_machdep.c diff -u src/sys/arch/x86/pci/pci_intr_machdep.c:1.49 src/sys/arch/x86/pci/pci_intr_machdep.c:1.50 --- src/sys/arch/x86/pci/pci_intr_machdep.c:1.49 Mon Feb 11 14:59:33 2019 +++ src/sys/arch/x86/pci/pci_intr_machdep.c Mon Jun 17 06:38:29 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: pci_intr_machdep.c,v 1.49 2019/02/11 14:59:33 cherry Exp $ */ +/* $NetBSD: pci_intr_machdep.c,v 1.50 2019/06/17 06:38:29 msaitoh Exp $ */ /*- * Copyright (c) 1997, 1998, 2009 The NetBSD Foundation, Inc. @@ -73,7 +73,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: pci_intr_machdep.c,v 1.49 2019/02/11 14:59:33 cherry Exp $"); +__KERNEL_RCSID(0, "$NetBSD: pci_intr_machdep.c,v 1.50 2019/06/17 06:38:29 msaitoh Exp $"); #include <sys/types.h> #include <sys/param.h> @@ -234,7 +234,7 @@ pci_intr_string(pci_chipset_tag_t pc, pc buf, len); } -#if defined(__HAVE_PCI_MSI_MSIX) +#if defined(__HAVE_PCI_MSI_MSIX) if (INT_VIA_MSI(ih)) return x86_pci_msi_string(pc, ih, buf, len); #endif @@ -266,9 +266,9 @@ pci_intr_setattr(pci_chipset_tag_t pc, p switch (attr) { case PCI_INTR_MPSAFE: if (data) { - *ih |= MPSAFE_MASK; + *ih |= MPSAFE_MASK; } else { - *ih &= ~MPSAFE_MASK; + *ih &= ~MPSAFE_MASK; } /* XXX Set live if already mapped. */ return 0; @@ -524,7 +524,7 @@ pci_intr_alloc(const struct pci_attach_a msi_count = 1; intx_count = 1; } else { - switch(max_type) { + switch (max_type) { case PCI_INTR_TYPE_MSIX: msix_count = counts[PCI_INTR_TYPE_MSIX]; /* FALLTHROUGH */ Index: src/sys/arch/x86/pci/pcib.c diff -u src/sys/arch/x86/pci/pcib.c:1.17 src/sys/arch/x86/pci/pcib.c:1.18 --- src/sys/arch/x86/pci/pcib.c:1.17 Mon Feb 11 14:59:33 2019 +++ src/sys/arch/x86/pci/pcib.c Mon Jun 17 06:38:29 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: pcib.c,v 1.17 2019/02/11 14:59:33 cherry Exp $ */ +/* $NetBSD: pcib.c,v 1.18 2019/06/17 06:38:29 msaitoh Exp $ */ /*- * Copyright (c) 1996, 1998 The NetBSD Foundation, Inc. @@ -30,7 +30,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: pcib.c,v 1.17 2019/02/11 14:59:33 cherry Exp $"); +__KERNEL_RCSID(0, "$NetBSD: pcib.c,v 1.18 2019/06/17 06:38:29 msaitoh Exp $"); #include <sys/types.h> #include <sys/param.h> @@ -167,7 +167,7 @@ pcibmatch(device_t parent, cfdata_t matc clock_broken_latch = 0; } #endif - return(1); + return (1); } break; } Index: src/sys/arch/x86/x86/idt.c diff -u src/sys/arch/x86/x86/idt.c:1.10 src/sys/arch/x86/x86/idt.c:1.11 --- src/sys/arch/x86/x86/idt.c:1.10 Mon Feb 11 14:59:33 2019 +++ src/sys/arch/x86/x86/idt.c Mon Jun 17 06:38:30 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: idt.c,v 1.10 2019/02/11 14:59:33 cherry Exp $ */ +/* $NetBSD: idt.c,v 1.11 2019/06/17 06:38:30 msaitoh Exp $ */ /*- * Copyright (c) 1996, 1997, 1998, 2000, 2009 The NetBSD Foundation, Inc. @@ -65,7 +65,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: idt.c,v 1.10 2019/02/11 14:59:33 cherry Exp $"); +__KERNEL_RCSID(0, "$NetBSD: idt.c,v 1.11 2019/06/17 06:38:30 msaitoh Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -77,11 +77,11 @@ __KERNEL_RCSID(0, "$NetBSD: idt.c,v 1.10 #include <machine/segments.h> -/* +/* * XEN PV and native have a different idea of what idt entries should * look like. */ -idt_descriptor_t *idt; +idt_descriptor_t *idt; static char idt_allocmap[NIDT]; @@ -92,13 +92,13 @@ void set_idtgate(struct trap_info *xen_idd, void *function, int ist, int type, int dpl, int sel) { - /* + /* * Find the page boundary in which the descriptor resides. * We make an assumption here, that the descriptor is part of * a table (array), which fits in a page and is page aligned. * * This assumption is from the usecases at early startup in - * machine/machdep.c + * machine/machdep.c * * Thus this function may not work in the "general" case of a * randomly located idt entry template (for eg:). @@ -109,13 +109,13 @@ set_idtgate(struct trap_info *xen_idd, v //kpreempt_disable(); #if defined(__x86_64__) /* Make it writeable, so we can update the values. */ - pmap_changeprot_local(xen_idt_vaddr, VM_PROT_READ|VM_PROT_WRITE); + pmap_changeprot_local(xen_idt_vaddr, VM_PROT_READ | VM_PROT_WRITE); #endif /* __x86_64 */ xen_idd->cs = sel; xen_idd->address = (unsigned long) function; xen_idd->flags = dpl; - /* + /* * Again we make the assumption that the descriptor is * implicitly part of an idt, which we infer as * xen_idt_vaddr. (See above). @@ -135,7 +135,7 @@ unset_idtgate(struct trap_info *xen_idd) vaddr_t xen_idt_vaddr = ((vaddr_t) xen_idd) & PAGE_MASK; /* Make it writeable, so we can update the values. */ - pmap_changeprot_local(xen_idt_vaddr, VM_PROT_READ|VM_PROT_WRITE); + pmap_changeprot_local(xen_idt_vaddr, VM_PROT_READ | VM_PROT_WRITE); #endif /* __x86_64 */ /* Zero it */ Index: src/sys/arch/x86/x86/intr.c diff -u src/sys/arch/x86/x86/intr.c:1.145 src/sys/arch/x86/x86/intr.c:1.146 --- src/sys/arch/x86/x86/intr.c:1.145 Wed Jun 5 04:31:37 2019 +++ src/sys/arch/x86/x86/intr.c Mon Jun 17 06:38:30 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: intr.c,v 1.145 2019/06/05 04:31:37 knakahara Exp $ */ +/* $NetBSD: intr.c,v 1.146 2019/06/17 06:38:30 msaitoh 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.145 2019/06/05 04:31:37 knakahara Exp $"); +__KERNEL_RCSID(0, "$NetBSD: intr.c,v 1.146 2019/06/17 06:38:30 msaitoh Exp $"); #include "opt_intrdebug.h" #include "opt_multiprocessor.h" @@ -361,12 +361,13 @@ intr_calculatemasks(struct cpu_info *ci) * by MI code and intrctl(8). */ const char * -intr_create_intrid(int legacy_irq, struct pic *pic, int pin, char *buf, size_t len) +intr_create_intrid(int legacy_irq, struct pic *pic, int pin, char *buf, + size_t len) { int ih = 0; #if NPCI > 0 -#if defined(__HAVE_PCI_MSI_MSIX) +#if defined(__HAVE_PCI_MSI_MSIX) if ((pic->pic_type == PIC_MSI) || (pic->pic_type == PIC_MSIX)) { uint64_t pih; int dev, vec; @@ -383,7 +384,7 @@ intr_create_intrid(int legacy_irq, struc return x86_pci_msi_string(NULL, pih, buf, len); } -#endif /* __HAVE_PCI_MSI_MSIX */ +#endif /* __HAVE_PCI_MSI_MSIX */ #endif if (pic->pic_type == PIC_XEN) { @@ -958,10 +959,10 @@ intr_establish_xname(int legacy_irq, str mutex_exit(&cpu_lock); if (bootverbose || cpu_index(ci) != 0) - aprint_verbose("allocated pic %s type %s pin %d level %d to %s slot %d " - "idt entry %d\n", - pic->pic_name, type == IST_EDGE ? "edge" : "level", pin, level, - device_xname(ci->ci_dev), slot, idt_vec); + aprint_verbose("allocated pic %s type %s pin %d level %d to " + "%s slot %d idt entry %d\n", + pic->pic_name, type == IST_EDGE ? "edge" : "level", pin, + level, device_xname(ci->ci_dev), slot, idt_vec); return (ih); } @@ -1033,7 +1034,8 @@ intr_disestablish_xcall(void *arg1, void * */ if (source->is_handlers == NULL) - (*pic->pic_delroute)(pic, ci, ih->ih_pin, idtvec, source->is_type); + (*pic->pic_delroute)(pic, ci, ih->ih_pin, idtvec, + source->is_type); else (*pic->pic_hwunmask)(pic, ih->ih_pin); @@ -1279,7 +1281,7 @@ cpu_intr_init(struct cpu_info *ci) */ istack = uvm_km_alloc(kernel_map, INTRSTACKSIZE + redzone_const_or_zero(2 * PAGE_SIZE), 0, - UVM_KMF_WIRED|UVM_KMF_ZERO); + UVM_KMF_WIRED | UVM_KMF_ZERO); if (redzone_const_or_false(true)) { pmap_kremove(istack, PAGE_SIZE); pmap_kremove(istack + INTRSTACKSIZE + PAGE_SIZE, PAGE_SIZE); @@ -1334,9 +1336,10 @@ intr_printconfig(void) isp = ci->ci_isources[i]; if (isp == NULL) continue; - (*pr)("%s source %d is pin %d from pic %s type %d maxlevel %d\n", - device_xname(ci->ci_dev), i, isp->is_pin, - isp->is_pic->pic_name, isp->is_type, isp->is_maxlevel); + (*pr)("%s source %d is pin %d from pic %s type %d " + "maxlevel %d\n", device_xname(ci->ci_dev), i, + isp->is_pin, isp->is_pic->pic_name, isp->is_type, + isp->is_maxlevel); for (ih = isp->is_handlers; ih != NULL; ih = ih->ih_next) (*pr)("\thandler %p level %d\n", @@ -1843,8 +1846,8 @@ intr_set_affinity(struct intrsource *isp err = intr_find_unused_slot(newci, &newslot); if (err) { - DPRINTF(("failed to allocate interrupt slot for PIC %s intrid %s\n", - isp->is_pic->pic_name, isp->is_intrid)); + DPRINTF(("failed to allocate interrupt slot for PIC %s intrid " + "%s\n", isp->is_pic->pic_name, isp->is_intrid)); return err; } @@ -2123,7 +2126,8 @@ interrupt_construct_intrids(const kcpuse return 0; /* - * Count the number of interrupts which affinity to any cpu of "cpuset". + * Count the number of interrupts which affinity to any cpu of + * "cpuset". */ count = 0; mutex_enter(&cpu_lock); Index: src/sys/arch/x86/x86/ioapic.c diff -u src/sys/arch/x86/x86/ioapic.c:1.61 src/sys/arch/x86/x86/ioapic.c:1.62 --- src/sys/arch/x86/x86/ioapic.c:1.61 Fri Jun 14 09:23:42 2019 +++ src/sys/arch/x86/x86/ioapic.c Mon Jun 17 06:38:30 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: ioapic.c,v 1.61 2019/06/14 09:23:42 msaitoh Exp $ */ +/* $NetBSD: ioapic.c,v 1.62 2019/06/17 06:38:30 msaitoh Exp $ */ /*- * Copyright (c) 2000, 2009 The NetBSD Foundation, Inc. @@ -44,11 +44,11 @@ * documentation and/or other materials provided with the distribution. * 3. All advertising materials mentioning features or use of this software * must display the following acknowledgement: - * This product includes software developed by the NetBSD - * Foundation, Inc. and its contributors. - * 4. Neither the name of The NetBSD Foundation nor the names of its - * contributors may be used to endorse or promote products derived - * from this software without specific prior written permission. + * This product includes software developed by the NetBSD + * Foundation, Inc. and its contributors. + * 4. Neither the name of The NetBSD Foundation nor the names of its + * contributors may be used to endorse or promote products derived + * from this software without specific prior written permission. * * THIS SOFTWARE IS PROVIDED BY AUTHOR AND CONTRIBUTORS ``AS IS'' AND ANY * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE @@ -64,7 +64,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: ioapic.c,v 1.61 2019/06/14 09:23:42 msaitoh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: ioapic.c,v 1.62 2019/06/17 06:38:30 msaitoh Exp $"); #include "opt_ddb.h"