From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757192AbZEUOCz (ORCPT ); Thu, 21 May 2009 10:02:55 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756881AbZEUOCY (ORCPT ); Thu, 21 May 2009 10:02:24 -0400 Received: from e28smtp07.in.ibm.com ([59.145.155.7]:52975 "EHLO e28smtp07.in.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755616AbZEUOCX (ORCPT ); Thu, 21 May 2009 10:02:23 -0400 Date: Thu, 21 May 2009 19:32:21 +0530 From: "K.Prasad" To: Ingo Molnar , Frederic Weisbecker Cc: Linux Kernel Mailing List , Alan Stern , "K.Prasad" Subject: [Patch 07/12] Modify signal handling code to refrain from re-enabling HW Breakpoints Message-ID: <20090521140221.GH13849@in.ibm.com> References: <20090521095613.834622717@prasadkr_t60p.in.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline; filename=modify_signal_07 User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch disables re-enabling of Hardware Breakpoint registers through the signal handling code. This is now done during hw_breakpoint_handler(). Original-patch-by: Alan Stern Signed-off-by: K.Prasad Reviewed-by: Alan Stern --- arch/x86/kernel/signal.c | 9 --------- 1 file changed, 9 deletions(-) Index: linux-2.6-tip.hbkpt/arch/x86/kernel/signal.c =================================================================== --- linux-2.6-tip.hbkpt.orig/arch/x86/kernel/signal.c +++ linux-2.6-tip.hbkpt/arch/x86/kernel/signal.c @@ -799,15 +799,6 @@ static void do_signal(struct pt_regs *re signr = get_signal_to_deliver(&info, &ka, regs, NULL); if (signr > 0) { - /* - * Re-enable any watchpoints before delivering the - * signal to user space. The processor register will - * have been cleared if the watchpoint triggered - * inside the kernel. - */ - if (current->thread.debugreg7) - set_debugreg(current->thread.debugreg7, 7); - /* Whee! Actually deliver the signal. */ if (handle_signal(signr, &info, &ka, oldset, regs) == 0) { /*