Fix nits in previous commit:
- Remove initializer for badstack_sbuf_size; it gets set unconditionally.
- Remove meaningless comment.
- Group witness_count and its sysctl together.
- Fix spacing in for statements (space after for and within condition).
- Change *all* M_NOWAIT usages in witness_initialize() to M_WAITOK; not just those that were newly introduced -- the allocation is assumed to succeed for all allocations.
- Avoid using uint8_t as the base type in sizeof() expressions; Use the variable name (w_rmatrix) as much as possible.
Pointed out by: jhb@ (thanks!)