Re: [PATCHv3 linux-next] hrtimer: Add notifier when clock_was_setwas called

[Fan Du]

On 2013å08æ18æ 17:05, Thomas Gleixner wrote:
> On Wed, 14 Aug 2013, Fan Du wrote:
>
> >  From e3929d4fdfad5b40fd8cad0e217597670d1aef54 Mon Sep 17 00:00:00 2001
> > From: Fan Du<fan.du@xxxxxxxxxxxxx>
> > Date: Wed, 14 Aug 2013 16:39:23 +0800
> > Subject: [PATCHv3 linux-next] hrtimer: Add notifier when clock_was_set was
> > called
> >
> > When clock_was_set is called in case of system wall time change
> > or host resume from suspend state, use this notifier for places
> > where interested in this action, e.g Ipsec SA lifetime management.
>
> Sigh. These notifiers have been proposed in the past and we always
> rejected them. Why do you need this? There should be nothing except
> the core timekeeping code which needs to know about clock_was_set.
>
> Can you please explain what kind of users you have in mind and WHY
> they need to know about it.

Hi, Thomas


Thanks for your patience. Please let me take a few seconds try to
explain this.

Current xfrm layers has *one* hrtimer to guard Ipsec keys timeout,
The timeout could be measured in either of below two ways:

 (1) The timer is started once the keys is created, but this
     key is not necessary actually used right now. In detail,
     record the get_seconds() when this key is created.

 (2) Starting the timer when this key is actually used, e.g when
     an IP packet need to be encrypted. In details, recored the
     get_seconds() when this key is first used.

So in the hrtimer handler, the code get current get_seconds, and
check against with what saved in (1)or(2), and notify the timeout
up to user land.

So the pitfall is using one hrtimer for two timeout events,
most importantly using get_seconds to check timeout, once system
clock is changed by user intentionally, the key timeout could
misbehave wildly.

A refractor has been proposed to get rid of depending on system wall
clock by cleaning up the hrtimer handler. Unfortunately David frowned
on it in (3), and suggest once system clock is changed, adjust the
timeout of the key.


(3): http://www.spinics.net/lists/netdev/msg245169.html


> Thanks,
>
> 	tglx

--
ææéæåèäæç

--fan
--
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/