From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752710AbcEJPka (ORCPT ); Tue, 10 May 2016 11:40:30 -0400 Received: from mail-oi0-f66.google.com ([209.85.218.66]:34217 "EHLO mail-oi0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751352AbcEJPk3 (ORCPT ); Tue, 10 May 2016 11:40:29 -0400 MIME-Version: 1.0 In-Reply-To: References: <25b7bea4-e319-99b8-ac4b-019f5b2a1904@kernel.org> Date: Tue, 10 May 2016 18:40:25 +0300 Message-ID: Subject: Re: Is it really correct to check for breakpoint in kernel space against ptracer's address space? From: Ruslan Kabatsayev To: Andy Lutomirski Cc: Andy Lutomirski , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 10, 2016 at 6:23 PM, Andy Lutomirski wrote: > Anyway, what kernel are you on? I think I fixed your specific issue in: > > commit 27747f8bc355a2808ca9e490ab6866acd85b4c16 > Author: Andy Lutomirski > Date: Thu Jul 30 20:32:42 2015 -0700 > > perf/x86/hw_breakpoints: Fix check for kernel-space breakpoints > > If that does fix it and it's a problem for you on older kernels, you > could ask for a backport. Indeed, I was using a 3.12 kernel. This should indeed fix it, thanks. BTW, in that commit, why isn't va>=TASK_SIZE_MAX redundant with the second operand of ||? If va is aligned to a multiple of len (so va+len-1 doesn't overflow), then in what cases would the first condition be true with the second being false?