On Wed, 24 Mar 2021 15:04:01 +0200
"Tzvetomir Stoyanov (VMware)" <tz.stoyanov@xxxxxxxxx> wrote:
> @@ -5904,7 +5965,15 @@ static void parse_record_options(int argc,
> break;
> case 'C':
> check_instance_die(ctx->instance, "-C");
> - ctx->instance->clock = optarg;
> + if (!strncmp(optarg, TSCNSEC_CLOCK, strlen(TSCNSEC_CLOCK))) {
> + ret = get_tsc_nsec(&ctx->tsc2nsec.shift,
> + &ctx->tsc2nsec.mult);
> + if (ret || !clock_is_supported(NULL, TSC_CLOCK))
> + die("Clock %s is not supported", optarg);
> + ctx->instance->flags |= BUFFER_FL_TSC2NSEC;
> + ctx->instance->clock = strdup(TSC_CLOCK);
> + } else
> + ctx->instance->clock = optarg;
> ctx->instance->flags |= BUFFER_FL_HAS_CLOCK;
> if (is_top_instance(ctx->instance))
I actually expected in the previous patch to have:
case 'C':
if (!clock_is_supported(optarg))
die("Clock %s is not supported", optarg);
Then in this patch, change it to:
} else if (!clock_is_supported(optarg)) {
die("Clock %s is not supported", optarg);
} else {
ctx->instance->clock = optarg;
}
-- Steve
|