Bug 430424

Description of problem:

procps-3.2.3-8.9 "vmstat.c" code has a few style problems.

Patch provided. This patch includes the Red Hat "procps-3.2.3-vmstat-pgpg.patch"
which transforms "unsigned long" to "unsigned long long" in some cases.

What is fixed:

Most important: 

Changed the usage output fully - hopefully made clearer, added '-f' explanation
which was missing.

Most problematic:

Output "-s" displays "KB", "KiB", "MB", "MiB" instead of "k", "K", "m", "M",
which is nicer but may break some programs which parse the output of vmstat.

Style fixes:

The "unit selectors" (dataUnit,szDataUnit) are transformed from global vars to
local var passed as arguments.

The "number of updates" (num_updates) is transformed from global var to local
var passed as argument.

The "display forks since boot" mode is implemented as a mode, not as a call and
exit().

The "display active/inactive pages" (a_option) is transformed from global var to
local var passed as argument.

Added function count32Bits() to check whether user requested more than one mode.
Removed the "FIXME" remark for that.

unitConvert() has a special case for SLABSTAT mode which never occurs. Commented
out.

Used "strncpy" instead of "strcpy" to read partition name form argument line.

In main(), "argc" was horribly recycled as "counter of integer arguments". Fixed.

The printf() call for the vmstat results has been rearranged to always used
"delta" values; it's just clearer.

Some variables have been renamed; this makes the diff messy but the code nicer.