Improve header self-containment

[jfree]

Summary

For each header below, add appropriate #includes to make them self-contained.

vfp.h:

• sys/types.h
• sys/ucontext.h: mcontext32_vfp_t

debug_monitor.h:

• sys/types.h

_lock.h:

• sys/types.h

_mutex.h:

• sys/types.h
• sys/_lock.h: struct lock_object

resourcevar.h:

• sys/types.h
• sys/time.h: struct itimerval

user.h:

• sys/param.h: MAXCOMLEN
• sys/priority.h: struct priority
• sys/syslimits.h: PATH_MAX

priority.h:

• sys/types.h

freebsd32.h:

• sys/mount.h: fsid_t

[imp]

There's a general preference to include sys/_types.h instead of sys/types.h
Otherwise, this looks good... I'd be tempted to do one commit per file to make MFC and backout easier.

[jfree]

In D40177#914781, @imp wrote:

There's a general preference to include sys/_types.h instead of sys/types.h

It looks like including sys/_types.h does not define types like uint32_t and uint64_t. I still get compilation errors for every file in this review when replacing sys/types.h with sys/_types.h.

[imp]

In D40177#914817, @jfree wrote:

In D40177#914781, @imp wrote:

There's a general preference to include sys/_types.h instead of sys/types.h

It looks like including sys/_types.h does not define types like uint32_t and uint64_t. I still get compilation errors for every file in this review when replacing sys/types.h with sys/_types.h.

There's two answers for that... if it's user visible, those should be __uint32_t, etc... But if it is kernel private (the whole file), then maybe sys/types.h is fine..

[jfree]

Update diff to include context