From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anton Vorontsov Subject: Re: [PATCH v7 0/11] KGDB/KDB FIQ (NMI) debugger Date: Thu, 13 Sep 2012 09:55:05 -0700 Message-ID: <20120913165505.GA6517@lizard> References: <20120913150133.GA16238@lizard> <20120913180457.1396637b@bob.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <20120913180457.1396637b@bob.linux.org.uk> Sender: linux-kernel-owner@vger.kernel.org To: Alan Cox Cc: Andrew Morton , Russell King , Jason Wessel , Greg Kroah-Hartman , Arve =?utf-8?B?SGrDuG5uZXbDpWc=?= , Colin Cross , Brian Swetland , John Stultz , Thomas Gleixner , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linaro-kernel@lists.linaro.org, patches@linaro.org, kernel-team@android.com, kgdb-bugreport@lists.sourceforge.net, linux-serial@vger.kernel.org List-Id: linux-serial@vger.kernel.org On Thu, Sep 13, 2012 at 06:04:57PM +0100, Alan Cox wrote: > On Thu, 13 Sep 2012 08:01:33 -0700 > Anton Vorontsov wrote: > > > Hi all, > > > > Here comes the lucky v7: > > > > - Per Alan Cox's suggestion added hangup method and removed a small > > leftover; > > - Per Colin Cross' suggestion moved IRQ quiescing logic into > > poll_get_char routine. IIUC, Alan is less unhappy about it. As a > > result, clear_irq() callback dropped. > > Ok that has my ack. Some of it is not pretty but debugger hooks that > have to run behind the OS while debugging it never are. > > Acked-by: Alan Cox Great! Much thanks for reviews, Alan.