Fix dbus_append_timestamp() for 32 bit systems.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Reported-by: Krishna Harathi <kharathi@exablox.com>
Acked-by: Jim Lieb <lieb@sea-troll.net>
Use dbus_bool_t rather than an int to pass to dbus boolean type

dbus_bool_t is probably an int, but having the correct type is better.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Jim Lieb <lieb@sea-troll.net>
export_id is uint16 bit integer, so we should pass DBUS_TYPE_UINT16

Passing DBUS_TYPE_INT32 will make the dbus code access 2 bytes of
incorrect memory space!

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Reviewed-by: Jim Lieb <lieb@sea-troll.net>
ishealthy is calculated when needed, so avoid having it the structure.

get_ganesha_health() now returns a boolean rather than updating a field in
a structure. The boolean is converted to a dbus_bool_t before passing it
to dbus code.

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>
Acked-by: Jim Lieb <lieb@sea-troll.net>
Fix record_stats definition to accommodate its callers usage!

Fixed the record_stats() definition to comply with its callers! This
patch fixes some incorrect statistics reports (latency and queue wait
time).

Reported-by: Frank Geisenheiner <FGEISEN@de.ibm.com>

Signed-off-by: Malahal Naineni <malahal@us.ibm.com>