Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752551AbbBEX17 (ORCPT ); Thu, 5 Feb 2015 18:27:59 -0500 Received: from mail-qa0-f53.google.com ([209.85.216.53]:33794 "EHLO mail-qa0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751046AbbBEX15 (ORCPT ); Thu, 5 Feb 2015 18:27:57 -0500 MIME-Version: 1.0 In-Reply-To: <54D3EAF4.4030304@osg.samsung.com> References: <1423118232-14096-1-git-send-email-john.stultz@linaro.org> <54D3EAF4.4030304@osg.samsung.com> Date: Thu, 5 Feb 2015 15:27:57 -0800 Message-ID: Subject: Re: [RFC][PATCH 0/7] Start adding timetests to selftest From: John Stultz To: Shuah Khan Cc: Linux Kernel Mailing List , Prarit Bhargava , Thomas Gleixner , Richard Cochran Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2100 Lines: 47 On Thu, Feb 5, 2015 at 2:13 PM, Shuah Khan wrote: > On 02/04/2015 11:37 PM, John Stultz wrote: >> I've maintained a suite of timekeeping and timer tests for awhile >> here: https://github.com/johnstultz-work/timetests >> >> I've been meaning to get them reworked and submitted into the >> selftest infrastructure, but haven't had much time recently. >> >> I'm going to have a long flight coming up so I figured I'd >> get started and get some initial feedback. >> >> This series adds all the non-destructive tests from my >> timetests suite. The suite has more tests which actually tweak >> time state and validate things behave as expected, but I wasn't >> sure how to integrate that into the selftest infrastructure >> as I'm not sure everyone would want the tests to be mucking >> with their system time. (Currently my plan is to add a >> script similar to the runall.sh in timetests, so folks >> can go in and manually run it if they're feeling daring) > > Providing a way to run destructive tests as a special > option is great. Non-destructive tests can be run in > default mode. > > Thanks for the tests. Looks good to me. I am in the middle > of review at the moment, will send you comments if any. Yea, I realized after sending it that I didn't checkpatch things and there's a bunch of style issues to be resolved. I've already fixed up most of those, and will re-submit next week. If you have any other feedback (other then white-space/style quirks), do let me know and I'll also try to add that in. I know its close to the merge window, and I'm in no rush to get these in for 3.20, but wanted to send them out for initial thoughts now because doing this has been on my list since kernel summit and was finally able to steal a few evening hours to start and hopefully get some momentum going on it. :) thanks -john -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/