Re: [patch 1/3] timerfd: Implement show_fdinfo method

[Cyrill Gorcunov]

On Tue, Apr 08, 2014 at 10:17:14AM +0400, Vladimir Davydov wrote:
...
> > +static int timerfd_show(struct seq_file *m, struct file *file)
> > +{
> > +	struct timerfd_ctx *ctx = file->private_data;
> > +	struct itimerspec t;
> > +
> > +	spin_lock_irq(&ctx->wqh.lock);
> > +	t.it_value = ktime_to_timespec(timerfd_get_remaining(ctx));
> > +	t.it_interval = ktime_to_timespec(ctx->tintv);
> > +	spin_unlock_irq(&ctx->wqh.lock);
> > +
> > +	return seq_printf(m,
> > +			  "clockid: %d\n"
> > +			  "ticks: %llu\n"
> > +			  "settime flags: 0%o\n"
> > +			  "it_value: (%llu, %llu)\n"
> > +			  "it_interval: (%llu, %llu)\n",
> 
> IMO, one would expect to setup the timer on restore by passing the
> values of settime_flags, it_value, and it_interval obtained from the
> fdinfo to sys_timerfd_settime, but this will be incorrect, because AFAIU
> the it_value you report here is always relative to the current time, no
> matter whether TFD_TIMER_ABSTIME was set in settime_flags or not. Is it OK?

Hi Vladimir! Well it_value returns remaining time so it's up to user to
adjust this value when restore with abs time. That said one can examinate
if abs flag was set and restore accordingly. If this is vague I'm open
to change it to more clear way. Ideas?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/