On Fri, 2015-09-18 at 18:18 +0800, Ning Yu wrote: > On Fri, Sep 18, 2015 at 5:34 PM, Ian Kent <raven@xxxxxxxxxx> wrote: > > This is what I get using "stg mail" when mailing patch series. > > > > I've made a few changes to te breakdown of the patches. > > I tried to implement the alarm module change we discussued. > > I changed the mount and expire condition to use a monotonic clock > > as well for both the direc and indirect modules for completeness. > > And I changed the patch descriptions a bit. > > > > I'll add my "Signed-off-by" after we've verified the series is OK > > and I've completed some samity testing. > > > > The series should apply, I don't think there are conflicts with > > patches in my queue that are before them. The CHANGELOG hunk will > > be a problem though, as expected, and will likely change due to > > me changing the patch order before actually committing them. > > > > Let me know if you can see any problem with the series. > > > > It looks much more better than my original version, thanks very much. Really it's much the same, but thanks, > I'm OK with it, and hope it won't cause any regression issue ;) Yeah, but this is very much about changes to expire and that's fairly easy to test, we'll see. > > > --- > > > > Ian Kent (1): > > autofs-5.1.1 - remove unused function elapsed() > > > > Yu Ning (7): > > autofs-5.1.1 - define monotonic clock helper functions > > autofs-5.1.1 - use monotonic clock for alarm thread condition wait > > autofs-5.1.1 - define pending condition init helper function > > autofs-5.1.1 - use monotonic clock for direct mount condition > > autofs-5.1.1 - use monotonic clock for indirect mount condition > > autofs-5.1.1 - change remaining gettimeofday() to use clock_gettime() > > autofs-5.1.1 - change time() to use monotonic_clock() > > > > > > CHANGELOG | 7 +++++++ > > daemon/automount.c | 4 ++-- > > daemon/direct.c | 24 ++++++++---------------- > > daemon/indirect.c | 24 ++++++++---------------- > > daemon/lookup.c | 6 +++--- > > daemon/state.c | 2 +- > > include/automount.h | 31 +++++++++++++++++++++++++++++++ > > include/rpc_subs.h | 1 + > > lib/alarm.c | 28 ++++++++++++++++++++++------ > > lib/cache.c | 2 +- > > lib/master.c | 4 ++-- > > lib/rpc_subs.c | 20 +++++++++++--------- > > modules/dclist.c | 2 +- > > modules/lookup_file.c | 8 ++++---- > > modules/lookup_hesiod.c | 6 +++--- > > modules/lookup_hosts.c | 4 ++-- > > modules/lookup_ldap.c | 10 +++++----- > > modules/lookup_nisplus.c | 10 +++++----- > > modules/lookup_program.c | 10 +++++----- > > modules/lookup_sss.c | 6 +++--- > > modules/lookup_userhome.c | 2 +- > > modules/lookup_yp.c | 8 ++++---- > > modules/mount_autofs.c | 2 +- > > modules/parse_amd.c | 2 +- > > modules/replicated.c | 34 ++++++++++++++++------------------ > > 25 files changed, 148 insertions(+), 109 deletions(-) > > > > -- > > Ian -- To unsubscribe from this list: send the line "unsubscribe autofs" in
