From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751504AbaLQVZY (ORCPT ); Wed, 17 Dec 2014 16:25:24 -0500 Received: from mx1.redhat.com ([209.132.183.28]:47286 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750995AbaLQVZW (ORCPT ); Wed, 17 Dec 2014 16:25:22 -0500 Date: Wed, 17 Dec 2014 16:25:10 -0500 From: Don Zickus To: Andrew Morton Cc: Ingo Molnar , LKML , uobergfe@redhat.com Subject: Re: [PATCH 0/9] watchdog: internally split softlockup and hardlockup Message-ID: <20141217212510.GN35311@redhat.com> References: <1415118024-194247-1-git-send-email-dzickus@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1415118024-194247-1-git-send-email-dzickus@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Andrew, Poke. What can I do to move this patch forward? Cheers, Don On Tue, Nov 04, 2014 at 11:20:15AM -0500, Don Zickus wrote: > The hardlockup and softockup had always been tied together. Due to the request > of KVM folks, they had a need to have one enabled but not the other. > Internally rework the code to split things apart more cleanly. > > There is a bunch of churn here, but the end result should be code that should > be easier to maintain and fix without knowing the internals of what is going > on. > > Tested by Uli and myself. > > Ulrich Obergfell (9): > watchdog: new definitions and variables, initialization > watchdog: introduce the proc_watchdog_update() function > watchdog: move definition of 'watchdog_proc_mutex' outside of > proc_dowatchdog() > watchdog: introduce the proc_watchdog_common() function > watchdog: introduce separate handlers for parameters in > /proc/sys/kernel > watchdog: implement error handling for failure to set up hardware > perf events > watchdog: enable the new user interface of the watchdog mechanism > watchdog: clean up some function names and arguments > watchdog: introduce the hardlockup_detector_disable() function > > arch/x86/kernel/kvm.c | 2 +- > include/linux/nmi.h | 21 ++-- > kernel/sysctl.c | 35 +++++-- > kernel/watchdog.c | 277 ++++++++++++++++++++++++++++++++++++------------- > 4 files changed, 240 insertions(+), 95 deletions(-) >