On Thu, Jul 13, 2017 at 12:16:51PM -0600, Jon Derrick wrote: > + status = nvme_set_features(ctrl, NVME_FEAT_TIMESTAMP, > + 0, &ts, 8, NULL); Minor nit: instead of hardcoded '8', sizeof(ts) makes it obvious what that parameter means.