instead of always working on the job_ptr->tres_req_array

The sacctmgr now accept "resource" and "reservation" as arguments,
  and some calls in the test just used "res", which now failed.
  Expanded some function names for clarity and changed the sacctmgr
  arguments to spell out "resource" rather than just "res".

Change some code format from Linux kernel to K&R, for better or worse...

of the whole type

duplicate code.

When a persistent burst buffer is created, capture its session ID
  and create time from DataWarp and use that information in the
  reservation used for accounting purposes.

change after this point.

Remove some vestigial limits functions and convert to use new
  functions where appropriate. NOTE: burst_buffer/generic limits
  are now no longer functional. The plugin will need to be rebuild
  later based upon burst_buffer/cray.

Power management error codes were NOT added to end of enum,
resulting in inconsistent error codes/messages

Signed-off-by: Danny Auble <da@schedmd.com>

Add tres_id to state (in addition to tres_pos).
Add calls to post persistent buffer create/delete to accounting
  using advanced reservation data structure.
Add ID to buffer for use in accounting.
Fix short malloc bug.
Fix for unitialized variable.
Fix locking problem (wrong variable previous used).