From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21315 invoked by alias); 13 May 2013 14:17:39 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 21305 invoked by uid 89); 13 May 2013 14:17:38 -0000 X-Spam-SWARE-Status: No, score=-5.3 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RP_MATCHES_RCVD,TW_HW autolearn=ham version=3.3.1 Received: from e24smtp01.br.ibm.com (HELO e24smtp01.br.ibm.com) (32.104.18.85) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Mon, 13 May 2013 14:17:38 +0000 Received: from /spool/local by e24smtp01.br.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 13 May 2013 11:17:35 -0300 Received: from d24dlp01.br.ibm.com (9.18.248.204) by e24smtp01.br.ibm.com (10.172.0.143) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Mon, 13 May 2013 11:17:32 -0300 Received: from d24relay02.br.ibm.com (d24relay02.br.ibm.com [9.13.184.26]) by d24dlp01.br.ibm.com (Postfix) with ESMTP id 843F73520055 for ; Mon, 13 May 2013 10:17:31 -0400 (EDT) Received: from d24av02.br.ibm.com (d24av02.br.ibm.com [9.8.31.93]) by d24relay02.br.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r4DEGOU428311768 for ; Mon, 13 May 2013 11:16:24 -0300 Received: from d24av02.br.ibm.com (loopback [127.0.0.1]) by d24av02.br.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id r4DEHUHD008397 for ; Mon, 13 May 2013 11:17:31 -0300 Received: from [9.8.0.182] ([9.8.0.182]) by d24av02.br.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id r4DEHUmT007145; Mon, 13 May 2013 11:17:30 -0300 Message-ID: <5190F5D8.3050108@linux.vnet.ibm.com> Date: Mon, 13 May 2013 14:17:00 -0000 From: Edjunior Barbosa Machado User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 MIME-Version: 1.0 To: Ulrich Weigand CC: gdb-patches@sourceware.org Subject: Re: [PATCH] Fix hardware watchpoints on PowerPC servers References: <201305131319.r4DDJKnN029814@d06av02.portsmouth.uk.ibm.com> In-Reply-To: <201305131319.r4DDJKnN029814@d06av02.portsmouth.uk.ibm.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-TM-AS-MML: No X-Content-Scanned: Fidelis XPS MAILER x-cbid: 13051314-1524-0000-0000-00000618EB12 X-SW-Source: 2013-05/txt/msg00435.txt.bz2 On 05/13/2013 10:19 AM, Ulrich Weigand wrote: > So the kernel now reports it supports PPC_DEBUG_FEATURE_DATA_BP_RANGE, > but then rejects any attempt to set a range BP of more than 8 bytes? > And there's no way to check for this ahead of time except for checking > the BookE hwcaps bits? That seems unfortunate ... > > But I guess if that's the way the kernel interface is now, we'll have to > add that check. Yes, unfortunately this is the info we have from the kernel interface. booke_debug_info.data_bp_alignment informs the alignment, which, in the case of current bookS processors, is also the max length for the watchpoint (8 bytes). > >> * ppc-linux-nat.c (ppc_linux_region_ok_for_hw_watchpoint): Check if the >> region is ok for a hardware watchpoint using the new ptrace interface >> on Power servers. > > This is OK. > > As noted by Luis, it would be good to rename the "booke" terminology, > maybe using something like "have_ptrace_hwdebug" instead of > have_ptrace_booke_interface. But that should be a separate patch. I'll work on a patch following these suggestions you and Luis mentioned. Thanks for the feedback, Ulrich. -- Edjunior