On Mon, Mar 25, 2013 at 01:08:10PM -0700, John Stultz wrote: > This patchset makes the lock ownership lines less obvious, but I've > been sure to keep the ntp state static to ntp.c and instead provided > some accessors via ntp-internal.h that timekeping code can use to > make changes. The only really ugly part is that do_adjtimex() has > to split some of the logic between timekeeping.c and ntp.c in order > to really get the locking done correctly.
I didn't find this too ugly or troublesome. The reshuffling you have here looks straightforward to me. Thanks, Richard -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/