Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932594Ab3DYOvf (ORCPT ); Thu, 25 Apr 2013 10:51:35 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47626 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932158Ab3DYOve (ORCPT ); Thu, 25 Apr 2013 10:51:34 -0400 Date: Thu, 25 Apr 2013 16:48:18 +0200 From: Oleg Nesterov To: "H. Peter Anvin" Cc: Ingo Molnar , Linus Torvalds , Cyrill Gorcunov , Peter Zijlstra , Thomas Gleixner , David Miller , "Theodore Ts'o" , Linux Kernel Mailing List , the arch/x86 maintainers , Network Development , "linux-ext4@vger.kernel.org" , Fr??d??ric Weisbecker Subject: Re: [PATCH] x86: make DR*_RESERVED unsigned long Message-ID: <20130425144818.GA25921@redhat.com> References: <20130424072630.GB1780@gmail.com> <20130424170702.GA1867@redhat.com> <5178282D.9030902@zytor.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5178282D.9030902@zytor.com> 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 Content-Length: 1246 Lines: 32 On 04/24, H. Peter Anvin wrote: > > On 04/24/2013 10:07 AM, Oleg Nesterov wrote: > > DR6_RESERVED and DR_CONTROL_RESERVED are used to clear the set > > bits in the "unsigned long" data, make them long to ensure that > > "&~" doesn't clear the upper bits. > > > > do_debug() and ptrace_write_dr7() which use DR*_RESERVED look > > safe, but probably it makes sense to cleanup anyway. > > > > Reported-by: Linus Torvalds > > Signed-off-by: Oleg Nesterov > > Can you please put in the description if this is a manifest or > non-manifest bug and, if manifest, what the issues are? It greatly > affects what we otherwise have to do to address the bug. Sorry if it was not clear, I tried to explain in the changelog that this is only cleanup. Yes, dr6 should have all zeroes in 32-64 so the usage of DR6_RESERVED is safe. > Also, the -UL suffix is usually capitalized in our codebase. OK, iiuc otherwise you agree with this change. I'll send v2. Oleg. -- 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/