ztrace: events take const char*

because event() and keyval() are likely to be called with string
literals, which are passed on to the c interface as c strings, take
their arguments as const char* to avoid constructing std::strings when
it isn't necessary

Signed-off-by: Casey Bodley <casey@cohortfs.com>