Page MenuHomeFreeBSD

vm_object: drop unnecessary vm_object.h header
ClosedPublic

Authored by dougm on Wed, Apr 30, 7:05 AM.

Details

Summary

The header vm_object.h is included in vm_phys.h and uma_core.h, where it is not necessary. Remove it.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dougm requested review of this revision.Wed, Apr 30, 7:05 AM
dougm created this revision.

Did you checked that vm/vm_object.h does not get pulled through any other header? E.g. you can add a temporal fragment like

#ifdef _VM_OBJECT_
#error "vm/vm_object.h still included"
#endif

to these two .c files to be sure.

This revision is now accepted and ready to land.Wed, Apr 30, 8:12 AM
In D50081#1142343, @kib wrote:

Did you checked that vm/vm_object.h does not get pulled through any other header?

I ran this command:

:src $ grep -r --include=*.[h] vm_object.h .
./sys/compat/linuxkpi/common/include/linux/list.h:#include <vm/vm_object.h>
./sys/compat/linuxkpi/common/include/linux/gfp.h:#include <vm/vm_object.h>
./sys/contrib/openzfs/include/os/freebsd/spl/sys/vnode.h:#include <vm/vm_object.h>
./sys/contrib/openzfs/include/os/freebsd/zfs/sys/zfs_znode_impl.h:#include <vm/vm_object.h>
./sys/dev/drm2/drmP.h:#include <vm/vm_object.h>
./sys/fs/nfs/nfsport.h:#include <vm/vm_object.h>

and concluded that neither of these files was including a file that included vm_object.h.

vm_phys.h and uma_core.h

Presumably you meant .c.

This revision was automatically updated to reflect the committed changes.