include.
ClosedPublic
Actions

Authored by skra on Feb 21 2016, 10:33 AM.

Details

Reviewers

kib
alc
jhibbits
cem

Group Reviewers

manpages

Commits

rS295880: As <machine/pmap.h> is included from <vm/pmap.h>, there is no need to

Summary

<machine/pmap> is included from <vm/pmap.h> so there is no need to include it explicitly when <vm/pmap.h> is already included.

Note that there are still files where <machine/pmap> is included solo.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

skra updated this revision to Diff 13560.Feb 21 2016, 10:33 AM

skra retitled this revision from to Remove redundant <machine/pmap.h> include..

skra updated this object.

skra edited the test plan for this revision. (Show Details)

skra added reviewers: alc, kib.

skra set the repository for this revision to rS FreeBSD src repository - subversion.

Herald added reviewers: cem, jhibbits, manpages. · View Herald TranscriptFeb 21 2016, 10:33 AM

Herald added subscribers: rpokala, emaste, andrew, imp. · View Herald Transcript

kib accepted this revision.Feb 21 2016, 11:57 AM

kib edited edge metadata.

This revision is now accepted and ready to land.Feb 21 2016, 11:57 AM

alc accepted this revision.Feb 21 2016, 5:14 PM

alc edited edge metadata.

You might also check for cases where both <vm/vm_param.h> and <machine/vmparam.h> are included. This isn't nearly as common, but I have seen it.

Closed by commit rS295880: As <machine/pmap.h> is included from <vm/pmap.h>, there is no need to (authored by skra). · Explain WhyFeb 22 2016, 9:02 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

share/

man/

man9/

bios.9

1 line

sys/

amd64/

cloudabi64/

cloudabi64_sysvec.c

1 line

vmm/

amd/

npt.c

2 lines

svm.c

1 line

arm/

annapurna/

alpine/

alpine_machdep.c

1 line

arm/

mp_machdep.c

1 line

arm64/

minidump_machdep.c

1 line

cloudabi64/

cloudabi64_sysvec.c

1 line

compat/

linuxkpi/

common/

include/

linux/

dma-mapping.h

1 line

list.h

1 line

src/

linux_compat.c

1 line

linux_pci.c

1 line

dev/

ce/

if_ce.c

1 line

cp/

if_cp.c

1 line

drm/

drmP.h

1 line

drm2/

drmP.h

1 line

fb/

machfb.c

1 line

isci/

isci_oem_parameters.c

1 line

ntb/

if_ntb/

if_ntb.c

1 line

ntb_hw/

ntb_hw.c

1 line

pms/

freebsd/

driver/

ini/

src/

agtiapi.c

1 line

rt/

if_rt.c

1 line

siba/

siba_pcib.c

1 line

vt/

hw/

efifb/

efifb.c

1 line

i386/

bios/

mca_machdep.c

1 line

pci/

pci_cfgreg.c

1 line

mips/

adm5120/

admpci.c

1 line

atheros/

1 line

1 line

1 line

1 line

1 line

1 line

cavium/

cvmx_config.h

1 line

octopci.c

1 line

idt/

idtpci.c

1 line

malta/

gt_pci.c

1 line

mips/

minidump_machdep.c

1 line

nexus.c

1 line

nlm/

xlp_simplebus.c

1 line

rt305x/

rt305x_pci.c

1 line

sibyte/

sb_zbpci.c

1 line

powerpc/

aim/

slb.c

1 line

ofw/

ofw_real.c

1 line

rtas.c

1 line

powermac/

macgpio.c

1 line

macio.c

1 line

platform_powermac.c

1 line

powerpc/

genassym.c

1 line

trap.c

1 line

ps3/

1 line

1 line

1 line

1 line

1 line

1 line

pseries/

platform_chrp.c

1 line

psim/

iobus.c

1 line

sparc64/

pci/

fire.c

1 line

sparc64/

iommu.c

1 line

tlb.c

1 line

x86/

acpica/

acpi_wakeup.c

1 line

x86/

nexus.c

1 line

Diff 13560

View Options

share/man/man9/bios.9

View Options

sys/amd64/cloudabi64/cloudabi64_sysvec.c

View Options

sys/amd64/vmm/amd/npt.c

View Options

sys/amd64/vmm/amd/svm.c

View Options

sys/arm/annapurna/alpine/alpine_machdep.c

View Options

sys/arm/arm/mp_machdep.c

View Options

sys/arm64/arm64/minidump_machdep.c

View Options

sys/arm64/cloudabi64/cloudabi64_sysvec.c

View Options

sys/compat/linuxkpi/common/include/linux/dma-mapping.h

View Options

sys/compat/linuxkpi/common/include/linux/list.h

View Options

sys/compat/linuxkpi/common/src/linux_compat.c

View Options

sys/compat/linuxkpi/common/src/linux_pci.c

View Options

sys/dev/ce/if_ce.c

View Options

sys/dev/cp/if_cp.c

View Options

sys/dev/drm/drmP.h

View Options

sys/dev/drm2/drmP.h

View Options

sys/dev/fb/machfb.c

View Options

sys/dev/isci/isci_oem_parameters.c

View Options

sys/dev/ntb/if_ntb/if_ntb.c

View Options

sys/dev/ntb/ntb_hw/ntb_hw.c

View Options

sys/dev/pms/freebsd/driver/ini/src/agtiapi.c

View Options

sys/dev/rt/if_rt.c

View Options

sys/dev/siba/siba_pcib.c

View Options

sys/dev/vt/hw/efifb/efifb.c

View Options

sys/i386/bios/mca_machdep.c

View Options

sys/i386/pci/pci_cfgreg.c

View Options

sys/mips/adm5120/admpci.c

View Options

sys/mips/atheros/ar71xx_fixup.c

View Options

sys/mips/atheros/ar71xx_pci.c

View Options

sys/mips/atheros/ar71xx_spi.c

View Options

sys/mips/atheros/ar724x_pci.c

View Options

sys/mips/atheros/if_arge.c

View Options

sys/mips/atheros/qca955x_pci.c

View Options

sys/mips/cavium/cvmx_config.h

View Options

sys/mips/cavium/octopci.c

View Options

sys/mips/idt/idtpci.c

View Options

sys/mips/malta/gt_pci.c

View Options

sys/mips/mips/minidump_machdep.c

View Options

sys/mips/mips/nexus.c

View Options

sys/mips/nlm/xlp_simplebus.c

View Options

sys/mips/rt305x/rt305x_pci.c

View Options

sys/mips/sibyte/sb_zbpci.c

View Options

sys/powerpc/aim/slb.c

View Options

sys/powerpc/ofw/ofw_real.c

View Options

sys/powerpc/ofw/rtas.c

View Options

sys/powerpc/powermac/macgpio.c

View Options

sys/powerpc/powermac/macio.c

View Options

sys/powerpc/powermac/platform_powermac.c

View Options

sys/powerpc/powerpc/genassym.c

View Options

sys/powerpc/powerpc/trap.c

View Options

sys/powerpc/ps3/if_glc.c

View Options

sys/powerpc/ps3/platform_ps3.c

View Options

sys/powerpc/ps3/ps3_syscons.c

View Options

sys/powerpc/ps3/ps3bus.c

View Options

sys/powerpc/ps3/ps3cdrom.c

View Options

sys/powerpc/ps3/ps3disk.c

View Options

sys/powerpc/pseries/platform_chrp.c

View Options

sys/powerpc/psim/iobus.c

View Options

sys/sparc64/pci/fire.c

View Options

sys/sparc64/sparc64/iommu.c

View Options

sys/sparc64/sparc64/tlb.c

View Options

sys/x86/acpica/acpi_wakeup.c

View Options

Remove redundant <machine/pmap.h> include.ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 13560

share/man/man9/bios.9

sys/amd64/cloudabi64/cloudabi64_sysvec.c

sys/amd64/vmm/amd/npt.c

sys/amd64/vmm/amd/svm.c

sys/arm/annapurna/alpine/alpine_machdep.c

sys/arm/arm/mp_machdep.c

sys/arm64/arm64/minidump_machdep.c

sys/arm64/cloudabi64/cloudabi64_sysvec.c

sys/compat/linuxkpi/common/include/linux/dma-mapping.h

sys/compat/linuxkpi/common/include/linux/list.h

sys/compat/linuxkpi/common/src/linux_compat.c

sys/compat/linuxkpi/common/src/linux_pci.c

sys/dev/ce/if_ce.c

sys/dev/cp/if_cp.c

sys/dev/drm/drmP.h

sys/dev/drm2/drmP.h

sys/dev/fb/machfb.c

sys/dev/isci/isci_oem_parameters.c

sys/dev/ntb/if_ntb/if_ntb.c

sys/dev/ntb/ntb_hw/ntb_hw.c

sys/dev/pms/freebsd/driver/ini/src/agtiapi.c

sys/dev/rt/if_rt.c

sys/dev/siba/siba_pcib.c

sys/dev/vt/hw/efifb/efifb.c

sys/i386/bios/mca_machdep.c

sys/i386/pci/pci_cfgreg.c

sys/mips/adm5120/admpci.c

sys/mips/atheros/ar71xx_fixup.c

sys/mips/atheros/ar71xx_pci.c

sys/mips/atheros/ar71xx_spi.c

sys/mips/atheros/ar724x_pci.c

sys/mips/atheros/if_arge.c

sys/mips/atheros/qca955x_pci.c

sys/mips/cavium/cvmx_config.h

sys/mips/cavium/octopci.c

sys/mips/idt/idtpci.c

sys/mips/malta/gt_pci.c

sys/mips/mips/minidump_machdep.c

sys/mips/mips/nexus.c

sys/mips/nlm/xlp_simplebus.c

sys/mips/rt305x/rt305x_pci.c

sys/mips/sibyte/sb_zbpci.c

sys/powerpc/aim/slb.c

sys/powerpc/ofw/ofw_real.c

sys/powerpc/ofw/rtas.c

sys/powerpc/powermac/macgpio.c

sys/powerpc/powermac/macio.c

sys/powerpc/powermac/platform_powermac.c

sys/powerpc/powerpc/genassym.c

sys/powerpc/powerpc/trap.c

sys/powerpc/ps3/if_glc.c

sys/powerpc/ps3/platform_ps3.c

sys/powerpc/ps3/ps3_syscons.c

sys/powerpc/ps3/ps3bus.c

sys/powerpc/ps3/ps3cdrom.c

sys/powerpc/ps3/ps3disk.c

sys/powerpc/pseries/platform_chrp.c

sys/powerpc/psim/iobus.c

sys/sparc64/pci/fire.c

sys/sparc64/sparc64/iommu.c

sys/sparc64/sparc64/tlb.c

sys/x86/acpica/acpi_wakeup.c

sys/x86/x86/nexus.c

Remove redundant <machine/pmap.h> include.
ClosedPublic
Actions

Revision Contents
Changeset List